Patchwork [U-Boot] M28: Fix the use of gpmi-nand in mtdparts

login
register
mail settings
Submitter Marek Vasut
Date Aug. 29, 2012, 1:12 a.m.
Message ID <1346202768-19058-1-git-send-email-marex@denx.de>
Download mbox | patch
Permalink /patch/180602/
State Awaiting Upstream
Delegated to: Stefano Babic
Headers show

Comments

Marek Vasut - Aug. 29, 2012, 1:12 a.m.
The mtd name of the NAND in Linux is "gpmi-nand", not "gpmi-nand.0" as
it would be expected, since the controller doesn't support multiple NANDs
attached to it as of now. Rectify this flub by adjusting default mtdparts.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Fabio Estevam <fabio.estevam@freescale.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/m28evk.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Stefano Babic - Aug. 29, 2012, 7:41 a.m.
On 29/08/2012 03:12, Marek Vasut wrote:
> The mtd name of the NAND in Linux is "gpmi-nand", not "gpmi-nand.0" as
> it would be expected, since the controller doesn't support multiple NANDs
> attached to it as of now. Rectify this flub by adjusting default mtdparts.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Fabio Estevam <fabio.estevam@freescale.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  include/configs/m28evk.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/configs/m28evk.h b/include/configs/m28evk.h
> index d5cd5a8..40df437 100644
> --- a/include/configs/m28evk.h
> +++ b/include/configs/m28evk.h
> @@ -172,9 +172,9 @@
>  #define	CONFIG_LZO
>  #define	CONFIG_MTD_DEVICE
>  #define	CONFIG_MTD_PARTITIONS
> -#define	MTDIDS_DEFAULT			"nand0=gpmi-nand.0"
> +#define	MTDIDS_DEFAULT			"nand0=gpmi-nand"
>  #define	MTDPARTS_DEFAULT			\
> -	"mtdparts=gpmi-nand.0:"			\
> +	"mtdparts=gpmi-nand:"			\
>  		"3m(bootloader)ro,"		\
>  		"512k(environment),"		\
>  		"512k(redundant-environment),"	\
> 

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

Best regards,
Stefano Babic
Stefano Babic - Sept. 4, 2012, 10:01 a.m.
On 29/08/2012 03:12, Marek Vasut wrote:
> The mtd name of the NAND in Linux is "gpmi-nand", not "gpmi-nand.0" as
> it would be expected, since the controller doesn't support multiple NANDs
> attached to it as of now. Rectify this flub by adjusting default mtdparts.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Fabio Estevam <fabio.estevam@freescale.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Otavio Salvador <otavio@ossystems.com.br>
> ---


Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic

Patch

diff --git a/include/configs/m28evk.h b/include/configs/m28evk.h
index d5cd5a8..40df437 100644
--- a/include/configs/m28evk.h
+++ b/include/configs/m28evk.h
@@ -172,9 +172,9 @@ 
 #define	CONFIG_LZO
 #define	CONFIG_MTD_DEVICE
 #define	CONFIG_MTD_PARTITIONS
-#define	MTDIDS_DEFAULT			"nand0=gpmi-nand.0"
+#define	MTDIDS_DEFAULT			"nand0=gpmi-nand"
 #define	MTDPARTS_DEFAULT			\
-	"mtdparts=gpmi-nand.0:"			\
+	"mtdparts=gpmi-nand:"			\
 		"3m(bootloader)ro,"		\
 		"512k(environment),"		\
 		"512k(redundant-environment),"	\