Skip to content
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

west.yml: update Zephyr to efc32081893 #8582

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

dbaluta
Copy link
Collaborator

@dbaluta dbaluta commented Dec 6, 2023

Update Zephyr to pull in imx8ulp support. This includes following zephyr commits:

  • cfb68f827202 ("boards: xtensa: adsp: add support for imx8ulp board")
  • a9a0c28282c3 ("dts/xtensa/nxp: Add dtsi for imx8ulp")
  • 4b33c65f7108 ("soc: xtensa: adsp: add support for NXP ADSP for i.MX8ULP")
  • ceae1d83ca4b ("west: sign: add support for NXP i.MX8ULP board")

@marc-hb
Copy link
Collaborator

marc-hb commented Dec 6, 2023

I checked the internal site and the only MTL failure in https://sof-ci.01.org/sofpr/PR8582/build800/devicetest/index.html is the well known GSC firmware problem

I'm more worried about https://sof-ci.01.org/sofpr/PR8582/build801/devicetest/index.html?model=ADLP_RVP_NOCODEC-ipc4&testcase=multiple-pause-resume-50

@dbaluta
Copy link
Collaborator Author

dbaluta commented Dec 7, 2023

@marc-hb @lgirdwood Can someone at Intel have a look?

@lgirdwood let me try with an older Zephyr, I don't need main exactly for imx8ulp changes

Update Zephyr to pull in imx8ulp support. This includes following zephyr
commits:

- cfb68f827202 ("boards: xtensa: adsp: add support for imx8ulp board")
- a9a0c28282c3 ("dts/xtensa/nxp: Add dtsi for imx8ulp")
- 4b33c65f7108 ("soc: xtensa: adsp: add support for NXP ADSP for i.MX8ULP")
- efc32081893d ("west: sign: add support for NXP i.MX8ULP board")

Signed-off-by: Daniel Baluta <[email protected]>
@kv2019i
Copy link
Collaborator

kv2019i commented Dec 7, 2023

@wszypelt I checked the Intel internal test and I can see test 000_01_TestMulticoreHostTopology failing. The FW log shows not data received by the pipeline and the resulting wave file for the secondary pipe is empty. However, I see same test failing with this PR and also with #7688 , so it would seem this is not related to the pull-requests, but either issue in the DUT or a regression that has slipped him.

@dbaluta
Copy link
Collaborator Author

dbaluta commented Dec 7, 2023

SOFCI TEST

@dbaluta
Copy link
Collaborator Author

dbaluta commented Dec 7, 2023

@kv2019i I se that build job is also failing but cannot see any details at: https://sof-ci.01.org/sof-pr-viewer/#/build/PR8582/build13236928

@wszypelt
Copy link

wszypelt commented Dec 7, 2023

@kv2019i @dbaluta You're right, this error is strange, I think the problem may be with the test execution, I ran PR again and the results should be available soon, I will also check this test carefully on this FW on another machine with a larger number of iterations

@kv2019i
Copy link
Collaborator

kv2019i commented Dec 7, 2023

Thanks @wszypelt , now seems to indeed pass! Proceeding with merge.

@kv2019i kv2019i merged commit a41f26d into thesofproject:main Dec 7, 2023
42 of 44 checks passed
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.

6 participants