[PATCH 06/10] [ARM] pxa: clean code in hardware.h

Eric Miao eric.y.miao at gmail.com
Wed May 5 02:00:56 EDT 2010


On Thu, Apr 29, 2010 at 4:18 PM, Haojian Zhuang
<haojian.zhuang at gmail.com> wrote:
> From 214c0dd2536c7eb55359bb46deb9b1126e1af822 Mon Sep 17 00:00:00 2001
> From: Haojian Zhuang <haojian.zhuang at marvell.com>
> Date: Fri, 12 Mar 2010 08:53:47 -0500
> Subject: [PATCH] [ARM] pxa: clean code in hardware.h
>
> Avoid too much conditional compilation in hardware.h.
>

Mmm.... this avoids the trivial optimization of each CPU being
compiled in or not.

> Signed-off-by: Haojian Zhuang <haojian.zhuang at marvell.com>
> ---
>  arch/arm/mach-pxa/include/mach/hardware.h |   27 ++++++---------------------
>  1 files changed, 6 insertions(+), 21 deletions(-)
>
> diff --git a/arch/arm/mach-pxa/include/mach/hardware.h
> b/arch/arm/mach-pxa/include/mach/hardware.h
> index 3d8d8cb..6694643 100644
> --- a/arch/arm/mach-pxa/include/mach/hardware.h
> +++ b/arch/arm/mach-pxa/include/mach/hardware.h
> @@ -148,63 +148,48 @@
>  #define __cpu_is_pxa27x(id)    (0)
>  #endif
>
> -#ifdef CONFIG_CPU_PXA300
> +#ifdef CONFIG_PXA3xx
>  #define __cpu_is_pxa300(id)                            \
>        ({                                              \
>                unsigned int _id = (id) >> 4 & 0xfff;   \
>                _id == 0x688;                           \
>         })
> -#else
> -#define __cpu_is_pxa300(id)    (0)
> -#endif
>
> -#ifdef CONFIG_CPU_PXA310
>  #define __cpu_is_pxa310(id)                            \
>        ({                                              \
>                unsigned int _id = (id) >> 4 & 0xfff;   \
>                _id == 0x689;                           \
>         })
> -#else
> -#define __cpu_is_pxa310(id)    (0)
> -#endif
>
> -#ifdef CONFIG_CPU_PXA320
>  #define __cpu_is_pxa320(id)                            \
>        ({                                              \
>                unsigned int _id = (id) >> 4 & 0xfff;   \
>                _id == 0x603 || _id == 0x682;           \
>         })
> -#else
> -#define __cpu_is_pxa320(id)    (0)
> -#endif
>
> -#ifdef CONFIG_CPU_PXA930
>  #define __cpu_is_pxa930(id)                            \
>        ({                                              \
>                unsigned int _id = (id) >> 4 & 0xfff;   \
>                _id == 0x683;                           \
>         })
> -#else
> -#define __cpu_is_pxa930(id)    (0)
> -#endif
>
> -#ifdef CONFIG_CPU_PXA935
>  #define __cpu_is_pxa935(id)                            \
>        ({                                              \
>                unsigned int _id = (id) >> 4 & 0xfff;   \
>                _id == 0x693;                           \
>         })
> -#else
> -#define __cpu_is_pxa935(id)    (0)
> -#endif
>
> -#ifdef CONFIG_CPU_PXA950
>  #define __cpu_is_pxa950(id)                             \
>        ({                                              \
>                unsigned int _id = (id) >> 4 & 0xfff;   \
>                _id == 0x697;                           \
>         })
>  #else
> +#define __cpu_is_pxa300(id)    (0)
> +#define __cpu_is_pxa310(id)    (0)
> +#define __cpu_is_pxa320(id)    (0)
> +#define __cpu_is_pxa930(id)    (0)
> +#define __cpu_is_pxa935(id)    (0)
>  #define __cpu_is_pxa950(id)    (0)
>  #endif
>
> --
> 1.5.6.5
>


More information about the linux-arm-kernel mailing list