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

Remove common_hdmi_codec_drv variable #5158

Merged
merged 3 commits into from
Aug 27, 2024

Conversation

brentlu
Copy link

@brentlu brentlu commented Aug 23, 2024

Remove common_hdmi_codec_drv variable from snd_soc_acpi_mach_params structure and SOF machine drivers since we don't need it anymore.

The variable common_hdmi_codec_drv is always true on SOF platform so
we could remove the reference in machine driver.

Signed-off-by: Brent Lu <brent.lu@intel.com>
The variable common_hdmi_codec_drv is always true on SOF platform so
we could remove the reference in machine driver.

Signed-off-by: Brent Lu <brent.lu@intel.com>
Do not set common_hdmi_codec_drv in SOF platform driver since no
machine driver needs it. Remove member variable common_hdmi_codec_drv
from snd_soc_acpi_mach_params structure.

Signed-off-by: Brent Lu <brent.lu@intel.com>
@bardliao
Copy link
Collaborator

Some failures on LNLM_RVP_HDA https://sof-ci.01.org/linuxpr/PR5158/build4423/devicetest/index.html. @brentlu FYI

@bardliao
Copy link
Collaborator

SOFCI TEST

Copy link
Collaborator

@bardliao bardliao left a comment

Choose a reason for hiding this comment

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

Let's wait for CI test result

Copy link
Collaborator

@bardliao bardliao left a comment

Choose a reason for hiding this comment

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

CI test result looks good

@bardliao bardliao merged commit bfe13c2 into thesofproject:topic/sof-dev Aug 27, 2024
11 of 14 checks passed
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.

3 participants