Hi Chris, On Thu, Aug 25, 2016 at 5:31 PM, Chris Brandt <chris.brandt@xxxxxxxxxxx> wrote: > Add the initial device tree for the RZ/A1 based development board (RSK). Thanks for your patch! > Signed-off-by: Chris Brandt <chris.brandt@xxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- /dev/null > +++ b/arch/arm/boot/dts/r7s72100-rskrza1.dts > + compatible = "renesas,rskrza1", "renesas,r7s72100"; scripts/checkpatch.pl says: DT compatible string "renesas,rskrza1" appears un-documented -- check ./Documentation/devicetree/bindings/ Please send a follow-up patch to add this compatible value to Documentation/devicetree/bindings/arm/shmobile.txt. > +&scif2 { > + status = "okay"; > +}; > + Please drop the trailing empty line. 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