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: dma: Change the property names in the bindings #83769

Merged
merged 2 commits into from
Mar 24, 2025

Conversation

rruuaanng
Copy link
Collaborator

Change the property names in the bindings and DTS
to use hyphens(-) for separation instead of underscores(_).

@rruuaanng rruuaanng force-pushed the dts-style-1 branch 2 times, most recently from 9d4241d to e1d8392 Compare January 14, 2025 07:18
@rruuaanng rruuaanng marked this pull request as ready for review January 14, 2025 07:18
@rruuaanng rruuaanng changed the title dts: bindings dma: Change the property names in the bindings dts: bindings: dma: Change the property names in the bindings Jan 14, 2025
@zephyrbot zephyrbot added Release Notes To be mentioned in the release notes platform: Silabs Silicon Labs platform: NXP NXP area: DMA Direct Memory Access platform: NXP Drivers NXP Semiconductors, drivers labels Jan 14, 2025
jhedberg
jhedberg previously approved these changes Jan 14, 2025
@rruuaanng
Copy link
Collaborator Author

Emm, it looks like I accidentally modified the cell properties during implementation.

jhedberg
jhedberg previously approved these changes Feb 15, 2025
decsny
decsny previously approved these changes Feb 17, 2025
@fabiobaltieri fabiobaltieri modified the milestones: v4.1.0, v4.2.0 Feb 17, 2025
@rruuaanng rruuaanng dismissed stale reviews from decsny, jerome-pouiller, and jhedberg via 68379d2 March 12, 2025 01:34
Copy link
Member

Choose a reason for hiding this comment

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

wrong file, 4.2

decsny
decsny previously approved these changes Mar 12, 2025
Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

Needs to update the current migration guide

Unify property names in bindings and DTS, using
hyphens (-) instead of underscores (_) as separators.

Signed-off-by: James Roy <rruuaanng@outlook.com>
Rename the nxp,a_on and dma_channels properties
to nxp,a-on and dma-channels in the devicetree
and bindings of the DMA subsystem.

Signed-off-by: James Roy <rruuaanng@outlook.com>
@rruuaanng rruuaanng dismissed stale reviews from jerome-pouiller and decsny via 5f08377 March 19, 2025 04:30
@rruuaanng rruuaanng requested a review from kartben March 19, 2025 04:30
@kartben kartben merged commit a07eb18 into zephyrproject-rtos:main Mar 24, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: DMA Direct Memory Access platform: NXP Drivers NXP Semiconductors, drivers platform: NXP NXP platform: Silabs Silicon Labs Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants