Re: [PATCH 1/3] thermal: imx: remove cpufreq cooling registration

From: Bastian Stender
Date: Mon May 14 2018 - 05:18:23 EST


On 05/14/2018 11:10 AM, Anson Huang wrote:
-----Original Message-----
From: Bastian Stender [mailto:bst@xxxxxxxxxxxxxx]
Sent: Monday, May 14, 2018 4:37 PM
To: Anson Huang <anson.huang@xxxxxxx>; shawnguo@xxxxxxxxxx;
s.hauer@xxxxxxxxxxxxxx; kernel@xxxxxxxxxxxxxx; Fabio Estevam
<fabio.estevam@xxxxxxx>; robh+dt@xxxxxxxxxx; mark.rutland@xxxxxxx;
rjw@xxxxxxxxxxxxx; viresh.kumar@xxxxxxxxxx; rui.zhang@xxxxxxxxx;
edubezval@xxxxxxxxx
Cc: devicetree@xxxxxxxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; dl-linux-imx
<linux-imx@xxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
linux-kernel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH 1/3] thermal: imx: remove cpufreq cooling registration

Hi,

On 05/14/2018 10:09 AM, Anson Huang wrote:
This patch removes cpufreq cooling registration in thermal .probe
function, cpufreq cooling should be done in cpufreq driver when it is
ready.

Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>

It seems you are trying to achieve something similar to a patch I sent a couple
of month ago. Unfortunately I did not have the time to rework it yet:


https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch
work.kernel.org%2Fpatch%2F10059085%2F&data=02%7C01%7Canson.huang
%40nxp.com%7C179da3635cab4a14deef08d5b975ecb6%7C686ea1d3bc2b4c6f
a92cd99c5c301635%7C0%7C0%7C636618838508978629&sdata=BcQ9tc%2BE
CZ%2Fk4AsZFxshgmvSsPg7eRN0ASzP8LO8yBI%3D&reserved=0

Some of the comments might apply here too.

Ah, I did NOT notice this thread, so how to proceed, will you continue to finish your patch? If yes, then you can just ignore/skip
my patch, thanks.
Yes, I will revisit the patch in a couple of weeks. It would be nice to
get your feedback then.

Regards,
Bastian

--
Pengutronix e.K.
Industrial Linux Solutions
http://www.pengutronix.de/
Peiner Str. 6-8, 31137 Hildesheim, Germany
Amtsgericht Hildesheim, HRA 2686