Re: [PATCH 0/2] thermal: Replace thermal_notify_framework with thermal_zone_device_update

[Thara Gopinath]

On 1/19/21 9:56 PM, Zhang, Rui wrote:
> Hi, Thara,
>
> Thanks for the cleanup. I've proposed similar patches previously.
> https://patchwork.kernel.org/project/linux-pm/patch/20200430063229.6182-2-rui.zhang@xxxxxxxxx/
> https://patchwork.kernel.org/project/linux-pm/patch/20200430063229.6182-3-rui.zhang@xxxxxxxxx/
> can you please also address the comments in the previous discussion, like doc cleanup?

Hi Rui,

Thanks for the quick feedback. I am sorry, I was not aware of your 
earlier series. I will add a patch cleaning up the Documentation and
send the v2. I have not tested this patch much. But I see from the
comments on your patches that this has been tested out. So that is good.

> Thanks,
> rui
>
> > -----Original Message-----
> > From: Thara Gopinath <thara.gopinath@xxxxxxxxxx>
> > Sent: Tuesday, January 19, 2021 10:06 PM
> > To: Zhang, Rui <rui.zhang@xxxxxxxxx>; daniel.lezcano@xxxxxxxxxx;
> > kvalo@xxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; kuba@xxxxxxxxxx; Coelho,
> > Luciano <luciano.coelho@xxxxxxxxx>
> > Cc: linux-wireless@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> > netdev@xxxxxxxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; amitk@xxxxxxxxxx;
> > Errera, Nathan <nathan.errera@xxxxxxxxx>
> > Subject: [PATCH 0/2] thermal: Replace thermal_notify_framework with
> > thermal_zone_device_update
> > Importance: High
> >
> > thermal_notify_framework just updates for a single trip point where as
> > thermal_zone_device_update does other bookkeeping like updating the
> > temperature of the thermal zone, running through the list of trip points and
> > setting the next trip point etc. Since  the later is a more thorough version of
> > former, replace thermal_notify_framework with
> > thermal_zone_device_update.
> >
> > Thara Gopinath (2):
> >    net: wireless: intel: iwlwifi: mvm: tt: Replace
> >      thermal_notify_framework
> >    drivers: thermal: Remove thermal_notify_framework
> >
> >   drivers/net/wireless/intel/iwlwifi/mvm/tt.c |  4 ++--
> >   drivers/thermal/thermal_core.c              | 18 ------------------
> >   include/linux/thermal.h                     |  4 ----
> >   3 files changed, 2 insertions(+), 24 deletions(-)
> >
> > --
> > 2.25.1

--
Warm Regards
Thara