Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757661AbcCXNBS (ORCPT ); Thu, 24 Mar 2016 09:01:18 -0400 Received: from mail-lf0-f68.google.com ([209.85.215.68]:36731 "EHLO mail-lf0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757445AbcCXNBJ (ORCPT ); Thu, 24 Mar 2016 09:01:09 -0400 MIME-Version: 1.0 In-Reply-To: <1458796038-6062-1-git-send-email-jszhang@marvell.com> References: <1458796038-6062-1-git-send-email-jszhang@marvell.com> Date: Thu, 24 Mar 2016 14:01:02 +0100 X-Google-Sender-Auth: 1hZQsMpASUZ4Abaq6fxD06Tj63c Message-ID: Subject: Re: [PATCH] cpuidle: arm: make enter idle operation a bit more efficient From: "Rafael J. Wysocki" To: Jisheng Zhang Cc: "Rafael J. Wysocki" , Daniel Lezcano , "linux-pm@vger.kernel.org" , Linux Kernel Mailing List , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1816 Lines: 54 On Thu, Mar 24, 2016 at 6:07 AM, Jisheng Zhang wrote: > Currently, entering idle need to check the idx every time to choose the > real entering idle routine. But this check could be avoided by pointing > the idle enter function pointer of each idle states to the routines > suitable for each states directly. > > Signed-off-by: Jisheng Zhang This looks reasonable to me, but I need an ACK from the maintainer of this code. > --- > drivers/cpuidle/cpuidle-arm.c | 14 ++++++++------ > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/drivers/cpuidle/cpuidle-arm.c b/drivers/cpuidle/cpuidle-arm.c > index 545069d..48a620f 100644 > --- a/drivers/cpuidle/cpuidle-arm.c > +++ b/drivers/cpuidle/cpuidle-arm.c > @@ -23,6 +23,13 @@ > > #include "dt_idle_states.h" > > +static int arm_enter_wfi_state(struct cpuidle_device *dev, > + struct cpuidle_driver *drv, int idx) > +{ > + cpu_do_idle(); > + return 0; > +} > + > /* > * arm_enter_idle_state - Programs CPU to enter the specified state > * > @@ -38,11 +45,6 @@ static int arm_enter_idle_state(struct cpuidle_device *dev, > { > int ret; > > - if (!idx) { > - cpu_do_idle(); > - return idx; > - } > - > ret = cpu_pm_enter(); > if (!ret) { > /* > @@ -69,7 +71,7 @@ static struct cpuidle_driver arm_idle_driver = { > * handler for idle state index 0. > */ > .states[0] = { > - .enter = arm_enter_idle_state, > + .enter = arm_enter_wfi_state, > .exit_latency = 1, > .target_residency = 1, > .power_usage = UINT_MAX, > --