-
Notifications
You must be signed in to change notification settings - Fork 309
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
app: fix filter in sample.yaml #8768
Conversation
nxp_adsp_imx8ulp needs to be in allowed platforms. Signed-off-by: Anas Nashif <[email protected]>
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.
@dbaluta and I asked @TangleZ to do this a couple months ago but they never did:
It does not help that SOF and Zephyr are different git repos. This file is in the SOF repo but used only by Zephyr CI. A one-line manifest change is enough to submit a PR in one repo to the CI tests of the other repo but that unfortunately that requires people being interested in getting their code tested :-( zephyrproject-rtos/zephyr#67393 (comment)
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.
@dbaluta fyi.
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.
The Intel CI fail is unlikeliy to be related, so looks good. But let's have NXP folks have a chance to review.
@lgirdwood all green |
nxp_adsp_imx8ulp needs to be in allowed platforms.
Signed-off-by: Anas Nashif [email protected]