diff mbox series

ASoC: amd: acp: Fix for ACP SOF dmic tplg component load failure

Message ID 20241104091312.1108299-1-venkataprasad.potturu@amd.com
State Accepted
Commit 224b898f7c5ff575b02911e21383f271761bdeb6
Headers show
Series ASoC: amd: acp: Fix for ACP SOF dmic tplg component load failure | expand

Commit Message

potturu venkata prasad Nov. 4, 2024, 9:13 a.m. UTC
Stream name mismatch with topology file causes tplg load failure.

As SOF framework assigns dailink->stream name, overriding stream name
other than link name causes SOF dmic component load failure.

[   35.474995] snd_sof_amd_acp70 0000:c4:00.5: error: can't connect DAI ACPDMIC0.IN stream acp-dmic-codec
[   35.475001] snd_sof_amd_acp70 0000:c4:00.5: failed to add widget type 28 name : ACPDMIC0.IN stream acp-dmic-codec
[   35.475013] sof_mach acp70-dsp: ASoC: failed to load widget ACPDMIC0.IN
[   35.475018] sof_mach acp70-dsp: ASoC: topology: could not load header: -22
[   35.475072] snd_sof_amd_acp70 0000:c4:00.5: error: tplg component load failed -22
[   35.475083] snd_sof_amd_acp70 0000:c4:00.5: error: failed to load DSP topology -22
[   35.475090] snd_sof_amd_acp70 0000:c4:00.5: ASoC: error at snd_soc_component_probe on 0000:c4:00.5: -22
[   35.475117] sof_mach acp70-dsp: ASoC: failed to instantiate card -22
[   35.475254] sof_mach acp70-dsp: error -EINVAL: Failed to register card(sof-acp70-dsp)
[   35.475261] sof_mach acp70-dsp: probe with driver sof_mach failed with error -22

Fixes: b2385de2ae11 ("ASoC: amd: acp: Add stream name to ACP PDM DMIC devices")

Signed-off-by: Venkata Prasad Potturu <venkataprasad.potturu@amd.com>
---
 sound/soc/amd/acp/acp-mach-common.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Markus Elfring Nov. 4, 2024, 9:35 a.m. UTC | #1
> As SOF framework assigns dailink->stream name, overriding stream name
> other than link name causes SOF dmic component load failure.
…

Will another imperative wording be helpful for an improved change description?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.12-rc5#n94

Regards,
Markus
Mark Brown Nov. 4, 2024, 12:16 p.m. UTC | #2
On Mon, Nov 04, 2024 at 10:35:23AM +0100, Markus Elfring wrote:
> …
> > As SOF framework assigns dailink->stream name, overriding stream name
> > other than link name causes SOF dmic component load failure.
> …

> Will another imperative wording be helpful for an improved change description?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.12-rc5#n94

Feel free to ignore Markus, he has a long history of sending
unhelpful review comments and continues to ignore repeated requests
to stop.
Mario Limonciello Nov. 4, 2024, 2:22 p.m. UTC | #3
On 11/4/2024 03:13, Venkata Prasad Potturu wrote:
> Stream name mismatch with topology file causes tplg load failure.
> 
> As SOF framework assigns dailink->stream name, overriding stream name
> other than link name causes SOF dmic component load failure.
> 
> [   35.474995] snd_sof_amd_acp70 0000:c4:00.5: error: can't connect DAI ACPDMIC0.IN stream acp-dmic-codec
> [   35.475001] snd_sof_amd_acp70 0000:c4:00.5: failed to add widget type 28 name : ACPDMIC0.IN stream acp-dmic-codec
> [   35.475013] sof_mach acp70-dsp: ASoC: failed to load widget ACPDMIC0.IN
> [   35.475018] sof_mach acp70-dsp: ASoC: topology: could not load header: -22
> [   35.475072] snd_sof_amd_acp70 0000:c4:00.5: error: tplg component load failed -22
> [   35.475083] snd_sof_amd_acp70 0000:c4:00.5: error: failed to load DSP topology -22
> [   35.475090] snd_sof_amd_acp70 0000:c4:00.5: ASoC: error at snd_soc_component_probe on 0000:c4:00.5: -22
> [   35.475117] sof_mach acp70-dsp: ASoC: failed to instantiate card -22
> [   35.475254] sof_mach acp70-dsp: error -EINVAL: Failed to register card(sof-acp70-dsp)
> [   35.475261] sof_mach acp70-dsp: probe with driver sof_mach failed with error -22
> 
> Fixes: b2385de2ae11 ("ASoC: amd: acp: Add stream name to ACP PDM DMIC devices")
> 
> Signed-off-by: Venkata Prasad Potturu <venkataprasad.potturu@amd.com>

Thanks. I was aiming to sort out (null) from `arecord -l` but 
functionality is more important.

Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>

Mark,

Up to you but alternatively, you can also drop my existing patch and 
force push your for-next branch.

> ---
>   sound/soc/amd/acp/acp-mach-common.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/sound/soc/amd/acp/acp-mach-common.c b/sound/soc/amd/acp/acp-mach-common.c
> index 67aa0ad83486..d314253207d5 100644
> --- a/sound/soc/amd/acp/acp-mach-common.c
> +++ b/sound/soc/amd/acp/acp-mach-common.c
> @@ -1561,7 +1561,6 @@ int acp_sofdsp_dai_links_create(struct snd_soc_card *card)
>   
>   	if (drv_data->dmic_cpu_id == DMIC) {
>   		links[i].name = "acp-dmic-codec";
> -		links[i].stream_name = "DMIC capture";
>   		links[i].id = DMIC_BE_ID;
>   		links[i].codecs = dmic_codec;
>   		links[i].num_codecs = ARRAY_SIZE(dmic_codec);
Mario Limonciello Nov. 4, 2024, 2:40 p.m. UTC | #4
On 11/4/2024 08:35, Mukunda,Vijendar wrote:
> On 04/11/24 19:52, Mario Limonciello wrote:
>> On 11/4/2024 03:13, Venkata Prasad Potturu wrote:
>>> Stream name mismatch with topology file causes tplg load failure.
>>>
>>> As SOF framework assigns dailink->stream name, overriding stream name
>>> other than link name causes SOF dmic component load failure.
>>>
>>> [   35.474995] snd_sof_amd_acp70 0000:c4:00.5: error: can't connect DAI
>>> ACPDMIC0.IN stream acp-dmic-codec
>>> [   35.475001] snd_sof_amd_acp70 0000:c4:00.5: failed to add widget type 28
>>> name : ACPDMIC0.IN stream acp-dmic-codec
>>> [   35.475013] sof_mach acp70-dsp: ASoC: failed to load widget ACPDMIC0.IN
>>> [   35.475018] sof_mach acp70-dsp: ASoC: topology: could not load header: -22
>>> [   35.475072] snd_sof_amd_acp70 0000:c4:00.5: error: tplg component load
>>> failed -22
>>> [   35.475083] snd_sof_amd_acp70 0000:c4:00.5: error: failed to load DSP
>>> topology -22
>>> [   35.475090] snd_sof_amd_acp70 0000:c4:00.5: ASoC: error at
>>> snd_soc_component_probe on 0000:c4:00.5: -22
>>> [   35.475117] sof_mach acp70-dsp: ASoC: failed to instantiate card -22
>>> [   35.475254] sof_mach acp70-dsp: error -EINVAL: Failed to register
>>> card(sof-acp70-dsp)
>>> [   35.475261] sof_mach acp70-dsp: probe with driver sof_mach failed with
>>> error -22
>>>
>>> Fixes: b2385de2ae11 ("ASoC: amd: acp: Add stream name to ACP PDM DMIC devices")
>>>
>>> Signed-off-by: Venkata Prasad Potturu <venkataprasad.potturu@amd.com>
>>
>> Thanks. I was aiming to sort out (null) from `arecord -l` but functionality is
>> more important.
>>
>> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
>>
>> Mark,
>>
>> Up to you but alternatively, you can also drop my existing patch and force
>> push your for-next branch.
> 
> @Mario: Your changes are required for Legacy machine driver.
> This patch can go as a fix on top of your commit.

Thanks!

> 
>>
>>> ---
>>>    sound/soc/amd/acp/acp-mach-common.c | 1 -
>>>    1 file changed, 1 deletion(-)
>>>
>>> diff --git a/sound/soc/amd/acp/acp-mach-common.c
>>> b/sound/soc/amd/acp/acp-mach-common.c
>>> index 67aa0ad83486..d314253207d5 100644
>>> --- a/sound/soc/amd/acp/acp-mach-common.c
>>> +++ b/sound/soc/amd/acp/acp-mach-common.c
>>> @@ -1561,7 +1561,6 @@ int acp_sofdsp_dai_links_create(struct snd_soc_card *card)
>>>          if (drv_data->dmic_cpu_id == DMIC) {
>>>            links[i].name = "acp-dmic-codec";
>>> -        links[i].stream_name = "DMIC capture";
>>>            links[i].id = DMIC_BE_ID;
>>>            links[i].codecs = dmic_codec;
>>>            links[i].num_codecs = ARRAY_SIZE(dmic_codec);
>>
>
Mark Brown Nov. 5, 2024, 4:38 p.m. UTC | #5
On Mon, 04 Nov 2024 14:43:10 +0530, Venkata Prasad Potturu wrote:
> Stream name mismatch with topology file causes tplg load failure.
> 
> As SOF framework assigns dailink->stream name, overriding stream name
> other than link name causes SOF dmic component load failure.
> 
> [   35.474995] snd_sof_amd_acp70 0000:c4:00.5: error: can't connect DAI ACPDMIC0.IN stream acp-dmic-codec
> [   35.475001] snd_sof_amd_acp70 0000:c4:00.5: failed to add widget type 28 name : ACPDMIC0.IN stream acp-dmic-codec
> [   35.475013] sof_mach acp70-dsp: ASoC: failed to load widget ACPDMIC0.IN
> [   35.475018] sof_mach acp70-dsp: ASoC: topology: could not load header: -22
> [   35.475072] snd_sof_amd_acp70 0000:c4:00.5: error: tplg component load failed -22
> [   35.475083] snd_sof_amd_acp70 0000:c4:00.5: error: failed to load DSP topology -22
> [   35.475090] snd_sof_amd_acp70 0000:c4:00.5: ASoC: error at snd_soc_component_probe on 0000:c4:00.5: -22
> [   35.475117] sof_mach acp70-dsp: ASoC: failed to instantiate card -22
> [   35.475254] sof_mach acp70-dsp: error -EINVAL: Failed to register card(sof-acp70-dsp)
> [   35.475261] sof_mach acp70-dsp: probe with driver sof_mach failed with error -22
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/1] ASoC: amd: acp: Fix for ACP SOF dmic tplg component load failure
      commit: 224b898f7c5ff575b02911e21383f271761bdeb6

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
diff mbox series

Patch

diff --git a/sound/soc/amd/acp/acp-mach-common.c b/sound/soc/amd/acp/acp-mach-common.c
index 67aa0ad83486..d314253207d5 100644
--- a/sound/soc/amd/acp/acp-mach-common.c
+++ b/sound/soc/amd/acp/acp-mach-common.c
@@ -1561,7 +1561,6 @@  int acp_sofdsp_dai_links_create(struct snd_soc_card *card)
 
 	if (drv_data->dmic_cpu_id == DMIC) {
 		links[i].name = "acp-dmic-codec";
-		links[i].stream_name = "DMIC capture";
 		links[i].id = DMIC_BE_ID;
 		links[i].codecs = dmic_codec;
 		links[i].num_codecs = ARRAY_SIZE(dmic_codec);