-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
tests: drivers: build_all: ieee802154: Add native_sim configuration #78518
tests: drivers: build_all: ieee802154: Add native_sim configuration #78518
Conversation
c7b81ce
to
fef004e
Compare
The mcr20a driver referencing `MCR20Overwrites.h` which is under `hal/nxp/mcux`. So, put `depends HAS_MCUX` for clarify the dependency. Signed-off-by: TOKITA Hiroshi <[email protected]>
fef004e
to
39b78ea
Compare
@@ -0,0 +1,80 @@ | |||
/* |
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.
I'm a bit puzzled here. This seems to be a copy of the overlay file for native_posix
platform:
https://github.com/zephyrproject-rtos/zephyr/blob/main/tests/drivers/build_all/ieee802154/boards/native_posix.overlay
Why to enforce this overlay for every other platform/configuration?
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.
I was considering using app.overlay as I do with other build_all-s,
but I submitted the PR incompletely.
I uploaded it with only the necessary fixes.
I'm sorry, but please check again.
The native_sim has already been added to the test cases of `drivers.ieee802154.build.external` is not being appropriately processed because there is no corresponding overlay. Create an overlay as an alias of native_posix.overlay. Also, add the native_sim/native/64 platforms in the same way. Signed-off-by: TOKITA Hiroshi <[email protected]>
39b78ea
to
69c5749
Compare
The native_sim has already been added to the test cases of
drivers.ieee802154.build.external
is not being appropriatelyprocessed because there is no corresponding overlay.
Create an overlay as an alias of native_posix.overlay.
Also, add the native_sim/native/64 platforms in the same way.
The mcr20a driver referencing
MCR20Overwrites.h
which is underhal/nxp/mcux
.So, put
depends HAS_MCUX
for clarify the dependency.