From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Andersson Subject: Re: [PATCH v4 08/14] ARM: dts: qcom: pm8941: add interrupt controller properties Date: Mon, 14 Jan 2019 16:42:41 -0800 Message-ID: <20190115004240.GL9278@minitux> References: <20190113154716.5145-1-masneyb@onstation.org> <20190113154716.5145-9-masneyb@onstation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190113154716.5145-9-masneyb@onstation.org> Sender: linux-kernel-owner@vger.kernel.org To: Brian Masney Cc: linus.walleij@linaro.org, sboyd@kernel.org, andy.gross@linaro.org, marc.zyngier@arm.com, shawnguo@kernel.org, dianders@chromium.org, linux-gpio@vger.kernel.org, nicolas.dechesne@linaro.org, niklas.cassel@linaro.org, david.brown@linaro.org, robh+dt@kernel.org, mark.rutland@arm.com, thierry.reding@gmail.com, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org List-Id: linux-gpio@vger.kernel.org On Sun 13 Jan 07:47 PST 2019, Brian Masney wrote: > Add interrupt controller properties now that spmi-gpio is a proper > hierarchical IRQ chip. The interrupts property is no longer needed so > remove it. Code was tested on the LG Nexus 5 (hammerhead) phone. > Reviewed-by: Bjorn Andersson > Signed-off-by: Brian Masney > Reviewed-by: Stephen Boyd > --- > Changes since v3: > - None > > Changes since v2: > - Remove interrupt-parent property > > Changes since v1: > - Interrupts are now two cells instead of four cells. > - Drop unnecessary interrupts property. > > arch/arm/boot/dts/qcom-pm8941.dtsi | 38 ++---------------------------- > 1 file changed, 2 insertions(+), 36 deletions(-) > > diff --git a/arch/arm/boot/dts/qcom-pm8941.dtsi b/arch/arm/boot/dts/qcom-pm8941.dtsi > index 2515c5c217ac..e65049e0fb45 100644 > --- a/arch/arm/boot/dts/qcom-pm8941.dtsi > +++ b/arch/arm/boot/dts/qcom-pm8941.dtsi > @@ -64,42 +64,8 @@ > reg = <0xc000>; > gpio-controller; > #gpio-cells = <2>; > - interrupts = <0 0xc0 0 IRQ_TYPE_NONE>, > - <0 0xc1 0 IRQ_TYPE_NONE>, > - <0 0xc2 0 IRQ_TYPE_NONE>, > - <0 0xc3 0 IRQ_TYPE_NONE>, > - <0 0xc4 0 IRQ_TYPE_NONE>, > - <0 0xc5 0 IRQ_TYPE_NONE>, > - <0 0xc6 0 IRQ_TYPE_NONE>, > - <0 0xc7 0 IRQ_TYPE_NONE>, > - <0 0xc8 0 IRQ_TYPE_NONE>, > - <0 0xc9 0 IRQ_TYPE_NONE>, > - <0 0xca 0 IRQ_TYPE_NONE>, > - <0 0xcb 0 IRQ_TYPE_NONE>, > - <0 0xcc 0 IRQ_TYPE_NONE>, > - <0 0xcd 0 IRQ_TYPE_NONE>, > - <0 0xce 0 IRQ_TYPE_NONE>, > - <0 0xcf 0 IRQ_TYPE_NONE>, > - <0 0xd0 0 IRQ_TYPE_NONE>, > - <0 0xd1 0 IRQ_TYPE_NONE>, > - <0 0xd2 0 IRQ_TYPE_NONE>, > - <0 0xd3 0 IRQ_TYPE_NONE>, > - <0 0xd4 0 IRQ_TYPE_NONE>, > - <0 0xd5 0 IRQ_TYPE_NONE>, > - <0 0xd6 0 IRQ_TYPE_NONE>, > - <0 0xd7 0 IRQ_TYPE_NONE>, > - <0 0xd8 0 IRQ_TYPE_NONE>, > - <0 0xd9 0 IRQ_TYPE_NONE>, > - <0 0xda 0 IRQ_TYPE_NONE>, > - <0 0xdb 0 IRQ_TYPE_NONE>, > - <0 0xdc 0 IRQ_TYPE_NONE>, > - <0 0xdd 0 IRQ_TYPE_NONE>, > - <0 0xde 0 IRQ_TYPE_NONE>, > - <0 0xdf 0 IRQ_TYPE_NONE>, > - <0 0xe0 0 IRQ_TYPE_NONE>, > - <0 0xe1 0 IRQ_TYPE_NONE>, > - <0 0xe2 0 IRQ_TYPE_NONE>, > - <0 0xe3 0 IRQ_TYPE_NONE>; > + interrupt-controller; > + #interrupt-cells = <2>; > > boost_bypass_n_pin: boost-bypass { > pins = "gpio21"; > -- > 2.17.2 >