[alsa-devel] Question about pinctrl_pm_select_xxx()

Kuninori Morimoto kuninori.morimoto.gx at renesas.com
Fri Jun 21 03:15:25 CEST 2019


Hi Mark, again

> > > Thus, calling it from DAI only is very enough, I think.
> > 
> > I'd expect that only the DAI should have pins so probably yes, but we
> > can't guarantee that this won't break anything in practice.
> 
> Yes, indeed.
> But we can just revert it if it breaks something.

I could confirm. Original code was for Codec,
and it was converted to Component by this patch.

9178feb4538e055bf22be44c38b90cc31d2baf99
("ASoC: add Component level suspend/resume")

...
-		pinctrl_pm_select_sleep_state(codec->dev);
+		pinctrl_pm_select_sleep_state(component->dev);
...

I will add this to git-log message

Thank you for your help !!
Best regards
---
Kuninori Morimoto


More information about the Alsa-devel mailing list