Re: [PATCH V3] cpufreq: Simplify cpufreq_add_dev()

2013-01-29 Thread Viresh Kumar
On 30 January 2013 03:15, Rafael J. Wysocki wrote: > Please don't add my sign-offs in advance. Yes, I have signed off a *previous* > version of that patch, but that doesn't apply to *this* version automatically. > > While you can do such things with Acked-by or Reviewed-by tags (that's not > ver

Re: [PATCH V3] cpufreq: Simplify cpufreq_add_dev()

2013-01-29 Thread Rafael J. Wysocki
On Tuesday, January 29, 2013 08:09:08 PM Viresh Kumar wrote: > Currently cpufreq_add_dev() firsts allocates policy, calls driver->init() and > then checks if this cpu is already managed or not. And if it is already > managed, > free its policy. > > We can save all this if we somehow know cpu is m

[PATCH V3] cpufreq: Simplify cpufreq_add_dev()

2013-01-29 Thread Viresh Kumar
Currently cpufreq_add_dev() firsts allocates policy, calls driver->init() and then checks if this cpu is already managed or not. And if it is already managed, free its policy. We can save all this if we somehow know cpu is managed or not in advance. policy->related_cpus contains list of all valid