Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
Date:   Fri, 30 Dec 2022 09:16:26 +0000
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
From:   "Yajun Deng" <yajun.deng@linux.dev>
Message-ID: <fc13e5c0dba372ddb1435c959e7dc41f@linux.dev>
Subject: Re: [PATCH] ARM: Remove redundant arch_cpu_idle_prepare()
To:     rmk+kernel@armlinux.org.uk, keescook@chromium.org,
        gregkh@linuxfoundation.org, Jason@zx2c4.com, djwong@kernel.org,
        ebiederm@xmission.com, thunder.leizhen@huawei.com, ardb@kernel.org,
        wangkefeng.wang@huawei.com
Cc:     linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
In-Reply-To: <20221201031306.2782204-1-yajun.deng@linux.dev>
References: <20221201031306.2782204-1-yajun.deng@linux.dev>
Precedence: bulk

December 1, 2022 11:13 AM, "Yajun Deng" <yajun.deng@linux.dev> wrote:=0A=
=0A> arch_cpu_idle_prepare() was only called by cpu_startup_entry() and i=
t is=0A> just a wrapper for local_fiq_enable().=0A> =0A> The local_fiq_en=
able() was already called before cpu_startup_entry().=0A> =0A> So remove =
redundant arch_cpu_idle_prepare().=0A> =0A> Signed-off-by: Yajun Deng <ya=
jun.deng@linux.dev>=0A> ---=0A> arch/arm/kernel/process.c | 5 -----=0A> 1=
 file changed, 5 deletions(-)=0A> =0A> diff --git a/arch/arm/kernel/proce=
ss.c b/arch/arm/kernel/process.c=0A> index f811733a8fc5..e3f490ab95f4 100=
644=0A> --- a/arch/arm/kernel/process.c=0A> +++ b/arch/arm/kernel/process=
.c=0A> @@ -81,11 +81,6 @@ void arch_cpu_idle(void)=0A> raw_local_irq_enab=
le();=0A> }=0A> =0A> -void arch_cpu_idle_prepare(void)=0A> -{=0A> - local=
_fiq_enable();=0A> -}=0A> -=0A> void arch_cpu_idle_enter(void)=0A> {=0A> =
ledtrig_cpu(CPU_LED_IDLE_START);=0A> -- =0A> 2.25.1=0A=0APing.