Skip to content

Conversation

kylebonnici
Copy link
Contributor

This PR aligns all the bindings types for Nexus nodes.

Now all are the same as ./zephyr/dts/bindings/pwm/pwm-nexus.yaml

This PR aligns all the bindings types for Nexus nodes.

Now all are the same as ./zephyr/dts/bindings/pwm/pwm-nexus.yaml

Signed-off-by: Kyle Bonnici <[email protected]>
@kylebonnici kylebonnici force-pushed the align-nexus-binding-types branch from eff2c5e to 6017fe9 Compare October 20, 2025 12:21
@sonarqubecloud
Copy link

@decsny
Copy link
Member

decsny commented Oct 22, 2025

what was the motivation of this PR?

@kylebonnici
Copy link
Contributor Author

kylebonnici commented Oct 22, 2025

AFAIK if a property does not respect the binding type, building with that dts file would fail.

This change helps make the type less generic improving validation.

Copy link
Contributor

@mbolivar mbolivar left a comment

Choose a reason for hiding this comment

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

DTSpec is annoyingly vague about the format for these properties in 2.5.1 Nexus Node Properties, but this is consistent with the spec and aligns with every example I could find in Linux, so I agree it's worth having our bindings match and actually validate that the format of these properties is an array.

@jhedberg jhedberg merged commit 0df11fc into zephyrproject-rtos:main Oct 23, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants