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: arm: nxp lpc55xx: fix nxp,ctimer-pwm init procedure (attach clock) #76769

Merged

Conversation

maximevince
Copy link
Contributor

@maximevince maximevince commented Aug 7, 2024

Currently, when using an nxp,ctimer-pwm DTS node, Zephyr boot will crash the system, because it's trying to access registers of the CTIMER peripheral that are not clocked.
This is very annoying to debug, too, since the system completely locks up and SWD access is difficult.

This patchs fixes that for the LPC55xx SoCs.

Fixes #77096

Add clock init for nxp-ctimer-pwm DTS nodes.

Signed-off-by: Maxime Vincent <maxime@veemax.be>
@maximevince maximevince force-pushed the lpc55xx-ctimer-support branch from a28618e to 8325fd3 Compare August 7, 2024 07:45
@maximevince maximevince changed the title soc: arm: lpc55xx: fix nxp,ctimer-pwm init procedure (attach clock) soc: arm: nxp lpc55xx: fix nxp,ctimer-pwm init procedure (attach clock) Aug 7, 2024
Copy link
Collaborator

@mmahadevan108 mmahadevan108 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the patch

@dleach02 dleach02 added the backport v3.7-branch Request backport to the v3.7-branch label Aug 14, 2024
Copy link
Member

@dleach02 dleach02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution. I created an issue and also marked this for backport to v3.7.

@MaureenHelm MaureenHelm merged commit 3a895ec into zephyrproject-rtos:main Aug 14, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport v3.7-branch Request backport to the v3.7-branch platform: NXP NXP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nxp lpc55xx will crash when trying to use a nxp,ctimer-pwm DTS node
5 participants