Skip to content

app: fix filter in sample.yaml - #8768

Merged
dbaluta merged 1 commit into
thesofproject:mainfrom
nashif:topic/sample/allow_filter
Jan 29, 2024
Merged

dbaluta merged 1 commit into
thesofproject:mainfrom
nashif:topic/sample/allow_filter

Conversation

@nashif

@nashif nashif commented Jan 22, 2024

Copy link
Copy Markdown
Contributor

nxp_adsp_imx8ulp needs to be in allowed platforms.

Signed-off-by: Anas Nashif anas.nashif@intel.com

nxp_adsp_imx8ulp needs to be in allowed platforms.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>

@marc-hb marc-hb left a comment •

Copy link
Copy Markdown
Collaborator

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)

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@dbaluta fyi.

@kv2019i kv2019i left a comment

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member

@lrudyX @wszypelt good to merge ? We think this is not tested by CI ?

@wszypelt

Copy link
Copy Markdown

@lgirdwood all green

@dbaluta
dbaluta merged commit 8b7c75a into thesofproject:main Jan 29, 2024
@dbaluta

dbaluta commented Jan 29, 2024

Copy link
Copy Markdown
Collaborator

@dbaluta and I asked @TangleZ to do this a couple months ago but they never did:

Thanks @marc-hb for keeping an eye on this. It just fall through the cracks on our side.

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.

7 participants