Messages in this thread | | | Date | Wed, 22 Nov 2017 06:55:32 +0530 | From | Viresh Kumar <> | Subject | Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff |
| |
On 21-11-17, 18:00, Javi Merino wrote: > As Ionela pointed out earlier in the thread, the cpufreq driver for Juno > was not acceptable for mainline because it used platform specific code.
Can we get a link to that thread? I don't remember what I have commented earlier but the above doesn't seem to be entirely true.
The basic idea is to use as much common stuff as possible and so to use cpufreq-dt.c if possible. Its not that we are against platform specific bits, they are fine if they are really required.
> When it was converted to cpufreq-dt, the static power was left behind > because it can't be represented in device tree. This is because there > isn't a function that works for every SoC, different process nodes > (among other things) will need different functions. So it can't be just > a bunch of coefficients in DT, we need a function. Hence the callback.
Sure thing. And we can make this happen if we need. We aren't blocking it.
> In a nutshell, mainline does not want platform specific code, but we
Not really. We don't want platform specific code in arch/arm64, but we can have that in drivers/opp/ for example if required.
Please start a discussion (in a separate thread if you want) and we can get cpufreq support updated for Juno very easily.
And don't worry about this patch here. We can surely drop the patch if someone is serious enough to start using it. But there needs to be a commitment, nothing more.
-- viresh
| |