Re: [PATCH 3/4] thermal/drivers/cpuidle_cooling: Change the registration function

From: Viresh Kumar
Date: Mon Mar 30 2020 - 05:58:58 EST


On 30-03-20, 00:03, Daniel Lezcano wrote:
> Today, there is no user for the cpuidle cooling device. The targetted
> platform is ARM and ARM64.
>
> The cpuidle and the cpufreq cooling device are based on the device tree.
>
> As the cpuidle cooling device can have its own configuration depending
> on the platform and the available idle states. The DT node description
> will give the optional properties to set the cooling device up.
>
> Do no longer rely on the CPU node which is prone to error and will
> lead to a confusion in the DT because the cpufreq cooling device is
> also using it. Let initialize the cpuidle cooling device with the DT
> binding.
>
> This was tested on:
> - hikey960
> - hikey6220
> - rock960
> - db845c
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> ---
> drivers/thermal/cpuidle_cooling.c | 58 +++++++++++++++++++++++++------
> include/linux/cpu_cooling.h | 7 ----
> 2 files changed, 47 insertions(+), 18 deletions(-)

Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

--
viresh