Hi Dinh, On Thu, Oct 03, 2013 at 09:58:41AM -0500, Dinh Nguyen wrote: > On Wed, 2013-10-02 at 09:35 +0200, Steffen Trumtrar wrote: > > +/include/ "socfpga_cyclone5.dtsi" > > + > > +/ { > > + model = "Altera SOCFPGA Cyclone V SoC Development Kit"; > > + compatible = "altr,socfpga-cyclone5", "altr,socfpga"; > > + > > + chosen { > > + bootargs = "console=ttyS0,57600"; > > Thanks for doing this. It looks good. While you're at it can you make > this 115200? 57600 was early in the bring-up stage. > I can also completely remove this. The question is: does u-boot need this line or does it add this line itself? We added basic support for SoCFPGA to barebox (well, it will go live with the october release AFAIK. With SoCKit and SoCrates boards) and we normally do not need to put the bootargs in the DT. So, I would be okay to remove this line OR if u-boot uses it, change it to 115200. What do you think? > > + }; > > + > > + memory { > > + name = "memory"; > > + device_type = "memory"; > > + reg = <0x0 0x40000000>; /* 1GB */ > > + }; > > + > > + aliases { > > + /* this allow the ethaddr uboot environmnet variable contents > > + * to be added to the gmac1 device tree blob. > > + */ > > + ethernet0 = &gmac1; > > + }; > > +}; > > With the change to 115200, > > Acked-by: Dinh Nguyen <dinguyen@xxxxxxxxxx> > Thanks. Regards, Steffen -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- 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