Re: [PATCH 6/6] arm64: dts: renesas: r8a77965: Add #interrupt-cells property
On Fri, Feb 23, 2018 at 11:06:07AM +0100, jacopo mondi wrote: > Hi Geert, > > On Fri, Feb 23, 2018 at 10:27:16AM +0100, Geert Uytterhoeven wrote: > > On Thu, Feb 22, 2018 at 8:06 PM, Jacopo Mondi> > wrote: > > > Add "#interrupt-cells" property and "interrupt-controller" label to > > > "interrupt-controller@e61c" device node. > > > > > > This silences the following DTC compiler warnings: > > > Warning (interrupts_property): Missing interrupt-controller or > > > interrupt-map property in /soc/interrupt-controller@e61c > > > Warning (interrupts_property): Missing #interrupt-cells in > > > interrupt-parent /soc/interrupt-controller@e61c000 > > > > > > Signed-off-by: Jacopo Mondi > > > > Reviewed-by: Geert Uytterhoeven > > > > > --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi > > > +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > > > @@ -316,6 +316,8 @@ > > > }; > > > > > > intc_ex: interrupt-controller@e61c { > > > + #interrupt-cells = <2>; > > > + interrupt-controller; > > > reg = <0 0xe61c 0 0x200>; > > > /* placeholder */ > > > > (bikeshedding) Should the placeholder be at the top of the node? > > I would like it more too, but in M3-W dtsi: > > pciec0: pcie@fe00 { > reg = <0 0xfe00 0 0x8>; > /* placeholder */ > }; > > So I kept using the same style everywhere here. > > I can change it everywhere in this series to have placeholder at the > beginning of the node definition if you agree with that! Either way is fine by me. But if we do change it lets change it everywhere.
Re: [PATCH 6/6] arm64: dts: renesas: r8a77965: Add #interrupt-cells property
Hi Geert, On Fri, Feb 23, 2018 at 10:27:16AM +0100, Geert Uytterhoeven wrote: > On Thu, Feb 22, 2018 at 8:06 PM, Jacopo Mondi> wrote: > > Add "#interrupt-cells" property and "interrupt-controller" label to > > "interrupt-controller@e61c" device node. > > > > This silences the following DTC compiler warnings: > > Warning (interrupts_property): Missing interrupt-controller or > > interrupt-map property in /soc/interrupt-controller@e61c > > Warning (interrupts_property): Missing #interrupt-cells in > > interrupt-parent /soc/interrupt-controller@e61c000 > > > > Signed-off-by: Jacopo Mondi > > Reviewed-by: Geert Uytterhoeven > > > --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi > > +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > > @@ -316,6 +316,8 @@ > > }; > > > > intc_ex: interrupt-controller@e61c { > > + #interrupt-cells = <2>; > > + interrupt-controller; > > reg = <0 0xe61c 0 0x200>; > > /* placeholder */ > > (bikeshedding) Should the placeholder be at the top of the node? I would like it more too, but in M3-W dtsi: pciec0: pcie@fe00 { reg = <0 0xfe00 0 0x8>; /* placeholder */ }; So I kept using the same style everywhere here. I can change it everywhere in this series to have placeholder at the beginning of the node definition if you agree with that! Thanks j > > > }; > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- > ge...@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like > that. > -- Linus Torvalds
Re: [PATCH 6/6] arm64: dts: renesas: r8a77965: Add #interrupt-cells property
On Thu, Feb 22, 2018 at 8:06 PM, Jacopo Mondiwrote: > Add "#interrupt-cells" property and "interrupt-controller" label to > "interrupt-controller@e61c" device node. > > This silences the following DTC compiler warnings: > Warning (interrupts_property): Missing interrupt-controller or > interrupt-map property in /soc/interrupt-controller@e61c > Warning (interrupts_property): Missing #interrupt-cells in > interrupt-parent /soc/interrupt-controller@e61c000 > > Signed-off-by: Jacopo Mondi Reviewed-by: Geert Uytterhoeven > --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > @@ -316,6 +316,8 @@ > }; > > intc_ex: interrupt-controller@e61c { > + #interrupt-cells = <2>; > + interrupt-controller; > reg = <0 0xe61c 0 0x200>; > /* placeholder */ (bikeshedding) Should the placeholder be at the top of the node? > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds