Re: [U-Boot] [PATCH] powerpc/8xxx: Introduce 85xx, 86xx, QorIQ config headers

2011-01-19 Thread Kumar Gala
Wolfgang,

Please take a look.  I'd like to see this go in for v2011.03 to clean up the 
both the order issue, but also establish a cleaner means to handle SoC specific 
defines for 85xx/86xx/QorIQ going forward.

(will be helpful for things like SoC specific errata not having to get put into 
every board config.h, as well as remove duplication in those files -- which can 
wait for post v2011.03).

- k

On Jan 19, 2011, at 3:27 AM, Kumar Gala wrote:

 Add new headers that capture common defines for a given SoC/processor
 rather than duplicating that information in board config.h and random
 other places.
 
 Eventually this should be handled by Kconfig  defconfigs
 
 Signed-off-by: Kumar Gala ga...@kernel.crashing.org
 ---
 * I know its late for such a patch, but I think it best to introduce right 
 after
  the merge window closes so that patches for the next release deal with it and
  I didn't require everyone else to deal with for this merge window
 
  Also I think its a beneficial cleanup to address several long standing
  code duplication problems.
 
 arch/powerpc/include/asm/config.h |   41 ++-
 arch/powerpc/include/asm/config_mpc85xx.h |  187 +
 arch/powerpc/include/asm/config_mpc86xx.h |   38 ++
 drivers/misc/fsl_law.c|   22 +
 include/configs/P4080DS.h |   16 ---
 5 files changed, 236 insertions(+), 68 deletions(-)
 create mode 100644 arch/powerpc/include/asm/config_mpc85xx.h
 create mode 100644 arch/powerpc/include/asm/config_mpc86xx.h
 
 diff --git a/arch/powerpc/include/asm/config.h 
 b/arch/powerpc/include/asm/config.h
 index 1b9c47b..2b6b233 100644
 --- a/arch/powerpc/include/asm/config.h
 +++ b/arch/powerpc/include/asm/config.h
 @@ -21,6 +21,14 @@
 #ifndef _ASM_CONFIG_H_
 #define _ASM_CONFIG_H_
 
 +#ifdef CONFIG_MPC85xx
 +#include asm/config_mpc85xx.h
 +#endif
 +
 +#ifdef CONFIG_MPC86xx
 +#include asm/config_mpc86xx.h
 +#endif
 +
 #define CONFIG_LMB
 #define CONFIG_SYS_BOOT_RAMDISK_HIGH
 #define CONFIG_SYS_BOOT_GET_CMDLINE
 @@ -43,19 +51,7 @@
 #endif
 #endif
 
 -#if defined(CONFIG_MPC8572) || defined(CONFIG_P1020) || \
 - defined(CONFIG_P1021) || defined(CONFIG_P1022) || \
 - defined(CONFIG_P2020) || defined(CONFIG_MPC8641)
 -#define CONFIG_MAX_CPUS  2
 -#elif defined(CONFIG_PPC_P2040)
 -#define CONFIG_MAX_CPUS  4
 -#elif defined(CONFIG_PPC_P3041)
 -#define CONFIG_MAX_CPUS  4
 -#elif defined(CONFIG_PPC_P4080)
 -#define CONFIG_MAX_CPUS  8
 -#elif defined(CONFIG_PPC_P5020)
 -#define CONFIG_MAX_CPUS  2
 -#else
 +#ifndef CONFIG_MAX_CPUS
 #define CONFIG_MAX_CPUS   1
 #endif
 
 @@ -69,30 +65,13 @@
 #endif
 #endif
 
 -/* Enable TSEC2.0 for the platforms that have it if we are using TSEC */
 -#if defined(CONFIG_TSEC_ENET)  \
 -(defined(CONFIG_P1010) || defined(CONFIG_P1014) || \
 - defined(CONFIG_P1020) || defined(CONFIG_P1011))
 -#define CONFIG_TSECV2
 -#endif
 -
 /*
  * SEC (crypto unit) major compatible version determination
  */
 -#if defined(CONFIG_FSL_CORENET) || \
 -defined(CONFIG_P1010) || defined(CONFIG_P1014)
 -#define CONFIG_SYS_FSL_SEC_COMPAT4
 -#elif defined(CONFIG_MPC85xx) || defined(CONFIG_MPC83xx)
 +#if defined(CONFIG_MPC83xx)
 #define CONFIG_SYS_FSL_SEC_COMPAT 2
 #endif
 
 -/* Number of TLB CAM entries we have on FSL Book-E chips */
 -#if defined(CONFIG_E500MC)
 -#define CONFIG_SYS_NUM_TLBCAMS   64
 -#elif defined(CONFIG_E500)
 -#define CONFIG_SYS_NUM_TLBCAMS   16
 -#endif
 -
 /* Since so many PPC SOCs have a semi-common LBC, define this here */
 #if defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx) || \
   defined(CONFIG_MPC83xx)
 diff --git a/arch/powerpc/include/asm/config_mpc85xx.h 
 b/arch/powerpc/include/asm/config_mpc85xx.h
 new file mode 100644
 index 000..8ba6c7e
 --- /dev/null
 +++ b/arch/powerpc/include/asm/config_mpc85xx.h
 @@ -0,0 +1,187 @@
 +/*
 + * Copyright 2011 Freescale Semiconductor, Inc.
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License as
 + * published by the Free Software Foundation; either version 2 of
 + * the License, or (at your option) any later version.
 + *
 + * This program is distributed in the hope that it will be useful,
 + * but WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 + * GNU General Public License for more details.
 + *
 + * You should have received a copy of the GNU General Public License
 + * along with this program; if not, write to the Free Software
 + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
 + * MA 02111-1307 USA
 + *
 + */
 +
 +#ifndef _ASM_MPC85xx_CONFIG_H_
 +#define _ASM_MPC85xx_CONFIG_H_
 +
 +/* SoC specific defines for Freescale MPC85xx (PQ3) and QorIQ processors */
 +
 +/* Number of TLB CAM entries we have on FSL Book-E chips */
 +#if 

Re: [U-Boot] [PATCH] powerpc/8xxx: Introduce 85xx, 86xx, QorIQ config headers

2011-01-19 Thread Wolfgang Denk
Dear Kumar Gala,

In message 1295429250-25788-1-git-send-email-ga...@kernel.crashing.org you 
wrote:
 Add new headers that capture common defines for a given SoC/processor
 rather than duplicating that information in board config.h and random
 other places.
 
 Eventually this should be handled by Kconfig  defconfigs
 
 Signed-off-by: Kumar Gala ga...@kernel.crashing.org
 ---
 * I know its late for such a patch, but I think it best to introduce right 
 after
   the merge window closes so that patches for the next release deal with it 
 and
   I didn't require everyone else to deal with for this merge window
 
   Also I think its a beneficial cleanup to address several long standing
   code duplication problems.
 
  arch/powerpc/include/asm/config.h |   41 ++-
  arch/powerpc/include/asm/config_mpc85xx.h |  187 
 +
  arch/powerpc/include/asm/config_mpc86xx.h |   38 ++
  drivers/misc/fsl_law.c|   22 +
  include/configs/P4080DS.h |   16 ---
  5 files changed, 236 insertions(+), 68 deletions(-)
  create mode 100644 arch/powerpc/include/asm/config_mpc85xx.h
  create mode 100644 arch/powerpc/include/asm/config_mpc86xx.h

Acked-by: Wolfgang Denk w...@denx.de

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: w...@denx.de
It's a small world, but I wouldn't want to paint it.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] powerpc/8xxx: Introduce 85xx, 86xx, QorIQ config headers

2011-01-19 Thread Kumar Gala

On Jan 19, 2011, at 3:40 PM, Wolfgang Denk wrote:

 Dear Kumar Gala,
 
 In message 1295429250-25788-1-git-send-email-ga...@kernel.crashing.org you 
 wrote:
 Add new headers that capture common defines for a given SoC/processor
 rather than duplicating that information in board config.h and random
 other places.
 
 Eventually this should be handled by Kconfig  defconfigs
 
 Signed-off-by: Kumar Gala ga...@kernel.crashing.org
 ---
 * I know its late for such a patch, but I think it best to introduce right 
 after
  the merge window closes so that patches for the next release deal with it 
 and
  I didn't require everyone else to deal with for this merge window
 
  Also I think its a beneficial cleanup to address several long standing
  code duplication problems.
 
 arch/powerpc/include/asm/config.h |   41 ++-
 arch/powerpc/include/asm/config_mpc85xx.h |  187 
 +
 arch/powerpc/include/asm/config_mpc86xx.h |   38 ++
 drivers/misc/fsl_law.c|   22 +
 include/configs/P4080DS.h |   16 ---
 5 files changed, 236 insertions(+), 68 deletions(-)
 create mode 100644 arch/powerpc/include/asm/config_mpc85xx.h
 create mode 100644 arch/powerpc/include/asm/config_mpc86xx.h
 
 Acked-by: Wolfgang Denk w...@denx.de

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot