scripts: add tool to list bus-related bindings #24
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| binding_files = [str(path) for path in find_binding_files(bindings_root)] | ||
|
|
||
| bindings = bindings_from_paths(binding_files) | ||
|
|
There was a problem hiding this comment.
The script gathers every YAML file under the bindings root and feeds them to bindings_from_paths(). Many files in dts/bindings are include fragments such as base.yaml that intentionally omit a compatible key; Binding() raises EDTError for those by default. With the current call the tool aborts before producing any output as soon as it encounters such a file, so the default invocation (list_bus_nodes.py with no arguments) cannot succeed even when PyYAML is installed. Consider filtering the list to only bindings with compatible or pass ignore_errors=True so include-only YAML does not kill the script.
Useful? React with 👍 / 👎.
1 participant
Summary
Testing
https://chatgpt.com/codex/tasks/task_e_68dfa52c6c148322bb77c9c68f88cdd9