From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH 06/16] ARM idle: delete pm_idle Date: Tue, 12 Feb 2013 14:04:37 -0800 Message-ID: <87pq05i3yi.fsf@linaro.org> References: <1360475903-30007-1-git-send-email-lenb@kernel.org> <2b219d07e0f287c2c713f5465fc8646158fa986e.1360475150.git.len.brown@intel.com> <9838a4aeee256adeaef90efe56df2c9988206982.1360475150.git.len.brown@intel.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <9838a4aeee256adeaef90efe56df2c9988206982.1360475150.git.len.brown@intel.com> (Len Brown's message of "Sun, 10 Feb 2013 00:58:13 -0500") Sender: linux-kernel-owner@vger.kernel.org To: Len Brown Cc: linux-pm@vger.kernel.org, Len Brown , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: linux-pm@vger.kernel.org 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-kernel@lists.infradead.org Looks good to me. Tested this patch along with 15/16 (removes global declaration) on an ARM platform (TI OMAP3) with and without CPUidle, and things continue to work as expected: Reviewed-by: Kevin Hilman Tested-by: Kevin Hilman Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@linaro.org (Kevin Hilman) Date: Tue, 12 Feb 2013 14:04:37 -0800 Subject: [PATCH 06/16] ARM idle: delete pm_idle In-Reply-To: <9838a4aeee256adeaef90efe56df2c9988206982.1360475150.git.len.brown@intel.com> (Len Brown's message of "Sun, 10 Feb 2013 00:58:13 -0500") References: <1360475903-30007-1-git-send-email-lenb@kernel.org> <2b219d07e0f287c2c713f5465fc8646158fa986e.1360475150.git.len.brown@intel.com> <9838a4aeee256adeaef90efe56df2c9988206982.1360475150.git.len.brown@intel.com> Message-ID: <87pq05i3yi.fsf@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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-kernel at lists.infradead.org Looks good to me. Tested this patch along with 15/16 (removes global declaration) on an ARM platform (TI OMAP3) with and without CPUidle, and things continue to work as expected: Reviewed-by: Kevin Hilman Tested-by: Kevin Hilman Kevin