topology2: cavs-sdw: add multiband DRC playback pipeline variant#10969
Open
singalsu wants to merge 1 commit into
Open
topology2: cavs-sdw: add multiband DRC playback pipeline variant#10969singalsu wants to merge 1 commit into
singalsu wants to merge 1 commit into
Conversation
Add an alternative BE speaker playback pipeline class mixout-gain-eqiir-eqfir-mbdrc-alh-dai-copier-playback that mirrors the existing mixout-gain-eqiir-eqfir-drc-alh-dai-copier-playback but uses the multiband_drc effect in place of the single-band drc. This lets an SDW amp topology run per-band dynamics processing without duplicating the surrounding gain, EQ IIR and EQ FIR chain. The SDW enhanced speaker playback path in the sdw-amp-generic platform partial is now gated by IncludeByKey.EFX_DRC_COMPONENT with "multiband" and "singleband" branches, both in the pipeline instantiation and in the corresponding route block. The multiband branch selects the new pipeline class and exposes the multiband_drc bytes and switch as "Post Mixer <name> MDRC bytes" / "MDRC switch" so it does not collide with the single-band DRC controls. EFX_DRC_COMPONENT still defaults to "singleband" in efx-default.conf, so existing targets are unaffected. Two development topology targets are added to exercise the new path: sof-mtl-rt713-l0-rt1316-l12-mbdrc and sof-arl-cs42l43-l0-cs35l56-l23-mbdrc, both with EFX_DRC_COMPONENT=multiband. Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds an SDW speaker BE playback pipeline variant that uses multiband_drc instead of the existing single-band drc, and wires it into the sdw-amp-generic platform partial via IncludeByKey.EFX_DRC_COMPONENT so a topology can select either implementation.
Changes:
- Added a new pipeline class
mixout-gain-eqiir-eqfir-mbdrc-alh-dai-copier-playback. - Gated SDW enhanced speaker playback pipeline instantiation and routing on
EFX_DRC_COMPONENT(singlebandvsmultiband). - Added two development topology targets that set
EFX_DRC_COMPONENT=multiband.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tools/topology/topology2/platform/intel/sdw-amp-generic.conf | Adds EFX_DRC_COMPONENT-gated selection between single-band DRC and multiband DRC pipelines, plus corresponding route changes. |
| tools/topology/topology2/include/pipelines/cavs/mixout-gain-eqiir-eqfir-mbdrc-alh-dai-copier-playback.conf | Introduces the new BE playback pipeline class that inserts multiband_drc after EQ FIR. |
| tools/topology/topology2/development/tplg-targets.cmake | Adds two development targets to build SDW topologies with EFX_DRC_COMPONENT=multiband. |
| tools/topology/topology2/cavs-sdw.conf | Registers the new pipeline include and adds an MBDRC params default. |
Comment on lines
117
to
+120
| EFX_SPK_IIR_PARAMS "highpass_100hz_0db_48khz" | ||
| EFX_SPK_FIR_PARAMS "passthrough" | ||
| EFX_SPK_DRC_PARAMS "speaker_default" | ||
| EFX_SPK_MBDRC_PARAMS "passthrough" |
Collaborator
Author
There was a problem hiding this comment.
OK, I'll also rename it to EFX_SPK_DRC_COMPONENT to be more specific to where it applies.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add an alternative BE speaker playback pipeline class mixout-gain-eqiir-eqfir-mbdrc-alh-dai-copier-playback that mirrors the existing mixout-gain-eqiir-eqfir-drc-alh-dai-copier-playback but uses the multiband_drc effect in place of the single-band drc. This lets an SDW amp topology run per-band dynamics processing without duplicating the surrounding gain, EQ IIR and EQ FIR chain.
The SDW enhanced speaker playback path in the sdw-amp-generic platform partial is now gated by IncludeByKey.EFX_DRC_COMPONENT with "multiband" and "singleband" branches, both in the pipeline instantiation and in the corresponding route block. The multiband branch selects the new pipeline class and exposes the multiband_drc bytes and switch as "Post Mixer MDRC bytes" / "MDRC switch" so it does not collide with the single-band DRC controls. EFX_DRC_COMPONENT still defaults to "singleband" in efx-default.conf, so existing targets are unaffected.
Two development topology targets are added to exercise the new path: sof-mtl-rt713-l0-rt1316-l12-mbdrc and
sof-arl-cs42l43-l0-cs35l56-l23-mbdrc, both with
EFX_DRC_COMPONENT=multiband.