On Tue, Dec 18, 2018 at 9:40 AM Chris Brandt <Chris.Brandt@xxxxxxxxxxx> wrote: > > On Tuesday, December 18, 2018 1, Rob Herring wrote: > > > +/ { > > > + compatible = "renesas,r7s9210"; > > > + interrupt-parent = <&gic>; > > > + #address-cells = <1>; > > > + #size-cells = <1>; > > > + > > > + bsid: chipid@fcfe8004 { > > > > Put all the mmio peripherals under a simple-bus node or nodes. > > Just to be clear on what you mean, basically put all the peripherals > under a 'soc' node like the other Renesas parts? Yes. > > soc { > compatible = "simple-bus"; > interrupt-parent = <&gic>; > > #address-cells = <2>; > #size-cells = <2>; > ranges; > > > Thanks, > Chris >