Skip to content

Conversation

nordic-krch
Copy link
Contributor

Remove redundant enabling of NRF_RTC_TIMER in SoC specific files and replace it with default y in the NRF_RTC_TIMER definition.

Remove redundant enabling of NRF_RTC_TIMER in SoC specific files and replace it with default y in the NRF_RTC_TIMER definition. Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
bool "nRF Real Time Counter (NRF_RTC1) Timer"
depends on CLOCK_CONTROL
depends on SOC_COMPATIBLE_NRF
depends on !DT_HAS_NORDIC_NRF_RTC_ENABLED && !DT_HAS_NORDIC_NRF_GRTC_ENABLED
Copy link
Member

@anangl anangl Oct 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This dependency looks wrong. It should be possible to use the RTC1 based timer when some other RTC instances are enabled. It's a clean-up PR, so maybe this could be corrected here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catch. That should be narrowed down to rtc1.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Labels

area: Boards/SoCs area: Timer Timer platform: nRF BSIM Nordic Semiconductors, nRF BabbleSim platform: nRF Nordic nRFx

4 participants