From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vladimir Zapolskiy Subject: Re: [PATCH v3 3/4] arm: dts: lpc32xx: reparent SIC1 and SIC2 interrupts from MIC Date: Tue, 26 Apr 2016 21:38:50 +0300 Message-ID: <571FB5BA.1010903@mleia.com> References: <1461546041-25713-1-git-send-email-vz@mleia.com> <1461618143-12310-1-git-send-email-vz@mleia.com> <1461673563.18338.20.camel@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1461673563.18338.20.camel@localhost> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sylvain Lemieux Cc: Rob Herring , Mark Rutland , Russell King , Arnd Bergmann , Roland Stigge , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Sylvain, On 26.04.2016 15:26, Sylvain Lemieux wrote: > Hi Vladimir, > > On Tue, 2016-04-26 at 00:02 +0300, Vladimir Zapolskiy wrote: >> The change adds separate device nodes for SIC1 and SIC2 interrupt >> controllers and reparents all defined SIC1 and SIC2 interrupt >> producers to the correspondent interrupt controller, this is needed to >> perform switching to a new LPC32xx MIC/SIC interrupt controller driver. >> >> Signed-off-by: Vladimir Zapolskiy >> --- >> Changes from v2 to v3: >> * fix io region size for MIC controller after moving SIC1/SIC2 >> controllers out, >> * move MIC 'reg' property up to follow 'compatible' property. >> >> Changes from v1 to v2: >> * squashed two step change in lpc32xx.dtsi - new irqchip driver can be >> used (i.e. board boots with access to userspace) with old DTS file, >> so there is no need to fix DTS firstly and then switch to a new driver, >> * removed "interrupt-controller-name" and "wakeup-sources" properties >> from all interrupt controllers, >> * removed outdated comment. >> >> arch/arm/boot/dts/lpc32xx.dtsi | 60 +++++++++++++++++++++++++++++------------- >> 1 file changed, 42 insertions(+), 18 deletions(-) >> > Please use capital letter for the word ARM in the patch title. right, I'll change it on application, thank you for review. > Acked-by: Sylvain Lemieux > -- With best wishes, Vladimir -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: vz@mleia.com (Vladimir Zapolskiy) Date: Tue, 26 Apr 2016 21:38:50 +0300 Subject: [PATCH v3 3/4] arm: dts: lpc32xx: reparent SIC1 and SIC2 interrupts from MIC In-Reply-To: <1461673563.18338.20.camel@localhost> References: <1461546041-25713-1-git-send-email-vz@mleia.com> <1461618143-12310-1-git-send-email-vz@mleia.com> <1461673563.18338.20.camel@localhost> Message-ID: <571FB5BA.1010903@mleia.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Sylvain, On 26.04.2016 15:26, Sylvain Lemieux wrote: > Hi Vladimir, > > On Tue, 2016-04-26 at 00:02 +0300, Vladimir Zapolskiy wrote: >> The change adds separate device nodes for SIC1 and SIC2 interrupt >> controllers and reparents all defined SIC1 and SIC2 interrupt >> producers to the correspondent interrupt controller, this is needed to >> perform switching to a new LPC32xx MIC/SIC interrupt controller driver. >> >> Signed-off-by: Vladimir Zapolskiy >> --- >> Changes from v2 to v3: >> * fix io region size for MIC controller after moving SIC1/SIC2 >> controllers out, >> * move MIC 'reg' property up to follow 'compatible' property. >> >> Changes from v1 to v2: >> * squashed two step change in lpc32xx.dtsi - new irqchip driver can be >> used (i.e. board boots with access to userspace) with old DTS file, >> so there is no need to fix DTS firstly and then switch to a new driver, >> * removed "interrupt-controller-name" and "wakeup-sources" properties >> from all interrupt controllers, >> * removed outdated comment. >> >> arch/arm/boot/dts/lpc32xx.dtsi | 60 +++++++++++++++++++++++++++++------------- >> 1 file changed, 42 insertions(+), 18 deletions(-) >> > Please use capital letter for the word ARM in the patch title. right, I'll change it on application, thank you for review. > Acked-by: Sylvain Lemieux > -- With best wishes, Vladimir