Re: [PATCH] thermal: ti-soc-thermal: Enable addition power management

From: Adam Ford
Date: Fri Nov 08 2019 - 16:23:46 EST


On Fri, Nov 8, 2019 at 3:17 PM Tony Lindgren <tony@xxxxxxxxxxx> wrote:
>
> * Adam Ford <aford173@xxxxxxxxx> [191108 21:00]:
> > +static int bandgap_omap_cpu_notifier(struct notifier_block *nb,
> > + unsigned long cmd, void *v);
> >
> > /*** Helper functions to access registers and their bitfields ***/
> >
> > @@ -1025,6 +1033,9 @@ int ti_bandgap_probe(struct platform_device *pdev)
> > }
> > }
> >
> > + bgp->nb.notifier_call = bandgap_omap_cpu_notifier;
> > + cpu_pm_register_notifier(&bgp->nb);
> > +
>
> Hmm looks like you're missing the related call to
> cpu_pm_unregister_notifier(), right?

Good catch. I'm new to this PM stuff. :-)

>
> Other than that, it also works on droid4, so please
> feel free to add:

Awesome!

>
> Reviewed-by: Tony Lindgren <tony@xxxxxxxxxxx>
> Tested-by: Tony Lindgren <tony@xxxxxxxxxxx>

Thank you. I have already resent V2

adam

>