From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sat, 30 Aug 2014 16:20:02 +0200 Subject: [U-Boot] [PATCH] ARM: mx6: Add support for Kosagi Novena In-Reply-To: References: <1409304613-5657-1-git-send-email-marex@denx.de> <201408301440.24751.marex@denx.de> Message-ID: <201408301620.02617.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Saturday, August 30, 2014 at 03:55:48 PM, Fabio Estevam wrote: > On Sat, Aug 30, 2014 at 9:40 AM, Marek Vasut wrote: > > On Friday, August 29, 2014 at 07:08:54 PM, Fabio Estevam wrote: > > [...] > > > >> > +/* Extra U-Boot environment. */ > >> > +#define CONFIG_EXTRA_ENV_SETTINGS > >> > \ + "fdt_high=0xffffffff\0" > >> > \ + "initrd_high=0xffffffff\0" > >> > \ + "consdev=ttymxc1\0" > >> > \ + "baudrate=115200\0" > >> > \ + "bootdev=/dev/mmcblk0p1\0" > >> > \ + "rootdev=/dev/mmcblk0p2\0" > >> > \ + "netdev=eth0\0" > >> > \ + > >> > "rootpath=/opt/eldk-5.5/armv7a-hf/rootfs-qte-sdk\0" \ > >> > >> This seems to be a too specific rootpath name. Not everyone will use > >> this same rootpath. > > > > You're right, but what kind of sane default do you suggest ? The user > > will overwrite this anyway ... > > I would say just get rid of this default rootpath entry. > > Users will have to add their own ipaddr, serverip and rootpath anyway. Sure, but having some default value is better than having all the scripts broken because the variable is not set at all, right ? Best regards, Marek Vasut