Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Mbed OS to 6.11.0 #101

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

gpsimenos
Copy link

@gpsimenos gpsimenos commented Oct 13, 2020

Build successes:
  * K64F::GCC_ARM::MBED-BUILD
  * K64F::GCC_ARM::TESTS-API-DIGITALIO
  * K64F::GCC_ARM::TESTS-ASSUMPTIONS-ANALOGIN
  * K64F::GCC_ARM::TESTS-ASSUMPTIONS-ANALOGOUT
  * K64F::GCC_ARM::TESTS-ASSUMPTIONS-DIGITALIO
  * K64F::GCC_ARM::TESTS-ASSUMPTIONS-I2C
  * K64F::GCC_ARM::TESTS-ASSUMPTIONS-PWM
  * K64F::GCC_ARM::TESTS-ASSUMPTIONS-SPI

@MarceloSalazar
Copy link

@ARMmbed/team-st-mcd would you like to test a few of your boards with Mbed OS 6 and provide feedback?

Copy link
Contributor

@jeromecoutant jeromecoutant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I started from a clean view.
This patch set "latest" label in mbed-os which is mbed-os-5.15.1 :-(

@jeromecoutant
Copy link
Contributor

After a manual checkout mbed-os-6.4.0,
I add #44 which is still missing in master

Then compilation with ARM is OK except few warnings:

[Warning] PWM_rise_fall.cpp@77,31: 'read_us' is deprecated: Use the Chrono-based elapsed_time method.  If integer microseconds are needed, you can use `elapsed_time().count()` [since mbed-os-6.0.0] [-Wdeprecated-declarations]
[Warning] PWM_rise_fall.cpp@89,59: 'read_us' is deprecated: Use the Chrono-based elapsed_time method.  If integer microseconds are needed, you can use `elapsed_time().count()` [since mbed-os-6.0.0] [-Wdeprecated-declarations]
[Warning] SPI.cpp@96,39: format string is not a string literal (potentially insecure) [-Wformat-security]
[Warning] Comms.cpp@96,31: format string is not a string literal (potentially insecure) [-Wformat-security]
[Warning] Comms.cpp@161,31: format string is not a string literal (potentially insecure) [-Wformat-security]
[Warning] Mixed.cpp@99,31: format string is not a string literal (potentially insecure) [-Wformat-security]
[Warning] Mixed.cpp@282,35: format string is not a string literal (potentially insecure) [-Wformat-security]

@jeromecoutant
Copy link
Contributor

GCC warnings:

[Warning] BusInOut.cpp@29,9: variable 'x' set but not used [-Wunused-but-set-variable]
[Warning] I2C.cpp@94,9: variable 'r' set but not used [-Wunused-but-set-variable]
[Warning] I2C.cpp@95,9: variable 'w' set but not used [-Wunused-but-set-variable]
[Warning] PWM_rise_fall.cpp@77,39: 'int mbed::TimerBase::read_us() const' is deprecated: Use the Chrono-based elapsed_time method.  If integer microseconds are needed, you can use `elapsed_time().count()` [since mbed-os-6.0.0] [-Wdeprecated-declarations]
[Warning] PWM_rise_fall.cpp@89,67: 'int mbed::TimerBase::read_us() const' is deprecated: Use the Chrono-based elapsed_time method.  If integer microseconds are needed, you can use `elapsed_time().count()` [since mbed-os-6.0.0] [-Wdeprecated-declarations]
[Warning] PWM_rise_fall.cpp@122,9: unused variable 'one_percent' [-Wunused-variable]
[Warning] SPI.cpp@86,9: variable 'error' set but not used [-Wunused-but-set-variable]
[Warning] SPI.cpp@110,9: variable 'error' set but not used [-Wunused-but-set-variable]

@gpsimenos
Copy link
Author

Thanks @jeromecoutant for testing with mbed os 6.4.0. Will update the patch.

@0xc0170
Copy link
Collaborator

0xc0170 commented May 18, 2021

@gpsimenos Still worth updating to the recent Mbed OS version?

@gpsimenos gpsimenos changed the title Update Mbed OS to 6.3.0 Update Mbed OS to 6.11.0 Jun 3, 2021
@jeromecoutant
Copy link
Contributor

jeromecoutant commented Jun 3, 2021

Note goal of this PR should be not to update mbed-os version... but to remove deprecated API and match the latest standard...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants