public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Andrew F. Davis <afd@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] ti_armv7_common: env: Add NFS loading support to default enviroment
Date: Sun, 13 Mar 2016 12:18:52 -0500	[thread overview]
Message-ID: <56E5A0FC.4010901@ti.com> (raw)
In-Reply-To: <56E3E841.3060708@ti.com>

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
> 

  reply	other threads:[~2016-03-13 17:18 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-11 21:04 [U-Boot] [PATCH v2] ti_armv7_common: env: Add NFS loading support to default enviroment Andrew F. Davis
2016-03-12  9:58 ` Lokesh Vutla
2016-03-13 17:18   ` Andrew F. Davis [this message]
2016-03-14  4:58     ` Lokesh Vutla
2016-03-17  2:05 ` [U-Boot] [U-Boot, " Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56E5A0FC.4010901@ti.com \
    --to=afd@ti.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox