[PATCH V5 4/7] cpufreq: add clk-reg cpufreq driver
richard.zhao at linaro.org
Wed Dec 28 07:40:56 EST 2011
On Wed, Dec 28, 2011 at 12:14:04PM +0000, Mark Brown wrote:
> On Wed, Dec 28, 2011 at 08:05:20PM +0800, Richard Zhao wrote:
> Looks like the problem with your mail client is that it's wrapping at
> exactly 80 characters which is too little - you need to leave space for
> being quoted.
I'm using vim. any suggestion for auto-wrapping?
> > On Wed, Dec 28, 2011 at 11:42:37AM +0000, Mark Brown wrote:
> > > You can't usefully work with voltages without knowing what the actual
> > > voltages are
if people don't enable REGULATOR, the stubs (if you mean the dummy func)
are only be used to pass build. The code is optimized out by compiler.
> - the only sensible stubs we could provide would return
> > > errors but then any driver using the stubs would probably fail to do
> > > whatever it was doing.
In this case, If regulator_get return NULL, I won't call other regulator
functions at runtime.
> With enable and disable we can sensibly stub
> > > things out with an always on regulator.
> > Sorry, I can not get your point here. Let me describe the problem I met:
> > - regulator_is_supported_voltage is not exported. when I build clk-reg-cpufreq
> > as kernel module, there's a link error.
> This is an oversight, I've just fixed it.
> > - I saw linux/regulator/consumer.h has some dummy functions if !REGULATOR. I
> > tried to make clk-reg-cpufreq driver work even !REGULATOR. I think that's
> > why the dummy functions are there. If regulator_get return NULL, it'll avoid
> > calling other regulator functions. But regulator_is_supported_voltage and
> > regulator_set_voltage_time don't have such dummy ones. Undefined functions.
> I can only repeat what I wrote above explaining why no stubs are
One word. You mean I have to always depends on REGULATOR config, right?
More information about the linux-arm-kernel