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

soc: nordic: enable DPPI and PPIB nodes by default #82254

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

e-rk
Copy link
Collaborator

@e-rk e-rk commented Nov 28, 2024

The DPPI and PPIB peripheral nodes must be enabled to allow the CONFIG_HAS_HW_NRF_DPPIC to be set. This change is consistent with what was done on nRF5340 and does not introduce any additional memory overhead, because there is no Zephyr driver behind the nrf-dppic and nrf-ppib bindings.

ankuns
ankuns previously approved these changes Nov 28, 2024
anangl
anangl previously approved these changes Nov 29, 2024
The UART_xyz_ENHANCED_POLL_OUT Kconfig was using an outdated property
for checking whether the peripheral has the endtx->stoptx short. The
property is now updated.

Signed-off-by: Rafał Kuźnia <[email protected]>
The DPPI and PPIB peripheral nodes must be enabled to allow the
CONFIG_HAS_HW_NRF_DPPIC to be set. This change is consistent with what
was done on nRF5340 and does not introduce any additional memory
overhead, because there is no Zephyr driver behind the nrf-dppic and
nrf-ppib bindings.

Signed-off-by: Rafał Kuźnia <[email protected]>
@e-rk e-rk dismissed stale reviews from anangl and ankuns via 3137f13 November 29, 2024 09:18
@zephyrbot zephyrbot added the area: UART Universal Asynchronous Receiver-Transmitter label Nov 29, 2024
@e-rk
Copy link
Collaborator Author

e-rk commented Nov 29, 2024

This change uncovered one bug in Kconfig depends statements. The fix was added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: UART Universal Asynchronous Receiver-Transmitter platform: nRF Nordic nRFx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants