Re-implement us_ticker and lp_ticker for Silicon Labs targets #6471
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Tested with #6472 on the EFM32PG12_STK3402 👍 |
geky
approved these changes
Mar 27, 2018
0xc0170
approved these changes
Mar 27, 2018
|
Jenkins should be fixed via #6469 (once merged we shall rebase this and should be green) |
bulislaw
force-pushed
the
feature-hal-spec-ticker
branch
from
afd66ab to
6daa7fc
Compare
mbed RTC specifications now dictate that the RTC needs to retain and keep on counting through reset. On Silicon Labs parts, this means the RTC API can not be backed by the Silicon Labs RTC peripheral, since that doesn't provide retention functionality. Therefore: * On EFM32GG, EFM32WG, EFM32LG: mbed RTC API is now backed by BURTC. * On EFM32PG, EFR32MG, EFM32PG12, EFR32MG12: mbed RTC API is now backed by RTCC. * On EFM32ZG, EFM32HG: mbed RTC API is sadly no longer supported, since these chips don't have retained memory. # Conflicts: # targets/TARGET_Silicon_Labs/TARGET_EFM32/lp_ticker.c # targets/TARGET_Silicon_Labs/TARGET_EFM32/rtc_api.c # targets/targets.json
Re-implemented both us_ticker and lp_ticker to match the new API and specifications. Details: * On EFM32GG, EFM32WG, EFM32LG, EFM32HG, EFM32ZG: Use the RTC peripheral to back lp_ticker, and a TIMER to back us_ticker. * On EFM32PG, EFR32MG, EFM32PG12, EFR32MG12: Use the RTCC peripheral to back lp_ticker (dual-purpose, also used to back RTC), and a TIMER to back us_ticker.
stevew817
force-pushed
the
feature-hal-spec-ticker
branch
from
343bbed to
fdb4a8e
Compare
|
/morph build |
Build : SUCCESSBuild number : 1611 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 1241 |
Test : SUCCESSBuild number : 1395 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Re-implemented both us_ticker and lp_ticker to match the new API and specifications.
Details:
Blockers
This PR will fail automatic testing until the test methodology is fixed, see issue #6470