[PATCH 7/8] cpufreq: st: Provide runtime initialised driver for ST's platforms

On Tue, 23 Jun 2015, Viresh Kumar wrote:

On 23-06-15, 09:27, Lee Jones wrote:

quoted

Okay, but the reasoning is the same.  I consider the function to have
failed, but the over-all failure culminates in just a warning that
voltage scaling has indeed failed, but we can still go on with
frequency scaling.

Ahh, I thought that the other opp-table will also have voltages.

Nope.  See patch 1.

quoted

Unless his is a big blocker for you, I would like to keep these
semantics.

No, the print is actually fine.

Perfect, thanks.

quoted

So technically you are correct, but it makes the code slightly more
confusing IMHO.  Yes, it's one more line of code, but it's worth it to
add clarity.

Your call :)

Thanks again.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog