Messages in this thread | | | Date | Mon, 16 Apr 2018 15:20:06 +0530 | From | Viresh Kumar <> | Subject | Re: [PATCH v3 6/7] thermal/drivers/cpu_cooling: Introduce the cpu idle cooling driver |
| |
On 16-04-18, 11:45, Daniel Lezcano wrote: > Can you elaborate a bit ? I'm not sure to get the point.
Sure. With your current code on Hikey960 (big/LITTLE), you end up creating two cooling devices, one for the big cluster and one for small cluster. Which is the right thing to do, as we also have two cpufreq cooling devices.
But with the change Sudeep is referring to, the helper you used to get cluster id will return 0 (SoC id) for all the 8 CPUs. So your code will end up creating a single cpuidle cooling device for all the CPUs. Which would be wrong.
> BTW, Am I asked to change my code to stick to something which is not > merged ?
Sudeep looked pretty confident on how the meaning of this routine is going to change very soon. I will let him respond on what guarantees we have that it will get merged :)
-- viresh
| |