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

kmod: Add modules useful for imx platforms #1236

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AnneOnciulescu
Copy link

Add snd_soc_simple_card module for insert/remove testing. Also, snd_sof_utils was missing from sof_insert.sh.

Add snd_soc_simple_card module for insert/remove testing.
Also, snd_sof_utils was missing from sof_insert.sh.

Signed-off-by: Anne Onciulescu <anne.onciulescu@gmail.com>
@AnneOnciulescu AnneOnciulescu requested review from plbossart and a team as code owners October 11, 2024 16:08
@sofci
Copy link
Collaborator

sofci commented Oct 11, 2024

Can one of the admins verify this patch?

reply test this please to run this test once


# core SOF driver
insert_module snd_sof
insert_module snd_sof_utils
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be at least a separate commit. Maybe even a separate PR if there's any debate about snd_soc_simple_card?

Copy link
Member

Choose a reason for hiding this comment

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

unless simple_card depends on utils ? @AnneOnciulescu pls confirm.

Copy link
Collaborator

Choose a reason for hiding this comment

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

The commit message says "Also, snd_sof_utils was missing from sof_insert.sh." which is a pretty clear sign this is a separate bug fix.

@marc-hb
Copy link
Collaborator

marc-hb commented Oct 16, 2024

SOFCI TEST

@marc-hb
Copy link
Collaborator

marc-hb commented Oct 16, 2024

Unrelated to this PR and off-topic but these are NOT the "usual LNL failures"
https://sof-ci.01.org/softestpr/PR1236/build782/devicetest/index.html

More info in (possibly unrelated too)

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