Skip to content

app: add logging_mipisystcat_overlay.conf overlay config#7838

Merged
lgirdwood merged 1 commit intothesofproject:mainfrom
kv2019i:202306-mipisyst-overlay
Jul 3, 2023
Merged

app: add logging_mipisystcat_overlay.conf overlay config#7838
lgirdwood merged 1 commit intothesofproject:mainfrom
kv2019i:202306-mipisyst-overlay

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Jun 20, 2023

Add an overlay config to build SOF with MIPI Sys-T Catalog logging enabled.

This adds MIPI Sys-T Catalog as an overlay option. There are still issues (tracked in #7703 ) with using this output in SOF, and some of the SOF CI infra is not prepare to handle binary-encoded logs, so this option cannot yet be enabled across the board.

Add an overlay config to build SOF with MIPI Sys-T Catalog
logging enabled.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@btian1
Copy link
Contributor

btian1 commented Jun 26, 2023

Shall we expect with this PR and MIPI device, SOF logs can be routed out to host like mtrace?

@kv2019i
Copy link
Collaborator Author

kv2019i commented Jun 26, 2023

@btian1 wrote:

Shall we expect with this PR and MIPI device, SOF logs can be routed out to host like mtrace?

This PR only changes en encoding of the logs. E.g. one can still use "mtrace-reader.py" to capture logs with SOF Linux driver. The only change is that one needs to use additional tools to parse the captured (binary) logs.

@btian1
Copy link
Contributor

btian1 commented Jul 3, 2023

please send out a mail(once #7703 resolved) for how to use tool to decode mipi logs, one more ask is: what's the benefit to add one more log format? windows required?

@lgirdwood lgirdwood merged commit 3fef488 into thesofproject:main Jul 3, 2023
@marc-hb
Copy link
Collaborator

marc-hb commented Jul 5, 2023

please send out a mail(once #7703 resolved)

There is a "subscribe" button.

one more ask is: what's the benefit to add one more log format?

https://docs.zephyrproject.org/latest/services/logging/index.html

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.

4 participants