[PATCHv2 3/3] picoxcell: add the DTS for the PC7302 board

[21cnbao@gmail.com (Barry Song)]

2011/9/15 Jamie Iles <jamie@jamieiles.com>:
> On Thu, Sep 15, 2011 at 08:32:02AM -0300, Benjamin Herrenschmidt wrote:
>> On Wed, 2011-08-24 at 15:41 +0100, Jamie Iles wrote:
>>
>> > + ? chosen {
>> > + ? ? ? ? ? bootargs = "console=ttyS0,115200 earlyprintk loglevel=9 root=ubi0:rootfs rw rootfstype=ubifs ubi.mtd=5,2048";
>> > + ? ? ? ? ? linux,stdout-path = &uart0;
>> > + ? };
>>
>> Hrm... we don't normally put the bootargs in the device-tree.
>>
>> Either you have a way to pass it from a previous firmware (which
>> can then slap it into the device-tree at runtime), or you
>> can have a way to compile it in the kernel image but the device-tree
>> isn't the right place for it.
>
> OK, that's fair enough. ?A few other ARM platforms (tegra, prima2 and
> zynq) have bootargs in the chosen node and that's where I got it from,
> but our bootloader has fdt support so this can easily be removed.

some powerpc platforms also do that:

asp834x-redboot.dts:		bootargs = "console=ttyS0,38400
root=/dev/mtdblock3 rootfstype=jffs2";
gamecube.dts:		bootargs = "root=/dev/gcnsda2 rootwait udbg-immortal";
prpmc2800.dts:		bootargs = "ip=on";
rainier.dts:		bootargs = "console=ttyS0,115200";
sequoia.dts:		bootargs = "console=ttyS0,115200";
virtex440-ml507.dts:		bootargs = "console=ttyS0 root=/dev/ram";
virtex440-ml510.dts:		bootargs = "console=ttyS0 root=/dev/ram";
wii.dts:		bootargs = "root=/dev/mmcblk0p2 rootwait udbg-immortal";

if we have no bootargs in bootloader, kernel will use that one in DT.

>
> Thanks,
>
> Jamie

-barry