diff mbox

[U-Boot] ARM: tam3517-common: fix nand spl boot

Message ID 1387645389-26217-1-git-send-email-jeroen@myspectrum.nl
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Jeroen Hofstee Dec. 21, 2013, 5:03 p.m. UTC
commit f9095aac793aa8917ab9b915c5d449e6dc8d3d30, "mtd: nand:
omap: add CONFIG_NAND_OMAP_ECCSCHEME for selection of ecc-scheme"
removed CONFIG_SPL_NAND_SOFTECC from the tam3517 common config,
causing the spl nand boot to fail. Add it back, so derived
boards boot again.

Cc: Pekon Gupta <pekon@ti.com>
Cc: Scott Wood <scottwood@freescale.com>
Cc: Raphael Assenat <raph@8d.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Tapani Utriainen <tapani@technexion.com>
Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
---
 include/configs/tam3517-common.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Stefano Babic Dec. 27, 2013, 10:53 a.m. UTC | #1
Hi Jeroen,

On 21/12/2013 18:03, Jeroen Hofstee wrote:
> commit f9095aac793aa8917ab9b915c5d449e6dc8d3d30, "mtd: nand:
> omap: add CONFIG_NAND_OMAP_ECCSCHEME for selection of ecc-scheme"
> removed CONFIG_SPL_NAND_SOFTECC from the tam3517 common config,
> causing the spl nand boot to fail. Add it back, so derived
> boards boot again.
> 
> Cc: Pekon Gupta <pekon@ti.com>
> Cc: Scott Wood <scottwood@freescale.com>
> Cc: Raphael Assenat <raph@8d.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Tapani Utriainen <tapani@technexion.com>
> Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
> ---
>  include/configs/tam3517-common.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/configs/tam3517-common.h b/include/configs/tam3517-common.h
> index 439fc47..d44b5c0 100644
> --- a/include/configs/tam3517-common.h
> +++ b/include/configs/tam3517-common.h
> @@ -224,6 +224,7 @@
>  #define CONFIG_SPL_BOARD_INIT
>  #define CONFIG_SPL_CONSOLE
>  #define CONFIG_SPL_NAND_SIMPLE
> +#define CONFIG_SPL_NAND_SOFTECC
>  #define CONFIG_SPL_NAND_WORKSPACE	0x8f07f000 /* below BSS */
>  
>  #define CONFIG_SPL_LIBCOMMON_SUPPORT
> 

Thanks for fixing it.

Acked-by: Stefano Babic <sbabic@denx.de>

Best regards,
Stefano Babic
Tom Rini Jan. 8, 2014, 1:34 p.m. UTC | #2
On Sat, Dec 21, 2013 at 06:03:09PM +0100, Jeroen Hofstee wrote:

> commit f9095aac793aa8917ab9b915c5d449e6dc8d3d30, "mtd: nand:
> omap: add CONFIG_NAND_OMAP_ECCSCHEME for selection of ecc-scheme"
> removed CONFIG_SPL_NAND_SOFTECC from the tam3517 common config,
> causing the spl nand boot to fail. Add it back, so derived
> boards boot again.
> 
> Cc: Pekon Gupta <pekon@ti.com>
> Cc: Scott Wood <scottwood@freescale.com>
> Cc: Raphael Assenat <raph@8d.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Tapani Utriainen <tapani@technexion.com>
> Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
> Acked-by: Stefano Babic <sbabic@denx.de>

Applied to u-boot-ti/master, thanks!
diff mbox

Patch

diff --git a/include/configs/tam3517-common.h b/include/configs/tam3517-common.h
index 439fc47..d44b5c0 100644
--- a/include/configs/tam3517-common.h
+++ b/include/configs/tam3517-common.h
@@ -224,6 +224,7 @@ 
 #define CONFIG_SPL_BOARD_INIT
 #define CONFIG_SPL_CONSOLE
 #define CONFIG_SPL_NAND_SIMPLE
+#define CONFIG_SPL_NAND_SOFTECC
 #define CONFIG_SPL_NAND_WORKSPACE	0x8f07f000 /* below BSS */
 
 #define CONFIG_SPL_LIBCOMMON_SUPPORT