On 26 February 2014 02:41, Saravana Kannan [off-list ref] wrote:

On 02/25/2014 05:04 AM, Rafael J. Wysocki wrote:

quoted

On Tuesday, February 25, 2014 02:20:57 PM Viresh Kumar wrote:

I think there's been a misunderstanding of what I'm proposing. The reference
to policy->clk is only to get rid of the dependency that
cpufreq_generic_get() has on the per cpu policy variable being filled. You
can do that by just removing calls to get _IF_ clk is filled in.

cpufreq_cpu_get() can be called by other drivers as well, which may not have
clock interface implemented for them. We can't stop them from calling it.

I'll look at the patches later today and respond. Although, I would have
been nice you had helped me fix any issues with my patch than coming up with
new ones. Kinda removes to motivation for submitting patches upstream.

Sorry if I demotivated you at all :)

I just wanted to get to a quick-fix and wasn't interested in getting
my patch count
up. Thought that isn't always bad :)

I sent my patches as they were very different then your approach. Obviously, I
wouldn't have done this otherwise :)