[U-Boot] [PATCH v2] mx6qsabre_common: Let mmc partition be board specific

[U-Boot] [PATCH v2] mx6qsabre_common: Let mmc partition be board specific

[Fabio Estevam]

From: Fabio Estevam <fabio.estevam@freescale.com>

commit 49ea0ff5 (49ea0ff5) introduced CONFIG_SYS_MMC_ENV_PART into mx6qsabresd.h
to store the mmc partition, but in order for it to have effect we should place
it into 'mmcpart' variable.

Also add CONFIG_SYS_MMC_ENV_PART into mx6qsabreauto.h.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
Changes since v1:
- Drop duplicated mmcpart
 include/configs/mx6qsabre_common.h |    2 +-
 include/configs/mx6qsabreauto.h    |    1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/include/configs/mx6qsabre_common.h b/include/configs/mx6qsabre_common.h
index cda4512..d76357c 100644
--- a/include/configs/mx6qsabre_common.h
+++ b/include/configs/mx6qsabre_common.h
@@ -91,7 +91,7 @@
 	"fdt_high=0xffffffff\0"	  \
 	"initrd_high=0xffffffff\0" \
 	"mmcdev=" __stringify(CONFIG_SYS_MMC_ENV_DEV) "\0" \
-	"mmcpart=1\0" \
+	"mmcpart=" __stringify(CONFIG_SYS_MMC_ENV_PART) "\0" \
 	"mmcroot=" CONFIG_MMCROOT " rootwait rw\0" \
 	"mmcargs=setenv bootargs console=${console},${baudrate} " \
 		"root=${mmcroot}\0" \
diff --git a/include/configs/mx6qsabreauto.h b/include/configs/mx6qsabreauto.h
index f4a082a..bdf36b3 100644
--- a/include/configs/mx6qsabreauto.h
+++ b/include/configs/mx6qsabreauto.h
@@ -24,6 +24,7 @@
 #define CONFIG_SYS_FSL_USDHC_NUM	2
 #if defined(CONFIG_ENV_IS_IN_MMC)
 #define CONFIG_SYS_MMC_ENV_DEV		0
+#define CONFIG_SYS_MMC_ENV_PART		1	/* Boot partition 1 */
 #endif
 
 #endif                         /* __MX6QSABREAUTO_CONFIG_H */
-- 
1.7.9.5

[U-Boot] [PATCH v2] mx6qsabre_common: Let mmc partition be board specific

[Fabio Estevam]

Hi Stefano,

On Mon, Jan 14, 2013 at 4:59 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> commit 49ea0ff5 (49ea0ff5) introduced CONFIG_SYS_MMC_ENV_PART into mx6qsabresd.h
> to store the mmc partition, but in order for it to have effect we should place
> it into 'mmcpart' variable.
>
> Also add CONFIG_SYS_MMC_ENV_PART into mx6qsabreauto.h.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> Changes since v1:
> - Drop duplicated mmcpart
>  include/configs/mx6qsabre_common.h |    2 +-
>  include/configs/mx6qsabreauto.h    |    1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)

Does this one look good now?

Thanks,

Fabio Estevam

[U-Boot] [PATCH v2] mx6qsabre_common: Let mmc partition be board specific

[Stefano Babic]

On 22/01/2013 15:42, Fabio Estevam wrote:
> Hi Stefano,
> 

Hi Fabio,

> On Mon, Jan 14, 2013 at 4:59 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> From: Fabio Estevam <fabio.estevam@freescale.com>
>>
>> commit 49ea0ff5 (49ea0ff5) introduced CONFIG_SYS_MMC_ENV_PART into mx6qsabresd.h
>> to store the mmc partition, but in order for it to have effect we should place
>> it into 'mmcpart' variable.
>>
>> Also add CONFIG_SYS_MMC_ENV_PART into mx6qsabreauto.h.
>>
>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
>> ---
>> Changes since v1:
>> - Drop duplicated mmcpart
>>  include/configs/mx6qsabre_common.h |    2 +-
>>  include/configs/mx6qsabreauto.h    |    1 +
>>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> Does this one look good now?

Yes, it is, it is in my queue for next merge.

Regards,
Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

[U-Boot] [PATCH v2] mx6qsabre_common: Let mmc partition be board specific

[Stefano Babic]

On 14/01/2013 19:59, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> commit 49ea0ff5 (49ea0ff5) introduced CONFIG_SYS_MMC_ENV_PART into mx6qsabresd.h
> to store the mmc partition, but in order for it to have effect we should place
> it into 'mmcpart' variable.
> 
> Also add CONFIG_SYS_MMC_ENV_PART into mx6qsabreauto.h.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---

Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================