Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752256AbcCYL0g (ORCPT ); Fri, 25 Mar 2016 07:26:36 -0400 Received: from mail-wm0-f44.google.com ([74.125.82.44]:35902 "EHLO mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751127AbcCYL0c (ORCPT ); Fri, 25 Mar 2016 07:26:32 -0400 Subject: Re: [PATCH 0/4] ARM: cpuidle: use const and __initconst for cpuidle_ops To: Jisheng Zhang , linux@arm.linux.org.uk, mark.rutland@arm.com, lorenzo.pieralisi@arm.com, andy.gross@linaro.org, david.brown@linaro.org References: <1458657763-2561-1-git-send-email-jszhang@marvell.com> Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org From: Daniel Lezcano Message-ID: <56F52063.6020002@linaro.org> Date: Fri, 25 Mar 2016 12:26:27 +0100 User-Agent: Mozilla/5.0 (X11; Linux i686; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <1458657763-2561-1-git-send-email-jszhang@marvell.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1114 Lines: 31 On 03/22/2016 03:42 PM, Jisheng Zhang wrote: > These trivial patches are similar as Masahiro posted in[1]. The main > purpose is let cpuidle_ops structure be constified and replace > "__initdata" with "__initconst". > > [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2015-August/365826.html > > Jisheng Zhang (4): > ARM: cpuidle: add const qualifier to cpuidle_ops member in structures > ARM: cpuidle: constify return value of arm_cpuidle_get_ops() > soc: qcom: spm: use const and __initconst for qcom_cpuidle_ops > drivers: firmware: psci: use const and __initconst for > psci_cpuidle_ops > > arch/arm/include/asm/cpuidle.h | 2 +- > arch/arm/kernel/cpuidle.c | 6 +++--- > drivers/firmware/psci.c | 2 +- > drivers/soc/qcom/spm.c | 2 +- > 4 files changed, 6 insertions(+), 6 deletions(-) Sounds reasonable. -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog