On Mon, Feb 16, 2015 at 10:54:56PM +0000, Cooper Jr., Franklin wrote:
> 
> 
> > -----Original Message-----
> > From: Menon, Nishanth
> > Sent: Monday, February 16, 2015 4:03 PM
> > To: Cooper Jr., Franklin; u-boot@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 <tr...@ti.com>
> > >
> > > Signed-off-by: Tom Rini <tr...@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.

And at that point adopt some of the generic distro boot command patches
that have been posted for am335x to be to ti_armv7_common.h and update
that generic framework as needed for netboot, etc?

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to