Skip to content

Conversation

@zejiang0jason
Copy link
Contributor

  • Enable get_supported_reset_cause for the chips which don't have register RCM->PARAM.
  • Enable hwinfo for MCXE24x and KE1xZ

@zejiang0jason zejiang0jason changed the title Feature/enable hwinfo rcm mcxe24 ke1xz enable hwinfo rcm mcxe24 ke1xz Nov 25, 2025
@zejiang0jason zejiang0jason changed the title enable hwinfo rcm mcxe24 ke1xz enable hwinfo_rcm for mcxe24 and ke1xz Nov 25, 2025
Holt-Sun
Holt-Sun previously approved these changes Nov 25, 2025
@dleach02 dleach02 removed their assignment Dec 3, 2025
@iuliana-prodan
Copy link
Contributor

@zejiang0jason this PR LGTM, but since we are here, can you add another commit to fix this wrong comment:

diff --git a/drivers/hwinfo/hwinfo_mcux_rcm.c b/drivers/hwinfo/hwinfo_mcux_rcm.c
index ed8122ac73b..1e61e9981b1 100644
--- a/drivers/hwinfo/hwinfo_mcux_rcm.c
+++ b/drivers/hwinfo/hwinfo_mcux_rcm.c
@@ -94,7 +94,7 @@ int z_impl_hwinfo_get_reset_cause(uint32_t *cause)
        sources = RCM_GetStickyResetSources(RCM) & kRCM_SourceAll;
 #else /* (defined(FSL_FEATURE_RCM_HAS_SSRS) && FSL_FEATURE_RCM_HAS_SSRS) */
        sources = RCM_GetPreviousResetSources(RCM) & kRCM_SourceAll;
-#endif /* !(defined(FSL_FEATURE_RCM_HAS_PARAM) && FSL_FEATURE_RCM_HAS_PARAM) */
+#endif /* !(defined(FSL_FEATURE_RCM_HAS_SSRS) && FSL_FEATURE_RCM_HAS_SSRS) */
 
        *cause = hwinfo_mcux_rcm_xlate_reset_sources(sources);

Thanks!

iuliana-prodan
iuliana-prodan previously approved these changes Dec 4, 2025
@decsny decsny removed their request for review December 4, 2025 19:13
@mmahadevan108
Copy link
Contributor

@zejiang0jason , please rebase

@zejiang0jason
Copy link
Contributor Author

@zejiang0jason , please rebase

done

Remove the conditional compilation guard FSL_FEATURE_RCM_HAS_PARAM.
`hwinfo_mcux_rcm_xlate_reset_sources` has handled the
supported cause using `FSL_FEATURE_xxx` macros, passing `UINT32_MAX`
can get all supported reset cause.

Signed-off-by: Jason Yu <zejiang.yu@nxp.com>
Enables the MCUX Reset Control Module (RCM) to support hwinfo

Signed-off-by: Jason Yu <zejiang.yu@nxp.com>
Correct mismatched #endif comment to reference the correct macro
FSL_FEATURE_RCM_HAS_SSRS

Signed-off-by: Jason Yu <zejiang.yu@nxp.com>
@zejiang0jason zejiang0jason force-pushed the feature/enable_hwinfo_rcm_mcxe24_ke1xz branch from 53311a3 to a960abd Compare December 8, 2025 01:31
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 8, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants