-
Notifications
You must be signed in to change notification settings - Fork 3k
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
GR_LYCHEE,RZ_A1H,VK_RZ_A1H: Fix greentea ticker test case failures #10213
Conversation
@d-kato, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@d-kato bbfee59 looks good to me
mbedgt: test suite report:
| target | platform_name | test suite | result | elapsed_time (sec) | copy_method |
|-------------------|---------------|------------------------------------|--------|--------------------|-------------|
| GR_LYCHEE-GCC_ARM | GR_LYCHEE | tests-mbed_drivers-ticker | OK | 83.67 | default |
| GR_LYCHEE-GCC_ARM | GR_LYCHEE | tests-mbed_drivers-timeout | OK | 96.42 | default |
| GR_LYCHEE-GCC_ARM | GR_LYCHEE | tests-mbed_hal-common_tickers_freq | OK | 75.23 | default |
| GR_LYCHEE-GCC_ARM | GR_LYCHEE | tests-mbed_platform-wait_ns | OK | 34.11 | default |
| GR_LYCHEE-GCC_ARM | GR_LYCHEE | tests-mbedmicro-rtos-mbed-basic | OK | 53.42 | default |
mbedgt: test suite results: 5 OK
Waiting on #10221 for astyle fix. |
Hmm... The astyle fix is missing this fix: #10222 |
please re-base when you have time, it should resolve the astyle issue |
The OS timer of RZ/A1 uses P0 clock, so until now it has been set the value of P0 clock in SystemCoreClock. Changed the system clock value to set to SystemCoreClock. Changed to refer to P0 clock macro instead of SystemCoreClock in OS timer processing.
Since GPIO.PPR0 can not check clock mode, I changed it to set a fixed value for each board.
@NirSonnenschein, All right. I rebased. |
CI started |
GR_LYCHEE,RZ_A1H,VK_RZ_A1H: Fix greentea ticker test case failures
Test run: SUCCESSSummary: 13 of 13 test jobs passed |
Description
Fix for issue #10119
Fixed the greentea test failur of mbed-os-tests-mbed_platform-wait_ns.
Pull request type
Reviewers
@VeijoPesonen
Release Notes