linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: bjorn.andersson@linaro.org (Bjorn Andersson)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] ARM: dts: rename MSM8660/APQ8060 pmicintc to pm8058
Date: Thu, 3 Nov 2016 14:50:46 -0700	[thread overview]
Message-ID: <20161103215046.GT25787@tuxbot> (raw)
In-Reply-To: <1478164390-21613-1-git-send-email-linus.walleij@linaro.org>

On Thu 03 Nov 02:13 PDT 2016, Linus Walleij wrote:

> The name "pmicintc" is ambiguous: there is a second power
> management IC named PM8901 on these systems, and it is also
> an interrupt controller. To make things clear, just name the
> node alias "pm8058", this in unambigous and has all information
> we need.
> 
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Regards,
Bjorn

> ---
>  arch/arm/boot/dts/qcom-apq8060-dragonboard.dts |  2 +-
>  arch/arm/boot/dts/qcom-msm8660-surf.dts        |  2 +-
>  arch/arm/boot/dts/qcom-msm8660.dtsi            | 12 ++++++------
>  3 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8060-dragonboard.dts b/arch/arm/boot/dts/qcom-apq8060-dragonboard.dts
> index 4b8872cc8bf9..4a532ddab53a 100644
> --- a/arch/arm/boot/dts/qcom-apq8060-dragonboard.dts
> +++ b/arch/arm/boot/dts/qcom-apq8060-dragonboard.dts
> @@ -412,7 +412,7 @@
>  				 * The second interrupt is the PME interrupt
>  				 * for network wakeup, connected to the TLMM.
>  				 */
> -				interrupts-extended = <&pmicintc 198 IRQ_TYPE_EDGE_FALLING>,
> +				interrupts-extended = <&pm8058 198 IRQ_TYPE_EDGE_FALLING>,
>  						    <&tlmm 29 IRQ_TYPE_EDGE_RISING>;
>  				reset-gpios = <&tlmm 30 GPIO_ACTIVE_LOW>;
>  				vdd33a-supply = <&dragon_veth>;
> diff --git a/arch/arm/boot/dts/qcom-msm8660-surf.dts b/arch/arm/boot/dts/qcom-msm8660-surf.dts
> index 23de764558ab..1adc04978a47 100644
> --- a/arch/arm/boot/dts/qcom-msm8660-surf.dts
> +++ b/arch/arm/boot/dts/qcom-msm8660-surf.dts
> @@ -48,7 +48,7 @@
>  	};
>  };
>  
> -&pmicintc {
> +&pm8058 {
>  	keypad at 148 {
>  		linux,keymap = <
>  			MATRIX_KEY(0, 0, KEY_FN_F1)
> diff --git a/arch/arm/boot/dts/qcom-msm8660.dtsi b/arch/arm/boot/dts/qcom-msm8660.dtsi
> index 4d828f810746..91c9a62ae725 100644
> --- a/arch/arm/boot/dts/qcom-msm8660.dtsi
> +++ b/arch/arm/boot/dts/qcom-msm8660.dtsi
> @@ -163,7 +163,7 @@
>  			reg = <0x500000 0x1000>;
>  			qcom,controller-type = "pmic-arbiter";
>  
> -			pmicintc: pmic at 0 {
> +			pm8058: pmic at 0 {
>  				compatible = "qcom,pm8058";
>  				interrupt-parent = <&tlmm>;
>  				interrupts = <88 8>;
> @@ -176,7 +176,7 @@
>  					compatible = "qcom,pm8058-gpio",
>  						     "qcom,ssbi-gpio";
>  					reg = <0x150>;
> -					interrupt-parent = <&pmicintc>;
> +					interrupt-parent = <&pm8058>;
>  					interrupts = <192 IRQ_TYPE_NONE>,
>  						     <193 IRQ_TYPE_NONE>,
>  						     <194 IRQ_TYPE_NONE>,
> @@ -232,7 +232,7 @@
>  					reg = <0x50>;
>  					gpio-controller;
>  					#gpio-cells = <2>;
> -					interrupt-parent = <&pmicintc>;
> +					interrupt-parent = <&pm8058>;
>  					interrupts =
>  					<128 IRQ_TYPE_NONE>,
>  					<129 IRQ_TYPE_NONE>,
> @@ -251,7 +251,7 @@
>  				pwrkey at 1c {
>  					compatible = "qcom,pm8058-pwrkey";
>  					reg = <0x1c>;
> -					interrupt-parent = <&pmicintc>;
> +					interrupt-parent = <&pm8058>;
>  					interrupts = <50 1>, <51 1>;
>  					debounce = <15625>;
>  					pull-up;
> @@ -260,7 +260,7 @@
>  				keypad at 148 {
>  					compatible = "qcom,pm8058-keypad";
>  					reg = <0x148>;
> -					interrupt-parent = <&pmicintc>;
> +					interrupt-parent = <&pm8058>;
>  					interrupts = <74 1>, <75 1>;
>  					debounce = <15>;
>  					scan-delay = <32>;
> @@ -270,7 +270,7 @@
>  				rtc at 1e8 {
>  					compatible = "qcom,pm8058-rtc";
>  					reg = <0x1e8>;
> -					interrupt-parent = <&pmicintc>;
> +					interrupt-parent = <&pm8058>;
>  					interrupts = <39 1>;
>  					allow-set-time;
>  				};
> -- 
> 2.7.4
> 

      parent reply	other threads:[~2016-11-03 21:50 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-03  9:13 [PATCH 1/3] ARM: dts: rename MSM8660/APQ8060 pmicintc to pm8058 Linus Walleij
2016-11-03  9:13 ` [PATCH 2/3] ARM: dts: reference PM8058 as IRQ parent Linus Walleij
2016-11-03 21:53   ` Bjorn Andersson
2016-11-03  9:13 ` [PATCH 3/3] ARM: dts: Add gyro and accel to APQ8060 Dragonboard Linus Walleij
2016-11-03 21:54   ` Bjorn Andersson
2016-11-03 21:50 ` Bjorn Andersson [this message]

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=20161103215046.GT25787@tuxbot \
    --to=bjorn.andersson@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).