Skip to content

ASoC: SOF: ipc3-loader: Handle PROBE_INFO ext_manifest element#5183

Merged
ujfalusi merged 2 commits intothesofproject:topic/sof-devfrom
ujfalusi:peter/sof/pr/ipc3-probe-info-extmanifest-01
Oct 4, 2024
Merged

ASoC: SOF: ipc3-loader: Handle PROBE_INFO ext_manifest element#5183
ujfalusi merged 2 commits intothesofproject:topic/sof-devfrom
ujfalusi:peter/sof/pr/ipc3-probe-info-extmanifest-01

Conversation

@ujfalusi
Copy link
Collaborator

Hi,

every kernel log contains the following info print:
unknown sof_ext_man header type 3 size 0x30

It is true that we don't handle the PROBE_INFO (type 3), but it is not true that it is an unknown element.

Add enum entry and an empty case to 'handle' the PROBE_INFO and silence the print.

Elem type 3 is PROBE_INFO in ext_manifest, add it to the list.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
…rsing

Every time when the firmware is loaded we see the following printed in
info level:
unknown sof_ext_man header type 3 size 0x30

This is the PROBE_INFO element, which is not parsed, but it is not
something that we should treat as surprise, aka unknown type.

Add an empty case with a debug print to 'handle' this ext_manifest type and
silence the confusing print in kernel log.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
@marc-hb
Copy link
Collaborator

marc-hb commented Sep 19, 2024

I don't know why Jenkins failed this one. I hope it's not a nasty side effect of #5182 (comment)

EDIT: nope, this was CI change 207. Reverted for now.

@marc-hb
Copy link
Collaborator

marc-hb commented Sep 19, 2024

SOFCI TEST

@ujfalusi ujfalusi merged commit c510fcc into thesofproject:topic/sof-dev Oct 4, 2024
@ujfalusi ujfalusi deleted the peter/sof/pr/ipc3-probe-info-extmanifest-01 branch December 13, 2024 09:03
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