Skip to content

dts: only generate getters for okay nodes #106

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 1 commit into
base: main
Choose a base branch
from

Conversation

dsseng
Copy link

@dsseng dsseng commented Jun 9, 2025

Code generated for disabled nodes produces Rust compiler errors, thus
we must not generate getters for unavailable nodes.

Fixes building for RP2350 (otherwise passes and works).

A node is considered enabled if its status property
is either "okay" or not defined
For backwards compatibility, the value "ok"
is treated the same as "okay"

Signed-off-by: Dmitrii Sharshakov [email protected]

Code generated for disabled nodes produces Rust compiler errors, thus
we must not generate getters for unavailable nodes.

Fixes building for RP2350 (otherwise passes and works).

> A node is considered enabled if its status property
> is either ``"okay"`` or not defined
> For backwards compatibility, the value ``"ok"``
> is treated the same as ``"okay"``

Signed-off-by: Dmitrii Sharshakov <[email protected]>
@dsseng
Copy link
Author

dsseng commented Jun 9, 2025

Examples tested on RP2350: async-philosophers, blinky

@go2sh
Copy link

go2sh commented Jun 11, 2025

I tested it with a custom tricore port as well. Fixes the build errors.

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.

2 participants