Skip to content

Comments

Revert "topology2: tplg-targets-sdca-generic: add echo ref to jack an…#10560

Merged
lgirdwood merged 1 commit intothesofproject:mainfrom
ujfalusi:peter/pr/revert-echoref-sdca
Feb 20, 2026
Merged

Revert "topology2: tplg-targets-sdca-generic: add echo ref to jack an…#10560
lgirdwood merged 1 commit intothesofproject:mainfrom
ujfalusi:peter/pr/revert-echoref-sdca

Conversation

@ujfalusi
Copy link
Contributor

…d amp"

This reverts commit 96ab79a.

The commit will break audio probing on any kernel older than 7.0 and using function topology.

We need to figure out something else for echoref, but this must not be released to public..

…d amp"

This reverts commit 96ab79a.

The commit will break audio probing on any kernel older than 7.0 and
using function topology.

We need to figure out something else for echoref, but this must not
be released to public..

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
@ujfalusi ujfalusi requested a review from ranj063 as a code owner February 20, 2026 09:02
Copilot AI review requested due to automatic review settings February 20, 2026 09:02
@ujfalusi ujfalusi requested a review from jsarha as a code owner February 20, 2026 09:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Reverts production topology2 SDCA generic target parameters that enabled echo reference support, because they break audio probing on kernels older than 7.0 when using function topology.

Changes:

  • Remove SDW_ECHO_REF_DAI / SDW_JACK_ECHO_REF flags from the SDCA jack target.
  • Remove JACK_RATE / SDW_SPK_ECHO_REF (and related echo-ref variants) from the SDCA amp targets.
  • Drop the *-ref_dai-* split-topology variants from the production target list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

ok, this can be addressed when @bardliao is back.
@udaymb @naveen-manohar fyi - may not impact you atm.

@lgirdwood lgirdwood merged commit 89b2177 into thesofproject:main Feb 20, 2026
54 of 56 checks passed
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.

3 participants