Skip to content

[RFC] ADL-RVP: add definitions for RT711 2xRT1316 RT714 in SDW mode#11

Merged
plbossart merged 1 commit intothesofproject:masterfrom
libinyang:adl-p
Apr 9, 2021
Merged

[RFC] ADL-RVP: add definitions for RT711 2xRT1316 RT714 in SDW mode#11
plbossart merged 1 commit intothesofproject:masterfrom
libinyang:adl-p

Conversation

@libinyang
Copy link

@libinyang libinyang commented Apr 2, 2021

RT711 is on link0
RT1316-1 is on link1
RT1316-2 is on link2
RT714 is on link3

Signed-off-by: Libin Yang libin.yang@intel.com


updated:
RT711 is on link0
RT1316-1 is on link1
RT714 is on link2
RT1316-2 is on link3

@plbossart
Copy link
Member

@libinyang if this refers to the 3-in-1 board, this looks incorrect?

@libinyang
Copy link
Author

@libinyang if this refers to the 3-in-1 board, this looks incorrect?

@plbossart You are right. After checking the HW design, I confirmed that topology is wrong. I will update the patch and test the new topology tomorrow. Thanks for pointing it outl

@libinyang
Copy link
Author

patch is updated to the new topology:
RT711 is on link0
RT1316-1 is on link1
RT714 is on link2
RT1316-2 is on link3

@@ -0,0 +1,45 @@
/** @file
Copy link
Member

Choose a reason for hiding this comment

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

would you mind renaming the file as adl_sdca_3_in_1 to match what the Linux PR does:

thesofproject/linux#2811

Copy link
Author

Choose a reason for hiding this comment

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

@plbossart The patch is updated to rename to adl_sdca_3_in_1.asl

RT711 is on link0
RT1316-1 is on link1
RT714 is on link2
RT1316-2 is on link3

Signed-off-by: Libin Yang <libin.yang@intel.com>
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.

thank you @libinyang

@plbossart plbossart merged commit 5c5ef1f into thesofproject:master Apr 9, 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.

2 participants