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

From: Tom Rini trini@ti.com
Signed-off-by: Tom Rini trini@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; " \

On 02/16/2015 02:59 PM, Franklin S Cooper Jr wrote:
From: Tom Rini trini@ti.com
Signed-off-by: Tom Rini trini@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}" \
"ramroot=/dev/ram0 rw\0" \ "ramrootfstype=ext2\0" \ "mmcargs=setenv bootargs console=${console} " \"::off\0" \
@@ -178,6 +182,11 @@ "${optargs} " \ "root=${usbroot} " \ "rootfstype=${usbrootfstype}\0" \
- "netargs=setenv bootargs console=${console} " \
"${optargs} " \
"root=/dev/nfs " \
"nfsroot=${serverip}:${rootpath},${nfsopts} rw " \
"bootenv=uEnv.txt\0" \ "loadbootenv=load ${devtype} ${devnum} ${loadaddr} ${bootenv}\0" \ "importbootenv=echo Importing environment from mmc ...; " \"ip=dhcp\0" \
@@ -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; " \
"findfdt="\ "if test $board_name = AM43EPOS; then " \ "setenv fdtfile am43x-epos-evm.dtb; fi; " \"bootz ${loadaddr} - ${fdtaddr}\0" \
Should we start introducing generic macros in v7_common.h ? that'd let all network enabled platforms to uniformly provide this?

-----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 trini@ti.com
Signed-off-by: Tom Rini trini@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}"
\
"ramroot=/dev/ram0 rw\0" \ "ramrootfstype=ext2\0" \ "mmcargs=setenv bootargs console=${console} " \ @@ -178,6 +182,11"::off\0" \
@@
"${optargs} " \ "root=${usbroot} " \ "rootfstype=${usbrootfstype}\0" \
- "netargs=setenv bootargs console=${console} " \
"${optargs} " \
"root=/dev/nfs " \
"nfsroot=${serverip}:${rootpath},${nfsopts} rw " \
"bootenv=uEnv.txt\0" \ "loadbootenv=load ${devtype} ${devnum} ${loadaddr} ${bootenv}\0" \ "importbootenv=echo Importing environment from mmc ...; " \ @@"ip=dhcp\0" \
-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; " \
"findfdt="\ "if test $board_name = AM43EPOS; then " \ "setenv fdtfile am43x-epos-evm.dtb; fi; " \"bootz ${loadaddr} - ${fdtaddr}\0" \
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

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 trini@ti.com
Signed-off-by: Tom Rini trini@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}"
\
"ramroot=/dev/ram0 rw\0" \ "ramrootfstype=ext2\0" \ "mmcargs=setenv bootargs console=${console} " \ @@ -178,6 +182,11"::off\0" \
@@
"${optargs} " \ "root=${usbroot} " \ "rootfstype=${usbrootfstype}\0" \
- "netargs=setenv bootargs console=${console} " \
"${optargs} " \
"root=/dev/nfs " \
"nfsroot=${serverip}:${rootpath},${nfsopts} rw " \
"bootenv=uEnv.txt\0" \ "loadbootenv=load ${devtype} ${devnum} ${loadaddr} ${bootenv}\0" \ "importbootenv=echo Importing environment from mmc ...; " \ @@"ip=dhcp\0" \
-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; " \
"findfdt="\ "if test $board_name = AM43EPOS; then " \ "setenv fdtfile am43x-epos-evm.dtb; fi; " \"bootz ${loadaddr} - ${fdtaddr}\0" \
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?
participants (4)
-
Cooper Jr., Franklin
-
Franklin S Cooper Jr
-
Nishanth Menon
-
Tom Rini