[U-Boot] [PATCH 1/1] arm: unify some CONFIGs

Wolfgang Denk wd at denx.de
Sun May 17 23:57:48 CEST 2009


Dear Jean-Christophe PLAGNIOL-VILLARD,

In message <1242593505-27370-1-git-send-email-plagnioj at jcrosoft.com> you wrote:
> all arm boards share the same following define
> 
> size in bytes reserved for initial data
> CONFIG_SYS_GBL_DATA_SIZE
> 
> max number of command args
> CONFIG_SYS_MAXARGS
> 
> IRQ stack
> CONFIG_STACKSIZE_IRQ
> 
> FIQ stack
> CONFIG_STACKSIZE_FIQ

I think many boards share other defines as well; I don't think that
splitting these few lines off is really something that improves the
code.

> diff --git a/include/asm-arm/config.h b/include/asm-arm/config.h
> index 049c44e..795b53d 100644
> --- a/include/asm-arm/config.h
> +++ b/include/asm-arm/config.h
> @@ -21,4 +21,16 @@
>  #ifndef _ASM_CONFIG_H_
>  #define _ASM_CONFIG_H_
>  
> +/* size in bytes reserved for initial data */
> +#define CONFIG_SYS_GBL_DATA_SIZE	128
> +
> +/* max number of command args */
> +#define CONFIG_SYS_MAXARGS		16
> +
> +/* IRQ stack */
> +#define CONFIG_STACKSIZE_IRQ		(4 * 1024)
> +
> +/* FIQ stack */
> +#define CONFIG_STACKSIZE_FIQ		(4 * 1024)

NAK. All config related information shall remain in the
include/configs/ directory only.


Also, this patch makes it impossible for a board to chose different
configurations.

Assume a board wants to use more than 16 arguments. With your patch, a
global, common file would need to be changed for such a simple thing.

This is contra-productive.


Sorry, but full NAK.

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
If something is different, it's either better or worse,  and  usually
both.                                                    - Larry Wall


More information about the U-Boot mailing list