Skip to content

Conversation

@gongjun-song
Copy link

No description provided.

bardliao
bardliao previously approved these changes Nov 1, 2022
Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

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

nit-pick below @gongjun-song

I also think we are missing some platforms, no? there's only 2 skus here.

@gongjun-song
Copy link
Author

nit-pick below @gongjun-song

I also think we are missing some platforms, no? there's only 2 skus here.

@plbossart Yes, Dell has three platforms, but the 0C11 is still being debugged.
@mengdonglin recommends submitting PR first for two platforms that are already working well.

SKU 0C10 product supports SoundWire capture from local microphone
and two SoundWire amplifiers(no headset codec).

Signed-off-by: Gongjun Song <gongjun.song@intel.com>
Audio hardware configuration of SKU 0C10 product is rt714 on link0,
two rt1316s on link1 and link2

Signed-off-by: Gongjun Song <gongjun.song@intel.com>
SKU 0C40 product supports a SoundWire headset codec, SoundWire
capture from local microphones and two SoundWire amplifiers.

Signed-off-by: Gongjun Song <gongjun.song@intel.com>
Audio hardware configuration of SKU 0C40 product is rt711 on link2,
two rt1316s on link0 and link1, rt714 on link 3.

Signed-off-by: Gongjun Song <gongjun.song@intel.com>
@plbossart plbossart requested a review from bardliao November 2, 2022 15:55
Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Looks good, thanks @gongjun-song !

@plbossart
Copy link
Member

SOFCI TEST

@plbossart plbossart merged commit cb13f81 into thesofproject:topic/sof-dev Nov 4, 2022
@gongjun-song gongjun-song deleted the support_dell-rpl-products branch December 7, 2022 05:23
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