Thread (27 messages) 27 messages, 5 authors, 2015-03-18

Re: [PATCH v2 3/4] cpufreq: mediatek: add Mediatek cpufreq driver

From: Viresh Kumar <hidden>
Date: 2015-03-12 09:28:59
Also in: linux-arm-kernel, linux-pm, lkml

On 11 March 2015 at 18:15, Mark Brown [off-list ref] wrote:
Ugh, no - that's a hideous bodge which is only going to create trouble
later.  Remember, DT is an ABI and should describe the hardware so if
we're doing bodges that are visible there to shoehorn things onto our
implementation that's bad.  The concerns that Pi-Cheng had about what
happens if the PMIC gets changed definitely seem relevant here too.
Hmm..
Why not just write a custom cpufreq driver if it's too hard to abstract?
Hmm, probably all that can be solved with the new OPP bindings where
we can have support for multiple regulator or clock sources to the CPU.

@Pi-cheng: How are you going to pass voltages for both the regulators
as OPPs today only support a single regulator, in case you have to write
your own driver.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help