Skip to content
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.

Sign up for GitHub

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

Tools: Topology2: Add pre-processing to ACE 1/2/3.x PCH DMIC topologies #9690

Merged
merged 4 commits into from
Dec 3, 2024
Merged

Tools: Topology2: Add pre-processing to ACE 1/2/3.x PCH DMIC topologies #9690

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
1b21c03
Tools: Topology2: Add pre-processing to ACE1.x PCM DMIC topologies
singalsu Nov 28, 2024
eefdb9e
Tools: Topology2: Add pre-processing to ACE2.x PCM DMIC topologies
singalsu Nov 28, 2024
7bcd9a9
Tools: Topology2: Add pre-processing to ACE3.x PCM DMIC topologies
singalsu Nov 28, 2024
05c75dd
Tools: Topology2: SDW: Use for PCH DMIC IIR type with +20 dB gain
singalsu Nov 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion tools/topology/topology2/production/tplg-targets-ace3.cmake
View file
Open in desktop
Copy link
Contributor

Choose a reason for hiding this comment

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

@singalsu , do we need to enable nhlt-sof-ptl-rt721-4ch.bin as well?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I was wondering what it is due to comment "RT721 eval board with PCH-DMIC" so I did not change it. Is it OK to add the processing to it too?

I can do a separate PR to address it. This PR should be merged quick to main branch to be able to make the v2.11-stable update for the IIR gain.

Copy link
Contributor

Choose a reason for hiding this comment

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

@bardliao ,I assume this would also be a customer board configuration for future use. What are your thoughts?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I was wondering if it's a topology for a Chromebook or a PC. In this step I'm adding processing to PCs running Linux. With Chromebooks similar type processing has been on the user space and/or in the closed firmware component, so it has not been enabled.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think the sof-ptl-rt721-4ch topology is for chrome EVB.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks! OK, I'll leave it without this change for now.

Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,8 @@ JACK_RATE=96000,DEEP_BUF_JACK_RATE=96000"
PDM1_MIC_A_ENABLE=1,PDM1_MIC_B_ENABLE=1,DMIC0_ID=5,DMIC1_ID=6,HDMI1_ID=7,HDMI2_ID=8,HDMI3_ID=9,\
SDW_AMP_FEEDBACK=false,SDW_SPK_STREAM=Playback-SmartAmp,SDW_DMIC_STREAM=Capture-SmartMic,\
SDW_JACK_OUT_STREAM=Playback-SimpleJack,SDW_JACK_IN_STREAM=Capture-SimpleJack,\
PREPROCESS_PLUGINS=nhlt,NHLT_BIN=nhlt-sof-ptl-rt722-4ch.bin"
PREPROCESS_PLUGINS=nhlt,NHLT_BIN=nhlt-sof-ptl-rt722-4ch.bin,DMIC0_ENHANCED_CAPTURE=true,\
EFX_DMIC0_TDFB_PARAMS=line4_pass,EFX_DMIC0_DRC_PARAMS=dmic_default"

"cavs-sdw\;sof-ptl-rt712-l2-rt1320-l1\;PLATFORM=ptl,SDW_DMIC=1,NUM_SDW_AMP_LINKS=2,\
SDW_AMP_FEEDBACK=false,SDW_SPK_STREAM=Playback-SmartAmp,SDW_DMIC_STREAM=Capture-SmartMic,\
Expand Down
Loading