public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Chin Liang See <clsee@altera.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
Date: Fri, 27 Nov 2015 20:18:42 +0800	[thread overview]
Message-ID: <1448626722.2125.12.camel@altera.com> (raw)
In-Reply-To: <201511271314.55367.marex@denx.de>

On Fri, 2015-11-27 at 13:14 +0100, Marek Vasut wrote:
> On Friday, November 27, 2015 at 01:10:21 PM, Chin Liang See wrote:
> > On Fri, 2015-11-27 at 11:35 +0100, Marek Vasut wrote:
> > > On Friday, November 27, 2015 at 03:27:51 AM, Chin Liang See
> > > wrote:
> > > > Remove fsloadcmd / ext2load as we are using load command
> > > > which use the corresponding latest file system command.
> > > > 
> > > > Signed-off-by: Chin Liang See <clsee@altera.com>
> > > > Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
> > > > Cc: Dinh Nguyen <dinh.linux@gmail.com>
> > > > Cc: Pavel Machek <pavel@denx.de>
> > > > Cc: Marek Vasut <marex@denx.de>
> > > > Cc: Stefan Roese <sr@denx.de>
> > > > ---
> > > > 
> > > >  include/configs/socfpga_arria5_socdk.h   | 1 -
> > > >  include/configs/socfpga_cyclone5_socdk.h | 1 -
> > > >  include/configs/socfpga_de0_nano_soc.h   | 1 -
> > > >  include/configs/socfpga_sockit.h         | 1 -
> > > >  include/configs/socfpga_socrates.h       | 1 -
> > > >  5 files changed, 5 deletions(-)
> > > > 
> > > > diff --git a/include/configs/socfpga_arria5_socdk.h
> > > > b/include/configs/socfpga_arria5_socdk.h index 0506336..9337a58
> > > > 100644
> > > > --- a/include/configs/socfpga_arria5_socdk.h
> > > > +++ b/include/configs/socfpga_arria5_socdk.h
> > > > @@ -81,7 +81,6 @@
> > > > 
> > > >  	"fdt_addr=100\0" \
> > > >  	"fdtimage=socfpga.dtb\0" \
> > > >  	"fdtimagesize=0x7000\0" \
> > > > 
> > > > -	"fsloadcmd=ext2load\0" \
> > > > 
> > > >  	"bootm ${loadaddr} - ${fdt_addr}\0" \
> > > >  	"mmcroot=/dev/mmcblk0p2\0" \
> > > >  	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \
> > > 
> > > This doesn't apply to u-boot-socfpga/master . Any idea why ?
> > 
> > Oh, this patch was on top of the QSPI patch. As that patch is
> > acked, I
> > am thinking that will be applied first than this (if acked).
> 
> Um, what did I smoke what I acked that patch ? :O
> 

Oh its actually acked by Pavel. But let me know if you want me to let
this patch go first than QSPI.

> btw. if you have some inter-patch deps, it'd be real nice if you
> could
> put a note about that somewhere next time.
> 

Noted.

Thanks
Chin Liang

> Best regards,
> Marek Vasut

  reply	other threads:[~2015-11-27 12:18 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-27  2:27 [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment Chin Liang See
2015-11-27  3:18 ` Stefan Roese
2015-11-27 10:35 ` Marek Vasut
2015-11-27 12:10   ` Chin Liang See
2015-11-27 12:14     ` Marek Vasut
2015-11-27 12:18       ` Chin Liang See [this message]
2015-11-27 12:52     ` Marek Vasut
2015-11-27 13:09       ` Chin Liang See
2015-11-27 13:16         ` Marek Vasut

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=1448626722.2125.12.camel@altera.com \
    --to=clsee@altera.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