public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lokesh Vutla <a0131933@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: Sat, 12 Mar 2016 15:28:25 +0530	[thread overview]
Message-ID: <56E3E841.3060708@ti.com> (raw)
In-Reply-To: <1457730243-24897-1-git-send-email-afd@ti.com>



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

  reply	other threads:[~2016-03-12  9:58 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 [this message]
2016-03-13 17:18   ` Andrew F. Davis
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=56E3E841.3060708@ti.com \
    --to=a0131933@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