-
Notifications
You must be signed in to change notification settings - Fork 349
Topology2: Add sof-ptl-rt712-l3-rt1320-l3 support #9704
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
lgirdwood
left a comment
There was a problem hiding this 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
|
@macchian ping |
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 is validation now done ? |
yes, it's verified. |
|
test results missing in CI - rerun. |
|
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
left a comment
There was a problem hiding this 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,\ |
There was a problem hiding this comment.
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
|
For reference, the kernel part was merged as thesofproject/linux#5258 |
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.