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 zephyr updates #1974

Merged
merged 2 commits into from
Sep 17, 2024

Conversation

AndrewD
Copy link
Collaborator

@AndrewD AndrewD commented May 30, 2024

This PR improves the support for indentation so those functions can be reused to generate a nicer Linux DTS. See #1960.

It also changes the formatting of the reg in dts_reg() to be dts schema compliant but I haven't tested if the old format is mandated by zephyr. Can someone please confirm, maybe @maass-hamburg or @trabucayre?

@maass-hamburg
Copy link
Contributor

It also changes the formatting of the reg in dts_reg() to be dts schema compliant but I haven't tested if the old format is mandated by zephyr.

In the zephyr docs it states thats its fine:

Array and similar type property values can be split into several <> blocks

https://docs.zephyrproject.org/latest/build/dts/intro-syntax-structure.html#writing-property-values
the type of reg is array.

@AndrewD
Copy link
Collaborator Author

AndrewD commented Jun 5, 2024

Thanks for the confirmation and link: my focus has been on other docs.

@AndrewD AndrewD force-pushed the dts_zephyr_updates branch from 398c75b to 2c24e52 Compare June 17, 2024 23:38
@AndrewD AndrewD force-pushed the dts_zephyr_updates branch from 2c24e52 to 1962c27 Compare July 21, 2024 05:19
Andrew Dennison added 2 commits July 29, 2024 09:16
To support linux dts generation.
addr size pairs should be enclosed '< >' and comma separated to be
dts compliant.
@AndrewD AndrewD force-pushed the dts_zephyr_updates branch from 1962c27 to e19dfa8 Compare July 28, 2024 23:19
@maass-hamburg
Copy link
Contributor

@enjoy-digital is this going to be merged anytime soon?

@enjoy-digital
Copy link
Owner

@maass-hamburg: It this looks for you, we can merge yes.

@maass-hamburg
Copy link
Contributor

@enjoy-digital looks fine from my side

@maass-hamburg
Copy link
Contributor

@enjoy-digital merge?

@enjoy-digital enjoy-digital merged commit 9bacbe1 into enjoy-digital:master Sep 17, 2024
1 check passed
@enjoy-digital
Copy link
Owner

Done thanks.

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.

3 participants