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: sof_sdw: Add support for Fatcat board in PTL platform #5261

Open
wants to merge 1 commit into
base: topic/sof-dev
Choose a base branch
from

Conversation

jairaj-arava
Copy link

Add an entry in the soundwire quirk table for Fatcat boards in PTL RVP

Add an entry in the soundwire quirk table for Fatcat boards in PTL RVP

Signed-off-by: Uday M Bhat <[email protected]>
Signed-off-by: Jairaj Arava <[email protected]>
@ranj063
Copy link
Collaborator

ranj063 commented Dec 4, 2024

@jairaj-arava there's another PR for this #5258 . Can you please sync with @macchian

@bardliao
Copy link
Collaborator

bardliao commented Dec 4, 2024

The quirk adds PCH DMIC support and nothing else. Am I correct?

@macchian
Copy link

macchian commented Dec 4, 2024

@jairaj-arava there's another PR for this #5258 . Can you please sync with @macchian
Synced with @jairaj-arava and @udaymb. Should be no conflicts, I've modified #5258.

{
.callback = sof_sdw_quirk_cb,
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Google"),
Copy link

Choose a reason for hiding this comment

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

Are we still remaining the SYS_VENDOR string in coreboot? Perhaps DMI_PRODUCT_FAMILY or DMI_MANUFACTURER instead?

Copy link
Author

Choose a reason for hiding this comment

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

I hope manufacturer string in coreboot is used for SYS_VENDOR. We can use DMI_PRODUCT_FAMILY too, however, we are sticking to the way we defined in MTL platform for REX boards.

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