topology2: ace3: Add enhanced audio features to ptl-rt721-4ch target - #10203
naveen-manohar wants to merge 2 commits into
Conversation
Add support to extend ptl-rt721-4ch conf with audio processing: - Enhanced DMIC0 capture with TDFB & DRC processing - Add Bluetooth PCM support (ID 20, device ID 10) - Enable deep buffer speaker processing with 10ms DMA buffer - Configure TDFB line4_pass and DRC dmic_default parameters Signed-off-by: Naveen Manohar <naveen.m@intel.com>
| PREPROCESS_PLUGINS=nhlt,NHLT_BIN=nhlt-sof-ptl-rt721-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,\ | ||
| DEEPBUFFER_FW_DMA_MS=10,DEEP_BUF_SPK=true" |
There was a problem hiding this comment.
Hmm, we have a problem with the BT enabling, see thesofproject/linux#5518
This topology is pretty common and could match non-chromebook devices but the BT quirk is only set for chromebook boards in Linux kernel, so this could break devices upstream.
@bardliao ok if we branch out and put any new SDW topologies with BT with a separate suffix. Chromebooks use a udev rule to pick the topology file anywas, so this should be managable, right @naveen-manohar ?
We have sof-ptl-rt712-l3-rt1320-l3-4ch already with BT support, but I think that's the only one so far.
So sof-ptl-rt721-4ch-bt-ssp2.tplg , with ther identical contents but adds the BT PCM. Then non-chromebook devices can load the base variant, and chromebooks can use the one with BT PCM added.
There was a problem hiding this comment.
I would refrain to extend generic topologies which might be used by normal users, sof-ptl-rt721-4ch would be loaded on user machines with rt721 and 4channel PCH-DMIC and will likely not work as intended for generic Linux.
There was a problem hiding this comment.
Apologies this is last year PR, I should have closed.
Add deep buffer PCM35 on amps for ptl-rt721-4ch tplg. Signed-off-by: Naveen Manohar <naveen.m@intel.com>
PR 10203: test resultsRun date: 2026-08-31 17:04 UTC Tested commit: b6b17977a440abeda25c9c06e031c1054634921a
|
| DEEP_BUFFER_PCM_ID_2 35 | ||
| DEEP_BUFFER_PIPELINE_SRC_2 'mixin.16.1' | ||
| DEEP_BUFFER_PIPELINE_SINK_2 'mixout.21.1' | ||
| DEEP_BUFFER_PCM_NAME_2 'Deepbuffer Amps' |
There was a problem hiding this comment.
why these are defined here? They are need for the previous patch? Would the previous patch compile w/o this change?
There was a problem hiding this comment.
Apologies this is last year PR, I should have closed.###
topology2: ace3: Add enhanced audio features to ptl-rt721-4ch target
Add support to extend ptl-rt721-4ch conf with audio processing: