On 28 February 2017 at 21:31, Jon Mason <jon.mason@xxxxxxxxxxxx> wrote: > @@ -70,10 +70,19 @@ > clocks = <&periph_clk>; > }; > > - local-timer@20600 { > + timer@20600 { Is there a reason for this rename? I don't really have preference for any name, it's just we shouldn't keep renaming this depending of personal preferences. It also doesn't seem like adjusting to any schema: bcm63138.dtsi uses local-timer@1e600 bcm-nsp.dtsi uses twd-timer@20600 berlin2cd.dtsi uses local-timer@ad0600 berlin2.dtsi uses local-timer@ad0600 berlin2q.dtsi uses local-timer@ad0600 > compatible = "arm,cortex-a9-twd-timer"; > - reg = <0x20600 0x100>; > - interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>; > + reg = <0x20600 0x20>; > + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(2) | > + IRQ_TYPE_LEVEL_HIGH)>; > + clocks = <&periph_clk>; > + }; The rest looks OK, thanks! :) -- Rafał -- 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