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

dts: bindings: misc: nrf: Support disabling lfosc #1987

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

karstenkoenig
Copy link
Contributor

The BICR LFOSC.LFXOCONFIG.MODE field supports disabling the usage of an external crystal oscillator to disable LFXO as LF clock source. This adds the enum to the binding, the rest of the codebase already supports the feature.

This is already supported in nrf-regtool here https://github.com/nrfconnect/nrf-regtool/commit/b23cdefbd0166b064fa946102b61bc6e097fd749

@anangl
Copy link
Contributor

anangl commented Sep 3, 2024

Please add:
nrf-squash! [nrf noup] dts: nordic: Add support for BICR generation
as the first line of the commit message body.

@karstenkoenig karstenkoenig force-pushed the bicr_bindings_lfosc_disabled branch from 3be6443 to 9fc97e6 Compare September 6, 2024 06:13
nrf-squash! [nrf noup] dts: nordic: Add support for BICR generation

The BICR LFOSC.LFXOCONFIG.MODE field supports disabling the usage of an
external crystal oscillator to disable LFXO as LF clock source. This
adds the enum to the binding, the rest of the codebase already supports
the feature.

Signed-off-by: Karsten Koenig <[email protected]>
@karstenkoenig karstenkoenig force-pushed the bicr_bindings_lfosc_disabled branch from 9fc97e6 to 88b7e77 Compare September 6, 2024 08:33
@karstenkoenig
Copy link
Contributor Author

@anangl fixed, could you please check again? Corresponding sdk-nrf build is also green nrfconnect/sdk-nrf#17098

@anangl anangl merged commit 681f57f into nrfconnect:main Sep 6, 2024
16 checks passed
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.

4 participants