ti_armv7_common: env: Add NFS loading support to default enviroment

PATCH Superseded Andrew F. Davis 3 participants 5 messages Mar 11, 2016 – Mar 17, 2016

Share & tools

Andrew F. Davis

author

2016-03-11 21:04 UTC #0 lore raw

PATCH v2 1/1 ti_armv7_common: env: Add NFS loading support to default enviroment

NFS loading is similar to net loading except initial files are loaded over NFS instead of TFTP, this removes the need for multiple different protocol servers running on the host and allows the use of a single network file system containing boot related files in their usual in-filesystem directory. Add defaults for this boot style here.

Signed-off-by Andrew F. Davis <[email protected]>

Reviewed-by Tom Rini <[email protected]>

---
Changes from v1:
- Remove definition of CONFIG_CMD_NFS, enabled by default

 include/configs/ti_armv7_keystone2.h | 6 ++++++
 1 file changed, 6 insertions(+)

include/configs/ti_armv7_keystone2.h +6 -0

index a7206f4..17d5aa4 100644
@@ -237,16 +237,21 @@
 	"run_mon=mon_install ${addr_mon}\0"				\
 	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
 	"init_net=run args_all args_net\0"				\
+	"init_nfs=setenv autoload no; dhcp; run args_all args_net\0"	\
 	"init_ubi=run args_all args_ubi; "				\
 		"ubi part ubifs; ubifsmount ubi:boot;"			\
 		"ubifsload ${addr_secdb_key} securedb.key.bin;\0"       \
 	"get_fdt_net=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
+	"get_fdt_nfs=nfs ${fdtaddr} ${nfs_root}/boot/${name_fdt}\0"	\
 	"get_fdt_ubi=ubifsload ${fdtaddr} ${name_fdt}\0"		\
 	"get_kern_net=dhcp ${loadaddr} ${tftp_root}/${name_kern}\0"	\
+	"get_kern_nfs=nfs ${loadaddr} ${nfs_root}/boot/${name_kern}\0"	\
 	"get_kern_ubi=ubifsload ${loadaddr} ${name_kern}\0"		\
 	"get_mon_net=dhcp ${addr_mon} ${tftp_root}/${name_mon}\0"	\
+	"get_mon_nfs=nfs ${addr_mon} ${nfs_root}/boot/${name_mon}\0"	\
 	"get_mon_ubi=ubifsload ${addr_mon} ${name_mon}\0"		\
 	"get_uboot_net=dhcp ${loadaddr} ${tftp_root}/${name_uboot}\0"	\
+	"get_uboot_nfs=nfs ${loadaddr} ${nfs_root}/boot/${name_uboot}\0" \
 	"burn_uboot_spi=sf probe; sf erase 0 0x80000; "		\
 		"sf write ${loadaddr} 0 ${filesize}\0"		\
 	"burn_uboot_nand=nand erase 0 0x100000; "			\

@@ -261,6 +266,7 @@
 	"get_mon_ramfs=dhcp ${addr_mon} ${tftp_root}/${name_mon}\0"	\
 	"get_fs_ramfs=dhcp ${rdaddr} ${tftp_root}/${name_fs}\0"	\
 	"get_ubi_net=dhcp ${addr_ubi} ${tftp_root}/${name_ubi}\0"	\
+	"get_ubi_nfs=nfs ${addr_ubi} ${nfs_root}/boot/${name_ubi}\0"	\
 	"burn_ubi=nand erase.part ubifs; "				\
 		"nand write ${addr_ubi} ubifs ${filesize}\0"		\
 	"init_ramfs=run args_all args_ramfs get_fs_ramfs\0"		\

Signature · 1 line

2.7.2

Lokesh Vutla

2016-03-12 09:58 UTC #1 lore raw

PATCH v2 1/1 ti_armv7_common: env: Add NFS loading support to default enviroment

Quoted 5 lines · Andrew F. Davis

NFS loading is similar to net loading except initial files are loaded over NFS instead of TFTP, this removes the need for multiple different protocol servers running on the host and allows the use of a single network file system containing boot related files in their usual in-filesystem directory. Add defaults for this boot style here.

I guess this patch is dependent on [1]. Please send it as a series and
[1] should be applied before this.

Quoted 15 lines

Signed-off-by Andrew F. Davis <[email protected]>
Reviewed-by Tom Rini <[email protected]>
---
Changes from v1:
- Remove definition of CONFIG_CMD_NFS, enabled by default
 include/configs/ti_armv7_keystone2.h | 6 ++++++
 1 file changed, 6 insertions(+)
include/configs/ti_armv7_keystone2.h +0 -0
index a7206f4..17d5aa4 100644

include/configs/ti_armv7_keystone2.h line 245

Quoted 9 lines

include/configs/ti_armv7_keystone2.h +2 -0

@@ -237,16 +237,21 @@
 	"run_mon=mon_install ${addr_mon}\0"				\
 	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
 	"init_net=run args_all args_net\0"				\
+	"init_nfs=setenv autoload no; dhcp; run args_all args_net\0"	\
 	"init_ubi=run args_all args_ubi; "				\
 		"ubi part ubifs; ubifsmount ubi:boot;"			\
 		"ubifsload ${addr_secdb_key} securedb.key.bin;\0"       \
 	"get_fdt_net=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
+	"get_fdt_nfs=nfs ${fdtaddr} ${nfs_root}/boot/${name_fdt}\0"	\

Can you also add a default nfs_root variable?

[1] http://patchwork.ozlabs.org/patch/596509/

Thanks and regards,
Lokesh

Andrew F. Davis

author

2016-03-13 17:18 UTC #2 lore raw

PATCH v2 1/1 ti_armv7_common: env: Add NFS loading support to default enviroment

Quoted 12 lines · Lokesh Vutla

Quoted 5 lines · Andrew F. Davis
NFS loading is similar to net loading except initial files are loaded over NFS instead of TFTP, this removes the need for multiple different protocol servers running on the host and allows the use of a single network file system containing boot related files in their usual in-filesystem directory. Add defaults for this boot style here.
I guess this patch is dependent on [1]. Please send it as a series and
[1] should be applied before this.

No dependency, this is for k2x, [1] is for dra7x.

Quoted 11 lines

Quoted 11 lines
Signed-off-by Andrew F. Davis <[email protected]>
Reviewed-by Tom Rini <[email protected]>
---
Changes from v1:
- Remove definition of CONFIG_CMD_NFS, enabled by default
 include/configs/ti_armv7_keystone2.h | 6 ++++++
 1 file changed, 6 insertions(+)

Quoted 17 lines

Quoted 5 lines

include/configs/ti_armv7_keystone2.h +0 -0
index a7206f4..17d5aa4 100644

include/configs/ti_armv7_keystone2.h line 245

Quoted 9 lines

include/configs/ti_armv7_keystone2.h +2 -0

@@ -237,16 +237,21 @@
 	"run_mon=mon_install ${addr_mon}\0"				\
 	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
 	"init_net=run args_all args_net\0"				\
+	"init_nfs=setenv autoload no; dhcp; run args_all args_net\0"	\
 	"init_ubi=run args_all args_ubi; "				\
 		"ubi part ubifs; ubifsmount ubi:boot;"			\
 		"ubifsload ${addr_secdb_key} securedb.key.bin;\0"       \
 	"get_fdt_net=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
+	"get_fdt_nfs=nfs ${fdtaddr} ${nfs_root}/boot/${name_fdt}\0"	\

Can you also add a default nfs_root variable?

Already defined, used by net boot previously.

Thanks,
Andrew

Quoted 5 lines

[1] http://patchwork.ozlabs.org/patch/596509/
Thanks and regards,
Lokesh

Lokesh Vutla

2016-03-14 04:58 UTC #3 lore raw

PATCH v2 1/1 ti_armv7_common: env: Add NFS loading support to default enviroment

Quoted 10 lines · Andrew F. Davis

Quoted 9 lines · Lokesh Vutla
Quoted 5 lines · Andrew F. Davis
NFS loading is similar to net loading except initial files are loaded over NFS instead of TFTP, this removes the need for multiple different protocol servers running on the host and allows the use of a single network file system containing boot related files in their usual in-filesystem directory. Add defaults for this boot style here.

Quoted 5 lines

Quoted 3 lines
I guess this patch is dependent on [1]. Please send it as a series and
[1] should be applied before this.
No dependency, this is for k2x, [1] is for dra7x.

Oops...my bad. Sorry for the noise.

Thanks and regards,
Lokesh

Quoted 40 lines

Quoted 12 lines
Quoted 12 lines
Signed-off-by Andrew F. Davis <[email protected]>
Reviewed-by Tom Rini <[email protected]>
---
Changes from v1:
- Remove definition of CONFIG_CMD_NFS, enabled by default
 include/configs/ti_armv7_keystone2.h | 6 ++++++
 1 file changed, 6 insertions(+)
Quoted 17 lines
Quoted 5 lines
include/configs/ti_armv7_keystone2.h +0 -0
index a7206f4..17d5aa4 100644
include/configs/ti_armv7_keystone2.h line 245
Quoted 9 lines
include/configs/ti_armv7_keystone2.h +2 -0
@@ -237,16 +237,21 @@
 	"run_mon=mon_install ${addr_mon}\0"				\
 	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
 	"init_net=run args_all args_net\0"				\
+	"init_nfs=setenv autoload no; dhcp; run args_all args_net\0"	\
 	"init_ubi=run args_all args_ubi; "				\
 		"ubi part ubifs; ubifsmount ubi:boot;"			\
 		"ubifsload ${addr_secdb_key} securedb.key.bin;\0"       \
 	"get_fdt_net=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
+	"get_fdt_nfs=nfs ${fdtaddr} ${nfs_root}/boot/${name_fdt}\0"	\
Can you also add a default nfs_root variable?
Already defined, used by net boot previously.
Thanks,
Andrew
Quoted 5 lines
[1] http://patchwork.ozlabs.org/patch/596509/
Thanks and regards,
Lokesh

Tom Rini

2016-03-17 02:05 UTC #4 lore raw

PATCH v2 1/1 ti_armv7_common: env: Add NFS loading support to default enviroment

In reply to Andrew F. Davis

Quoted 8 lines · Andrew F. Davis

NFS loading is similar to net loading except initial files are loaded over NFS instead of TFTP, this removes the need for multiple different protocol servers running on the host and allows the use of a single network file system containing boot related files in their usual in-filesystem directory. Add defaults for this boot style here.
Signed-off-by Andrew F. Davis <[email protected]>
Reviewed-by Tom Rini <[email protected]>

Applied to u-boot/master, thanks!

Signature · 8 lines

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160316/9d70d6ea/attachment.sig>