[U-Boot] [PATCH 4/6] powerpc/p1021: add more P1021 defines.

2011-01-31 Thread Haiying.Wang
From: Haiying Wang haiying.w...@freescale.com

Signed-off-by: Haiying Wang haiying.w...@freescale.com
---
 arch/powerpc/include/asm/immap_85xx.h |6 ++
 arch/powerpc/include/asm/immap_qe.h   |9 +++--
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/arch/powerpc/include/asm/immap_85xx.h 
b/arch/powerpc/include/asm/immap_85xx.h
index 6bd83ba..77e3629 100644
--- a/arch/powerpc/include/asm/immap_85xx.h
+++ b/arch/powerpc/include/asm/immap_85xx.h
@@ -1948,6 +1948,12 @@ typedef struct ccsr_gur {
u8  res10b[76];
par_io_t qe_par_io[7];
u8  res10c[1600];
+#elif defined(CONFIG_P1021)
+   u8  res10b1[12];
+   u32 iovselsr;
+   u8  res10b2[60];
+   par_io_t qe_par_io[3];
+   u8  res10c[1496];
 #else
u8  res10b[1868];
 #endif
diff --git a/arch/powerpc/include/asm/immap_qe.h 
b/arch/powerpc/include/asm/immap_qe.h
index 531cfc8..0fffba2 100644
--- a/arch/powerpc/include/asm/immap_qe.h
+++ b/arch/powerpc/include/asm/immap_qe.h
@@ -3,7 +3,7 @@
  * The Internal Memory Map for devices with QE on them. This
  * is the superset of all QE devices (8360, etc.).
  *
- * Copyright (c) 2006-2009 Freescale Semiconductor, Inc.
+ * Copyright (c) 2006-2011 Freescale Semiconductor, Inc.
  * Author: Shlomi Gridih grid...@freescale.com
  *
  * This program is free software; you can redistribute  it and/or modify it
@@ -588,6 +588,9 @@ typedef struct qe_immap {
 #elif defined(CONFIG_MPC8569)
u8 muram[0x2];  /* 0x1_ -  0x3_ Multi-user RAM */
u8 res17[0x1];  /* 0x3_ -  0x4_ */
+#elif defined(CONFIG_P1021)
+   u8 muram[0x06000];  /* 0x1_ -  0x1_6000 Multi-user RAM */
+   u8 res17[0x1a000];  /* 0x1_6000 -  0x3_ */
 #else
u8 muram[0xC000];   /* 0x11 -  0x11C000 Multi-user RAM */
u8 res17[0x24000];  /* 0x11C000 -  0x14 */
@@ -601,13 +604,15 @@ extern qe_map_t *qe_immr;
 #define QE_MURAM_SIZE  0x1UL
 #elif defined(CONFIG_MPC8569)
 #define QE_MURAM_SIZE  0x2UL
+#elif defined(CONFIG_P1021)
+#define QE_MURAM_SIZE  0x6000UL
 #elif defined(CONFIG_MPC8360)
 #define QE_MURAM_SIZE  0xc000UL
 #elif defined(CONFIG_MPC832x)
 #define QE_MURAM_SIZE  0x4000UL
 #endif
 
-#if defined(CONFIG_MPC8323)
+#if defined(CONFIG_MPC8323) || defined(CONFIG_P1021)
 #define MAX_QE_RISC 1
 #define QE_NUM_OF_SNUM 28
 #elif defined(CONFIG_MPC8569)
-- 
1.7.3.1.50.g1e633


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


Re: [U-Boot] [PATCH 4/6] powerpc/p1021: add more P1021 defines.

2011-01-31 Thread Wolfgang Denk
Dear haiying.w...@freescale.com,

In message 1296499317-26616-5-git-send-email-haiying.w...@freescale.com you 
wrote:
 From: Haiying Wang haiying.w...@freescale.com
 
 Signed-off-by: Haiying Wang haiying.w...@freescale.com
 ---
  arch/powerpc/include/asm/immap_85xx.h |6 ++
  arch/powerpc/include/asm/immap_qe.h   |9 +++--
  2 files changed, 13 insertions(+), 2 deletions(-)
 
 diff --git a/arch/powerpc/include/asm/immap_85xx.h 
 b/arch/powerpc/include/asm/immap_85xx.h
 index 6bd83ba..77e3629 100644
 --- a/arch/powerpc/include/asm/immap_85xx.h
 +++ b/arch/powerpc/include/asm/immap_85xx.h
 @@ -1948,6 +1948,12 @@ typedef struct ccsr_gur {
   u8  res10b[76];
   par_io_t qe_par_io[7];
   u8  res10c[1600];
 +#elif defined(CONFIG_P1021)
 + u8  res10b1[12];
 + u32 iovselsr;
 + u8  res10b2[60];
 + par_io_t qe_par_io[3];
 + u8  res10c[1496];
  #else

res10b1? Two levels of insertions already. Isn't it time to renumber
the reserved fields?


 @@ -601,13 +604,15 @@ extern qe_map_t *qe_immr;
  #define QE_MURAM_SIZE0x1UL
  #elif defined(CONFIG_MPC8569)
  #define QE_MURAM_SIZE0x2UL
 +#elif defined(CONFIG_P1021)
 +#define QE_MURAM_SIZE  0x6000UL
  #elif defined(CONFIG_MPC8360)
  #define QE_MURAM_SIZE0xc000UL
  #elif defined(CONFIG_MPC832x)
  #define QE_MURAM_SIZE0x4000UL
  #endif

Can you please keep the if defined(..) list sorted? Thanks.

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
If you want strict real-time behavior, run in the real  time  schedu-
ling class.  But there are no seatbelts or airbags;  main(){for(;;);}
can hard hang your system.  -- Bart Smaalders
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 4/6] powerpc/p1021: add more P1021 defines.

2011-01-31 Thread Kumar Gala

On Jan 31, 2011, at 12:41 PM, haiying.w...@freescale.com 
haiying.w...@freescale.com wrote:

 From: Haiying Wang haiying.w...@freescale.com
 
 Signed-off-by: Haiying Wang haiying.w...@freescale.com
 ---
 arch/powerpc/include/asm/immap_85xx.h |6 ++
 arch/powerpc/include/asm/immap_qe.h   |9 +++--
 2 files changed, 13 insertions(+), 2 deletions(-)
 
 diff --git a/arch/powerpc/include/asm/immap_85xx.h 
 b/arch/powerpc/include/asm/immap_85xx.h
 index 6bd83ba..77e3629 100644
 --- a/arch/powerpc/include/asm/immap_85xx.h
 +++ b/arch/powerpc/include/asm/immap_85xx.h
 @@ -1948,6 +1948,12 @@ typedef struct ccsr_gur {
   u8  res10b[76];
   par_io_t qe_par_io[7];
   u8  res10c[1600];
 +#elif defined(CONFIG_P1021)
 + u8  res10b1[12];
 + u32 iovselsr;
 + u8  res10b2[60];
 + par_io_t qe_par_io[3];
 + u8  res10c[1496];
 #else
   u8  res10b[1868];
 #endif
 diff --git a/arch/powerpc/include/asm/immap_qe.h 
 b/arch/powerpc/include/asm/immap_qe.h
 index 531cfc8..0fffba2 100644
 --- a/arch/powerpc/include/asm/immap_qe.h
 +++ b/arch/powerpc/include/asm/immap_qe.h
 @@ -3,7 +3,7 @@
  * The Internal Memory Map for devices with QE on them. This
  * is the superset of all QE devices (8360, etc.).
  *
 - * Copyright (c) 2006-2009 Freescale Semiconductor, Inc.
 + * Copyright (c) 2006-2011 Freescale Semiconductor, Inc.
  * Author: Shlomi Gridih grid...@freescale.com
  *
  * This program is free software; you can redistribute  it and/or modify it
 @@ -588,6 +588,9 @@ typedef struct qe_immap {
 #elif defined(CONFIG_MPC8569)
   u8 muram[0x2];  /* 0x1_ -  0x3_ Multi-user RAM */
   u8 res17[0x1];  /* 0x3_ -  0x4_ */
 +#elif defined(CONFIG_P1021)
 + u8 muram[0x06000];  /* 0x1_ -  0x1_6000 Multi-user RAM */
 + u8 res17[0x1a000];  /* 0x1_6000 -  0x3_ */
 #else
   u8 muram[0xC000];   /* 0x11 -  0x11C000 Multi-user RAM */
   u8 res17[0x24000];  /* 0x11C000 -  0x14 */

Can we reduce this mess with using QE_MURAM_SIZE?

u8 muram[QE_MURAM_SIZE];
u8 res17[0x - QE_MURAM_SIZE];


 @@ -601,13 +604,15 @@ extern qe_map_t *qe_immr;
 #define QE_MURAM_SIZE 0x1UL
 #elif defined(CONFIG_MPC8569)
 #define QE_MURAM_SIZE 0x2UL
 +#elif defined(CONFIG_P1021)
 +#define QE_MURAM_SIZE  0x6000UL
 #elif defined(CONFIG_MPC8360)
 #define QE_MURAM_SIZE 0xc000UL
 #elif defined(CONFIG_MPC832x)
 #define QE_MURAM_SIZE 0x4000UL
 #endif
 
 -#if defined(CONFIG_MPC8323)
 +#if defined(CONFIG_MPC8323) || defined(CONFIG_P1021)
 #define MAX_QE_RISC 1
 #define QE_NUM_OF_SNUM28
 #elif defined(CONFIG_MPC8569)

We can move some of these into include/config_mpc85xx.h

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

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


Re: [U-Boot] [PATCH 4/6] powerpc/p1021: add more P1021 defines.

2011-01-31 Thread Timur Tabi
On Mon, Jan 31, 2011 at 3:08 PM, Kumar Gala ga...@kernel.crashing.org wrote:

 @@ -588,6 +588,9 @@ typedef struct qe_immap {
 #elif defined(CONFIG_MPC8569)
       u8 muram[0x2];      /* 0x1_ -  0x3_ Multi-user RAM */
       u8 res17[0x1];      /* 0x3_ -  0x4_ */
 +#elif defined(CONFIG_P1021)
 +     u8 muram[0x06000];      /* 0x1_ -  0x1_6000 Multi-user RAM */
 +     u8 res17[0x1a000];      /* 0x1_6000 -  0x3_ */
 #else
       u8 muram[0xC000];       /* 0x11 -  0x11C000 Multi-user RAM */
       u8 res17[0x24000];      /* 0x11C000 -  0x14 */

 Can we reduce this mess with using QE_MURAM_SIZE?

        u8 muram[QE_MURAM_SIZE];
        u8 res17[0x - QE_MURAM_SIZE];

I don't think we need res17, because nothing references it.  That will
simplify it even more.

-- 
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 4/6] powerpc/p1021: add more P1021 defines.

2011-01-31 Thread Kumar Gala

On Jan 31, 2011, at 5:36 PM, Timur Tabi wrote:

 On Mon, Jan 31, 2011 at 3:08 PM, Kumar Gala ga...@kernel.crashing.org wrote:
 
 @@ -588,6 +588,9 @@ typedef struct qe_immap {
 #elif defined(CONFIG_MPC8569)
   u8 muram[0x2];  /* 0x1_ -  0x3_ Multi-user RAM */
   u8 res17[0x1];  /* 0x3_ -  0x4_ */
 +#elif defined(CONFIG_P1021)
 + u8 muram[0x06000];  /* 0x1_ -  0x1_6000 Multi-user RAM */
 + u8 res17[0x1a000];  /* 0x1_6000 -  0x3_ */
 #else
   u8 muram[0xC000];   /* 0x11 -  0x11C000 Multi-user RAM */
   u8 res17[0x24000];  /* 0x11C000 -  0x14 */
 
 Can we reduce this mess with using QE_MURAM_SIZE?
 
u8 muram[QE_MURAM_SIZE];
u8 res17[0x - QE_MURAM_SIZE];
 
 I don't think we need res17, because nothing references it.  That will
 simplify it even more.

Looks like qe_immap isn't embedded anywhere so should be ok (was concerned if 
the struct is expected to be a given size.

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