On Fri, Jan 20, 2017 at 10:20 PM, Chris Brandt <chris.brandt@xxxxxxxxxxx> wrote: > Signed-off-by: Chris Brandt <chris.brandt@xxxxxxxxxxx> > --- > v2: > * wrap clock lines to avoid 80 char max > * split into 2 separate channel nodes > --- > arch/arm/boot/dts/r7s72100.dtsi | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi > index d5946df..47b2165 100644 > --- a/arch/arm/boot/dts/r7s72100.dtsi > +++ b/arch/arm/boot/dts/r7s72100.dtsi > @@ -505,4 +505,20 @@ > cap-sdio-irq; > status = "disabled"; > }; > + > + ostm0: ostm@fcfec000 { > + compatible = "renesas,r7s72100-ostm", "renesas,ostm"; > + reg = <0xfcfec000 0x30>; > + interrupts = <GIC_SPI 102 IRQ_TYPE_EDGE_RISING>; > + clocks = <&mstp5_clks R7S72100_CLK_OSTM0>; Missing power-domains property: power-domains = <&cpg_clocks>; > + status = "disabled"; > + }; > + > + ostm1: ostm@fcfec400 { > + compatible = "renesas,r7s72100-ostm", "renesas,ostm"; > + reg = <0xfcfec400 0x30>; > + interrupts = <GIC_SPI 103 IRQ_TYPE_EDGE_RISING>; > + clocks = <&mstp5_clks R7S72100_CLK_OSTM1>; Likewise. BTW, they're also missing for the last added device nodes (MMC and SDHI). > + status = "disabled"; > + }; > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- 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