Skip to content

Conversation

@jsarha
Copy link
Contributor

@jsarha jsarha commented Feb 12, 2026

Fix SRC DP regression when dp_data is present in the ext_init payload.

The only situation where we can assume that there is no module specific init after the ext_init payload is when it was offered as part of ext_init payload.

Completes fix abf1aa9 ("audio: module-adapter: fix SRC DP regression")

Fixes: d263649 ("module: Add support for module-specific init data")

This fix is required for thesofproject/linux#5537 to work as it shoud.

Copilot AI review requested due to automatic review settings February 12, 2026 15:23
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

Fixes an SRC DP initialization regression when dp_data is present in the ext_init payload by adjusting how the module adapter decides between legacy vs. extended init handling.

Changes:

  • Changes the legacy/extended init decision to hinge on whether module_data is present in the ext_init payload.
  • Adds a clarifying comment about assuming the legacy API when module data is absent.

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

Fix SRC DP regression when dp_data is present in the ext_init
payload.

The only situation where we can assume that there is no module
specific init after the ext_init payload is when it was offered as
part of ext_init payload.

Completes fix abf1aa9 ("audio: module-adapter: fix SRC DP regression")

Fixes: d263649 ("module: Add support for module-specific init data")

Signed-off-by: Jyri Sarha <jyri.sarha@linux.intel.com>
@jsarha jsarha force-pushed the assume_legacy_mod_data_if_no_ext_init_mod_data branch from 1d0803c to 4841dc6 Compare February 12, 2026 17:00
@jsarha
Copy link
Contributor Author

jsarha commented Feb 12, 2026

@lyakh & @ranj063 , can you take a look?

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.

1 participant