All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH V2] ARM: cpuidle: davinci: Fix target residency
@ 2013-06-27 15:10 ` Daniel Lezcano
  0 siblings, 0 replies; 6+ messages in thread
From: Daniel Lezcano @ 2013-06-27 15:10 UTC (permalink / raw)
  To: nsekhar, khilman
  Cc: sergei.shtylyov, rjw, linux-pm, linux-arm-kernel, patches, linaro-kernel

The commit 19976c2a88d125aec16b9255c7197c297bbdd637 changed the target
residency to 100000, assuming this is a careless mistake.

commit 19976c2a88d125aec16b9255c7197c297bbdd637
Author: Robert Lee <rob.lee@linaro.org>
Date:   Tue Mar 20 15:22:45 2012 -0500

    ARM: davinci: Consolidate time keeping and irq enable

    Enable core cpuidle timekeeping and irq enabling and remove that
    handling from this code.

    Signed-off-by: Robert Lee <rob.lee@linaro.org>
    Reviewed-by: Kevin Hilman <khilman@ti.com>
    Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
    Acked-by: Jean Pihet <j-pihet@ti.com>
    Signed-off-by: Len Brown <len.brown@intel.com>

The same happened to the at91's cpuidle driver.

Fix it by putting the initial value to 10000.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 arch/arm/mach-davinci/cpuidle.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-davinci/cpuidle.c b/arch/arm/mach-davinci/cpuidle.c
index 36aef3a..f1ac1c9 100644
--- a/arch/arm/mach-davinci/cpuidle.c
+++ b/arch/arm/mach-davinci/cpuidle.c
@@ -65,7 +65,7 @@ static struct cpuidle_driver davinci_idle_driver = {
 	.states[1]		= {
 		.enter			= davinci_enter_idle,
 		.exit_latency		= 10,
-		.target_residency	= 100000,
+		.target_residency	= 10000,
 		.flags			= CPUIDLE_FLAG_TIME_VALID,
 		.name			= "DDR SR",
 		.desc			= "WFI and DDR Self Refresh",
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH V2] ARM: cpuidle: davinci: Fix target residency
@ 2013-06-27 15:10 ` Daniel Lezcano
  0 siblings, 0 replies; 6+ messages in thread
From: Daniel Lezcano @ 2013-06-27 15:10 UTC (permalink / raw)
  To: linux-arm-kernel

The commit 19976c2a88d125aec16b9255c7197c297bbdd637 changed the target
residency to 100000, assuming this is a careless mistake.

commit 19976c2a88d125aec16b9255c7197c297bbdd637
Author: Robert Lee <rob.lee@linaro.org>
Date:   Tue Mar 20 15:22:45 2012 -0500

    ARM: davinci: Consolidate time keeping and irq enable

    Enable core cpuidle timekeeping and irq enabling and remove that
    handling from this code.

    Signed-off-by: Robert Lee <rob.lee@linaro.org>
    Reviewed-by: Kevin Hilman <khilman@ti.com>
    Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
    Acked-by: Jean Pihet <j-pihet@ti.com>
    Signed-off-by: Len Brown <len.brown@intel.com>

The same happened to the at91's cpuidle driver.

Fix it by putting the initial value to 10000.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 arch/arm/mach-davinci/cpuidle.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-davinci/cpuidle.c b/arch/arm/mach-davinci/cpuidle.c
index 36aef3a..f1ac1c9 100644
--- a/arch/arm/mach-davinci/cpuidle.c
+++ b/arch/arm/mach-davinci/cpuidle.c
@@ -65,7 +65,7 @@ static struct cpuidle_driver davinci_idle_driver = {
 	.states[1]		= {
 		.enter			= davinci_enter_idle,
 		.exit_latency		= 10,
-		.target_residency	= 100000,
+		.target_residency	= 10000,
 		.flags			= CPUIDLE_FLAG_TIME_VALID,
 		.name			= "DDR SR",
 		.desc			= "WFI and DDR Self Refresh",
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH V2] ARM: cpuidle: davinci: Fix target residency
  2013-06-27 15:10 ` Daniel Lezcano
@ 2013-06-28  6:42   ` Sekhar Nori
  -1 siblings, 0 replies; 6+ messages in thread
From: Sekhar Nori @ 2013-06-28  6:42 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: khilman, sergei.shtylyov, rjw, linux-pm, linux-arm-kernel,
	patches, linaro-kernel

On 6/27/2013 8:40 PM, Daniel Lezcano wrote:
> The commit 19976c2a88d125aec16b9255c7197c297bbdd637 changed the target
> residency to 100000, assuming this is a careless mistake.
> 
> commit 19976c2a88d125aec16b9255c7197c297bbdd637
> Author: Robert Lee <rob.lee@linaro.org>
> Date:   Tue Mar 20 15:22:45 2012 -0500
> 
>     ARM: davinci: Consolidate time keeping and irq enable
> 
>     Enable core cpuidle timekeeping and irq enabling and remove that
>     handling from this code.
> 
>     Signed-off-by: Robert Lee <rob.lee@linaro.org>
>     Reviewed-by: Kevin Hilman <khilman@ti.com>
>     Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>     Acked-by: Jean Pihet <j-pihet@ti.com>
>     Signed-off-by: Len Brown <len.brown@intel.com>
> 
> The same happened to the at91's cpuidle driver.
> 
> Fix it by putting the initial value to 10000.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

Acked-by: Sekhar Nori <nsekhar@ti.com>

Thanks,
Sekhar

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH V2] ARM: cpuidle: davinci: Fix target residency
@ 2013-06-28  6:42   ` Sekhar Nori
  0 siblings, 0 replies; 6+ messages in thread
From: Sekhar Nori @ 2013-06-28  6:42 UTC (permalink / raw)
  To: linux-arm-kernel

On 6/27/2013 8:40 PM, Daniel Lezcano wrote:
> The commit 19976c2a88d125aec16b9255c7197c297bbdd637 changed the target
> residency to 100000, assuming this is a careless mistake.
> 
> commit 19976c2a88d125aec16b9255c7197c297bbdd637
> Author: Robert Lee <rob.lee@linaro.org>
> Date:   Tue Mar 20 15:22:45 2012 -0500
> 
>     ARM: davinci: Consolidate time keeping and irq enable
> 
>     Enable core cpuidle timekeeping and irq enabling and remove that
>     handling from this code.
> 
>     Signed-off-by: Robert Lee <rob.lee@linaro.org>
>     Reviewed-by: Kevin Hilman <khilman@ti.com>
>     Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>     Acked-by: Jean Pihet <j-pihet@ti.com>
>     Signed-off-by: Len Brown <len.brown@intel.com>
> 
> The same happened to the at91's cpuidle driver.
> 
> Fix it by putting the initial value to 10000.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

Acked-by: Sekhar Nori <nsekhar@ti.com>

Thanks,
Sekhar

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH V2] ARM: cpuidle: davinci: Fix target residency
  2013-06-28  6:42   ` Sekhar Nori
@ 2013-07-11 11:53     ` Daniel Lezcano
  -1 siblings, 0 replies; 6+ messages in thread
From: Daniel Lezcano @ 2013-07-11 11:53 UTC (permalink / raw)
  To: Sekhar Nori
  Cc: khilman, sergei.shtylyov, rjw, linux-pm, linux-arm-kernel,
	patches, linaro-kernel

On 06/28/2013 08:42 AM, Sekhar Nori wrote:
> On 6/27/2013 8:40 PM, Daniel Lezcano wrote:
>> The commit 19976c2a88d125aec16b9255c7197c297bbdd637 changed the target
>> residency to 100000, assuming this is a careless mistake.
>>
>> commit 19976c2a88d125aec16b9255c7197c297bbdd637
>> Author: Robert Lee <rob.lee@linaro.org>
>> Date:   Tue Mar 20 15:22:45 2012 -0500
>>
>>     ARM: davinci: Consolidate time keeping and irq enable
>>
>>     Enable core cpuidle timekeeping and irq enabling and remove that
>>     handling from this code.
>>
>>     Signed-off-by: Robert Lee <rob.lee@linaro.org>
>>     Reviewed-by: Kevin Hilman <khilman@ti.com>
>>     Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>>     Acked-by: Jean Pihet <j-pihet@ti.com>
>>     Signed-off-by: Len Brown <len.brown@intel.com>
>>
>> The same happened to the at91's cpuidle driver.
>>
>> Fix it by putting the initial value to 10000.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> 
> Acked-by: Sekhar Nori <nsekhar@ti.com>

Hi Sekhar,

I applied this patch on my tree for the next PM pull request.

Thanks
  -- Daniel



-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH V2] ARM: cpuidle: davinci: Fix target residency
@ 2013-07-11 11:53     ` Daniel Lezcano
  0 siblings, 0 replies; 6+ messages in thread
From: Daniel Lezcano @ 2013-07-11 11:53 UTC (permalink / raw)
  To: linux-arm-kernel

On 06/28/2013 08:42 AM, Sekhar Nori wrote:
> On 6/27/2013 8:40 PM, Daniel Lezcano wrote:
>> The commit 19976c2a88d125aec16b9255c7197c297bbdd637 changed the target
>> residency to 100000, assuming this is a careless mistake.
>>
>> commit 19976c2a88d125aec16b9255c7197c297bbdd637
>> Author: Robert Lee <rob.lee@linaro.org>
>> Date:   Tue Mar 20 15:22:45 2012 -0500
>>
>>     ARM: davinci: Consolidate time keeping and irq enable
>>
>>     Enable core cpuidle timekeeping and irq enabling and remove that
>>     handling from this code.
>>
>>     Signed-off-by: Robert Lee <rob.lee@linaro.org>
>>     Reviewed-by: Kevin Hilman <khilman@ti.com>
>>     Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>>     Acked-by: Jean Pihet <j-pihet@ti.com>
>>     Signed-off-by: Len Brown <len.brown@intel.com>
>>
>> The same happened to the at91's cpuidle driver.
>>
>> Fix it by putting the initial value to 10000.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> 
> Acked-by: Sekhar Nori <nsekhar@ti.com>

Hi Sekhar,

I applied this patch on my tree for the next PM pull request.

Thanks
  -- Daniel



-- 
 <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-07-11 11:53 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-27 15:10 [PATCH V2] ARM: cpuidle: davinci: Fix target residency Daniel Lezcano
2013-06-27 15:10 ` Daniel Lezcano
2013-06-28  6:42 ` Sekhar Nori
2013-06-28  6:42   ` Sekhar Nori
2013-07-11 11:53   ` Daniel Lezcano
2013-07-11 11:53     ` Daniel Lezcano

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.