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.

Sign up for GitHub

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

Jump to bottom

doc: devicetree: clarify child node disabling #82064

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
+3 −0
Open

doc: devicetree: clarify child node disabling #82064

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions doc/build/dts/intro-syntax-structure.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -288,6 +288,9 @@ status
devices must be enabled for the corresponding ``struct device`` in the
Zephyr driver model to be allocated and initialized.

Note that a child node is not implicitly disabled when its parent node is
disabled, i.e. child nodes should be disabled explicitly if desired.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe we could use the full name instead of the Latin. Not everyone is familiar with it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The abbreviation i.e. has already been used two times in the same document, so I assumed it would be OK, but I will gladly change all occurences if you like?

Copy link
Collaborator

Choose a reason for hiding this comment

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

If you can, I think it's good. If I were from a country that doesn't use Latin, I would be puzzled when I saw it and said, what the hell is it?

Copy link
Collaborator

Choose a reason for hiding this comment

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

i.e. is used all over the place in zephyr, the documentation is in english, it's fine to stay

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i.e. is used all over the place in zephyr, the documentation is in english, it's fine to stay

Then I'll rather leave it as it is, I hope that's OK with everyone. (I tried to find different variations by replacing it with "in other words" or similar but it seemed a bit unfitting to me to be honest)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Then I'll rather leave it as it is, I hope that's OK with everyone

Don't worry about it. This is just a suggestion that you can't follow, insignificant :)
(zephyr designated language is English)


interrupts
Information about interrupts generated by the device, encoded as an array
of one or more *interrupt specifiers*. Each interrupt specifier has some
Expand Down