Message ID | 1404363351-25351-1-git-send-email-yamada.m@jp.panasonic.com |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
On 03.07.2014 06:55, Masahiro Yamada wrote: > 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> Thanks for catching this. Acked-by: Stefan Roese <sr@denx.de> Thanks, Stefan
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
On Thu, Jul 03, 2014 at 01:55:51PM +0900, Masahiro Yamada wrote: > 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> > Acked-by: Stefan Roese <sr@denx.de> > Acked-by: Sonic Zhang <sonic.zhang@analog.com> Applied to u-boot/master, thanks!
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)
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(-)