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

[CIONLY] west: test for PR 86473 in zephyr #9867

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tmleman
Copy link
Contributor

@tmleman tmleman commented Feb 28, 2025

I am only testing the PR in Zephyr.

@tmleman tmleman force-pushed the topic/upstream/cionly/zephyr_pr/86473 branch 2 times, most recently from 6843edb to 09a0fb4 Compare March 5, 2025 12:43
tmleman added 3 commits March 5, 2025 14:50
I am only testing the PR in Zephyr.

Signed-off-by: Tomasz Leman <[email protected]>
This patch updates the debug overlay configuration to enable the
`LOG_BACKEND_ADSP_MTRACE_AUTOSTART` option. By setting this option to
'y', the ADSP mtrace logging backend will automatically start in the
debug overlay, facilitating detailed logging for debugging purposes.
This change allows developers to capture comprehensive logs during
debugging sessions, aiding in the identification and resolution of
issues.

Signed-off-by: Tomasz Leman <[email protected]>
This patch addresses an issue where logs were not being collected after
an IPC message when the mtrace autostart was disabled in regular builds.

The problem was resolved by replacing `log_backend_activate` and
`log_backend_deactivate` with `log_backend_enable` and
`log_backend_disable`, respectively. These functions ensure that the
backend is properly enabled with the initial maximum filtering level and
disabled when necessary. This change ensures that logging behavior
aligns with the intended configuration, allowing logs to be collected as
expected after an IPC message.

Signed-off-by: Tomasz Leman <[email protected]>
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.

1 participant