All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pramod Gurav <pramod.gurav@smartplayin.com>
To: Lina Iyer <lina.iyer@linaro.org>
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
Subject: Re: [PATCH v7 5/7] qcom: cpuidle: Add cpuidle driver for QCOM cpus
Date: Mon, 29 Sep 2014 16:01:26 +0530	[thread overview]
Message-ID: <542934FE.1060506@smartplayin.com> (raw)
In-Reply-To: <1411779495-39724-6-git-send-email-lina.iyer@linaro.org>

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.

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


WARNING: multiple messages have this Message-ID (diff)
From: pramod.gurav@smartplayin.com (Pramod Gurav)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 5/7] qcom: cpuidle: Add cpuidle driver for QCOM cpus
Date: Mon, 29 Sep 2014 16:01:26 +0530	[thread overview]
Message-ID: <542934FE.1060506@smartplayin.com> (raw)
In-Reply-To: <1411779495-39724-6-git-send-email-lina.iyer@linaro.org>

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.

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

  parent reply	other threads:[~2014-09-29 10:31 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-27  0:58 [PATCH v7 0/7] QCOM 8974 and 8084 cpuidle driver Lina Iyer
2014-09-27  0:58 ` Lina Iyer
2014-09-27  0:58 ` [PATCH v7 1/7] qcom: spm: Add Subsystem Power Manager driver Lina Iyer
2014-09-27  0:58   ` Lina Iyer
2014-09-27  8:07   ` Pramod Gurav
2014-09-27  8:07     ` Pramod Gurav
2014-09-29 10:29   ` Pramod Gurav
2014-09-29 10:29     ` Pramod Gurav
2014-09-29 23:19   ` Stephen Boyd
2014-09-29 23:19     ` Stephen Boyd
2014-09-30 16:27     ` Lina Iyer
2014-09-30 16:27       ` Lina Iyer
2014-09-30 17:26   ` Kevin Hilman
2014-09-30 17:26     ` Kevin Hilman
2014-09-30 21:18     ` Lina Iyer
2014-09-30 21:18       ` Lina Iyer
2014-09-27  0:58 ` [PATCH v7 2/7] arm: dts: qcom: Add SPM device bindings for 8974 Lina Iyer
2014-09-27  0:58   ` Lina Iyer
2014-09-29 23:02   ` Stephen Boyd
2014-09-29 23:02     ` Stephen Boyd
2014-09-27  0:58 ` [PATCH v7 3/7] arm: dts: qcom: Add SPM device bindings for 8084 Lina Iyer
2014-09-27  0:58   ` Lina Iyer
2014-09-30 17:31   ` Kevin Hilman
2014-09-30 17:31     ` Kevin Hilman
2014-09-27  0:58 ` [PATCH v7 4/7] qcom: pm: Add cpu low power mode functions Lina Iyer
2014-09-27  0:58   ` Lina Iyer
2014-09-27  8:22   ` Pramod Gurav
2014-09-27  8:22     ` Pramod Gurav
2014-09-29 23:37   ` Stephen Boyd
2014-09-29 23:37     ` Stephen Boyd
2014-09-30 16:03     ` Lina Iyer
2014-09-30 16:03       ` Lina Iyer
2014-09-30 17:35       ` Kevin Hilman
2014-09-30 17:35         ` Kevin Hilman
2014-10-02  0:10       ` Stephen Boyd
2014-10-02  0:10         ` Stephen Boyd
2014-10-02  9:50   ` Lorenzo Pieralisi
2014-10-02  9:50     ` Lorenzo Pieralisi
2014-10-06 17:10     ` Lina Iyer
2014-10-06 17:10       ` Lina Iyer
2014-09-27  0:58 ` [PATCH v7 5/7] qcom: cpuidle: Add cpuidle driver for QCOM cpus Lina Iyer
2014-09-27  0:58   ` Lina Iyer
2014-09-27  8:18   ` Pramod Gurav
2014-09-27  8:18     ` Pramod Gurav
2014-09-29 10:31   ` Pramod Gurav [this message]
2014-09-29 10:31     ` Pramod Gurav
2014-09-29 15:04     ` Lina Iyer
2014-09-29 15:04       ` Lina Iyer
2014-09-29 15:31   ` Lorenzo Pieralisi
2014-09-29 15:31     ` Lorenzo Pieralisi
2014-09-29 16:16     ` Lina Iyer
2014-09-29 16:16       ` Lina Iyer
2014-09-29 17:22       ` Lorenzo Pieralisi
2014-09-29 17:22         ` Lorenzo Pieralisi
2014-09-30 17:41     ` Kevin Hilman
2014-09-30 17:41       ` Kevin Hilman
2014-09-30 17:51       ` Nicolas Pitre
2014-09-30 17:51         ` Nicolas Pitre
2014-09-30 18:06         ` Kevin Hilman
2014-09-30 18:06           ` Kevin Hilman
2014-09-30 18:30           ` Nicolas Pitre
2014-09-30 18:30             ` Nicolas Pitre
2014-09-30 18:41             ` Kevin Hilman
2014-09-30 18:41               ` Kevin Hilman
2014-09-30 20:36               ` Lina Iyer
2014-09-30 20:36                 ` Lina Iyer
2014-09-29 23:18   ` Stephen Boyd
2014-09-29 23:18     ` Stephen Boyd
2014-09-30  8:58     ` Lorenzo Pieralisi
2014-09-30  8:58       ` Lorenzo Pieralisi
2014-09-30 15:46       ` Lina Iyer
2014-09-30 15:46         ` Lina Iyer
2014-09-30 15:41     ` Lina Iyer
2014-09-30 15:41       ` Lina Iyer
2014-09-27  0:58 ` [PATCH v7 6/7] arm: dts: qcom: Add idle states device nodes for 8974 Lina Iyer
2014-09-27  0:58   ` Lina Iyer
2014-09-27  0:58 ` [PATCH v7 7/7] arm: dts: qcom: Add idle states device nodes for 8084 Lina Iyer
2014-09-27  0:58   ` Lina Iyer
2014-09-29 12:21 ` [PATCH v7 0/7] QCOM 8974 and 8084 cpuidle driver Pramod Gurav
2014-09-29 12:21   ` Pramod Gurav
2014-09-29 15:05   ` Lina Iyer
2014-09-29 15:05     ` Lina Iyer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=542934FE.1060506@smartplayin.com \
    --to=pramod.gurav@smartplayin.com \
    --cc=daniel.lezcano@linaro.org \
    --cc=galak@codeaurora.org \
    --cc=khilman@linaro.org \
    --cc=lina.iyer@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=msivasub@codeaurora.org \
    --cc=sboyd@codeaurora.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.