[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ZKK9QASAPdYjaa21@hovoldconsulting.com>
Date: Mon, 3 Jul 2023 14:21:20 +0200
From: Johan Hovold <johan@...nel.org>
To: Takashi Iwai <tiwai@...e.de>
Cc: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
Mark Brown <broonie@...nel.org>, johan+linaro@...nel.org,
perex@...ex.cz, tiwai@...e.com, lgirdwood@...il.com,
ckeepax@...nsource.cirrus.com, kuninori.morimoto.gx@...esas.com,
linux-kernel@...r.kernel.org, pierre-louis.bossart@...ux.intel.com,
alsa-devel@...a-project.org, Stable@...r.kernel.org
Subject: Re: [PATCH] ASoC: qdsp6: q6apm: use dai link pcm id as pcm device
number
On Mon, Jul 03, 2023 at 10:48:51AM +0200, Johan Hovold wrote:
> On Mon, Jul 03, 2023 at 10:19:29AM +0200, Takashi Iwai wrote:
> > On Mon, 03 Jul 2023 10:03:52 +0200,
> > Johan Hovold wrote:
>
> > > Ok, scratch that. I just hit the race again also with this patch
> > > applied:
> > >
> > > [ 11.815028] q6apm-dai 3000000.remoteproc:glink-edge:gpr:service@1:dais: Adding to iommu group 23
> > > [ 11.838667] snd-sc8280xp sound: ASoC: adding FE link failed
> > > [ 11.838774] snd-sc8280xp sound: ASoC: topology: could not load header: -517
> > > [ 11.838916] qcom-apm gprsvc:service:2:1: tplg component load failed-517
> > > [ 11.838996] qcom-apm gprsvc:service:2:1: ASoC: error at snd_soc_component_probe on gprsvc:service:2:1: -22
> > > [ 11.839430] snd-sc8280xp sound: ASoC: failed to instantiate card -22
> > > [ 11.844801] snd-sc8280xp: probe of sound failed with error -22
> >
> > Isn't it rather an issue about the error code passing in qcom driver?
> > How about the change like below?
>
> Indeed, and I tested a change like that here:
>
> https://lore.kernel.org/lkml/ZIHSGf18Aw7htb9o8@hovoldconsulting.com/
That link was missing was apparently broken, should have been:
https://lore.kernel.org/all/ZIHSGf18w7htb9o8@hovoldconsulting.com/
>
> but that only seems to make the problem worse currently.
>
> This should probably still be fixed, but I was just hoping that the DAI
> numbering could have been the cause for the probe deferral (which then
> triggers the other bugs).
Johan
Powered by blists - more mailing lists