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

[Lokesh Vutla <a0131933@ti.com>]

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