Patchwork [U-Boot,13/13] mx53evk: Place machine ID into board config

login
register
mail settings
Submitter Fabio Estevam
Date Sept. 22, 2011, 6:07 p.m.
Message ID <1316714842-18271-13-git-send-email-fabio.estevam@freescale.com>
Download mbox | patch
Permalink /patch/115994/
State Accepted
Commit 00e11a439729a1a63ec13fea85a4c091c2d0376c
Delegated to: Stefano Babic
Headers show

Comments

Fabio Estevam - Sept. 22, 2011, 6:07 p.m.
Let common code set the machine ID.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 board/freescale/mx53evk/mx53evk.c |    1 -
 include/configs/mx53evk.h         |    2 ++
 2 files changed, 2 insertions(+), 1 deletions(-)
Stefano Babic - Sept. 23, 2011, 8:31 a.m.
On 09/22/2011 08:07 PM, Fabio Estevam wrote:
> Let common code set the machine ID.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  board/freescale/mx53evk/mx53evk.c |    1 -
>  include/configs/mx53evk.h         |    2 ++
>  2 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/board/freescale/mx53evk/mx53evk.c b/board/freescale/mx53evk/mx53evk.c
> index 81857ff..fd821c0 100644
> --- a/board/freescale/mx53evk/mx53evk.c
> +++ b/board/freescale/mx53evk/mx53evk.c
> @@ -355,7 +355,6 @@ int board_early_init_f(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_arch_number = MACH_TYPE_MX53_EVK;
>  	/* address of boot parameters */
>  	gd->bd->bi_boot_params = PHYS_SDRAM_1 + 0x100;
>  
> diff --git a/include/configs/mx53evk.h b/include/configs/mx53evk.h
> index 7fb1d9a..b127b06 100644
> --- a/include/configs/mx53evk.h
> +++ b/include/configs/mx53evk.h
> @@ -29,6 +29,8 @@
>  #define CONFIG_DISPLAY_CPUINFO
>  #define CONFIG_DISPLAY_BOARDINFO
>  
> +#define CONFIG_MACH_TYPE	MACH_TYPE_MX53_EVK
> +
>  #include <asm/arch/imx-regs.h>
>  
>  #define CONFIG_CMDLINE_TAG		1	/* enable passing of ATAGs */

Forwarded to Jason.

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

Best regards,
Stefano Babic
Jason Liu - Sept. 23, 2011, 8:37 a.m.
Hi, Fabio,

On Fri, Sep 23, 2011 at 4:31 PM, Stefano Babic <sbabic@denx.de> wrote:
> On 09/22/2011 08:07 PM, Fabio Estevam wrote:
>> Let common code set the machine ID.
>>
>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

As Stefano said, please Cc me here.

>> ---
>>  board/freescale/mx53evk/mx53evk.c |    1 -
>>  include/configs/mx53evk.h         |    2 ++
>>  2 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/board/freescale/mx53evk/mx53evk.c b/board/freescale/mx53evk/mx53evk.c
>> index 81857ff..fd821c0 100644
>> --- a/board/freescale/mx53evk/mx53evk.c
>> +++ b/board/freescale/mx53evk/mx53evk.c
>> @@ -355,7 +355,6 @@ int board_early_init_f(void)
>>
>>  int board_init(void)
>>  {
>> -     gd->bd->bi_arch_number = MACH_TYPE_MX53_EVK;
>>       /* address of boot parameters */
>>       gd->bd->bi_boot_params = PHYS_SDRAM_1 + 0x100;
>>
>> diff --git a/include/configs/mx53evk.h b/include/configs/mx53evk.h
>> index 7fb1d9a..b127b06 100644
>> --- a/include/configs/mx53evk.h
>> +++ b/include/configs/mx53evk.h
>> @@ -29,6 +29,8 @@
>>  #define CONFIG_DISPLAY_CPUINFO
>>  #define CONFIG_DISPLAY_BOARDINFO
>>
>> +#define CONFIG_MACH_TYPE     MACH_TYPE_MX53_EVK
>> +
>>  #include <asm/arch/imx-regs.h>
>>
>>  #define CONFIG_CMDLINE_TAG           1       /* enable passing of ATAGs */
>
> Forwarded to Jason.

Acked-by: Jason Liu <jason.hui@linaro.org>

>
> Acked-by: Stefano Babic <sbabic@denx.de>
>
> 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-0 Fax: +49-8142-66989-80  Email: office@denx.de
> =====================================================================
>
Stefano Babic - Sept. 23, 2011, 8:54 a.m.
On 09/22/2011 08:07 PM, Fabio Estevam wrote:
> Let common code set the machine ID.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  board/freescale/mx53evk/mx53evk.c |    1 -
>  include/configs/mx53evk.h         |    2 ++
>  2 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/board/freescale/mx53evk/mx53evk.c b/board/freescale/mx53evk/mx53evk.c
> index 81857ff..fd821c0 100644
> --- a/board/freescale/mx53evk/mx53evk.c
> +++ b/board/freescale/mx53evk/mx53evk.c
> @@ -355,7 +355,6 @@ int board_early_init_f(void)
>  
>  int board_init(void)
>  {
> -	gd->bd->bi_arch_number = MACH_TYPE_MX53_EVK;
>  	/* address of boot parameters */
>  	gd->bd->bi_boot_params = PHYS_SDRAM_1 + 0x100;
>  
> diff --git a/include/configs/mx53evk.h b/include/configs/mx53evk.h
> index 7fb1d9a..b127b06 100644
> --- a/include/configs/mx53evk.h
> +++ b/include/configs/mx53evk.h
> @@ -29,6 +29,8 @@
>  #define CONFIG_DISPLAY_CPUINFO
>  #define CONFIG_DISPLAY_BOARDINFO
>  
> +#define CONFIG_MACH_TYPE	MACH_TYPE_MX53_EVK
> +
>  #include <asm/arch/imx-regs.h>
>  
>  #define CONFIG_CMDLINE_TAG		1	/* enable passing of ATAGs */

Applied to u-boot-imx, next branch, thanks.

Best regards,
Stefano Babic

Patch

diff --git a/board/freescale/mx53evk/mx53evk.c b/board/freescale/mx53evk/mx53evk.c
index 81857ff..fd821c0 100644
--- a/board/freescale/mx53evk/mx53evk.c
+++ b/board/freescale/mx53evk/mx53evk.c
@@ -355,7 +355,6 @@  int board_early_init_f(void)
 
 int board_init(void)
 {
-	gd->bd->bi_arch_number = MACH_TYPE_MX53_EVK;
 	/* address of boot parameters */
 	gd->bd->bi_boot_params = PHYS_SDRAM_1 + 0x100;
 
diff --git a/include/configs/mx53evk.h b/include/configs/mx53evk.h
index 7fb1d9a..b127b06 100644
--- a/include/configs/mx53evk.h
+++ b/include/configs/mx53evk.h
@@ -29,6 +29,8 @@ 
 #define CONFIG_DISPLAY_CPUINFO
 #define CONFIG_DISPLAY_BOARDINFO
 
+#define CONFIG_MACH_TYPE	MACH_TYPE_MX53_EVK
+
 #include <asm/arch/imx-regs.h>
 
 #define CONFIG_CMDLINE_TAG		1	/* enable passing of ATAGs */