Skip to content

Support Lenovo laptop with rt722 and rt1320#5692

Open
oortcomet wants to merge 3 commits intothesofproject:topic/sof-devfrom
oortcomet:lnv-rt722-rt1320
Open

Support Lenovo laptop with rt722 and rt1320#5692
oortcomet wants to merge 3 commits intothesofproject:topic/sof-devfrom
oortcomet:lnv-rt722-rt1320

Conversation

@oortcomet
Copy link

These commits support the combination with rt722 and rt1320 soundwire devices.

  • rt722 in Link3 -> UAJ
  • rt1320 in Link2 -> AMP & MIC

Mac Chiang and others added 3 commits March 12, 2026 10:43
…bles

Use functional topologies to support RT722 topologies with/without amplifiers,
e.g. sof-ptl-rt722.tplg, sof-ptl-rt722-rt1320.tplg...

If these entries are not removed, they will find the first same link of
sof-ptl-rt722.tplg.

Signed-off-by: Mac Chiang <mac.chiang@intel.com>
Add 'rt1320-aif2' dai info for rt1320 dmic function.

Signed-off-by: Derek Fang <derek.fang@realtek.com>
The Lenovo T14s Gen 7 system uses rt722 UAJ on link3 and rt1320 amp and
dmic on link2.
This patch supports the combination.

Signed-off-by: Derek Fang <derek.fang@realtek.com>
@sofci
Copy link
Collaborator

sofci commented Mar 12, 2026

Can one of the admins verify this patch?

reply test this please to run this test once

card->components = devm_kasprintf(card->dev, GFP_KERNEL,
"%s mic:%s", card->components,
mic_name);
if (rt1320_dmic_num > 0)
Copy link
Collaborator

Choose a reason for hiding this comment

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

What if dai->name == rt1320-aif2 but rt1320_dmic_num == 0?

Copy link
Author

Choose a reason for hiding this comment

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

The situation you said should never have happened.
Might use
if (!strcmp(dai->name, "rt1320-aif2"))
instead of
if (rt1320_dmic_num > 0)

@bardliao
Copy link
Collaborator

test this please

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.

3 participants