[U-Boot] [PATCH 1/1] arm: fix some arch_number definition

Wolfgang Denk wd at denx.de
Thu Jan 22 08:40:30 CET 2009


Dear Jean-Christophe PLAGNIOL-VILLARD,

In message <1232603494-14583-2-git-send-email-plagnioj at jcrosoft.com> you wrote:
> 
> diff --git a/board/armadillo/armadillo.c b/board/armadillo/armadillo.c
> index ca5bd1d..e9be758 100644
> --- a/board/armadillo/armadillo.c
> +++ b/board/armadillo/armadillo.c
> @@ -42,8 +42,7 @@ int board_init (void)
>  	/* Activate LED flasher */
>  	IO_LEDFLSH = 0x40;
>  
> -	/* arch number MACH_TYPE_ARMADILLO - not official*/
> -	gd->bd->bi_arch_number = 83;
> +	gd->bd->bi_arch_number = MACH_TYPE_ARMADILLO;
>  
>  	/* location of boot parameters */
>  	gd->bd->bi_boot_params = 0xc0000100;

You actually CHANGE the arch_number here; you need the board
maintainer's ACK for such a patch.

Note that there are other places where this is used - these must be
changed, too - for example:

"board/lpc2292sodimm/lpc2292sodimm.c":

 42         /* arch number MACH_TYPE_ARMADILLO - not official*/
 43         gd->bd->bi_arch_number = 83;

"board/siemens/SMN42/smn42.c":

 42         /* arch number MACH_TYPE_ARMADILLO - not official*/
 43         gd->bd->bi_arch_number = 83;

I recommend you got through this again, more thoroughly.

> index 3ad756d..205bd3a 100644
> --- a/board/cmc_pu2/cmc_pu2.c
> +++ b/board/cmc_pu2/cmc_pu2.c
> @@ -96,12 +96,7 @@ int board_init (void)
>  	piob->PIO_PER = AT91C_PIO_PB3 | AT91C_PIO_PB4 |
>  			AT91C_PIO_PB5 | AT91C_PIO_PB6;
>  
> -	/*
> -	 * arch number of CMC_PU2-Board. MACH_TYPE_CMC_PU2 is not supported in
> -	 * the linuxarm kernel, yet.
> -	 */
> -	/* gd->bd->bi_arch_number = MACH_TYPE_CMC_PU2; */
> -	gd->bd->bi_arch_number = 251;
> +	gd->bd->bi_arch_number = MACH_TYPE_CMC_PU2;
>  	/* adress of boot parameters */
>  	gd->bd->bi_boot_params = PHYS_SDRAM + 0x100;

You actually CHANGE the arch_number here; you need the board
maintainer's ACK for such a patch.

> diff --git a/board/imx31_phycore/imx31_phycore.c b/board/imx31_phycore/imx31_phycore.c
> index ae93444..4b72d57 100644
> --- a/board/imx31_phycore/imx31_phycore.c
> +++ b/board/imx31_phycore/imx31_phycore.c
> @@ -60,7 +60,7 @@ int board_init (void)
>  	mx31_gpio_mux(MUX_CSPI2_MOSI__I2C2_SCL);
>  	mx31_gpio_mux(MUX_CSPI2_MISO__I2C2_SDA);
>  
> -	gd->bd->bi_arch_number = 447;		/* board id for linux */
> +	gd->bd->bi_arch_number = MACH_TYPE_MX31ADS;	/* board id for linux */
>  	gd->bd->bi_boot_params = (0x80000100);	/* adress of boot parameters */

MX31ADS ??

> diff --git a/board/ns9750dev/ns9750dev.c b/board/ns9750dev/ns9750dev.c
> index fc46244..b8f377b 100644
> --- a/board/ns9750dev/ns9750dev.c
> +++ b/board/ns9750dev/ns9750dev.c
> @@ -68,7 +68,7 @@ int board_init( void )
>  #warning Please register your machine at http://www.arm.linux.org.uk/developer/machines/?action=new
>  	/* arch number of OMAP 1510-Board */
>  	/* to be changed for OMAP 1610 Board */
> -	gd->bd->bi_arch_number = 234;
> +	gd->bd->bi_arch_number = MACH_TYPE_NS9750;

You actually CHANGE the arch_number here; you need the board
maintainer's ACK for such a patch.



NAK as is. This needs to be reworked.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Real computer scientists don't comment their  code.  The  identifiers
are so long they can't afford the disk space.


More information about the U-Boot mailing list