Skip to content

ASoC: SOF: Add topology parsing and load of IPC4 ASRC component - #5335

Merged
lgirdwood merged 1 commit into
thesofproject:topic/sof-devfrom
singalsu:asrc_load_add
Feb 24, 2025
Merged

lgirdwood merged 1 commit into
thesofproject:topic/sof-devfrom
singalsu:asrc_load_add

Conversation

@singalsu

Copy link
Copy Markdown
Collaborator

This patch adds to IPC4 topology parsing the support for the asynchronous sample rate converter (ASRC) SOF component. It is applied for the DAPM widget type SND_SOC_TPLG_DAPM_ASRC.

The parsed SOF tokens for ASRC are SOF_TKN_ASRC_RATE_OUT and SOF_TKN_ASRC_OPERATION_MODE.

@singalsu

singalsu commented Feb 20, 2025 •

Copy link
Copy Markdown
Collaborator Author

WIP - it still fails with sof-hda-benchmark-asrc32.tplg load. This should address issue thesofproject/sof#9845 .

Comment thread sound/soc/sof/ipc4-topology.c Outdated
ranj063
ranj063 previously approved these changes Feb 20, 2025

@ranj063 ranj063 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM @singalsu

@ujfalusi ujfalusi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@singalsu, couple of nitpicks and a note that you need to handle the snd_soc_dapm_asrc in sof_ipc4_widget_setup() as well

Comment thread sound/soc/sof/ipc4-topology.h Outdated
Comment thread sound/soc/sof/ipc4-topology.h Outdated
Comment thread sound/soc/sof/ipc4-topology.c Outdated
Comment thread sound/soc/sof/ipc4-topology.c Outdated
Comment thread sound/soc/sof/ipc4-topology.c Outdated
@singalsu

Copy link
Copy Markdown
Collaborator Author

This version works with my test topology. @ujfalusi Thanks for the fix suggestion! I'll continue with addressing the review comments.

@singalsu
singalsu marked this pull request as ready for review February 21, 2025 10:51
@singalsu
singalsu requested review from ranj063 and ujfalusi February 21, 2025 10:51
Comment thread sound/soc/sof/ipc4-topology.c Outdated
@singalsu
singalsu requested a review from ujfalusi February 21, 2025 13:55
Comment thread sound/soc/sof/ipc4-topology.c
This patch adds to IPC4 topology parsing the support for the
asynchronous sample rate converter (ASRC) SOF component. It
is applied for the DAPM widget type SND_SOC_TPLG_DAPM_ASRC.

The parsed SOF tokens for ASRC are SOF_TKN_ASRC_RATE_OUT and
SOF_TKN_ASRC_OPERATION_MODE.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@lyakh

lyakh commented Feb 21, 2025

Copy link
Copy Markdown
Collaborator

Tested on MTL with an ASRC HDA topology - passed.

@lgirdwood

Copy link
Copy Markdown
Member

Test failures are unrelated to this PR and are know problems with some DUTs

@lgirdwood
lgirdwood merged commit bc653c5 into thesofproject:topic/sof-dev Feb 24, 2025
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.

6 participants