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: counter: Change the property names in the overlay #83770

Merged
merged 2 commits into from
Jan 14, 2025

Conversation

rruuaanng
Copy link
Collaborator

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

decsny
decsny previously approved these changes Jan 10, 2025
kartben

This comment was marked as duplicate.

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.

same two comments as #83840, both for commit message being "backwards", and need for migration guide entry

@rruuaanng rruuaanng changed the title test: drivers: counter: Change the property names in the overlay dts: bindings: counter: Change the property names in the overlay Jan 13, 2025
@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Jan 13, 2025
Rename the following properties in binding and overlay:
-- primary_source => primary-source
-- secondary_source => secondary-source
-- filter_count => filter-count
-- filter_period => filter-period

Signed-off-by: James Roy <rruuaanng@outlook.com>
fabiobaltieri
fabiobaltieri previously approved these changes Jan 13, 2025
@rruuaanng rruuaanng requested a review from kartben January 13, 2025 14:39
Rename the primary_source, secondary_source,
filter_count and filter_period property to
primary-source,  secondary-source, filter-count
and filter-period in the devicetree and bindings
of the counter subsystem.

Signed-off-by: James Roy <rruuaanng@outlook.com>
@kartben kartben merged commit 4cb8ec7 into zephyrproject-rtos:main Jan 14, 2025
24 checks passed
@rruuaanng rruuaanng deleted the dts-style-counter branch January 14, 2025 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Counter platform: NXP Drivers NXP Semiconductors, drivers Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants