Skip to content
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

nRF clock_control add ZLI API #2386

Merged

Conversation

ppryga-nordic
Copy link
Contributor

Implement the zero latency interrupt safe APIs to the HFXO clock commonly used by the Bluetooth stack from zero latency interrupt context.

The PR brings changes done in upstream (+ one cleanup change that is in a PR).

@nordicjm
Copy link
Contributor

Can you close the auto manifest PR if it is not needed and open PRs with the manifest PR skip line if they are not needed?

… APIs

Add zero latency interrupt safe APIs to allow requesting and
releasing HFXO. These will be used from components running
in zero latency interrupt context, like the bluetooth stack.

Co-authored-by: Piotr Pryga <[email protected]>

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
Signed-off-by: Piotr Pryga <[email protected]>
(cherry picked from commit 2aec438)
Signed-off-by: Piotr Pryga <[email protected]>
…sr API

Implement the zero latency interrupt safe APIs to the HFXO clock
commonly used by the bluetooth stach from zero latency interrupt
context.

Co-authored-by: Piotr Pryga <[email protected]>

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
Signed-off-by: Piotr Pryga <[email protected]>
(cherry picked from commit 73a45a7)
Signed-off-by: Piotr Pryga <[email protected]>
@ppryga-nordic ppryga-nordic force-pushed the nrf-clock-control-zli-api-dowstream branch 2 times, most recently from 0687e98 to c5923e4 Compare December 19, 2024 06:21
@carlescufi
Copy link
Contributor

@ppryga-nordic:

ERROR: PR #82995 is merged, please use [nrf fromtree] instead

There were redundant code in full_irq_lock(), full_irq_unlock()
functions that supposed to be used when ZLI IRQs are disabled.
These functions are compiled in only when CONFIG_ZERO_LATENCY_IRQS
is set, hence the non-ZLI execution path was never included
in final binaries.

Signed-off-by: Piotr Pryga <[email protected]>
(cherry picked from commit 233095c3f452c05683f0410140e737d89567d927)
Signed-off-by: Piotr Pryga <[email protected]>
@ppryga-nordic ppryga-nordic force-pushed the nrf-clock-control-zli-api-dowstream branch from c5923e4 to 0862348 Compare December 19, 2024 09:49
@ppryga-nordic
Copy link
Contributor Author

@carlescufi done, should I re-run manifest PR?

@jukkar jukkar merged commit 10a6bd0 into nrfconnect:main Dec 19, 2024
15 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.

7 participants