Skip to content

scripts: qemu-check.sh: update READY_IPC for imx8 - #6713

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
iuliana-prodan:imx_qemu_boot_fix
Dec 5, 2022
Merged

lgirdwood merged 1 commit into
thesofproject:mainfrom
iuliana-prodan:imx_qemu_boot_fix

Conversation

@iuliana-prodan

@iuliana-prodan iuliana-prodan commented Dec 5, 2022 •

Copy link
Copy Markdown
Contributor

Update READY_IPC value based on changes regarding MU reset. READY_IPC value comes from:

  • clear GP pending interrupt # 0 and # 1 from MU's xSR register;
  • enable GP # 0 and # 1 for Host -> DSP and DSP -> Host message notification from MU's xCR register;
  • now interrupt host to tell it we are done booting by setting GIRn bit in MU's xCR register.

So, "00 00 00 c0 00 00 04 c0" is the MU's xSR and xCR registers: xSR: c0000000 and xCR: c0040000

Signed-off-by: Iuliana Prodan iuliana.prodan@nxp.com

Update READY_IPC value based on changes regarding MU reset.
READY_IPC value comes from:
- clear GP pending interrupt #0 and thesofproject#1 from MU's xSR register;
- enable GP #0 and thesofproject#1 for Host -> DSP and DSP -> Host
message notification from MU's xCR register;
- now interrupt host to tell it we are done booting
by setting GIRn bit in MU's xCR register.

So, "00 00 00 c0 00 00 04 c0" is the MU's xSR and xCR registers:
xSR: c0000000 and xCR: c0040000

Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
@iuliana-prodan

iuliana-prodan commented Dec 5, 2022 •

Copy link
Copy Markdown
Contributor Author

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

So the problem was that it boots fine but the script couldn't detect it? Fair enough!

@lgirdwood
lgirdwood merged commit 6131901 into thesofproject:main Dec 5, 2022
@lgirdwood

Copy link
Copy Markdown
Member

So the problem was that it boots fine but the script couldn't detect it? Fair enough!

I would encourage adding more of the IP to qemu if you can, at least it would be a fall back virtual CI to prevent any critical breakages.

@fredoh9 @greg-intel I assume the CI container needs updated for this qemu update ?

@lgirdwood

Copy link
Copy Markdown
Member

@fredoh9 @greg-intel I assume the CI container needs updated for this qemu update ?

Please ignore - the script is automatic. I will rerun the CI for the blocked PR.

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.

3 participants