Re: [PATCH 06/16] ARM idle: delete pm_idle

2013-02-12 Thread Kevin Hilman
Len Brown writes: > From: Len Brown > > pm_idle() on ARM was a synonym for default_idle(), > so simply invoke default_idle() directly. > > Signed-off-by: Len Brown > Cc: linux-arm-ker...@lists.infradead.org Looks good to me. Tested this patch along with 15/16 (removes global declaration) on a

Re: [PATCH 06/16] ARM idle: delete pm_idle

2013-02-11 Thread Len Brown
On 02/11/2013 11:11 AM, Russell King - ARM Linux wrote: > On Mon, Feb 11, 2013 at 04:02:30PM +, Catalin Marinas wrote: >> On Sun, Feb 10, 2013 at 05:58:13AM +, Len Brown wrote: >>> pm_idle() on ARM was a synonym for default_idle(), >>> so simply invoke default_idle() directly. >> >> The cle

Re: [PATCH 06/16] ARM idle: delete pm_idle

2013-02-11 Thread Russell King - ARM Linux
On Mon, Feb 11, 2013 at 04:02:30PM +, Catalin Marinas wrote: > On Sun, Feb 10, 2013 at 05:58:13AM +, Len Brown wrote: > > pm_idle() on ARM was a synonym for default_idle(), > > so simply invoke default_idle() directly. > > The clean-up looks fine as we already have an arm_pm_idle but longe

Re: [PATCH 06/16] ARM idle: delete pm_idle

2013-02-11 Thread Catalin Marinas
On Sun, Feb 10, 2013 at 05:58:13AM +, Len Brown wrote: > pm_idle() on ARM was a synonym for default_idle(), > so simply invoke default_idle() directly. The clean-up looks fine as we already have an arm_pm_idle but longer term I was thinking about having a common declaration similar to pm_power

[PATCH 06/16] ARM idle: delete pm_idle

2013-02-09 Thread Len Brown
From: Len Brown pm_idle() on ARM was a synonym for default_idle(), so simply invoke default_idle() directly. Signed-off-by: Len Brown Cc: linux-arm-ker...@lists.infradead.org --- arch/arm/kernel/process.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/arch/ar