Skip to content

Conversation

@ujfalusi
Copy link
Contributor

@ujfalusi ujfalusi commented May 2, 2025

The purpose of PASSTHROUGH is to exclude processing modules from paths, but in case of the nocodec topologies the PASSTHROUGH enables formats on the SSP DAI side for which we don't have a blob generated and no hw_config either.
We only really support in this mode S32_LE and nothing else, S16_LE and S24_LE formats are invalid.

Support only S32_LE on dai side only and do the conversion if needed on the host side so all internal (non-)processing is 32bit.

Link: #9979

@ujfalusi ujfalusi requested review from kv2019i, lgirdwood and singalsu May 2, 2025 11:49
@ujfalusi ujfalusi requested review from jsarha and ranj063 as code owners May 2, 2025 11:49
@ujfalusi ujfalusi requested review from ranj063 and removed request for ranj063 May 2, 2025 11:49
The purpose of PASSTHROUGH is to exclude processing modules from paths, but
in case of the nocodec topologies the PASSTHROUGH enables formats on the
SSP DAI side for which we don't have a blob generated and no hw_config
either.
We only really support in this mode S32_LE and nothing else, S16_LE and
S24_LE formats are invalid.

Support only S32_LE on dai side only and do the conversion if needed on
the host side so all internal (non-)processing is 32bit.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Link: thesofproject#9979
@kv2019i kv2019i merged commit fa18fe5 into thesofproject:main May 5, 2025
42 of 49 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