Thread (33 messages) read the whole thread 33 messages, 5 authors, 2015-06-24

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

From: Viresh Kumar <viresh.kumar@linaro.org>
Date: 2015-06-23 08:31:19
Also in: linux-arm-kernel, linux-pm, lkml

On 23-06-15, 09:27, Lee Jones wrote:
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.
Unless his is a big blocker for you, I would like to keep these
semantics.
No, the print is actually fine.
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 :)

-- 
viresh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help