On 2020-11-12 11:38 PM, Pierre-Louis Bossart wrote: > For some reason this ops is missing in 2 out of the 3 broadwell > drivers. Add to make sure ASoC takes care of power management. > Changes provided with this patch are not connected to the overall idea behind your patchset. I suggest splitting pm-changes into a separate one. The very first sentence in commit message isn't providing any value, guess one-liner will suffice. > Tested-by: Cezary Rojewski <cezary.rojewski@xxxxxxxxx> > Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> > Reviewed-by: Ranjani Sridharan <ranjani.sridharan@xxxxxxxxxxxxxxx> > Reviewed-by: Rander Wang <rander.wang@xxxxxxxxxxxxxxx> > Reviewed-by: Guennadi Liakhovetski <guennadi.liakhovetski@xxxxxxxxxxxxxxx> > --- > sound/soc/intel/boards/bdw-rt5677.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/sound/soc/intel/boards/bdw-rt5677.c b/sound/soc/intel/boards/bdw-rt5677.c > index 7a3e773d0a1c..9cdd4164e1fb 100644 > --- a/sound/soc/intel/boards/bdw-rt5677.c > +++ b/sound/soc/intel/boards/bdw-rt5677.c > @@ -446,6 +446,7 @@ static struct platform_driver bdw_rt5677_audio = { > .probe = bdw_rt5677_probe, > .driver = { > .name = "bdw-rt5677", > + .pm = &snd_soc_pm_ops > }, > }; > >