Am Dienstag, 8. November 2016, 20:31:55 schrieb Andy Yan: > Hi Heiko: > > On 2016年11月04日 16:00, Heiko Stuebner wrote: > > Am Donnerstag, 3. November 2016, 20:40:48 CET schrieb Andy Yan: > >> + gic: interrupt-controller@32010000 { > >> + compatible = "arm,cortex-a15-gic"; > > > > compatible = "arm,gic-400"; ? > > > >> + interrupt-controller; > >> + #interrupt-cells = <3>; > >> + #address-cells = <0>; > >> + > >> + reg = <0x32011000 0x1000>, > >> + <0x32012000 0x1000>; > > > > please provide all 4 register areas and also the interrupt ( > > I only found 2 register areas in our rockchip linux 3.10 source > code. And haven't found the interrupt. From the arm,gic bindings, the > interrupt property is optional. So am not sure if we > really need it here. Devicetree is a hardware description, so it's not a factor if we "need" it but only if it is present in the hardware. And we really want this information to be complete, as these additional areas are necessary if someone wants to use the virtualization extensions the cortext-A7 does contain. The gic is a very standard component and the gic400 used here should definitly have those two additional areas as well as the interrupt. I think the memory areas are pretty standard and should be for the rk1108: reg = <0x32011000 0x1000>, <0x32012000 0x1000>, <0x32014000 0x2000>, <0x32016000 0x2000>; The TRM talks about 128 SPI and 3 PPI interrupts but the irq-list does not contain them, so this seems to be an error in the TRM, as the gic interrupt should be one of those PPI interrupts. Heiko -- 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