-
Notifications
You must be signed in to change notification settings - Fork 7k
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
Sanitycheck dts awarness #20204
Sanitycheck dts awarness #20204
Conversation
All checks are passing now. Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits. Nice to get rid of more .conf
stuff.
Is there any documentation that needs to be updated btw?
a9da96a
to
13fc0b8
Compare
Add the following functions to allow filtering based on device tree dt_compat_enabled(compat) - Returns true if a device tree node compatible matches 'compat' and the node is enabled. dt_alias_exists(alias) - Returns true if a device tree node exists with 'alias' and the node is enabled. dt_compat_enabled_with_alias - Returns true if a device tree node compatible matches 'compat' and the node has 'alias' and the node is enabled. Signed-off-by: Kumar Gala <kumar.gala@linaro.org>
convert sample and test yaml filters that utilize a DT_ define to instead use a dt_ function. The intent is to remove the Kconfig generated DT defines and just make directy queries into the device tree. Signed-off-by: Kumar Gala <kumar.gala@linaro.org>
All users of dt kconfig symbols for tests and samples are using dt functions instead so we can remove the support for parsing the generated file. Signed-off-by: Kumar Gala <kumar.gala@linaro.org>
13fc0b8
to
e248409
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine with me.
I think these macros ( EDIT: Would solve #9406 |
This seems to have broken something: |
PR zephyrproject-rtos#20204 broke master for boards without DTS PR zephyrproject-rtos#20365 tried to partially fix it, but the filtering in sanitycheck remained broken Let's quickly revert it to allow other PRs to come in, and then resubmit them in a separate PR. This reverts commit 556f3cb. Signed-off-by: Alberto Escolar Piedras <alpi@oticon.com>
PR zephyrproject-rtos#20204 broke master for boards without DTS PR zephyrproject-rtos#20365 tried to partially fix it, but the filtering in sanitycheck remained broken Let's quickly revert it to allow other PRs to come in, and then resubmit them in a separate PR. This reverts commit 4cc91fd. Signed-off-by: Alberto Escolar Piedras <alpi@oticon.com>
PR zephyrproject-rtos#20204 broke master for boards without DTS PR zephyrproject-rtos#20365 tried to partially fix it, but the filtering in sanitycheck remained broken Let's quickly revert it to allow other PRs to come in, and then resubmit them in a separate PR. This reverts commit 9b5720d. Signed-off-by: Alberto Escolar Piedras <alpi@oticon.com>
PR zephyrproject-rtos#20204 broke master for boards without DTS PR zephyrproject-rtos#20365 tried to partially fix it, but the filtering in sanitycheck remained broken Let's quickly revert it to allow other PRs to come in, and then resubmit them in a separate PR. This reverts commit 7733b94. Signed-off-by: Alberto Escolar Piedras <alpi@oticon.com>
Replace use of DT_ Kconfig defines with functions that query the dts/edt.