Skip to content

Topology2: Add sof-ptl-rt712-l3-rt1320-l3 support#9704

Merged
kv2019i merged 2 commits into
thesofproject:mainfrom
macchian:dev-sdw-rt712
Apr 29, 2025
Merged

Topology2: Add sof-ptl-rt712-l3-rt1320-l3 support#9704
kv2019i merged 2 commits into
thesofproject:mainfrom
macchian:dev-sdw-rt712

Conversation

@macchian

@macchian macchian commented Dec 3, 2024

Copy link
Copy Markdown
Contributor

The topology supports rt712_vb multi-function codec and rt1320 on SoundWire link 3 and shareing the sdw3-clock, sdw3-data[0] and sdw3-data[1] data lanes.

@macchian
macchian requested review from bardliao and kv2019i December 3, 2024 09:21

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@macchian pls change to ready when validation complete

@lgirdwood

Copy link
Copy Markdown
Member

@macchian ping

@macchian

Copy link
Copy Markdown
Contributor Author

@macchian pls change to ready when validation complete

not ready to validate, to be added multilane configuration into bios. stay tuned.

These necessary pipeline configurations apply only when
deep buffer speaker feature is enabled.

Following commit f35420 (Revert "topology2: Update deep buffer and
BT offload for ptl rt722 topology")

Signed-off-by: Mac Chiang <mac.chiang@intel.com>
@macchian
macchian marked this pull request as ready for review April 10, 2025 10:44
@macchian
macchian requested review from jsarha and ranj063 as code owners April 10, 2025 10:44
@macchian
macchian requested a review from ujfalusi April 10, 2025 10:44
@lgirdwood

Copy link
Copy Markdown
Member

@macchian is validation now done ?

@macchian

Copy link
Copy Markdown
Contributor Author

@macchian is validation now done ?

yes, it's verified.

@lgirdwood

Copy link
Copy Markdown
Member

test results missing in CI - rerun.

@lgirdwood

Copy link
Copy Markdown
Member

SOFCI TEST

The topology supports rt712_vb multi-function codec and rt1320
on SoundWire link 3 and shareing the sdw3-clock, sdw3-data[0],
sdw3-data[1] data lanes and without/with PCH_DMIC support.

Signed-off-by: Mac Chiang <mac.chiang@intel.com>

@kv2019i kv2019i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The BT PCM handling is a bit confused now (see inline), but given this matches with the merged kernel PRs, let's go and merge this one.

SDW_JACK_OUT_STREAM=Playback-SimpleJack,SDW_JACK_IN_STREAM=Capture-SimpleJack,\
PREPROCESS_PLUGINS=nhlt,NHLT_BIN=nhlt-sof-ptl-rt712-l3-rt1320-l3-4ch.bin,DMIC0_ENHANCED_CAPTURE=true,\
EFX_DMIC0_TDFB_PARAMS=line4_pass,EFX_DMIC0_DRC_PARAMS=dmic_default,\
BT_PCM_ID=20,BT_ID=10,BT_PCM_NAME=Bluetooth,ADD_BT=true,\

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@macchian This is a bit confusing BT offload is only enabled for the 4ch dmic variant.

FYI @bardliao this is also now a bit unconsistent when only some devices supported by kernel set the BT offload quirk bit. E.g. this sof-ptl-rt712-l3-rt1320-l3-4ch will only work for Chromebooks with current kernel.

Not sure how to best tackle this, but we need some systematic approach to naming these tplgs that can be used both in Chromebook sand desktop Linux. Maybe just enable BT always if there's any configuration that needs it (like in this case). FYI @ujfalusi @ranj063

@bardliao The kernel part is already in so I guess we need to proceed with

@kv2019i
kv2019i merged commit b6ee409 into thesofproject:main Apr 29, 2025
@kv2019i

kv2019i commented Apr 29, 2025

Copy link
Copy Markdown
Collaborator

For reference, the kernel part was merged as thesofproject/linux#5258

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.

4 participants