[U-Boot] [PATCH] am43xx_evm: Add netboot and related to environment

[Cooper Jr., Franklin]

> -----Original Message-----
> From: Menon, Nishanth
> Sent: Monday, February 16, 2015 4:03 PM
> To: Cooper Jr., Franklin; u-boot at lists.denx.de
> Cc: Rini, Tom
> Subject: Re: [U-Boot] [PATCH] am43xx_evm: Add netboot and related to
> environment
> 
> On 02/16/2015 02:59 PM, Franklin S Cooper Jr wrote:
> > From: Tom Rini <trini at ti.com>
> >
> > Signed-off-by: Tom Rini <trini at ti.com>
> > ---
> >  include/configs/am43xx_evm.h | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> >
> > diff --git a/include/configs/am43xx_evm.h
> > b/include/configs/am43xx_evm.h index 7ccbf36..44b6f93 100644
> > --- a/include/configs/am43xx_evm.h
> > +++ b/include/configs/am43xx_evm.h
> > @@ -168,6 +168,10 @@
> >  	"usbroot=/dev/sda2 rw\0" \
> >  	"usbrootfstype=ext4 rootwait\0" \
> >  	"usbdev=0\0" \
> > +	"rootpath=/export/rootfs\0" \
> > +	"nfsopts=nolock\0" \
> > +	"static_ip=${ipaddr}:${serverip}:${gatewayip}:${netmask}:${hostname}"
> \
> > +		"::off\0" \
> >  	"ramroot=/dev/ram0 rw\0" \
> >  	"ramrootfstype=ext2\0" \
> >  	"mmcargs=setenv bootargs console=${console} " \ @@ -178,6 +182,11
> @@
> >  		"${optargs} " \
> >  		"root=${usbroot} " \
> >  		"rootfstype=${usbrootfstype}\0" \
> > +	"netargs=setenv bootargs console=${console} " \
> > +		"${optargs} " \
> > +		"root=/dev/nfs " \
> > +		"nfsroot=${serverip}:${rootpath},${nfsopts} rw " \
> > +		"ip=dhcp\0" \
> >  	"bootenv=uEnv.txt\0" \
> >  	"loadbootenv=load ${devtype} ${devnum} ${loadaddr} ${bootenv}\0" \
> >  	"importbootenv=echo Importing environment from mmc ...; " \ @@
> > -229,6 +238,13 @@
> >  				"bootz ${loadaddr} - ${fdtaddr}; " \
> >  			"fi;" \
> >  		"fi\0" \
> > +	"netboot=echo Booting from network ...; " \
> > +		"setenv autoload no; " \
> > +		"dhcp; " \
> > +		"tftp ${loadaddr} ${bootfile}; " \
> > +		"tftp ${fdtaddr} ${fdtfile}; " \
> > +		"run netargs; " \
> > +		"bootz ${loadaddr} - ${fdtaddr}\0" \
> >  	"findfdt="\
> >  		"if test $board_name = AM43EPOS; then " \
> >  			"setenv fdtfile am43x-epos-evm.dtb; fi; " \
> >
> 
> Should we start introducing generic macros in v7_common.h ? that'd let all
> network enabled platforms to uniformly provide this?
[Franklin] I see no issue with that. We should probably do the same for mmc, usb and probably some other peripherals. It won't really change anything for 335x, 437x and am57x TI evms but users of non TI boards or other TI SOCs may have an issue.
> 
> --
> Regards,
> Nishanth Menon