2021-01-20 11:33:32

by Thara Gopinath

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



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/[email protected]/
> https://patchwork.kernel.org/project/linux-pm/patch/[email protected]/
> 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 <[email protected]>
>> Sent: Tuesday, January 19, 2021 10:06 PM
>> To: Zhang, Rui <[email protected]>; [email protected];
>> [email protected]; [email protected]; [email protected]; Coelho,
>> Luciano <[email protected]>
>> Cc: [email protected]; [email protected];
>> [email protected]; [email protected]; [email protected];
>> Errera, Nathan <[email protected]>
>> 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