On Wed, Jun 7, 2017 at 1:57 PM, Wesley Terpstra <wesley@xxxxxxxxxx> wrote: > On Wed, Jun 7, 2017 at 3:13 AM, Mark Rutland <mark.rutland@xxxxxxx> wrote: >>> > +RISC-V Hart-Level Interrupt Controller (HLIC) >>> > +--------------------------------------------- [...] >>> > + plic: interrupt-controller@c000000 { >>> > + #address-cells = <0>; >> >> This can go, given you don't have sub-nodes, nor a #size-cells property. > > The device-tree-specification seems to indicate that this is mandatory > for an interrupt-controller. Or have I understood this wrongly? When > you use interrupts-extended, doesn't it use the address-cells of the > interrupt controller? We should add that size-cells = 0, though. It's only needed if you have an interrupt-map property AIUI. #size-cells should never be needed (unless you have child nodes of this one). Rob -- 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