Re: [RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-05-16 Thread Chander Kashyap
On 16 May 2014 15:16, Rafael J. Wysocki wrote: > On 5/16/2014 10:16 AM, Chander Kashyap wrote: >> >> On 22 April 2014 18:08, Chander Kashyap >> wrote: >>> >>> In menu_select function we check for correction factor every time. >>> If it is zero we are initializing to unity. Hence move it to init

Re: [RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-05-16 Thread Rafael J. Wysocki
On 5/16/2014 10:16 AM, Chander Kashyap wrote: On 22 April 2014 18:08, Chander Kashyap wrote: In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence avoid repeated comparisons.

Re: [RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-05-16 Thread Chander Kashyap
On 22 April 2014 18:08, Chander Kashyap wrote: > In menu_select function we check for correction factor every time. > If it is zero we are initializing to unity. Hence move it to init function > and initialise by unity, hence avoid repeated comparisons. > > Signed-off-by: Chander Kashyap >

Re: [RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-05-16 Thread Chander Kashyap
On 22 April 2014 18:08, Chander Kashyap chander.kash...@linaro.org wrote: In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence avoid repeated comparisons. Signed-off-by:

Re: [RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-05-16 Thread Rafael J. Wysocki
On 5/16/2014 10:16 AM, Chander Kashyap wrote: On 22 April 2014 18:08, Chander Kashyap chander.kash...@linaro.org wrote: In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence

Re: [RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-05-16 Thread Chander Kashyap
On 16 May 2014 15:16, Rafael J. Wysocki rafael.j.wyso...@intel.com wrote: On 5/16/2014 10:16 AM, Chander Kashyap wrote: On 22 April 2014 18:08, Chander Kashyap chander.kash...@linaro.org wrote: In menu_select function we check for correction factor every time. If it is zero we are

[RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Chander Kashyap
In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence avoid repeated comparisons. Signed-off-by: Chander Kashyap Reviewed-by: Tuukka Tikkanen ---

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Chander Kashyap
On 22 April 2014 16:54, Rafael J. Wysocki wrote: > On Tuesday, April 22, 2014 03:24:21 PM Chander Kashyap wrote: >> +cc >> linux...@vger.kernel.org. > > And can you please resend the patch? So that it shows up in Patchwork, > for example? > Yes i will resend it again. > >> On 22 April 2014

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Rafael J. Wysocki
On Tuesday, April 22, 2014 03:24:21 PM Chander Kashyap wrote: > +cc > linux...@vger.kernel.org. And can you please resend the patch? So that it shows up in Patchwork, for example? > On 22 April 2014 15:33, Rafael J. Wysocki wrote: > > On Tuesday, April 22, 2014 08:27:53 AM Chander Kashyap

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Chander Kashyap
+cc linux...@vger.kernel.org. On 22 April 2014 15:33, Rafael J. Wysocki wrote: > On Tuesday, April 22, 2014 08:27:53 AM Chander Kashyap wrote: >> ping > > Please CC cpuidle patches to linux...@vger.kernel.org. +cc to linux...@vger.kernel.org. Thanks Rafael > > Thanks! > >> On 10 April 2014

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Rafael J. Wysocki
On Tuesday, April 22, 2014 08:27:53 AM Chander Kashyap wrote: > ping Please CC cpuidle patches to linux...@vger.kernel.org. Thanks! > On 10 April 2014 19:57, Tuukka Tikkanen wrote: > > On 10 April 2014 16:43, Chander Kashyap wrote: > >> In menu_select function we check for correction factor

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Rafael J. Wysocki
On Tuesday, April 22, 2014 08:27:53 AM Chander Kashyap wrote: ping Please CC cpuidle patches to linux...@vger.kernel.org. Thanks! On 10 April 2014 19:57, Tuukka Tikkanen tuukka.tikka...@linaro.org wrote: On 10 April 2014 16:43, Chander Kashyap chander.kash...@linaro.org wrote: In

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Chander Kashyap
+cc linux...@vger.kernel.org. On 22 April 2014 15:33, Rafael J. Wysocki r...@rjwysocki.net wrote: On Tuesday, April 22, 2014 08:27:53 AM Chander Kashyap wrote: ping Please CC cpuidle patches to linux...@vger.kernel.org. +cc to linux...@vger.kernel.org. Thanks Rafael Thanks! On 10 April

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Rafael J. Wysocki
On Tuesday, April 22, 2014 03:24:21 PM Chander Kashyap wrote: +cc linux...@vger.kernel.org. And can you please resend the patch? So that it shows up in Patchwork, for example? On 22 April 2014 15:33, Rafael J. Wysocki r...@rjwysocki.net wrote: On Tuesday, April 22, 2014 08:27:53 AM

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Chander Kashyap
On 22 April 2014 16:54, Rafael J. Wysocki r...@rjwysocki.net wrote: On Tuesday, April 22, 2014 03:24:21 PM Chander Kashyap wrote: +cc linux...@vger.kernel.org. And can you please resend the patch? So that it shows up in Patchwork, for example? Yes i will resend it again. On 22 April

[RESEND][PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-22 Thread Chander Kashyap
In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence avoid repeated comparisons. Signed-off-by: Chander Kashyap chander.kash...@linaro.org Reviewed-by: Tuukka Tikkanen

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-21 Thread Chander Kashyap
ping On 10 April 2014 19:57, Tuukka Tikkanen wrote: > On 10 April 2014 16:43, Chander Kashyap wrote: >> In menu_select function we check for correction factor every time. >> If it is zero we are initializing to unity. Hence move it to init function >> and initialise by unity, hence avoid

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-21 Thread Chander Kashyap
ping On 10 April 2014 19:57, Tuukka Tikkanen tuukka.tikka...@linaro.org wrote: On 10 April 2014 16:43, Chander Kashyap chander.kash...@linaro.org wrote: In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-10 Thread Tuukka Tikkanen
On 10 April 2014 16:43, Chander Kashyap wrote: > In menu_select function we check for correction factor every time. > If it is zero we are initializing to unity. Hence move it to init function > and initialise by unity, hence avoid repeated comparisons. > > Signed-off-by: Chander Kashyap > --- >

[PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-10 Thread Chander Kashyap
In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence avoid repeated comparisons. Signed-off-by: Chander Kashyap --- drivers/cpuidle/governors/menu.c | 15 --- 1

[PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-10 Thread Chander Kashyap
In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence avoid repeated comparisons. Signed-off-by: Chander Kashyap chander.kash...@linaro.org --- drivers/cpuidle/governors/menu.c

Re: [PATCH] cpuidle: governor: menu: move repeated correction factor check to init

2014-04-10 Thread Tuukka Tikkanen
On 10 April 2014 16:43, Chander Kashyap chander.kash...@linaro.org wrote: In menu_select function we check for correction factor every time. If it is zero we are initializing to unity. Hence move it to init function and initialise by unity, hence avoid repeated comparisons. Signed-off-by: