Thanks for the comments Prafulla! I'll update the patch according to them, but have some questions:
On Fri, 11 Sep 2009 03:43:04 -0700 Prafulla Wadaskar <prafu...@marvell.com> wrote: > > +#define CONFIG_ENV_SIZE 0x20000 /* 128k */ > > +#define CONFIG_ENV_ADDR 0xa0000 > > +#define CONFIG_ENV_OFFSET 0xa0000 /* env starts here */ > Do you really need this address? the u-boot binary is within 256kb, at the > most it can go till 512kb This is where the default environment (for the U-boot shipped by Marvell) resides. Like on Sheevaplug, it uses 4-bit ECC, so current U-boot won't be able to read it, but I know you're working on it. > > +#define MTDIDS_DEFAULT "nand0=nand_mtd" > > +#define MTDPARTS_DEFAULT > > "mtdparts=nand_mtd:0x100...@0x000000(uboot),0x400...@0x100000( > > uImage),"\ > > + "0x1fb00...@0x500000(rootfs)" > This is must be the default setup in board setup file for openrd_base (if it > is again derived from sheevaplug) > In that case this settings not required I don't quite understand this. This is also the default addresses shipped with the board. > > +#define CONFIG_EXTRA_ENV_SETTINGS "x_bootargs=console" \ > > + "=ttyS0,115200 "MTDPARTS_DEFAULT " rw ubi.mtd=2,2048\0" \ > > + "x_bootcmd_kernel=nand read 0x6400000 0x100000 0x300000\0" \ > > + "x_bootcmd_usb=usb start\0" \ > You must add x_bootcmd_eth for program MAC addrss in hardware before kernel > boot. Well, we need to figure out some generic way of giving Linux the MAC address on these boards, but I've understood that initializing the interface without using it is not popular, so in this case I'd just let the problem remain (no x_bootcmd_eth). Or did I misunderstand your comment? // Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
