On Tue, May 19, 2015 at 09:09:06AM +0200, Heiko Schocher wrote: > >>@@ -0,0 +1,124 @@ > >>+/* > >>+ * support fot the imx6 based aristainetos2 board > >>+ * > >>+ * Copyright (C) 2015 Heiko Schocher <hs@xxxxxxx> > >>+ * > >>+ * This program is free software; you can redistribute it and/or modify > >>+ * it under the terms of the GNU General Public License version 2 as > >>+ * published by the Free Software Foundation. > >>+ * > >>+ */ > > > >GPL/X11 dual licence is suggested for new DTS files. See > >arch/arm/boot/dts/imx6sl-warp.dts for an example. > > Ok, I ask the customer, if this is OK for him ... The dual license get non-Linux users be able to use the DTS file. [...] > >>+&i2c3 { > >>+ atmel: atmel_mxt_ts@4b { > > > >Node name should be generic. > > So, what name do you prefer? The node name with the example in bindings doc Documentation/devicetree/bindings/input/atmel,maxtouch.txt seems perfectly fine to me. [...] > >>+&i2c3 { > >>+ atmel: atmel_mxt_ts@4d { > >>+ compatible = "atmel,maxtouch"; > >>+ reg = <0x4d>; > >>+ interrupt-parent = <&gpio2>; > >>+ interrupts = <9 8>; > >>+ }; > > > >It seems the touch device is identical between > >imx6dl-aristainetos2_4.dts and imx6dl-aristainetos2_7.dts. Why don't > >you put it into imx6qdl-aristainetos2.dtsi? > > They have different i2c addresses ... Okay, I missed that. Shawn -- 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