Skip to content

[nrf fromtree] Cherry pick nrf auxpll refactor nrf clock control #2994

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

Hi-Im-David
Copy link
Contributor

Cherry pick of zephyrproject-rtos/zephyr#91065 apart from: a5ecca8564b30f9f247b04dbe7f1d179a20ddeed as this requires board files restructure commits

Currently rebased against: #2985
Will drop these commits once it is merged

@anangl
Copy link
Contributor

anangl commented Jul 1, 2025

[nrf fromtree] drivers: clock_control:refactor nrf_auxpll driver to nrf2 is not a valid fromtree commit (you cannot resolve merge conflicts while cherry-picking). You need to cherry-pick zephyrproject-rtos/zephyr@46b5d05 first and then zephyrproject-rtos/zephyr@49b0f1a can be cherry-picked cleanly.

VynDragon and others added 7 commits July 1, 2025 11:23
This introduces a clock_control driver for bl60x

Signed-off-by: Camille BAUD <[email protected]>
(cherry picked from commit 46b5d05)
Introduce Nordic NRF AUXPLL binding include.

Signed-off-by: David Jewsbury <[email protected]>
(cherry picked from commit 8b5ff6a)
…pll binding

Added new binding for macros of possible auxpll frequency settings.

Will be used in future products also.

Signed-off-by: David Jewsbury <[email protected]>
(cherry picked from commit b4a9b8e)
…nrf2

Refactor of previous clock_control_nrf_auxpll.c to use
the nrf2 clock control API

Signed-off-by: David Jewsbury <[email protected]>
(cherry picked from commit 49b0f1a)
Auxpll driver has been refactored to use nrf2 API instead.

Signed-off-by: David Jewsbury <[email protected]>
(cherry picked from commit 9087e82)
…f2_auxpll driver

Test added for new nrf2_auxpll driver. Frequency checks are
against known usecases of the auxpll (USB, CAN and AUDIO).
Also update test to be more modular, only running certain tests
if in the devicetree.

Signed-off-by: David Jewsbury <[email protected]>
(cherry picked from commit bd5aefb)
Move lfosc_get_accuracy away from common library as
not all devices need this function.

Signed-off-by: David Jewsbury <[email protected]>
(cherry picked from commit 0285cf4)
@Hi-Im-David Hi-Im-David force-pushed the cherry-pick-nrf-auxpll-refactor-nrf-clock-control branch from d01d16c to e0ec489 Compare July 1, 2025 10:25
Copy link

sonarqubecloud bot commented Jul 1, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Hi-Im-David
Copy link
Contributor Author

@anangl please could you advise what should I do about this CI failure since it is on a cherry-picked commit?

@anangl
Copy link
Contributor

anangl commented Jul 1, 2025

@anangl please could you advise what should I do about this CI failure since it is on a cherry-picked commit?

I don't really see anything that can be done to prevent this failure. I think we'll need to ignore this one check. @carlescufi What do you think?

@carlescufi carlescufi merged commit 1eb30be into nrfconnect:main Jul 3, 2025
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants