Re: [PATCH 2/6] ARM: omap: allow building omap44xx without SMP

2012-08-23 Thread Arnd Bergmann
On Thursday 23 August 2012, Shilimkar, Santosh wrote: > > On Thu, Aug 23, 2012 at 5:42 PM, Arnd Bergmann wrote: > > > > On Thursday 23 August 2012, Shilimkar, Santosh wrote: > > > I see your point but alternate patch is pushing down the fix to the low > > > level driver and that means you end up

Re: [PATCH 2/6] ARM: omap: allow building omap44xx without SMP

2012-08-23 Thread Shilimkar, Santosh
On Thu, Aug 23, 2012 at 5:42 PM, Arnd Bergmann wrote: > > On Thursday 23 August 2012, Shilimkar, Santosh wrote: > > I see your point but alternate patch is pushing down the fix to the low > > level driver and that means you end up patching more drivers when they > > use COUPLE idle infrastructure.

Re: [PATCH 2/6] ARM: omap: allow building omap44xx without SMP

2012-08-23 Thread Arnd Bergmann
On Thursday 23 August 2012, Shilimkar, Santosh wrote: > I see your point but alternate patch is pushing down the fix to the low > level driver and that means you end up patching more drivers when they > use COUPLE idle infrastructure. That was the only reason I was thinking > of suppressing the err

Re: [PATCH 2/6] ARM: omap: allow building omap44xx without SMP

2012-08-23 Thread Shilimkar, Santosh
On Wed, Aug 22, 2012 at 10:52 PM, Arnd Bergmann wrote: > > On Wednesday 22 August 2012, Shilimkar, Santosh wrote: > > > Was just thinking whether we should just take care of it at > > core cpuidle level itself. Will below be enough to kill the build > > error what you mentioned in the change log ?

Re: [PATCH 2/6] ARM: omap: allow building omap44xx without SMP

2012-08-22 Thread Arnd Bergmann
On Wednesday 22 August 2012, Shilimkar, Santosh wrote: > Was just thinking whether we should just take care of it at > core cpuidle level itself. Will below be enough to kill the build > error what you mentioned in the change log ? > > diff --git a/drivers/cpuidle/coupled.c b/drivers/cpuidle/coup

Re: [PATCH 2/6] ARM: omap: allow building omap44xx without SMP

2012-08-22 Thread Shilimkar, Santosh
On Wed, Aug 22, 2012 at 8:43 PM, Arnd Bergmann wrote: > The new omap4 cpuidle implementation currently requires > ARCH_NEEDS_CPU_IDLE_COUPLED, which only works on SMP. > > This patch makes it possible to build a non-SMP kernel > for that platform. This is not normally desired for > end-users but c

[PATCH 2/6] ARM: omap: allow building omap44xx without SMP

2012-08-22 Thread Arnd Bergmann
The new omap4 cpuidle implementation currently requires ARCH_NEEDS_CPU_IDLE_COUPLED, which only works on SMP. This patch makes it possible to build a non-SMP kernel for that platform. This is not normally desired for end-users but can be useful for testing. Without this patch, building rand-0y2jS