From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH 10/13] ARM: dts: r8a7792: add IRQC support Date: Wed, 1 Jun 2016 17:02:39 +0300 Message-ID: References: <13205049.n7pM8utpHF@wasted.cogentembedded.com> <3573091.BUvyGW3hVt@wasted.cogentembedded.com> <20160601011813.GJ20527@verge.net.au> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160601011813.GJ20527@verge.net.au> Sender: linux-renesas-soc-owner@vger.kernel.org To: Simon Horman Cc: linux-renesas-soc@vger.kernel.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, devicetree@vger.kernel.org, magnus.damm@gmail.com, linux@arm.linux.org.uk, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On 06/01/2016 04:18 AM, Simon Horman wrote: >> Describe the IRQC interrupt controller in the R8A7792 device tree. >> >> Signed-off-by: Sergei Shtylyov >> >> --- >> arch/arm/boot/dts/r8a7792.dtsi | 13 +++++++++++++ >> 1 file changed, 13 insertions(+) >> >> Index: renesas/arch/arm/boot/dts/r8a7792.dtsi >> =================================================================== >> --- renesas.orig/arch/arm/boot/dts/r8a7792.dtsi >> +++ renesas/arch/arm/boot/dts/r8a7792.dtsi >> @@ -62,6 +62,19 @@ >> IRQ_TYPE_LEVEL_HIGH)>; >> }; >> >> + irqc: interrupt-controller@e61c0000 { > > Could you consider using irqc0 as the node name for consistency with the > DT of other R-Car Gen2 SoCs? Frankly speaking I didn't get that name indexing thing at all. Could you share the reasoning behind it? > What follows looks correct to me. Thank you. :-) MBR, Sergei