Re: [PATCH] thermal: core: Add thermal zone enable/disable notification

[Daniel Lezcano]

On 29/07/2020 07:36, Amit Kucheria wrote:
> On Tue, Jul 28, 2020 at 4:40 AM Daniel Lezcano
>  wrote:
>>
>> Now the calls to enable/disable a thermal zone are centralized in a
>> call to a function, we can add in these the corresponding netlink
>> notifications.
>>
>> Signed-off-by: Daniel Lezcano 
> 
> Reviewed-by: Amit Kucheria 

I've applied the changes, thanks for the review.


-- 
 Linaro.org │ Open source software for ARM SoCs

Follow Linaro:   Facebook |
 Twitter |
 Blog

Re: [PATCH] thermal: core: Add thermal zone enable/disable notification

[Amit Kucheria]

On Tue, Jul 28, 2020 at 4:40 AM Daniel Lezcano
 wrote:
>
> Now the calls to enable/disable a thermal zone are centralized in a
> call to a function, we can add in these the corresponding netlink
> notifications.
>
> Signed-off-by: Daniel Lezcano 

Reviewed-by: Amit Kucheria 

> ---
>  drivers/thermal/thermal_core.c | 5 +
>  1 file changed, 5 insertions(+)
>
> diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
> index 9748fbb9a3a1..72bf159bcecc 100644
> --- a/drivers/thermal/thermal_core.c
> +++ b/drivers/thermal/thermal_core.c
> @@ -509,6 +509,11 @@ static int thermal_zone_device_set_mode(struct 
> thermal_zone_device *tz,
>
> thermal_zone_device_update(tz, THERMAL_EVENT_UNSPECIFIED);
>
> +   if (mode == THERMAL_DEVICE_ENABLED)
> +   thermal_notify_tz_enable(tz->id);
> +   else
> +   thermal_notify_tz_disable(tz->id);
> +
> return ret;
>  }
>
> --
> 2.17.1
>

Re: [PATCH] thermal: core: Add thermal zone enable/disable notification

[Zhang Rui]

On Tue, 2020-07-28 at 01:10 +0200, Daniel Lezcano wrote:
> Now the calls to enable/disable a thermal zone are centralized in a
> call to a function, we can add in these the corresponding netlink
> notifications.
> 
> Signed-off-by: Daniel Lezcano 

Acked-by: Zhang Rui 
> ---
>  drivers/thermal/thermal_core.c | 5 +
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/thermal/thermal_core.c
> b/drivers/thermal/thermal_core.c
> index 9748fbb9a3a1..72bf159bcecc 100644
> --- a/drivers/thermal/thermal_core.c
> +++ b/drivers/thermal/thermal_core.c
> @@ -509,6 +509,11 @@ static int thermal_zone_device_set_mode(struct
> thermal_zone_device *tz,
>  
>   thermal_zone_device_update(tz, THERMAL_EVENT_UNSPECIFIED);
>  
> + if (mode == THERMAL_DEVICE_ENABLED)
> + thermal_notify_tz_enable(tz->id);
> + else
> + thermal_notify_tz_disable(tz->id);
> +
>   return ret;
>  }
>  

[PATCH] thermal: core: Add thermal zone enable/disable notification

[Daniel Lezcano]

Now the calls to enable/disable a thermal zone are centralized in a
call to a function, we can add in these the corresponding netlink
notifications.

Signed-off-by: Daniel Lezcano 
---
 drivers/thermal/thermal_core.c | 5 +
 1 file changed, 5 insertions(+)

diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
index 9748fbb9a3a1..72bf159bcecc 100644
--- a/drivers/thermal/thermal_core.c
+++ b/drivers/thermal/thermal_core.c
@@ -509,6 +509,11 @@ static int thermal_zone_device_set_mode(struct 
thermal_zone_device *tz,
 
thermal_zone_device_update(tz, THERMAL_EVENT_UNSPECIFIED);
 
+   if (mode == THERMAL_DEVICE_ENABLED)
+   thermal_notify_tz_enable(tz->id);
+   else
+   thermal_notify_tz_disable(tz->id);
+
return ret;
 }
 
-- 
2.17.1