Skip to content

Test-case: Updated check-sof-logger.sh with SOF-specific codec - #1248

Merged
kv2019i merged 1 commit into
thesofproject:mainfrom
harajend:fix/sof-logger
Jan 27, 2025
Merged

Test-case: Updated check-sof-logger.sh with SOF-specific codec#1248
kv2019i merged 1 commit into
thesofproject:mainfrom
harajend:fix/sof-logger

Conversation

@harajend

@harajendharajend commented Jan 13, 2025

Copy link
Copy Markdown
Contributor

Updated SOF-specific codecs since the sof* regex also lists sofprobes

Should fix#1243

@harajend
harajend requested a review from a team as a code ownerJanuary 13, 2025 03:54
@sofci

Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

reply test this please to run this test once

@marc-hb

Copy link
Copy Markdown
Collaborator

test this please

Comment threadtest-case/check-sof-logger.sh Outdated
Comment threadtest-case/check-sof-logger.sh

@kv2019ikv2019i left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No blockers, PR seems good, but I'd encourage to look up old git commits to get a feel of the style on conventions used in this repo.

Comment threadtest-case/check-sof-logger.sh Outdated
Comment threadtest-case/check-sof-logger.sh Outdated
…SOF probes
We have multiple sof* devices whenever SOF probes are enabled
Signed-off-by: Hariprasad, Rajendra <hariprasad.rajendra@intel.com>
@kv2019i
kv2019i merged commit 79046ae into thesofproject:mainJan 27, 2025
Sign up for freeto 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.

[BUG] /proc/asound/sofprobes breaks "check-sof-logger" test

5 participants

@harajend@sofci@marc-hb@ujfalusi@kv2019i