Skip to content

Conversation

@anangl
Copy link
Contributor

@anangl anangl commented Oct 16, 2024

See commit messages.

@anangl anangl force-pushed the fix_nrf54h_clock_control_enabling_ncs branch 2 times, most recently from ebd427a to 997a0d0 Compare October 24, 2024 06:19
@anangl anangl changed the title [nrf fromtree] soc: nordic: Fix the way of enabling clock control for… [nrf fromtree] clock control related fixes Oct 24, 2024
@anangl anangl force-pushed the fix_nrf54h_clock_control_enabling_ncs branch from 997a0d0 to 76eb469 Compare October 24, 2024 07:29
… nRF54H Series

This is a follow-up to commit 7a2ce28.

Do not enable clock control by default on nRF54H Series SoCs when
the system timer is present, because clock control is not needed
for this purpose there.

Add missing `default y` in the CLOCK_CONTROL_NRF2 Kconfig option that
enables compilation of clock control drivers for nRF54H Series.
This way modules that actually require clock control (like drivers
that use radio) will be able to enable it using the generic option
(CLOCK_CONTROL), not the above one that is specific for nRF54H.

Update accordingly applications that referenced the CLOCK_CONTROL_NRF2
option.

Signed-off-by: Andrzej Głąbek <[email protected]>
(cherry picked from commit edc4f75)
…S is available

To check if DVFS can be used, the CONFIG_NRFS_DVFS_LOCAL_DOMAIN symbol
needs to be used, not CONFIG_NRFS_HAS_DVFS_SERVICE which only indicates
that DVFS is technically possible, not that its local domain part is
actually included in the build.

Signed-off-by: Andrzej Głąbek <[email protected]>
(cherry picked from commit f629f1e)
@anangl anangl force-pushed the fix_nrf54h_clock_control_enabling_ncs branch from 76eb469 to 31619e4 Compare October 25, 2024 06:37
@anangl anangl added the backport v3.7.99-ncs1-branch Relates to NCS v2.8-branch label Oct 29, 2024
@rlubos rlubos merged commit 5a8a8e3 into nrfconnect:main Oct 29, 2024
18 checks passed
@anangl anangl deleted the fix_nrf54h_clock_control_enabling_ncs branch October 29, 2024 10:08
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