Messages in this thread | | | Date | Mon, 10 Aug 2020 18:50:47 +0100 | From | Mark Brown <> | Subject | Re: [GIT PULL] sound updates for 5.9 |
| |
On Mon, Aug 10, 2020 at 06:06:14PM +0100, Srinivas Kandagatla wrote:
> In q6routing case all the dapm widgets reuse reg field in > snd_soc_dapm_widget to store offset information or routing table indexs or > some DSP related id and so on... These are not real registers.
> I think the core is trying to read the state of these widgets during > startup, Which will fail in qdsp6 case as we do not have any regmap or read > callback associated with this ASoC component.
Yes, it will try to figure out the current state of things during startup.
> Previously we never had chance to see these messages so we did not implement > any dummy read callback.
So I guess this is another instance of the issues with other things, just having an effect beyond the cosmetic this time :/
> Adding a dummy callback to q6routing and q6afe-dai ASoC component will fix > this issue at-least in Qualcomm case!
Yes, that's going to be better for robustness regardless of changes in the core - just pick a default state if the underlying thing is undefined. [unhandled content-type:application/pgp-signature] | |