Skip to content

Conversation

@macchian
Copy link
Contributor

@macchian macchian commented Dec 3, 2024

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
Copy link
Member

@lgirdwood lgirdwood left a comment

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
Member

@macchian ping

@macchian
Copy link
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
Member

@macchian is validation now done ?

@macchian
Copy link
Contributor Author

@macchian is validation now done ?

yes, it's verified.

@lgirdwood
Copy link
Member

test results missing in CI - rerun.

@lgirdwood
Copy link
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>
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.

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
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
42 of 49 checks passed
@kv2019i
Copy link
Collaborator

kv2019i commented Apr 29, 2025

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