Hello, I have a couple of minor comments. On Thu, Feb 07, 2013 at 02:11:37PM +0000, Anil Kumar wrote: > DevKit8000 is a beagle board clone from Timll, sold by > armkits.com. The DevKit8000 has RS232 serial port, LCD, DVI-D, > S-Video, Ethernet, SD/MMC, keyboard, camera, SPI, I2C, USB and > JTAG interface. > > This patch adds the basic DT support for devkit8000. At this time, Information > of twl4030, MMC1, I2C1, leds and there pim mux information are added. > > Signed-off-by: Anil Kumar <anilk4.v@xxxxxxxxx> > Tested-by: Thomas Weber <thomas@xxxxxxxxxxx> > --- > > -This patch is based on top of kernel 3.8-rc5. > > -Tested on Devkit8000. > [...] > diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts > new file mode 100644 > index 0000000..9864fd7 > --- /dev/null > +++ b/arch/arm/boot/dts/omap3-devkit8000.dts > @@ -0,0 +1,125 @@ > +/* > + * Anil Kumar <anilk4.v@xxxxxxxxx> > + * > + * 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. > + */ > +/dts-v1/; > + > +/include/ "omap3.dtsi" > +/ { > + model = "TI OMAP3 Devkit8000"; Should this not be "TimLL Devkit8000" ? > + compatible = "ti,omap3-devkit8000", "ti,omap3"; "timll,devkit8000" ? [...] Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html