drivers: hwinfo: cc23x0: Add support for cc23x0 in hwinfo #89508
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
alexanderwachter
requested changes
May 6, 2025
bogdanovs
force-pushed
the
baylibre/upstream-cc23x0-hwinfo
branch
from
a53fe97 to
2f3f683
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR adds hardware info support for the cc23x0 family by implementing two new functions to retrieve supported reset causes and the current reset cause for the device.
- Introduces z_impl_hwinfo_get_supported_reset_cause with combined bit flags.
- Implements z_impl_hwinfo_get_reset_cause using PMCTLGetResetReason() and a switch-case.
Files not reviewed (2)
- drivers/hwinfo/CMakeLists.txt: Language not supported
- drivers/hwinfo/Kconfig: Language not supported
bogdanovs
force-pushed
the
baylibre/upstream-cc23x0-hwinfo
branch
from
2f3f683 to
6b92c02
Compare
alexanderwachter
previously approved these changes
May 6, 2025
kartben
previously approved these changes
May 6, 2025
| case PMCTL_RESET_LOCKUP: | ||
| *cause = RESET_CPU_LOCKUP; | ||
| break; | ||
| default: |
There was a problem hiding this comment.
I believe a few more reset causes are supported:
- PMCTL_RESET_LFXT (LFXT clock loss) for RESET_CLOCK
- PMCTL_RESET_SYSTEM (Software triggered reset) for RESET_SOFTWARE
- PMCTL_RESET_TSD (Device woke up from thermal shutdown after temperature drop) for RESET_TEMPERATURE
- PMCTL_RESET_SWD (Device reset triggered by Serial Wire Debug reset event) for RESET_DEBUG
Contributor
Author
There was a problem hiding this comment.
- PMCTL_RESET_LFXT (LFXT clock loss) for RESET_CLOCK
I was not exactly sure , if this case is exact match. If you think it is I am adding it.
- PMCTL_RESET_SYSTEM (Software triggered reset) for RESET_SOFTWARE
This is already implemented
- PMCTL_RESET_TSD (Device woke up from thermal shutdown after temperature drop) for RESET_TEMPERATURE
I was not exactly sure , if this case is exact match. If you think it is I am adding it.
- PMCTL_RESET_SWD (Device reset triggered by Serial Wire Debug reset event) for RESET_DEBUG
Missed it because there was already SWD case - added now.
bogdanovs
force-pushed
the
baylibre/upstream-cc23x0-hwinfo
branch
from
6b92c02 to
230446a
Compare
Contributor
Author
|
Added couple more supported reset reasons as requested. PR is rebased and on latest main , since force push was required. |
Add support for get_reset_cause and get_supported_reset_cause. Signed-off-by: Stoyan Bogdanov <sbogdanov@baylibre.com>
bogdanovs
force-pushed
the
baylibre/upstream-cc23x0-hwinfo
branch
from
230446a to
1e52d3e
Compare
LarsMorstadTI
approved these changes
May 9, 2025
kartben
approved these changes
May 14, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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 hwinfo support for get_reset_cause and get_supported_reset_cause.
Datasheet: https://www.ti.com/lit/ds/symlink/cc2340r5.pdf