Re: [PATCH 5/5] ARM: exynos: enable/disable cpuidle when cpu1 is down/up

2013-01-18 Thread Daniel Lezcano
On 01/18/2013 04:51 AM, Kukjin Kim wrote: > Daniel Lezcano wrote: >> On 01/10/2013 11:33 PM, amit daniel kachhap wrote: >>> On Thu, Jan 10, 2013 at 1:32 PM, Daniel Lezcano >> wrote: On 01/10/2013 09:07 PM, amit daniel kachhap wrote: > Hi Daniel, Hi Amit Daniel, > This hotplu

RE: [PATCH 5/5] ARM: exynos: enable/disable cpuidle when cpu1 is down/up

2013-01-17 Thread Kukjin Kim
Daniel Lezcano wrote: > > On 01/10/2013 11:33 PM, amit daniel kachhap wrote: > > On Thu, Jan 10, 2013 at 1:32 PM, Daniel Lezcano > wrote: > >> On 01/10/2013 09:07 PM, amit daniel kachhap wrote: > >>> Hi Daniel, > >> > >> Hi Amit Daniel, > >> > >>> This hotplug noifiers looks fine. I suppose it sh

Re: [PATCH 5/5] ARM: exynos: enable/disable cpuidle when cpu1 is down/up

2013-01-15 Thread Daniel Lezcano
On 01/10/2013 11:33 PM, amit daniel kachhap wrote: > On Thu, Jan 10, 2013 at 1:32 PM, Daniel Lezcano > wrote: >> On 01/10/2013 09:07 PM, amit daniel kachhap wrote: >>> Hi Daniel, >> >> Hi Amit Daniel, >> >>> This hotplug noifiers looks fine. I suppose it should add extra state >>> C1 in cpu0. If

Re: [PATCH 5/5] ARM: exynos: enable/disable cpuidle when cpu1 is down/up

2013-01-10 Thread amit daniel kachhap
On Thu, Jan 10, 2013 at 1:32 PM, Daniel Lezcano wrote: > On 01/10/2013 09:07 PM, amit daniel kachhap wrote: >> Hi Daniel, > > Hi Amit Daniel, > >> This hotplug noifiers looks fine. I suppose it should add extra state >> C1 in cpu0. If it is done like below than for normal cases(when all >> cpu's a

Re: [PATCH 5/5] ARM: exynos: enable/disable cpuidle when cpu1 is down/up

2013-01-10 Thread Daniel Lezcano
On 01/10/2013 09:07 PM, amit daniel kachhap wrote: > Hi Daniel, Hi Amit Daniel, > This hotplug noifiers looks fine. I suppose it should add extra state > C1 in cpu0. If it is done like below than for normal cases(when all > cpu's are online) there wont be any statistics for C0 state I guess you

Re: [PATCH 5/5] ARM: exynos: enable/disable cpuidle when cpu1 is down/up

2013-01-10 Thread amit daniel kachhap
Hi Daniel, This hotplug noifiers looks fine. I suppose it should add extra state C1 in cpu0. If it is done like below than for normal cases(when all cpu's are online) there wont be any statistics for C0 state also which is required. Other patches look good. Thanks, Amit On Fri, Jan 4, 2013 at 8:

[PATCH 5/5] ARM: exynos: enable/disable cpuidle when cpu1 is down/up

2013-01-04 Thread Daniel Lezcano
What we have now is (1) cpu0 going always to WFI when cpu1 is up, (2) cpu0 going to all states when cpu1 is down. In other words, cpuidle is disabled when cpu1 is up and enabled when cpu1 is down. This patch use the cpu hotplug notifier to enable/disable cpuidle, when the cpu1 is plugged or unplu