On Mon, Jul 30, 2007 at 07:16:28PM +0400, Valentine Barshak wrote:
> AMCC 440EPx Sequoia board support.
> 
> Signed-off-by: Valentine Barshak <[EMAIL PROTECTED]>
> ---
>  arch/powerpc/kernel/cputable.c       |   36 +++++++++++++++++++
>  arch/powerpc/kernel/head_44x.S       |    2 -
>  arch/powerpc/platforms/44x/Kconfig   |   17 ++++++++-
>  arch/powerpc/platforms/44x/Makefile  |    1 
>  arch/powerpc/platforms/44x/sequoia.c |   66 
> +++++++++++++++++++++++++++++++++++
>  5 files changed, 120 insertions(+), 2 deletions(-)
> 
> diff -ruN linux.orig/arch/powerpc/kernel/cputable.c 
> linux/arch/powerpc/kernel/cputable.c
> --- linux.orig/arch/powerpc/kernel/cputable.c 2007-07-27 20:37:10.000000000 
> +0400
> +++ linux/arch/powerpc/kernel/cputable.c      2007-07-27 20:44:26.000000000 
> +0400
> @@ -1132,6 +1132,42 @@
>               .dcache_bsize           = 32,
>               .platform               = "ppc440",
>       },
> +     { /* 440EPX  - with Security/Kasumi  */
> +             .pvr_mask               = 0xf0000fff,
> +             .pvr_value              = 0x200008D0,
> +             .cpu_name               = "440EPX - with Security/Kasumi",
> +             .cpu_features           = CPU_FTRS_44X,
> +             .cpu_user_features      = COMMON_USER_BOOKE | 
> PPC_FEATURE_HAS_FPU, /* 440EPX has an FPU */
> +             .icache_bsize           = 32,
> +             .dcache_bsize           = 32,
> +     },
> +     { /* 440EPX  - without Security/Kasumi  */
> +             .pvr_mask               = 0xf0000fff,
> +             .pvr_value              = 0x200008D4,
> +             .cpu_name               = "440EPX - no Security/Kasumi",
> +             .cpu_features           = CPU_FTRS_44X,
> +             .cpu_user_features      = COMMON_USER_BOOKE | 
> PPC_FEATURE_HAS_FPU, /* 440EPX has an FPU */
> +             .icache_bsize           = 32,
> +             .dcache_bsize           = 32,
> +     },

Since the with/without Security/Kasumi versions have no differences in
their cputable entry other than the PVR, couldn't you just remove the
relevant PVR bit from the mask and use a single entry?

-- 
David Gibson                    | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
                                | _way_ _around_!
http://www.ozlabs.org/~dgibson
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to