Skip to content

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Jun 4, 2024

Set CONFIG_DMA_INTEL_ADSP_HDA_TIMING_L1_EXIT in board overlay as the Zephyr platform default is not correct in the current version of Zephyr.

Link: #9191

Set CONFIG_DMA_INTEL_ADSP_HDA_TIMING_L1_EXIT in board overlay
as the Zephyr platform default is not correct in the current
version of Zephyr.

Link: thesofproject#9191
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i
Copy link
Collaborator Author

kv2019i commented Jun 4, 2024

Zephyr updates are currently blocked due to a interface change in upstream Zephyr (see #9189), so to fix the L1_EXIT setting before v2.10 branch point (and increase test coverage earlier), here's a PR to change the setting on SOF side.

@marc-hb
Copy link
Collaborator

marc-hb commented Jun 4, 2024

Pause still failing but playback passed in LNL SDW https://sof-ci.01.org/sofpr/PR9194/build5077/devicetest/index.html

Reproduction of bank switch timeout thesofproject/linux#4823 was always much higher on pause/resume so this result probably does not mean anything.

LNL HDA was not available, will re-run later.

cc:

@marc-hb
Copy link
Collaborator

marc-hb commented Jun 4, 2024

SOFCI TEST

@marc-hb
Copy link
Collaborator

marc-hb commented Jun 4, 2024

@lyakh can you check this new, unrelated, Zephyr main incompatibility?

https://github.com/thesofproject/sof/actions/runs/9371200004/job/25799955329?pr=9194

/zep_workspace/sof/src/library_manager/llext_manager.c:212:63: error: 'struct llext_loader' has no member named 'prog_data_size'
  212 |         ctx->segment_size[SOF_MAN_SEGMENT_RODATA] = ebl.loader.prog_data_size;
      |                                                               ^

Off topic sorry.

@marc-hb
Copy link
Collaborator

marc-hb commented Jun 4, 2024

The "usual" LNL failures in https://sof-ci.01.org/sofpr/PR9194/build5081/devicetest/index.html

@lyakh
Copy link
Collaborator

lyakh commented Jun 5, 2024

@lyakh can you check this new, unrelated, Zephyr main incompatibility?

https://github.com/thesofproject/sof/actions/runs/9371200004/job/25799955329?pr=9194

/zep_workspace/sof/src/library_manager/llext_manager.c:212:63: error: 'struct llext_loader' has no member named 'prog_data_size'
  212 |         ctx->segment_size[SOF_MAN_SEGMENT_RODATA] = ebl.loader.prog_data_size;
      |                                                               ^

Off topic sorry.

@marc-hb known, dealt with

@kv2019i
Copy link
Collaborator Author

kv2019i commented Jun 5, 2024

Unfortunately this change alone is not sufficient, the dmic/capture error with no data is still seen with this PR applied: https://sof-ci.01.org/sofpr/PR9194/build5081/devicetest/index.html?model=LNLM_RVP_HDA&testcase=check-pause-resume-capture-100

@kv2019i
Copy link
Collaborator Author

kv2019i commented Jun 13, 2024

Not needed as Zephyr baseline update already changed the default.

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