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: can: infineon: xmc4xxx: rename clock_div8 to clock-div8 #83782

Conversation

henrikbrixandersen
Copy link
Member

Rename the Infineon XMC4xxx CAN node devicetree property clock_div8 to clock-div8 (prefering hyphens over underscores to separate devicetree property names).

Rename the Infineon XMC4xxx CAN node devicetree property clock_div8 to
clock-div8 (prefering hyphens over underscores to separate devicetree
property names).

Signed-off-by: Henrik Brix Andersen <hebad@vestas.com>
@henrikbrixandersen henrikbrixandersen added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Jan 10, 2025
@zephyrbot zephyrbot added platform: Infineon Infineon Technologies AG size: XS A PR changing only a single line of code area: CAN labels Jan 10, 2025
@kartben kartben merged commit 8998b1a into zephyrproject-rtos:main Jan 10, 2025
33 checks passed
@kartben
Copy link
Collaborator

kartben commented Jan 11, 2025

@henrikbrixandersen btw I think this is missing an entry in migration guide ; unless I am missing something this will break downstream users (although I think maybe the intent is that eventually, thanks to PR #83352, it will then "work again"? but in the mean time I think people need the heads-up)

@henrikbrixandersen
Copy link
Member Author

@henrikbrixandersen btw I think this is missing an entry in migration guide ; unless I am missing something this will break downstream users (although I think maybe the intent is that eventually, thanks to PR #83352, it will then "work again"? but in the mean time I think people need the heads-up)

Sure, I'll add a note to the migration guide. I left it out since I expected a more general note about all the property renames would be added later on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: CAN platform: Infineon Infineon Technologies AG size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants