From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752077AbcF0RRm (ORCPT ); Mon, 27 Jun 2016 13:17:42 -0400 Received: from mga04.intel.com ([192.55.52.120]:64386 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751784AbcF0RRk (ORCPT ); Mon, 27 Jun 2016 13:17:40 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,537,1459839600"; d="scan'208";a="1006090181" Message-ID: <1467047944.8970.79.camel@linux.intel.com> Subject: Re: [PATCH v2 1/3] intel_pstate: Fix incorrect placement of __initdata From: Srinivas Pandruvada To: Jisheng Zhang , lenb@kernel.org, rjw@rjwysocki.net, viresh.kumar@linaro.org Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org Date: Mon, 27 Jun 2016 10:19:04 -0700 In-Reply-To: <1467022038-924-2-git-send-email-jszhang@marvell.com> References: <1467022038-924-1-git-send-email-jszhang@marvell.com> <1467022038-924-2-git-send-email-jszhang@marvell.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.5.2 (3.18.5.2-1.fc23) Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2016-06-27 at 18:07 +0800, Jisheng Zhang wrote: > __initdata should be placed between the variable name and equal sign > (if there is) for the variable to be placed in the intended section. > > Signed-off-by: Jisheng Zhang > Acked-by: Viresh Kumar Acked-by: Srinivas Pandruvada > --- >  drivers/cpufreq/intel_pstate.c | 6 +++--- >  1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/cpufreq/intel_pstate.c > b/drivers/cpufreq/intel_pstate.c > index fe9dc17..44099e9 100644 > --- a/drivers/cpufreq/intel_pstate.c > +++ b/drivers/cpufreq/intel_pstate.c > @@ -1574,9 +1574,9 @@ static struct cpufreq_driver > intel_pstate_driver = { >   .name = "intel_pstate", >  }; >   > -static int __initdata no_load; > -static int __initdata no_hwp; > -static int __initdata hwp_only; > +static int no_load __initdata; > +static int no_hwp __initdata; > +static int hwp_only __initdata; >  static unsigned int force_load; >   >  static int intel_pstate_msrs_not_valid(void)