[U-Boot] [PATCH v2] ti_armv7_common: env: Add NFS loading support to default enviroment

[U-Boot] [PATCH v2] ti_armv7_common: env: Add NFS loading support to default enviroment

[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 <afd@ti.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
---
Changes from v1:
 - Remove definition of CONFIG_CMD_NFS, enabled by default

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

diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
index a7206f4..17d5aa4 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -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"		\
-- 
2.7.2

[U-Boot] [PATCH v2] ti_armv7_common: env: Add NFS loading support to default enviroment

[Lokesh Vutla]

On Saturday 12 March 2016 02:34 AM, Andrew F. Davis wrote:
> 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.


> 
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> ---
> Changes from v1:
>  - Remove definition of CONFIG_CMD_NFS, enabled by default
> 
>  include/configs/ti_armv7_keystone2.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
> index a7206f4..17d5aa4 100644
> --- a/include/configs/ti_armv7_keystone2.h
> +++ b/include/configs/ti_armv7_keystone2.h
> @@ -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

[U-Boot] [PATCH v2] ti_armv7_common: env: Add NFS loading support to default enviroment

[Andrew F. Davis]

On 03/12/2016 03:58 AM, Lokesh Vutla wrote:
> 
> 
> On Saturday 12 March 2016 02:34 AM, Andrew F. Davis wrote:
>> 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.

> 
>>
>> Signed-off-by: Andrew F. Davis <afd@ti.com>
>> Reviewed-by: Tom Rini <trini@konsulko.com>
>> ---
>> Changes from v1:
>>  - Remove definition of CONFIG_CMD_NFS, enabled by default
>>
>>  include/configs/ti_armv7_keystone2.h | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
>> index a7206f4..17d5aa4 100644
>> --- a/include/configs/ti_armv7_keystone2.h
>> +++ b/include/configs/ti_armv7_keystone2.h
>> @@ -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

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

[U-Boot] [PATCH v2] ti_armv7_common: env: Add NFS loading support to default enviroment

[Lokesh Vutla]

On Sunday 13 March 2016 10:48 PM, Andrew F. Davis wrote:
> On 03/12/2016 03:58 AM, Lokesh Vutla wrote:
>>
>>
>> On Saturday 12 March 2016 02:34 AM, Andrew F. Davis wrote:
>>> 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.

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

Thanks and regards,
Lokesh

> 
>>
>>>
>>> Signed-off-by: Andrew F. Davis <afd@ti.com>
>>> Reviewed-by: Tom Rini <trini@konsulko.com>
>>> ---
>>> Changes from v1:
>>>  - Remove definition of CONFIG_CMD_NFS, enabled by default
>>>
>>>  include/configs/ti_armv7_keystone2.h | 6 ++++++
>>>  1 file changed, 6 insertions(+)
>>>
>>> diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
>>> index a7206f4..17d5aa4 100644
>>> --- a/include/configs/ti_armv7_keystone2.h
>>> +++ b/include/configs/ti_armv7_keystone2.h
>>> @@ -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
> 
>> [1] http://patchwork.ozlabs.org/patch/596509/
>>
>> Thanks and regards,
>> Lokesh
>>

[U-Boot] [U-Boot, v2] ti_armv7_common: env: Add NFS loading support to default enviroment

[Tom Rini]

On Fri, Mar 11, 2016 at 03:04:03PM -0600, Andrew F. Davis wrote:

> 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 <afd@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!

-- 
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>