Skip to content

[SKIP SOF-TEST] app/sample.yml: add MTL and LNL to Zephyr CI - #8429

Merged
kv2019i merged 1 commit into
thesofproject:mainfrom
marc-hb:mtl-lnl-zephyr-ci
Nov 3, 2023
Merged

kv2019i merged 1 commit into
thesofproject:mainfrom
marc-hb:mtl-lnl-zephyr-ci

Conversation

@marc-hb

@marc-hb marc-hb commented Nov 1, 2023

Copy link
Copy Markdown
Collaborator

This will catch earlier compile-time regressions like this one: zephyrproject-rtos/zephyr#61166 (comment)

Zephyr commit 06cfbd4159fd ("drivers: power_domain: Introduce a gpio monitor driver")

This will catch earlier compile-time regressions like this one:
zephyrproject-rtos/zephyr#61166 (comment)

Zephyr commit 06cfbd4159fd ("drivers: power_domain: Introduce a gpio
monitor driver")

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb

marc-hb commented Nov 1, 2023

Copy link
Copy Markdown
Collaborator Author

I locally and successfully tested this with twister, both with and without the revert of zephyrproject-rtos/zephyr#61166 (comment)

@marc-hb

marc-hb commented Nov 1, 2023 •

Copy link
Copy Markdown
Collaborator Author

The build failures in https://github.com/thesofproject/sof/actions/runs/6725515097/job/18280066526?pr=8429 are precisely the regressions that escaped Zephyr CI.

I tried to avoid the harmless checkpatch warnings but I could not find a simple way.

Everything else is either green or irrelevant.

One timeout in https://sof-ci.01.org/sof-pr-viewer/#/build/PR8429/build13044359, all other tests passed (they don't use this file)

@keqiaozhang keqiaozhang 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.

LGTM

@kv2019i
kv2019i merged commit 9e080af into thesofproject:main Nov 3, 2023
@marc-hb
marc-hb deleted the mtl-lnl-zephyr-ci branch November 14, 2023 20:11
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.

5 participants