From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lina Iyer Subject: Re: [PATCH v7 5/7] qcom: cpuidle: Add cpuidle driver for QCOM cpus Date: Mon, 29 Sep 2014 09:04:41 -0600 Message-ID: <20140929150441.GB1170@ilina-mac.local> References: <1411779495-39724-1-git-send-email-lina.iyer@linaro.org> <1411779495-39724-6-git-send-email-lina.iyer@linaro.org> <542934FE.1060506@smartplayin.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Return-path: Content-Disposition: inline In-Reply-To: <542934FE.1060506@smartplayin.com> Sender: linux-arm-msm-owner@vger.kernel.org To: Pramod Gurav Cc: daniel.lezcano@linaro.org, khilman@linaro.org, sboyd@codeaurora.org, galak@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, lorenzo.pieralisi@arm.com, msivasub@codeaurora.org List-Id: linux-pm@vger.kernel.org On Mon, Sep 29 2014 at 04:27 -0600, Pramod Gurav wrote: >Hi Lina > >One sections mismatch warning here as well: > >WARNING: drivers/cpuidle/built-in.o(.text+0x2740): Section mismatch in >reference from the function qcom_cpuidle_probe() to the (unknown >reference) .init.rodata:(unknown) >The function qcom_cpuidle_probe() references >the (unknown reference) __initconst (unknown). >This is often because qcom_cpuidle_probe lacks a __initconst >annotation or the annotation of (unknown) is wrong. > Thanks, will fix it. >On Saturday 27 September 2014 06:28 AM, Lina Iyer wrote: >> Add cpuidle driver interface to allow cpus to go into C-States. Use the >> cpuidle DT interface common across ARM architectures to provide the >> C-State information to the cpuidle framework. >> >> > >.. > >> +}; >> + >> +static const struct of_device_id qcom_idle_state_match[] __initconst = { >This is causing it. > >> + { .compatible = "qcom,idle-state-wfi", .data = qcom_lpm_enter_wfi }, >> + { .compatible = "qcom,idle-state-spc", .data = qcom_lpm_enter_spc }, >> + { }, >> +}; >> + >Thanks >Pramod >