public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
@ 2015-11-27  2:27 Chin Liang See
  2015-11-27  3:18 ` Stefan Roese
  2015-11-27 10:35 ` Marek Vasut
  0 siblings, 2 replies; 9+ messages in thread
From: Chin Liang See @ 2015-11-27  2:27 UTC (permalink / raw)
  To: u-boot

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 \
diff --git a/include/configs/socfpga_cyclone5_socdk.h b/include/configs/socfpga_cyclone5_socdk.h
index d7369ee..3b2cfb0 100644
--- a/include/configs/socfpga_cyclone5_socdk.h
+++ b/include/configs/socfpga_cyclone5_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 \
diff --git a/include/configs/socfpga_de0_nano_soc.h b/include/configs/socfpga_de0_nano_soc.h
index b371a9b..16e146c 100644
--- a/include/configs/socfpga_de0_nano_soc.h
+++ b/include/configs/socfpga_de0_nano_soc.h
@@ -74,7 +74,6 @@
 	"bootimage=zImage\0" \
 	"fdt_addr=100\0" \
 	"fdtimage=socfpga.dtb\0" \
-		"fsloadcmd=ext2load\0" \
 	"bootm ${loadaddr} - ${fdt_addr}\0" \
 	"mmcroot=/dev/mmcblk0p2\0" \
 	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \
diff --git a/include/configs/socfpga_sockit.h b/include/configs/socfpga_sockit.h
index d26f53d..5bcee05 100644
--- a/include/configs/socfpga_sockit.h
+++ b/include/configs/socfpga_sockit.h
@@ -79,7 +79,6 @@
 	"bootimage=zImage\0" \
 	"fdt_addr=100\0" \
 	"fdtimage=socfpga.dtb\0" \
-		"fsloadcmd=ext2load\0" \
 	"bootm ${loadaddr} - ${fdt_addr}\0" \
 	"mmcroot=/dev/mmcblk0p2\0" \
 	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \
diff --git a/include/configs/socfpga_socrates.h b/include/configs/socfpga_socrates.h
index 6331251..16a2a86 100644
--- a/include/configs/socfpga_socrates.h
+++ b/include/configs/socfpga_socrates.h
@@ -75,7 +75,6 @@
 	"bootimage=zImage\0" \
 	"fdt_addr=100\0" \
 	"fdtimage=socfpga.dtb\0" \
-		"fsloadcmd=ext2load\0" \
 	"bootm ${loadaddr} - ${fdt_addr}\0" \
 	"mmcroot=/dev/mmcblk0p2\0" \
 	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \
-- 
2.2.0

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  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
  1 sibling, 0 replies; 9+ messages in thread
From: Stefan Roese @ 2015-11-27  3:18 UTC (permalink / raw)
  To: u-boot

On 27.11.2015 03:27, 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 \
> diff --git a/include/configs/socfpga_cyclone5_socdk.h b/include/configs/socfpga_cyclone5_socdk.h
> index d7369ee..3b2cfb0 100644
> --- a/include/configs/socfpga_cyclone5_socdk.h
> +++ b/include/configs/socfpga_cyclone5_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 \
> diff --git a/include/configs/socfpga_de0_nano_soc.h b/include/configs/socfpga_de0_nano_soc.h
> index b371a9b..16e146c 100644
> --- a/include/configs/socfpga_de0_nano_soc.h
> +++ b/include/configs/socfpga_de0_nano_soc.h
> @@ -74,7 +74,6 @@
>   	"bootimage=zImage\0" \
>   	"fdt_addr=100\0" \
>   	"fdtimage=socfpga.dtb\0" \
> -		"fsloadcmd=ext2load\0" \
>   	"bootm ${loadaddr} - ${fdt_addr}\0" \
>   	"mmcroot=/dev/mmcblk0p2\0" \
>   	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \
> diff --git a/include/configs/socfpga_sockit.h b/include/configs/socfpga_sockit.h
> index d26f53d..5bcee05 100644
> --- a/include/configs/socfpga_sockit.h
> +++ b/include/configs/socfpga_sockit.h
> @@ -79,7 +79,6 @@
>   	"bootimage=zImage\0" \
>   	"fdt_addr=100\0" \
>   	"fdtimage=socfpga.dtb\0" \
> -		"fsloadcmd=ext2load\0" \
>   	"bootm ${loadaddr} - ${fdt_addr}\0" \
>   	"mmcroot=/dev/mmcblk0p2\0" \
>   	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \
> diff --git a/include/configs/socfpga_socrates.h b/include/configs/socfpga_socrates.h
> index 6331251..16a2a86 100644
> --- a/include/configs/socfpga_socrates.h
> +++ b/include/configs/socfpga_socrates.h
> @@ -75,7 +75,6 @@
>   	"bootimage=zImage\0" \
>   	"fdt_addr=100\0" \
>   	"fdtimage=socfpga.dtb\0" \
> -		"fsloadcmd=ext2load\0" \
>   	"bootm ${loadaddr} - ${fdt_addr}\0" \
>   	"mmcroot=/dev/mmcblk0p2\0" \
>   	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  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
  1 sibling, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2015-11-27 10:35 UTC (permalink / raw)
  To: u-boot

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 ?

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  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:52     ` Marek Vasut
  0 siblings, 2 replies; 9+ messages in thread
From: Chin Liang See @ 2015-11-27 12:10 UTC (permalink / raw)
  To: u-boot

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

Thanks
Chin Liang

> 
> Best regards,
> Marek Vasut

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  2015-11-27 12:10   ` Chin Liang See
@ 2015-11-27 12:14     ` Marek Vasut
  2015-11-27 12:18       ` Chin Liang See
  2015-11-27 12:52     ` Marek Vasut
  1 sibling, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2015-11-27 12:14 UTC (permalink / raw)
  To: u-boot

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

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

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  2015-11-27 12:14     ` Marek Vasut
@ 2015-11-27 12:18       ` Chin Liang See
  0 siblings, 0 replies; 9+ messages in thread
From: Chin Liang See @ 2015-11-27 12:18 UTC (permalink / raw)
  To: u-boot

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  2015-11-27 12:10   ` Chin Liang See
  2015-11-27 12:14     ` Marek Vasut
@ 2015-11-27 12:52     ` Marek Vasut
  2015-11-27 13:09       ` Chin Liang See
  1 sibling, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2015-11-27 12:52 UTC (permalink / raw)
  To: u-boot

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

I'm inclined to pick this one up early and what for the QSPI situation to pan
out. Can you resubmit this one on top of u-boot-socfpga/master ? I can fix the
patch myself too tho...

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  2015-11-27 12:52     ` Marek Vasut
@ 2015-11-27 13:09       ` Chin Liang See
  2015-11-27 13:16         ` Marek Vasut
  0 siblings, 1 reply; 9+ messages in thread
From: Chin Liang See @ 2015-11-27 13:09 UTC (permalink / raw)
  To: u-boot

On Fri, 2015-11-27 at 13:52 +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).
> 
> I'm inclined to pick this one up early and what for the QSPI
> situation to pan
> out. Can you resubmit this one on top of u-boot-socfpga/master ? I
> can fix the
> patch myself too tho...
> 

Sure, let me do that as its a quick one :)

Thanks
Chin Liang

> Best regards,
> Marek Vasut

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [U-Boot] [PATCH] arm: socfpga: Remove fsloadcmd from environment
  2015-11-27 13:09       ` Chin Liang See
@ 2015-11-27 13:16         ` Marek Vasut
  0 siblings, 0 replies; 9+ messages in thread
From: Marek Vasut @ 2015-11-27 13:16 UTC (permalink / raw)
  To: u-boot

On Friday, November 27, 2015 at 02:09:29 PM, Chin Liang See wrote:
> On Fri, 2015-11-27 at 13:52 +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).
> > 
> > I'm inclined to pick this one up early and what for the QSPI
> > situation to pan
> > out. Can you resubmit this one on top of u-boot-socfpga/master ? I
> > can fix the
> > patch myself too tho...
> 
> Sure, let me do that as its a quick one :)

Thanks :)

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2015-11-27 13:16 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
2015-11-27 12:52     ` Marek Vasut
2015-11-27 13:09       ` Chin Liang See
2015-11-27 13:16         ` Marek Vasut

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox