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

ASoC: Intel: skl_hda_dsp_generic: Drop HDMI routes when HDMI is not a… #4649

Conversation

ujfalusi
Copy link
Collaborator

@ujfalusi ujfalusi commented Oct 23, 2023

…vailable

When the HDMI is not present due to disabled display support
we will use dummy codec and the HDMI routes will refer to non existent
DAPM widgets.

Trim the route list from the HDMI routes to be able to probe the card even
if the HDMI dais are not registered.

@ujfalusi
Copy link
Collaborator Author

@plbossart, @keqiaozhang, this patch will likely fix #4648

and it is also going to unblock #4620

We create dummy codec in case of HDMI is not available but we still had the routes and that is what fails. Trimming the routes will allow the card to probe and have the dummy HDMI PCMs (they are described in tplg and we cannot filter them out).

@ujfalusi ujfalusi force-pushed the peter/sof/pr/hda_generic_trim_hdmi_01 branch from 19259e2 to 19e6a95 Compare October 23, 2023 12:42
…vailable

When the HDMI is not present due to disabled display support
we will use dummy codec and the HDMI routes will refer to non existent
DAPM widgets.

Trim the route list from the HDMI routes to be able to probe the card even
if the HDMI dais are not registered.

Signed-off-by: Peter Ujfalusi <[email protected]>
@ujfalusi
Copy link
Collaborator Author

Changes since v1:

  • fix commit message

@plbossart plbossart merged commit 5cdeb11 into thesofproject:topic/sof-dev Oct 23, 2023
@keqiaozhang
Copy link
Collaborator

keqiaozhang commented Oct 24, 2023

this patch will likely fix #4648

It seems that #4649 is not very helpful for #4648. The issue still exists in daily test.

@ujfalusi
Copy link
Collaborator Author

this patch will likely fix #4648

It seems that #4649 is not very helpful for #4648. The issue still exists in daily test.

I see, it is on sdw device... Then #4639 is the patch that will help.

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