Re: [PATCH 2/3 V2] intel_idle: Removing the redundant calculating for dev->state_count

2013-03-11 Thread Daniel Lezcano
On 03/11/2013 11:44 AM, Chuansheng Liu wrote: > > In function intel_idle_cpu_init() and intel_idle_cpuidle_driver_init(), > they are having the same for(;;) loop to count the ->state_count. > > Although intel_idle_cpu_init() can be called at runtime CPU HOTPLUG case, > but max_cstate can not be c

[PATCH 2/3 V2] intel_idle: Removing the redundant calculating for dev->state_count

2013-03-10 Thread Chuansheng Liu
In function intel_idle_cpu_init() and intel_idle_cpuidle_driver_init(), they are having the same for(;;) loop to count the ->state_count. Although intel_idle_cpu_init() can be called at runtime CPU HOTPLUG case, but max_cstate can not be changed at runtime. So the dev->state_count should be == d