On Mon, Aug 06, 2018 at 02:59:48PM -0600, Rob Herring wrote: > > > +Required properties: > > > +- compatible : "sifive,plic0" > > > +- #address-cells : should be <0> > > > +- #interrupt-cells : should be <1> > > > +- interrupt-controller : Identifies the node as an interrupt controller > > > +- reg : Should contain 1 register range (address and length) > > > > The one in the real device tree has two entries. > > reg = <0x00000000 0x0c000000 0x00000000 0x04000000>; > > > > Is it intentional or just incorrect entry left over from earlier days? > > > > + reg = <0xc000000 0x4000000>; > > Looks to me like one has #size-cells and #address-cells set to 2 and > the example is using 1. Yes. And it seems like the real life device tree is simply bogus. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html