Skip to content

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Jan 8, 2021

Add rules to select SOF driver for Alder Lake systems if a digital
microphone or SoundWire codecs are present in the system. This is
following same rules as for older Tiger Lake systems.

Signed-off-by: Kai Vehmanen kai.vehmanen@linux.intel.com

Add rules to select SOF driver for Alder Lake systems if a digital
microphone or SoundWire codecs are present in the system. This is
following same rules as for older Tiger Lake systems.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i
Copy link
Collaborator Author

kv2019i commented Jan 8, 2021

@plbossart I split this dspconfig to a separate PR so we can perhaps discuss the approach. E.g. do we still keep adding separate kconfig options for each Intel PCI SOF platform? It feels a bit odd to keep adding the same PCI ids to three places, all guarded by the same ifdefs. OTOH, this probably is orthogonal to adding ADL support as a change to this approach would apply to old platforms as well.

@kv2019i kv2019i requested review from libinyang and xiulipan January 8, 2021 17:34
{
.flags = FLAG_SOF | FLAG_SOF_ONLY_IF_DMIC_OR_SOUNDWIRE,
.device = 0x51c8,
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think there were patches recently, enabling SOF on TGL unconditionally since SST doesn't support TGL. Does SST support ADL?

Choose a reason for hiding this comment

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

@lyakh We talked about is TGL+, so I think NO for ADL here.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think there were patches recently, enabling SOF on TGL unconditionally since SST doesn't support TGL. Does SST support ADL?

My bad, #2665 is confusing people. @lyakh Legacy HDA driver should still be the default even if SST doesn't support ADL. See #2672

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ack @bardliao @xiulipan , this is still needed to allow to select between non-DSP and DSP drivers.

It would be nice to avoid repeating all the PCI IDs here, but unless we change approach, this is not needed.

@kv2019i kv2019i merged commit 4f87f86 into thesofproject:topic/sof-dev Jan 13, 2021
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.

6 participants