Acked-by: Sonic Zhang sonic.zhang@analog.com

Thanks,

Sonic

-----Original Message----- From: Masahiro Yamada [mailto:yamada.m@jp.panasonic.com] Sent: Thursday, July 03, 2014 12:56 PM To: u-boot@lists.denx.de Cc: Masahiro Yamada; Stefan Roese; Zhang, Sonic Subject: [PATCH] blackfin,powerpc: remove redundant definitions of ARRAY_SIZE

Since ARRAY_SIZE macro is defined in include/common.h, re-defining it in arch- specific files is redundant.

Signed-off-by: Masahiro Yamada yamada.m@jp.panasonic.com Cc: Stefan Roese sr@denx.de Cc: Sonic Zhang sonic.zhang@analog.com

---

I built all Blackfin and Powerpc boards. I confirmed this patch has no bad impact.

arch/blackfin/include/asm/config-pre.h | 3 --- arch/powerpc/cpu/ppc4xx/speed.c | 2 -- 2 files changed, 5 deletions(-)

diff --git a/arch/blackfin/include/asm/config-pre.h b/arch/blackfin/include/asm/config- pre.h index d0fd537..2d8b293 100644 --- a/arch/blackfin/include/asm/config-pre.h +++ b/arch/blackfin/include/asm/config-pre.h @@ -9,9 +9,6 @@ #ifndef __ASM_BLACKFIN_CONFIG_PRE_H__ #define __ASM_BLACKFIN_CONFIG_PRE_H__

-/* Misc helper functions */ -#define ARRAY_SIZE(x) (sizeof(x) / sizeof((x)[0]))

/* Bootmode defines -- your config needs to select this via CONFIG_BFIN_BOOT_MODE.

• Depending on your cpu, some of these may not be valid, check your HRM.
• The actual values here are meaningless as long as they're unique.

diff --git a/arch/powerpc/cpu/ppc4xx/speed.c b/arch/powerpc/cpu/ppc4xx/speed.c index 7e077d5..4baee77 100644 --- a/arch/powerpc/cpu/ppc4xx/speed.c +++ b/arch/powerpc/cpu/ppc4xx/speed.c @@ -19,8 +19,6 @@ DECLARE_GLOBAL_DATA_PTR; #define DEBUGF(fmt,args...) #endif

-#define ARRAY_SIZE(x) (sizeof(x) / sizeof((x)[0]))

#if defined(CONFIG_405GP)

void get_sys_info (PPC4xx_SYS_INFO * sysInfo)

1.9.1