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

Lokesh Vutla
2016-03-14T04:58:59Z

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