RE: [PATCH v8 4/8] ARM: davinci: Consolidate time keeping and irq enable

2012-03-21 Thread Nori, Sekhar
Hi Amit, On Wed, Mar 21, 2012 at 20:06:29, Amit Kucheria wrote: > Rob, you should start a new '3.4-fixes' branch containing such > bugfixes that can be pushed to Len after -rc1. I think it is OK to send the fixes to Len right away so he can queue the fixes during merge window if he gets the chanc

RE: [PATCH v8 4/8] ARM: davinci: Consolidate time keeping and irq enable

2012-03-21 Thread Nori, Sekhar
Hi Rob, On Wed, Mar 21, 2012 at 01:52:45, Robert Lee wrote: > Enable core cpuidle timekeeping and irq enabling and remove that > handling from this code. > > Signed-off-by: Robert Lee > Reviewed-by: Kevin Hilman > Reviewed-by: Daniel Lezcano > Acked-by: Jean Pihet > static struct cpuidle_dri

Re: [PATCH v8 4/8] ARM: davinci: Consolidate time keeping and irq enable

2012-03-21 Thread Amit Kucheria
Rob, you should start a new '3.4-fixes' branch containing such bugfixes that can be pushed to Len after -rc1. On Wed, Mar 21, 2012 at 4:28 PM, Rob Lee wrote: > Sekhar tested this patch on Davinci last night and found a problem.  I > looked at the code again and found a mindless omission on my par

Re: [PATCH v8 4/8] ARM: davinci: Consolidate time keeping and irq enable

2012-03-21 Thread Rob Lee
On Wed, Mar 21, 2012 at 9:28 AM, Rob Lee wrote: > Sekhar tested this patch on Davinci last night and found a problem.  I > looked at the code again and found a mindless omission on my part (see > below).  Fix is trivial.  I've check all other platforms and confirmed > this problem does not exist f

Re: [PATCH v8 4/8] ARM: davinci: Consolidate time keeping and irq enable

2012-03-21 Thread Rob Lee
Sekhar tested this patch on Davinci last night and found a problem. I looked at the code again and found a mindless omission on my part (see below). Fix is trivial. I've check all other platforms and confirmed this problem does not exist for those. Will resend a v9 of the patchset shortly. On

[PATCH v8 4/8] ARM: davinci: Consolidate time keeping and irq enable

2012-03-21 Thread Robert Lee
Enable core cpuidle timekeeping and irq enabling and remove that handling from this code. Signed-off-by: Robert Lee Reviewed-by: Kevin Hilman Reviewed-by: Daniel Lezcano Acked-by: Jean Pihet --- arch/arm/mach-davinci/cpuidle.c | 82 --- 1 file changed, 33