use RTC as system clock for CC13X2/CC26X2 #19232
Merged
Conversation
This file contains 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
Add RTC timer driver for CC13X2/CC26X2, and use it instead of systick as system clock. It is necessary to use this timer for power management support, so that the system can exit from deep sleep upon expiry of timeouts. Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
The RTC on TI CC13X2/CC26X2 is a 32 KHz clock for which the minimum compare delay is 3 ticks. When using it as the system clock, we need to relax the upper bound to ensure the test succeeds. Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
Given the RTC is used for system clock, we need to plug its interrupt to avoid an exception that would occur if it is left unhandled. Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
vanti
requested review from
andrewboie,
andyross,
bwitherspoon,
galak,
ioannisg and
nashif
as code owners
vanti
changed the title
andyross
approved these changes
Sep 17, 2019
|
@galak Not sure why the checks are not finishing... |
zephyrbot
added
area: Devicetree
area: Timer
|
All checks are passing now. Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages. |
Adding @vanti as an owner. Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
|
Thanks for the help! |
vanti
added a commit
to vanti/zephyr
that referenced
this pull request
Oct 31, 2019
The SPI peripheral should be configured using the CPU clock speed and not the system clock speed. This used to be fine because they were the same before zephyrproject-rtos#19232, but now that the system clock is RTC-based (which has a different frequency), we can no longer make that assumption. Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
galak
pushed a commit
that referenced
this pull request
Nov 7, 2019
The SPI peripheral should be configured using the CPU clock speed and not the system clock speed. This used to be fine because they were the same before #19232, but now that the system clock is RTC-based (which has a different frequency), we can no longer make that assumption. Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
vanti
added a commit
to vanti/zephyr
that referenced
this pull request
Nov 8, 2019
The I2C peripheral should be configured using the CPU clock frequency and not the system clock frequency. This used to be fine because they were the same before zephyrproject-rtos#19232, but now that the system clock is RTC-based (which has a different frequency), we can no longer make that assumption. Fixes zephyrproject-rtos#20480 Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
galak
pushed a commit
that referenced
this pull request
Nov 11, 2019
The I2C peripheral should be configured using the CPU clock frequency and not the system clock frequency. This used to be fine because they were the same before #19232, but now that the system clock is RTC-based (which has a different frequency), we can no longer make that assumption. Fixes #20480 Signed-off-by: Vincent Wan <vincent.wan@linaro.org>
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.
Add RTC timer driver for CC13X2/CC26X2, and use it instead of systick
as system clock. It is necessary to use this timer for power
management support, so that the system can exit from deep sleep upon
expiry of timeouts.
A couple of tests also need to be adjusted as a result of using this timer.