Skip to content

[nrf fromlist] scripts: parse module.yml for dts bindings #3193

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

tejlmand
Copy link
Contributor

Change BoardYml compliance check from using get_module_setting_root() to load and parse module.yml directly.

get_module_setting_root cannot be used directly with module.yml. Instead adjust the code so that the module.yml file is loaded locally and the dts_root setting is extracted from the dictionary.

Upstream PR #: 94397

rruuaanng and others added 4 commits August 13, 2025 09:24
Implement a check in the CI pipeline to enforce
that property names in device tree bindings do
not contain underscores.

Signed-off-by: James Roy <[email protected]>
Signed-off-by: Martí Bolívar <[email protected]>
(cherry picked from commit 4240853)
… for Kconfig

Adds support for checking modules for disallow Kconfig's in boards
and SoCs, which have been defined in a Zephyr module file

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit 6d73a9c)
The git diff subprocess was leaking, ie.,
it was still running with it's file streams open,
and python was printing warnings about this. Fix by calling
communicate() on the object which will do the cleanup.

Signed-off-by: Declan Snyder <[email protected]>
(cherry picked from commit 3b5bfd9)
Change BoardYml compliance check from using get_module_setting_root()
to load and parse module.yml directly.

get_module_setting_root cannot be used directly with module.yml.
Instead adjust the code so that the module.yml file is loaded locally
and the dts_root setting is extracted from the dictionary.

Upstream PR #: 94397

Signed-off-by: Torsten Rasmussen <[email protected]>
@tejlmand tejlmand force-pushed the compliance_root_handling_module_yaml branch from cf95d1f to c4dcdcd Compare August 13, 2025 07:27
@tejlmand tejlmand requested a review from tokangas August 13, 2025 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants