* [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.