Re: [U-Boot-Users] [Patch 1/1] Re-Submit: QE UEC: Add MII Commands

2008-07-29 Thread Kim Phillips
On Tue, 29 Jul 2008 11:10:09 -0400
richardretanubun [EMAIL PROTECTED] wrote:

 Patch originally submitted by David Saada
 
 http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/38946
 
 Shamelessly copy-pasted below.
 
 Richard
 ===

the proper way to maintain correct attributions in this case is to
replace all the above with a simple:

From: David Saada [EMAIL PROTECTED]

as the first line in the email body.  git-am will now assign the
correct author when applying it to the tree.

 Add MII commands to the UEC driver. Note that once a UEC device is selected,
 any device on its MDIO bus can be addressed.
 
 Signed-off-by: David Saada david.saada at ecitele.com

please feel free to be a part of the community and add your Tested-by,
Reviewed-by, Acked-by, and/or signoff.  See
linux-2.6/Documentation/SubmittingPatches for further details.

Kim

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] fdt/fdt_fixup_ethernet

2008-07-28 Thread Kim Phillips
On Mon, 23 Jun 2008 16:00:28 +0200
Marc Leeman [EMAIL PROTECTED] wrote:

 Since I moved to FDT (from the previous OF implementation), I get the
 following behaviour.

..

 However, the MAC addresses that are passed to the kernel are a mix of
 the hardcoded defaults and the ones retrieved from the mac flash:

you probably need to #define CONFIG_HAS_ETH0 (and 1) in your board's
config file.  Make sure your device tree has ethernet aliases, too (see
existing dts alias entries in linux).

Kim

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Freescale MPC8349EMDS hang on boot

2008-07-17 Thread Kim Phillips
On Wed, 16 Jul 2008 15:28:21 -0700
Ira Snyder [EMAIL PROTECTED] wrote:

 During the development, I noticed that adding and removing certain
 pieces of debugging code (which did not change any program state) caused
 the board to hang on boot, like so:
 
 
 U-Boot 1.3.4-rc1-1-gb89881c-dirty (Jul 16 2008 - 14:54:56) MPC83XX
 
 Reset Status: External/Internal Soft, External/Internal Hard
 
 CPU:   e300c1, MPC8349EA, Rev: 3.0 at 528 MHz, CSB: 264 MHz
 Board: Freescale MPC8349EMDS
 I2C:   

try reverting this commit:

commit 1c3dd43338a077165e7e0309cb3994e65d2bdbf8
Author: Grant Likely [EMAIL PROTECTED]
Date:   Tue Nov 13 22:18:33 2007 -0700

powerpc: Backout relocation changes.

Ugh.  I *hate* to back this change out, but these compiler flags don't
work for relocation on all versions of GCC.  I've not been able to
reproduce the environment in my setup (and hence, not been able to
find a combination that *does* work), so I've got no choice but to go
back to the old gcc flags and linker script.

Signed-off-by: Grant Likely [EMAIL PROTECTED]

Kim

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH v4] fdt: add crypto node handling for MPC8{3, 5}xxE processors

2008-07-11 Thread Kim Phillips
On Mon, 7 Jul 2008 13:46:04 -0500
Kim Phillips [EMAIL PROTECTED] wrote:

 On Sun, 06 Jul 2008 00:32:03 +0200
 Wolfgang Denk [EMAIL PROTECTED] wrote:
 
  Dear Kim,
  
  in message [EMAIL PROTECTED] you wrote:
   Delete the crypto node if not on an E-processor.  If on 8360 or 834x 
   family,
   check rev and up-rev crypto node (to SEC rev. 2.4 property values)
   if on an 'EA' processor, e.g. MPC8349EA.
   
   Signed-off-by: Kim Phillips [EMAIL PROTECTED]
   ---
   diff from v3: prepended Delete the to the patch commit message.
   mumble.  Thanks gvb.
  
  Just for the record (note that I've been offline for the last ten
  days): I didn't apply this yet as I wait for v5 (see Kumar's
  comments).
 
 Hi WD, I addressed Kumar's comments in a separate patch (to go through
 the mpc85xx.git tree) here:
 
 http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/42725
 
 Andy, please apply.
 
 Kim

Wolfgang, please apply v4 of the patch in this thread.

Thanks,

Kim

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] 83xx: mpc8315erdb: fix silly thinko in fdt_tsec1_fixup

2008-07-11 Thread Kim Phillips
On Thu, 10 Jul 2008 17:20:51 +0400
Anton Vorontsov [EMAIL PROTECTED] wrote:

 The thinko was quite silly indeed, I messed with !ptr. Normally this
 would trigger some fault, but in U-Boot NULL pointer is equal to phys
 0, so the code was working still, just didn't actually test mpc8315erdb
 environment variable value. Heh.
 
 Signed-off-by: Anton Vorontsov [EMAIL PROTECTED]
 ---

applied to u-boot-mpc83xx.git#next.

Thanks Anton,

Kim

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] 8xxx-fdt: set ns16550 clock from CFG_NS16550_CLK, not bi_busfreq

2008-07-09 Thread Kim Phillips
On Wed,  9 Jul 2008 14:43:46 -0400
Paul Gortmaker [EMAIL PROTECTED] wrote:

 Some boards that have external 16550 UARTs don't have a direct
 tie between bi_busfreq and the clock used for the UARTs.  Boards
 that do have such a tie should set CFG_NS16550_CLK to be
 get_bus_freq(0) -- which most of them do already.
 
 Signed-off-by: Paul Gortmaker [EMAIL PROTECTED]

Acked-by: Kim Phillips [EMAIL PROTECTED]

Kim


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH v4] fdt: add crypto node handling for MPC8{3, 5}xxE processors

2008-07-07 Thread Kim Phillips
On Sun, 06 Jul 2008 00:32:03 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 Dear Kim,
 
 in message [EMAIL PROTECTED] you wrote:
  Delete the crypto node if not on an E-processor.  If on 8360 or 834x family,
  check rev and up-rev crypto node (to SEC rev. 2.4 property values)
  if on an 'EA' processor, e.g. MPC8349EA.
  
  Signed-off-by: Kim Phillips [EMAIL PROTECTED]
  ---
  diff from v3: prepended Delete the to the patch commit message.
  mumble.  Thanks gvb.
 
 Just for the record (note that I've been offline for the last ten
 days): I didn't apply this yet as I wait for v5 (see Kumar's
 comments).

Hi WD, I addressed Kumar's comments in a separate patch (to go through
the mpc85xx.git tree) here:

http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/42725

Andy, please apply.

Kim

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Release status - things to be done

2008-07-07 Thread Kim Phillips
On Sun, 06 Jul 2008 01:05:40 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 * We have incompatible definitions of the CPU_TYPE_ENTRY for MPC83xx
   systems:
   
   In include/asm-ppc/processor.h:
 
973 #define CPU_TYPE_ENTRY(n, v) \
974 { .name = #n, .soc_ver = SVR_##v, }
 
   But in cpu/mpc83xx/cpu.c:
 
47 #define CPU_TYPE_ENTRY(x) {#x, SPR_##x}
 
   This causes ``CPU_TYPE_ENTRY redefined'' warnings that need to get
   fixed.
 
   - for MPC83xx custodian
 
you have already pulled a fix for this (commit
4890246a2c5df90a74e2941e3673a49bbd36aee9).

  3564  06/09 Kim Phillips   [U-Boot-Users] [PATCH] fdt_support: add 
 crypto node handling for MPC8{3, 5}xxE processors
  3621  06/10 Kumar Gala Re: [U-Boot-Users] [PATCH] fdt_support: add 
 crypto node handling for MPC8{3, 5}xxE processors
  3630  06/10 Kim Phillips   [PATCH v2] fdt: add crypto node handling for 
 MPC8{3,5}xxE processors
  3631  06/10 Jerry Van BarenRe: [PATCH v2] fdt: add crypto node handling 
 for MPC8{3,5}xxE processors
  3895  06/16 Kim Phillips   [PATCH v3] fdt: add crypto node handling for 
 MPC8{3,5}xxE processors
  3896  06/16 Kim Phillips   [PATCH v4] fdt: add crypto node handling for 
 MPC8{3, 5}xxE processors
  3925  06/17 Kumar Gala Re: [PATCH v4] fdt: add crypto node handling 
 for MPC8{3, 5}xxE processors
   - asked submitter about state: resubmit cleaned up version?

I responded to your query in that thread; please go ahead and apply v4
as Kumar's comments are addressed in a separate (85xx) patch.

  3935  06/17 Kim Phillips   [U-Boot-Users] [PATCH 2/2] mpc83xx: move 
 CPU_TYPE_ENTRY over to processor.h
   - for 83xx custodian

you have already pulled this (same commit as above).

Kim

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] Please pull u-boot-mpc83xx.git

2008-07-03 Thread Kim Phillips
WD/Detlev,

please pull mostly MVBLM7 board fixes:

The following changes since commit 914f58c5766860373a7d232e961cee5a4b54a55b:
  Wolfgang Denk (1):
Merge branch 'master' of git://www.denx.de/git/u-boot-cfi-flash

are available in the git repository at:

  git://git.denx.de/u-boot-mpc83xx.git

Andre Schwarz (3):
  fix system config overwrite @ MPC834x and MPC8313
  fix non-working mvBL-M7
  update mvBL-M7 board config

Kim Phillips (1):
  mpc83xx: move CPU_TYPE_ENTRY over to processor.h

 board/mvblm7/mvblm7.c   |2 +-
 cpu/mpc83xx/cpu.c   |1 -
 cpu/mpc83xx/cpu_init.c  |5 +
 include/asm-ppc/processor.h |7 +++
 include/configs/MVBLM7.h|5 +++--
 5 files changed, 16 insertions(+), 4 deletions(-)

Thanks,

Kim

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] TSEC/PHY @ MPC834x stopped working

2008-07-02 Thread Kim Phillips
On Wed, 02 Jul 2008 16:36:11 +0200
Andre Schwarz [EMAIL PROTECTED] wrote:

 Ben,
 
 thanks for your quick reply.
 Looks like you're right - nothing changed regarding network yet.
 
 Maybe it's a cpu specific problem.
 Kim ? Can you help here ?
 

I know of nothing (barring this being the ETHADDR not specified
problem) that could have caused this. Can you try using git-bisect?

Kim

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] TSEC/PHY @ MPC834x stopped working

2008-07-02 Thread Kim Phillips
On Wed, 02 Jul 2008 18:01:01 +0200
Andre Schwarz [EMAIL PROTECTED] wrote:

 this is a fully configured board, i.e. serial# and ethaddr/eth1addr set
 up correctly.
 
 I'll give the bisect a try a soon as there'll be some time.
 
just to clarify, are you using the mainline u-boot.git tree, or
u-boot-mpc83xx.git?  you should be using the latter until I ask WD to
pull.  Actually, if you can confirm it corrects your problem, I have no
reason to delay it further (I was waiting for WD to get back from
vacation).

Kim

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] RRvision now uses the CFI driver so board/RRvision/flash.c gets deleted.

2008-06-25 Thread Kim Phillips
On Mon, 23 Jun 2008 17:28:25 +0200
Gary Jennejohn [EMAIL PROTECTED] wrote:

 
  Signed-off-by: Gary Jennejohn [EMAIL PROTECTED]
 
 ---
  board/RRvision/flash.c |  522 
 
  1 files changed, 0 insertions(+), 522 deletions(-)
  delete mode 100644 board/RRvision/flash.c

merely deleting a file required to build a board unnecessarily breaks
future git-bisect runs for it.

Suggest removing flash-related stuff from your next poorly titled
update patch and adding those changes to this one.

Also please indicate any dependency chain via Subject: [PATCH x/n]
subsystem:  nomenclature so everybody knows where the dependencies are
in your patchseries, and what subsystems are affected (mpc8xx in this
case?).

Thanks,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] fix system config overwrite @ MPC834x and MPC8313

2008-06-25 Thread Kim Phillips
On Mon, 23 Jun 2008 11:40:56 +0200
Andre Schwarz [EMAIL PROTECTED] wrote:

 During 83xx setup the System I/O configuration register high gets
 overwritten with user defined value if CFG_SICRH is defined.
 
 Regarding to the MPC834x manual (Table 5-28 reve.1) bits 28+29 of SICRH 
 must keep their reset value regardless of configuration.
 
 On my board (using RGMII) those bits are set after reset - yet it's 
 unclear where they come from.
 
 The patch keeps both bits on MPC834x and MPC8313.
 
 Signed-off-by: Andre Schwarz [EMAIL PROTECTED]

this has been applied after a slight rewrite to not cause the following:

cpu_init.c: In function ‘cpu_init_f’:
cpu_init.c:62: warning: unused variable ‘tmp_sicrh’

Thanks Andre,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] fix non-working mvBL-M7

2008-06-25 Thread Kim Phillips
On Mon, 23 Jun 2008 13:25:34 +0200
Andre Schwarz [EMAIL PROTECTED] wrote:

Hello Andre,

 diff --git a/include/configs/MVBLM7.h b/include/configs/MVBLM7.h
 index 349ca14..8c0e8b6 100644
 --- a/include/configs/MVBLM7.h
 +++ b/include/configs/MVBLM7.h
 @@ -27,7 +27,7 @@
  #ifndef __CONFIG_H
  #define __CONFIG_H
  
 -#define MV_VERSION  v1.0.1
 +#define MV_VERSION  v1.0.2

instead of incrementing this for each patch, can we just use
U_BOOT_VERSION instead?

  #define CONFIG_HAS_ETH0
 +#define CONFIG_ETHADDR  b6:b4:45:eb:fb:c0
  #define CONFIG_TSEC1_NAMETSEC0
  #define CONFIG_FEC1_PHY_NORXERR
  #define CFG_TSEC1_OFFSET 0x24000
 @@ -224,6 +225,7 @@
  #define TSEC1_FLAGS  (TSEC_GIGABIT|TSEC_REDUCED)
  
  #define CONFIG_HAS_ETH1
 +#define CONFIG_ETH1ADDR b6:b4:45:eb:fb:c2
  #define CONFIG_TSEC2_NAMETSEC1

I had removed this from your original board port because this goes
against recommended practice; see:

http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/41178

 +#define CONFIG_HIGH_BATS   1

this line applied.

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH 2/2] mpc83xx: move CPU_TYPE_ENTRY over to processor.h

2008-06-17 Thread Kim Phillips
to avoid this:

cpu.c:47:1: warning: CPU_TYPE_ENTRY redefined
In file included from cpu.c:33:
/home/kim/git/u-boot/include/asm/processor.h:982:1: warning: this is the 
location of the previous definition

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 cpu/mpc83xx/cpu.c   |1 -
 include/asm-ppc/processor.h |7 +++
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/cpu/mpc83xx/cpu.c b/cpu/mpc83xx/cpu.c
index 36de78d..cc03f8a 100644
--- a/cpu/mpc83xx/cpu.c
+++ b/cpu/mpc83xx/cpu.c
@@ -44,7 +44,6 @@ int checkcpu(void)
char buf[32];
int i;
 
-#define CPU_TYPE_ENTRY(x) {#x, SPR_##x}
const struct cpu_type {
char name[15];
u32 partid;
diff --git a/include/asm-ppc/processor.h b/include/asm-ppc/processor.h
index 4bfd9f9..cf44f4f 100644
--- a/include/asm-ppc/processor.h
+++ b/include/asm-ppc/processor.h
@@ -979,8 +979,15 @@ struct cpu_type {
 
 struct cpu_type *identify_cpu(uint ver);
 
+#if defined(CONFIG_MPC85xx)
 #define CPU_TYPE_ENTRY(n, v) \
{ .name = #n, .soc_ver = SVR_##v, }
+#else
+#if defined(CONFIG_MPC83XX)
+#define CPU_TYPE_ENTRY(x) {#x, SPR_##x}
+#endif
+#endif
+
 
 #ifndef CONFIG_MACH_SPECIFIC
 extern int _machine;
-- 
1.5.6.rc2.26.g8c37


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH 1/2] mpc85xx: use IS_E_PROCESSOR macro

2008-06-17 Thread Kim Phillips
Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
as requested by galak.

 cpu/mpc85xx/cpu.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/cpu/mpc85xx/cpu.c b/cpu/mpc85xx/cpu.c
index 2b7e753..a2b11cc 100644
--- a/cpu/mpc85xx/cpu.c
+++ b/cpu/mpc85xx/cpu.c
@@ -97,7 +97,7 @@ int checkcpu (void)
if (cpu) {
puts(cpu-name);
 
-   if (svr  0x8)
+   if (IS_E_PROCESSOR(svr))
puts(E);
} else {
puts(Unknown);
-- 
1.5.6.rc2.26.g8c37


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH v3] fdt: add crypto node handling for MPC8{3, 5}xxE processors

2008-06-16 Thread Kim Phillips
crypto node if not on an E-processor.  If on 8360 or 834x family,
check rev and up-rev crypto node (to SEC rev. 2.4 property values)
if on an 'EA' processor, e.g. MPC8349EA.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
fixed v2's off-by-one error when accumulating compatible entry offsets
that deleted the necessary '\0' sprintf adds between strings.

 common/fdt_support.c|   87 +++
 cpu/mpc83xx/fdt.c   |   18 +
 cpu/mpc85xx/fdt.c   |5 ++
 include/asm-ppc/processor.h |9 
 include/fdt_support.h   |6 +++
 include/mpc83xx.h   |7 +++-
 6 files changed, 131 insertions(+), 1 deletions(-)

diff --git a/common/fdt_support.c b/common/fdt_support.c
index e58b294..13dab63 100644
--- a/common/fdt_support.c
+++ b/common/fdt_support.c
@@ -442,3 +442,90 @@ void fdt_fixup_dr_usb(void *blob, bd_t *bd)
   prop, compat, fdt_strerror(err));
 }
 #endif /* CONFIG_HAS_FSL_DR_USB */
+
+#if defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx)
+/*
+ * update crypto node properties to a specified revision of the SEC
+ * called with sec_rev == 0 if not on an mpc8xxxE processor
+ */
+void fdt_fixup_crypto_node(void *blob, int sec_rev)
+{
+   const struct sec_rev_prop {
+   u32 sec_rev;
+   u32 num_channels;
+   u32 channel_fifo_len;
+   u32 exec_units_mask;
+   u32 descriptor_types_mask;
+   } sec_rev_prop_list [] = {
+   { 0x0200, 4, 24, 0x07e, 0x01010ebf }, /* SEC 2.0 */
+   { 0x0201, 4, 24, 0x0fe, 0x012b0ebf }, /* SEC 2.1 */
+   { 0x0202, 1, 24, 0x04c, 0x0122003f }, /* SEC 2.2 */
+   { 0x0204, 4, 24, 0x07e, 0x012b0ebf }, /* SEC 2.4 */
+   { 0x0300, 4, 24, 0x9fe, 0x03ab0ebf }, /* SEC 3.0 */
+   { 0x0303, 4, 24, 0x97c, 0x03ab0abf }, /* SEC 3.3 */
+   };
+   char compat_strlist[ARRAY_SIZE(sec_rev_prop_list) *
+   sizeof(fsl,secX.Y)];
+   int crypto_node, sec_idx, err;
+   char *p;
+   u32 val;
+
+   /* locate crypto node based on lowest common compatible */
+   crypto_node = fdt_node_offset_by_compatible(blob, -1, fsl,sec2.0);
+   if (crypto_node == -FDT_ERR_NOTFOUND)
+   return;
+
+   /* delete it if not on an E-processor */
+   if (crypto_node  0  !sec_rev) {
+   fdt_del_node(blob, crypto_node);
+   return;
+   }
+
+   /* else we got called for possible uprev */
+   for (sec_idx = 0; sec_idx  ARRAY_SIZE(sec_rev_prop_list); sec_idx++)
+   if (sec_rev_prop_list[sec_idx].sec_rev == sec_rev)
+   break;
+
+   if (sec_idx == ARRAY_SIZE(sec_rev_prop_list)) {
+   puts(warning: unknown SEC revision number\n);
+   return;
+   }
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].num_channels);
+   err = fdt_setprop(blob, crypto_node, fsl,num-channels, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].descriptor_types_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,descriptor-types-mask, val, 
4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].exec_units_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,exec-units-mask, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].channel_fifo_len);
+   err = fdt_setprop(blob, crypto_node, fsl,channel-fifo-len, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = 0;
+   while (sec_idx = 0) {
+   p = compat_strlist + val;
+   val += sprintf(p, fsl,sec%d.%d,
+   (sec_rev_prop_list[sec_idx].sec_rev  0xff00)  8,
+   sec_rev_prop_list[sec_idx].sec_rev  0x00ff) + 1;
+   sec_idx--;
+   }
+   err = fdt_setprop(blob, crypto_node, compatible, compat_strlist, 
val);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+}
+#endif /* defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx) */
diff --git a/cpu/mpc83xx/fdt.c b/cpu/mpc83xx/fdt.c
index 02c4d05..267ae6a 100644
--- a/cpu/mpc83xx/fdt.c
+++ b/cpu/mpc83xx/fdt.c
@@ -26,6 +26,7 @@
 #include common.h
 #include libfdt.h
 #include fdt_support.h
+#include asm/processor.h
 
 extern void ft_qe_setup(void *blob);
 
@@ -33,6 +34,23 @@ DECLARE_GLOBAL_DATA_PTR;
 
 void ft_cpu_setup(void *blob, bd_t *bd)
 {
+   immap_t

[U-Boot-Users] [PATCH v4] fdt: add crypto node handling for MPC8{3, 5}xxE processors

2008-06-16 Thread Kim Phillips
Delete the crypto node if not on an E-processor.  If on 8360 or 834x family,
check rev and up-rev crypto node (to SEC rev. 2.4 property values)
if on an 'EA' processor, e.g. MPC8349EA.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
diff from v3: prepended Delete the to the patch commit message.
mumble.  Thanks gvb.

 common/fdt_support.c|   87 +++
 cpu/mpc83xx/fdt.c   |   18 +
 cpu/mpc85xx/fdt.c   |5 ++
 include/asm-ppc/processor.h |9 
 include/fdt_support.h   |6 +++
 include/mpc83xx.h   |7 +++-
 6 files changed, 131 insertions(+), 1 deletions(-)

diff --git a/common/fdt_support.c b/common/fdt_support.c
index e58b294..13dab63 100644
--- a/common/fdt_support.c
+++ b/common/fdt_support.c
@@ -442,3 +442,90 @@ void fdt_fixup_dr_usb(void *blob, bd_t *bd)
   prop, compat, fdt_strerror(err));
 }
 #endif /* CONFIG_HAS_FSL_DR_USB */
+
+#if defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx)
+/*
+ * update crypto node properties to a specified revision of the SEC
+ * called with sec_rev == 0 if not on an mpc8xxxE processor
+ */
+void fdt_fixup_crypto_node(void *blob, int sec_rev)
+{
+   const struct sec_rev_prop {
+   u32 sec_rev;
+   u32 num_channels;
+   u32 channel_fifo_len;
+   u32 exec_units_mask;
+   u32 descriptor_types_mask;
+   } sec_rev_prop_list [] = {
+   { 0x0200, 4, 24, 0x07e, 0x01010ebf }, /* SEC 2.0 */
+   { 0x0201, 4, 24, 0x0fe, 0x012b0ebf }, /* SEC 2.1 */
+   { 0x0202, 1, 24, 0x04c, 0x0122003f }, /* SEC 2.2 */
+   { 0x0204, 4, 24, 0x07e, 0x012b0ebf }, /* SEC 2.4 */
+   { 0x0300, 4, 24, 0x9fe, 0x03ab0ebf }, /* SEC 3.0 */
+   { 0x0303, 4, 24, 0x97c, 0x03ab0abf }, /* SEC 3.3 */
+   };
+   char compat_strlist[ARRAY_SIZE(sec_rev_prop_list) *
+   sizeof(fsl,secX.Y)];
+   int crypto_node, sec_idx, err;
+   char *p;
+   u32 val;
+
+   /* locate crypto node based on lowest common compatible */
+   crypto_node = fdt_node_offset_by_compatible(blob, -1, fsl,sec2.0);
+   if (crypto_node == -FDT_ERR_NOTFOUND)
+   return;
+
+   /* delete it if not on an E-processor */
+   if (crypto_node  0  !sec_rev) {
+   fdt_del_node(blob, crypto_node);
+   return;
+   }
+
+   /* else we got called for possible uprev */
+   for (sec_idx = 0; sec_idx  ARRAY_SIZE(sec_rev_prop_list); sec_idx++)
+   if (sec_rev_prop_list[sec_idx].sec_rev == sec_rev)
+   break;
+
+   if (sec_idx == ARRAY_SIZE(sec_rev_prop_list)) {
+   puts(warning: unknown SEC revision number\n);
+   return;
+   }
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].num_channels);
+   err = fdt_setprop(blob, crypto_node, fsl,num-channels, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].descriptor_types_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,descriptor-types-mask, val, 
4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].exec_units_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,exec-units-mask, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].channel_fifo_len);
+   err = fdt_setprop(blob, crypto_node, fsl,channel-fifo-len, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = 0;
+   while (sec_idx = 0) {
+   p = compat_strlist + val;
+   val += sprintf(p, fsl,sec%d.%d,
+   (sec_rev_prop_list[sec_idx].sec_rev  0xff00)  8,
+   sec_rev_prop_list[sec_idx].sec_rev  0x00ff) + 1;
+   sec_idx--;
+   }
+   err = fdt_setprop(blob, crypto_node, compatible, compat_strlist, 
val);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+}
+#endif /* defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx) */
diff --git a/cpu/mpc83xx/fdt.c b/cpu/mpc83xx/fdt.c
index 02c4d05..267ae6a 100644
--- a/cpu/mpc83xx/fdt.c
+++ b/cpu/mpc83xx/fdt.c
@@ -26,6 +26,7 @@
 #include common.h
 #include libfdt.h
 #include fdt_support.h
+#include asm/processor.h
 
 extern void ft_qe_setup(void *blob);
 
@@ -33,6 +34,23 @@ DECLARE_GLOBAL_DATA_PTR;
 
 void ft_cpu_setup(void *blob, bd_t *bd)
 {
+   immap_t *immr = (immap_t *)CFG_IMMR

Re: [U-Boot-Users] [PATCH 1/8] New board SIMPC8313 support: MAINTAINERS, MAKEALL, Makefile

2008-06-10 Thread Kim Phillips
On Fri, 30 May 2008 11:07:38 -0700 (PDT)
Ron Madrid [EMAIL PROTECTED] wrote:

 New board SIMPC8313 support: MAINTAINERS, MAKEALL,
 Makefile
 
 Signed-off-by: Ron Madrid
 ---
  MAINTAINERS |5 +
  MAKEALL |2 ++
  Makefile|   18 ++
  3 files changed, 25 insertions(+), 0 deletions(-)
 
I can't apply this:

Applying New board SIMPC8313 support: MAINTAINERS, MAKEALL, Makefile
.dotest/patch:15: trailing whitespace.
 
fatal: corrupt patch at line 41
Patch failed at 0001.
When you have resolved this problem run git-am --resolved.
If you would prefer to skip this patch, instead run git-am --skip.

...

 $(obj)include/config.h ; \
 + $(MKCONFIG) -a SIMPC8313 ppc mpc83xx simpc8313
 sheldon; \

your mailer is obviously line-wrapping the patch.

Please refer to linux-2.6/Documentation/email-clients.txt on how to fix
this.

Thank you,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] 85xx: extended cpu identification

2008-06-10 Thread Kim Phillips
On Tue, 10 Jun 2008 08:23:46 -0500
Kumar Gala [EMAIL PROTECTED] wrote:

 
 On Jun 10, 2008, at 12:57 AM, Kim Phillips wrote:
 
  On Thu, 29 May 2008 03:20:08 -0500 (CDT)
  Kumar Gala [EMAIL PROTECTED] wrote:
 
  +struct cpu_type cpu_type_list [] = {
  +  CPU_TYPE_ENTRY(8533, 8533, 0),
  +  CPU_TYPE_ENTRY(8533, 8533_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8540, 8540, 0),
  +  CPU_TYPE_ENTRY(8541, 8541, 0),
  +  CPU_TYPE_ENTRY(8541, 8541_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8543, 8543, 0),
  +  CPU_TYPE_ENTRY(8543, 8543_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8544, 8544, 0),
  +  CPU_TYPE_ENTRY(8544, 8544_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8545, 8545, 0),
  +  CPU_TYPE_ENTRY(8545, 8545_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8547, 8547_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8548, 8548, 0),
  +  CPU_TYPE_ENTRY(8548, 8548_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8555, 8555, 0),
  +  CPU_TYPE_ENTRY(8555, 8555_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8560, 8560, 0),
  +  CPU_TYPE_ENTRY(8567, 8567, 0),
  +  CPU_TYPE_ENTRY(8567, 8567_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8568, 8568, 0),
  +  CPU_TYPE_ENTRY(8568, 8568_E, CPU_FTRS_HAS_CRYPTO),
  +  CPU_TYPE_ENTRY(8572, 8572, 0),
  +  CPU_TYPE_ENTRY(8572, 8572_E, CPU_FTRS_HAS_CRYPTO),
 
  this seems like overkill given all we have to do is check one bit (see
  IS_E_PROCESSOR macro in handle crypto node patch I just sent out).
 
 I don't trust our HW guys to keep w/that convention.  Plus we can use  
 this mechanism for other things if need be.

they've been pretty good so far, and until the other thing comes
around, we can compact this table instead of expanding it - the 85xx_E
entries themselves are unnecessary.

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] 85xx: extended cpu identification

2008-06-10 Thread Kim Phillips
On Tue, 10 Jun 2008 10:10:02 -0500
Kumar Gala [EMAIL PROTECTED] wrote:

 
 On Jun 10, 2008, at 8:48 AM, Kim Phillips wrote:
 
  On Tue, 10 Jun 2008 08:23:46 -0500
  Kumar Gala [EMAIL PROTECTED] wrote:
 
 
  On Jun 10, 2008, at 12:57 AM, Kim Phillips wrote:
 
  On Thu, 29 May 2008 03:20:08 -0500 (CDT)
  Kumar Gala [EMAIL PROTECTED] wrote:
 
  +struct cpu_type cpu_type_list [] = {
  +CPU_TYPE_ENTRY(8533, 8533, 0),
  +CPU_TYPE_ENTRY(8533, 8533_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8540, 8540, 0),
  +CPU_TYPE_ENTRY(8541, 8541, 0),
  +CPU_TYPE_ENTRY(8541, 8541_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8543, 8543, 0),
  +CPU_TYPE_ENTRY(8543, 8543_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8544, 8544, 0),
  +CPU_TYPE_ENTRY(8544, 8544_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8545, 8545, 0),
  +CPU_TYPE_ENTRY(8545, 8545_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8547, 8547_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8548, 8548, 0),
  +CPU_TYPE_ENTRY(8548, 8548_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8555, 8555, 0),
  +CPU_TYPE_ENTRY(8555, 8555_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8560, 8560, 0),
  +CPU_TYPE_ENTRY(8567, 8567, 0),
  +CPU_TYPE_ENTRY(8567, 8567_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8568, 8568, 0),
  +CPU_TYPE_ENTRY(8568, 8568_E, CPU_FTRS_HAS_CRYPTO),
  +CPU_TYPE_ENTRY(8572, 8572, 0),
  +CPU_TYPE_ENTRY(8572, 8572_E, CPU_FTRS_HAS_CRYPTO),
 
  this seems like overkill given all we have to do is check one bit  
  (see
  IS_E_PROCESSOR macro in handle crypto node patch I just sent out).
 
  I don't trust our HW guys to keep w/that convention.  Plus we can use
  this mechanism for other things if need be.
 
  they've been pretty good so far, and until the other thing comes
  around, we can compact this table instead of expanding it - the 85xx_E
  entries themselves are unnecessary.
 
 I disagree and would prefer to keep it as I've done as it provides the  
 most flexibility.

it's easier to do a IS_E_PROCESSOR(get_svr()) from cpu/mpc85xx/fdt.c
instead of getting ver, calling cpu = identify_cpu(ver), and then
checking cpu-features every time.  I don't know what other features
you have in mind (perhaps this patch should wait until then?), but
HAS_CRYPTO is either on or off, and the bit already exists in the h/w..

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] fdt: unshadow global working fdt variable

2008-06-10 Thread Kim Phillips
differentiate with local variables of the same name by renaming the
global 'fdt' variable 'working_fdt'.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
applies to current u-boot-fdt.git.

 common/cmd_fdt.c |  114 +++---
 common/fdt_support.c |2 +-
 include/libfdt_env.h |2 +-
 lib_ppc/bootm.c  |2 +-
 4 files changed, 64 insertions(+), 56 deletions(-)

diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
index 4285a96..5c3a0bb 100644
--- a/common/cmd_fdt.c
+++ b/common/cmd_fdt.c
@@ -62,7 +62,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
/*
 * Set the address [and length] of the fdt.
 */
-   fdt = (struct fdt_header *)simple_strtoul(argv[2], NULL, 16);
+   working_fdt = (struct fdt_header *)simple_strtoul(argv[2], 
NULL, 16);
 
if (!fdt_valid()) {
return 1;
@@ -75,15 +75,15 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
 * Optional new length
 */
len = simple_strtoul(argv[3], NULL, 16);
-   if (len  fdt_totalsize(fdt)) {
+   if (len  fdt_totalsize(working_fdt)) {
printf (New length %d  existing length %d, 
ignoring.\n,
-   len, fdt_totalsize(fdt));
+   len, fdt_totalsize(working_fdt));
} else {
/*
 * Open in place with a new length.
 */
-   err = fdt_open_into(fdt, fdt, len);
+   err = fdt_open_into(working_fdt, working_fdt, 
len);
if (err != 0) {
printf (libfdt fdt_open_into(): %s\n,
fdt_strerror(err));
@@ -92,7 +92,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
}
 
/
-* Move the fdt
+* Move the working_fdt
 /
} else if (strncmp(argv[1], mo, 2) == 0) {
struct fdt_header *newaddr;
@@ -107,7 +107,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
/*
 * Set the address and length of the fdt.
 */
-   fdt = (struct fdt_header *)simple_strtoul(argv[2], NULL, 16);
+   working_fdt = (struct fdt_header *)simple_strtoul(argv[2], 
NULL, 16);
if (!fdt_valid()) {
return 1;
}
@@ -119,13 +119,13 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
 * current length.
 */
if (argc = 4) {
-   len = fdt_totalsize(fdt);
+   len = fdt_totalsize(working_fdt);
} else {
len = simple_strtoul(argv[4], NULL, 16);
-   if (len  fdt_totalsize(fdt)) {
+   if (len  fdt_totalsize(working_fdt)) {
printf (New length 0x%X  existing length 
0x%X, aborting.\n,
-   len, fdt_totalsize(fdt));
+   len, fdt_totalsize(working_fdt));
return 1;
}
}
@@ -133,13 +133,13 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
/*
 * Copy to the new location.
 */
-   err = fdt_open_into(fdt, newaddr, len);
+   err = fdt_open_into(working_fdt, newaddr, len);
if (err != 0) {
printf (libfdt fdt_open_into(): %s\n,
fdt_strerror(err));
return 1;
}
-   fdt = newaddr;
+   working_fdt = newaddr;
 
/
 * Make a new node
@@ -161,7 +161,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
pathp = argv[2];
nodep = argv[3];
 
-   nodeoffset = fdt_path_offset (fdt, pathp);
+   nodeoffset = fdt_path_offset (working_fdt, pathp);
if (nodeoffset  0) {
/*
 * Not found or something else bad happened.
@@ -170,7 +170,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv

[U-Boot-Users] Please pull mpc83xx.git

2008-06-10 Thread Kim Phillips
Wolfgang,

please pull support for the MVBLM7 board and some local bus define
refactoring:

The following changes since commit 8155efbd7ae9c65564ca98affe94631d612ae088:
  Wolfgang Denk (1):
Merge branch 'master' of ssh://mercury/home/wd/git/u-boot/master

are available in the git repository at:

  git://git.denx.de/u-boot-mpc83xx.git

Andre Schwarz (2):
  add MPC8343 based board mvBlueLYNX-M7 (doc+config)
  add MPC8343 based board mvBlueLYNX-M7 (board+make files)

Anton Vorontsov (2):
  83xx/85xx/86xx: factor out Freescale Localbus defines out of mpc83xx.h
  83xx/85xx: further localbus cleanups

Kim Phillips (1):
  mpc83xx: MVBLM7: minor build fixups

Tor Krill (1):
  Add missing CSCONFIG_BANK_BIT_3 define to mpc83xx.h

 CREDITS|4 +
 MAINTAINERS|4 +
 MAKEALL|1 +
 Makefile   |3 +
 board/mvblm7/Makefile  |   48 
 board/mvblm7/config.mk |   25 ++
 board/mvblm7/fpga.c|  188 
 board/mvblm7/fpga.h|   34 +++
 board/mvblm7/mvblm7.c  |  157 +
 board/mvblm7/mvblm7.h  |   21 ++
 board/mvblm7/mvblm7_autoscript |   37 +++
 board/mvblm7/pci.c |  133 +++
 doc/README.mvblm7  |   85 +++
 include/asm-ppc/fsl_lbc.h  |  264 ++
 include/configs/MPC8315ERDB.h  |2 +-
 include/configs/MPC8349EMDS.h  |2 +-
 include/configs/MPC8349ITX.h   |4 +-
 include/configs/MPC8360EMDS.h  |2 +-
 include/configs/MPC8360ERDK.h  |2 +-
 include/configs/MPC837XEMDS.h  |2 +-
 include/configs/MVBLM7.h   |  479 
 include/configs/TQM834x.h  |2 +-
 include/mpc83xx.h  |  235 +---
 include/mpc85xx.h  |   37 +---
 include/mpc86xx.h  |2 +
 25 files changed, 1497 insertions(+), 276 deletions(-)
 create mode 100644 board/mvblm7/Makefile
 create mode 100644 board/mvblm7/config.mk
 create mode 100644 board/mvblm7/fpga.c
 create mode 100644 board/mvblm7/fpga.h
 create mode 100644 board/mvblm7/mvblm7.c
 create mode 100644 board/mvblm7/mvblm7.h
 create mode 100644 board/mvblm7/mvblm7_autoscript
 create mode 100644 board/mvblm7/pci.c
 create mode 100644 doc/README.mvblm7
 create mode 100644 include/asm-ppc/fsl_lbc.h
 create mode 100644 include/configs/MVBLM7.h

Thanks,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH v2] fdt: add crypto node handling for MPC8{3, 5}xxE processors

2008-06-10 Thread Kim Phillips
crypto node if not on an E-processor.  If on 8360 or 834x family,
check rev and up-rev crypto node (to SEC rev. 2.4 property values)
if on an 'EA' processor, e.g. MPC8349EA.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
change since v1: mpc85xx IS_E_PROCESSOR(svr) definition changed to (svr
 0x8).

(I'm not sure, but I think this goes through WD directly)

 common/fdt_support.c|   87 +++
 cpu/mpc83xx/fdt.c   |   18 +
 cpu/mpc85xx/fdt.c   |6 +++
 include/asm-ppc/processor.h |9 
 include/fdt_support.h   |6 +++
 include/mpc83xx.h   |7 +++-
 6 files changed, 132 insertions(+), 1 deletions(-)

diff --git a/common/fdt_support.c b/common/fdt_support.c
index 7507744..c5b4650 100644
--- a/common/fdt_support.c
+++ b/common/fdt_support.c
@@ -447,3 +447,90 @@ void fdt_fixup_dr_usb(void *blob, bd_t *bd)
   prop, compat, fdt_strerror(err));
 }
 #endif /* CONFIG_HAS_FSL_DR_USB */
+
+#if defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx)
+/*
+ * update crypto node properties to a specified revision of the SEC
+ * called with sec_rev == 0 if not on an mpc8xxxE processor
+ */
+void fdt_fixup_crypto_node(void *blob, int sec_rev)
+{
+   const struct sec_rev_prop {
+   u32 sec_rev;
+   u32 num_channels;
+   u32 channel_fifo_len;
+   u32 exec_units_mask;
+   u32 descriptor_types_mask;
+   } sec_rev_prop_list [] = {
+   { 0x0200, 4, 24, 0x07e, 0x01010ebf }, /* SEC 2.0 */
+   { 0x0201, 4, 24, 0x0fe, 0x012b0ebf }, /* SEC 2.1 */
+   { 0x0202, 1, 24, 0x04c, 0x0122003f }, /* SEC 2.2 */
+   { 0x0204, 4, 24, 0x07e, 0x012b0ebf }, /* SEC 2.4 */
+   { 0x0300, 4, 24, 0x9fe, 0x03ab0ebf }, /* SEC 3.0 */
+   { 0x0303, 4, 24, 0x97c, 0x03ab0abf }, /* SEC 3.3 */
+   };
+   char compat_strlist[ARRAY_SIZE(sec_rev_prop_list) *
+   sizeof(fsl,secX.Y)];
+   int crypto_node, sec_idx, err;
+   char *p;
+   u32 val;
+
+   /* locate crypto node based on lowest common compatible */
+   crypto_node = fdt_node_offset_by_compatible(blob, -1, fsl,sec2.0);
+   if (crypto_node == -FDT_ERR_NOTFOUND)
+   return;
+
+   /* delete it if not on an E-processor */
+   if (crypto_node  0  !sec_rev) {
+   fdt_del_node(blob, crypto_node);
+   return;
+   }
+
+   /* else we got called for possible uprev */
+   for (sec_idx = 0; sec_idx  ARRAY_SIZE(sec_rev_prop_list); sec_idx++)
+   if (sec_rev_prop_list[sec_idx].sec_rev == sec_rev)
+   break;
+
+   if (sec_idx == ARRAY_SIZE(sec_rev_prop_list)) {
+   puts(warning: unknown SEC revision number\n);
+   return;
+   }
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].num_channels);
+   err = fdt_setprop(blob, crypto_node, fsl,num-channels, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].descriptor_types_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,descriptor-types-mask, val, 
4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].exec_units_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,exec-units-mask, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].channel_fifo_len);
+   err = fdt_setprop(blob, crypto_node, fsl,channel-fifo-len, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = 0;
+   while (sec_idx = 0) {
+   p = compat_strlist + val;
+   val += sprintf(p, fsl,sec%d.%d,
+   (sec_rev_prop_list[sec_idx].sec_rev  0xff00)  8,
+   sec_rev_prop_list[sec_idx].sec_rev  0x00ff);
+   sec_idx--;
+   }
+   err = fdt_setprop(blob, crypto_node, compatible, compat_strlist, 
val);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+}
+#endif /* defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx) */
diff --git a/cpu/mpc83xx/fdt.c b/cpu/mpc83xx/fdt.c
index 02c4d05..267ae6a 100644
--- a/cpu/mpc83xx/fdt.c
+++ b/cpu/mpc83xx/fdt.c
@@ -26,6 +26,7 @@
 #include common.h
 #include libfdt.h
 #include fdt_support.h
+#include asm/processor.h
 
 extern void ft_qe_setup(void *blob);
 
@@ -33,6 +34,23 @@ DECLARE_GLOBAL_DATA_PTR;
 
 void ft_cpu_setup(void *blob, bd_t *bd

Re: [U-Boot-Users] [PATCH] libfdt: Move the working_fdt pointer to cmd_fdt.c

2008-06-10 Thread Kim Phillips
On Tue, 10 Jun 2008 22:36:07 -0400
Jerry Van Baren [EMAIL PROTECTED] wrote:

 I've applied your patch fdt: unshadow global working fdt variable.
 As a feeble attempt to redeem myself, I added a patch to move working_fdt
 to cmd_fdt.c... it isn't used (any longer) in fdt_support.c.

sounds good

 I started to hack bootm.c (if we moved the reference to working_fdt out
 of boot_relocate_fdt() or move that function, we could make working_fdt
 static), but it got too complex so I backed out of that change for now.

:)

I gave a shot at switching all MPC8[356]xx to MPC8[356]XX but the 100k
limit, stepping into 51xx space, and probably horrible timing (due to
the many unapplied patches out there) prevented me from doing so.

thanks,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] fdt_support: add crypto node handling for MPC8{3, 5}xxE processors

2008-06-09 Thread Kim Phillips
crypto node if not on an E-processor.  If on 8360 or 834x family,
check rev and up-rev crypto node (to SEC rev. 2.4) property values
if on an 'EA' processor, e.g. MPC8349EA.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 common/fdt_support.c|   87 +++
 cpu/mpc83xx/fdt.c   |   18 +
 cpu/mpc85xx/fdt.c   |6 +++
 include/asm-ppc/processor.h |9 
 include/fdt_support.h   |6 +++
 include/mpc83xx.h   |7 +++-
 6 files changed, 132 insertions(+), 1 deletions(-)

diff --git a/common/fdt_support.c b/common/fdt_support.c
index 7507744..c5b4650 100644
--- a/common/fdt_support.c
+++ b/common/fdt_support.c
@@ -447,3 +447,90 @@ void fdt_fixup_dr_usb(void *blob, bd_t *bd)
   prop, compat, fdt_strerror(err));
 }
 #endif /* CONFIG_HAS_FSL_DR_USB */
+
+#if defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx)
+/*
+ * update crypto node properties to a specified revision of the SEC
+ * called with sec_rev == 0 if not on an mpc8xxxE processor
+ */
+void fdt_fixup_crypto_node(void *blob, int sec_rev)
+{
+   const struct sec_rev_prop {
+   u32 sec_rev;
+   u32 num_channels;
+   u32 channel_fifo_len;
+   u32 exec_units_mask;
+   u32 descriptor_types_mask;
+   } sec_rev_prop_list [] = {
+   { 0x0200, 4, 24, 0x07e, 0x01010ebf }, /* SEC 2.0 */
+   { 0x0201, 4, 24, 0x0fe, 0x012b0ebf }, /* SEC 2.1 */
+   { 0x0202, 1, 24, 0x04c, 0x0122003f }, /* SEC 2.2 */
+   { 0x0204, 4, 24, 0x07e, 0x012b0ebf }, /* SEC 2.4 */
+   { 0x0300, 4, 24, 0x9fe, 0x03ab0ebf }, /* SEC 3.0 */
+   { 0x0303, 4, 24, 0x97c, 0x03ab0abf }, /* SEC 3.3 */
+   };
+   char compat_strlist[ARRAY_SIZE(sec_rev_prop_list) *
+   sizeof(fsl,secX.Y)];
+   int crypto_node, sec_idx, err;
+   char *p;
+   u32 val;
+
+   /* locate crypto node based on lowest common compatible */
+   crypto_node = fdt_node_offset_by_compatible(blob, -1, fsl,sec2.0);
+   if (crypto_node == -FDT_ERR_NOTFOUND)
+   return;
+
+   /* delete it if not on an E-processor */
+   if (crypto_node  0  !sec_rev) {
+   fdt_del_node(blob, crypto_node);
+   return;
+   }
+
+   /* else we got called for possible uprev */
+   for (sec_idx = 0; sec_idx  ARRAY_SIZE(sec_rev_prop_list); sec_idx++)
+   if (sec_rev_prop_list[sec_idx].sec_rev == sec_rev)
+   break;
+
+   if (sec_idx == ARRAY_SIZE(sec_rev_prop_list)) {
+   puts(warning: unknown SEC revision number\n);
+   return;
+   }
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].num_channels);
+   err = fdt_setprop(blob, crypto_node, fsl,num-channels, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].descriptor_types_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,descriptor-types-mask, val, 
4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].exec_units_mask);
+   err = fdt_setprop(blob, crypto_node, fsl,exec-units-mask, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = cpu_to_fdt32(sec_rev_prop_list[sec_idx].channel_fifo_len);
+   err = fdt_setprop(blob, crypto_node, fsl,channel-fifo-len, val, 4);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+
+   val = 0;
+   while (sec_idx = 0) {
+   p = compat_strlist + val;
+   val += sprintf(p, fsl,sec%d.%d,
+   (sec_rev_prop_list[sec_idx].sec_rev  0xff00)  8,
+   sec_rev_prop_list[sec_idx].sec_rev  0x00ff);
+   sec_idx--;
+   }
+   err = fdt_setprop(blob, crypto_node, compatible, compat_strlist, 
val);
+   if (err  0)
+   printf(WARNING: could not set crypto property: %s\n,
+  fdt_strerror(err));
+}
+#endif /* defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx) */
diff --git a/cpu/mpc83xx/fdt.c b/cpu/mpc83xx/fdt.c
index 02c4d05..267ae6a 100644
--- a/cpu/mpc83xx/fdt.c
+++ b/cpu/mpc83xx/fdt.c
@@ -26,6 +26,7 @@
 #include common.h
 #include libfdt.h
 #include fdt_support.h
+#include asm/processor.h
 
 extern void ft_qe_setup(void *blob);
 
@@ -33,6 +34,23 @@ DECLARE_GLOBAL_DATA_PTR;
 
 void ft_cpu_setup(void *blob, bd_t *bd)
 {
+   immap_t *immr = (immap_t *)CFG_IMMR;
+   int spridr = immr-sysconf.spridr;
+
+   /*
+* delete crypto node if not on an E-processor

Re: [U-Boot-Users] [PATCH] 85xx: extended cpu identification

2008-06-09 Thread Kim Phillips
On Thu, 29 May 2008 03:20:08 -0500 (CDT)
Kumar Gala [EMAIL PROTECTED] wrote:

 +struct cpu_type cpu_type_list [] = {
 + CPU_TYPE_ENTRY(8533, 8533, 0),
 + CPU_TYPE_ENTRY(8533, 8533_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8540, 8540, 0),
 + CPU_TYPE_ENTRY(8541, 8541, 0),
 + CPU_TYPE_ENTRY(8541, 8541_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8543, 8543, 0),
 + CPU_TYPE_ENTRY(8543, 8543_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8544, 8544, 0),
 + CPU_TYPE_ENTRY(8544, 8544_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8545, 8545, 0),
 + CPU_TYPE_ENTRY(8545, 8545_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8547, 8547_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8548, 8548, 0),
 + CPU_TYPE_ENTRY(8548, 8548_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8555, 8555, 0),
 + CPU_TYPE_ENTRY(8555, 8555_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8560, 8560, 0),
 + CPU_TYPE_ENTRY(8567, 8567, 0),
 + CPU_TYPE_ENTRY(8567, 8567_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8568, 8568, 0),
 + CPU_TYPE_ENTRY(8568, 8568_E, CPU_FTRS_HAS_CRYPTO),
 + CPU_TYPE_ENTRY(8572, 8572, 0),
 + CPU_TYPE_ENTRY(8572, 8572_E, CPU_FTRS_HAS_CRYPTO),

this seems like overkill given all we have to do is check one bit (see
IS_E_PROCESSOR macro in handle crypto node patch I just sent out).

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 2/2] Add board specific files for BUBBATWO

2008-06-03 Thread Kim Phillips
On Mon,  2 Jun 2008 15:16:02 +0200
Tor Krill [EMAIL PROTECTED] wrote:

 +U_BOOT_CMD (bubbacmd, 3, 1, do_bubbacmd,
 + bubbacmd- Board specific commands for Bubba TWO\n,
 + spindown [dev] - spin down disk dev\n
 + bubbacmd spinup [dev] - spin up disk dev\n
 + bubbacmd button - read button status\n);
 +#endif

I the spinup/spindown commands should be part of cmd_ide, no?  Or
rather, should they be eliminated in favour of Das U-Boot way of only
powering up devices when accesses are requested?  i.e, power up on a
disk read command and power down after it's done?

btw, this also prevents me from applying this board support patch
because of the sata_sil3114 driver dependency here.

 +int getboardversion (void)
 +{
 + volatile immap_t *im = (immap_t *) CFG_IMMR;
 + int rev = im-gpio[0].dat  (HW_ID0 | HW_ID1 | HW_ID2);

looks like the and op is unnecessary here..plus we're not doing
volatile i/o accesses any more - use {in,out}_be32(), {set,clr}bits32()
etc., this occurs many other places in this patch..

 +
 + rev =
 + ((rev  HW_ID0)  3) | ((rev  HW_ID1)  2) | ((rev  HW_ID2) 
 
 + 1);
 +
 + return rev;
 +}

return ((rev  HW_ID0)  3) | ((rev  HW_ID1)  2) |
   ((rev  HW_ID2)  1);


 +ulong board_flash_get_legacy (ulong base, int banknum, flash_info_t * info)
 +{
 + if (banknum == 0) { /* non-CFI boot flash */
 + info-portwidth = FLASH_CFI_16BIT;
 + info-chipwidth = FLASH_CFI_BY16;
 + info-interface = FLASH_CFI_X16;
 + return 1;
 + } else {
 + return 0;
 + }

else is superfluous

 +/* Lookup table for the different boardversions */
 +static struct {
 + u32 memsize;
 + u32 config;
 +} meminfo[] = {
 + /* 0 - revA  prototypes */
 + {
 + memsize: 128, config:CFG_DDR_CONFIG_128},
 + /* 1 - revB */
 + {
 + memsize: 256, config:CFG_DDR_CONFIG_256},
 + /* 2 - empty */
 + {
 + memsize: 128, config:CFG_DDR_CONFIG_128},
 + /* 3 - empty */
 + {
 + memsize: 128, config:CFG_DDR_CONFIG_128},
 + /* 4 - empty */
 + {
 + memsize: 128, config:CFG_DDR_CONFIG_128},
 + /* 5 - empty */
 + {
 + memsize: 128, config:CFG_DDR_CONFIG_128},
 + /* 6 - empty */
 + {
 + memsize: 128, config:CFG_DDR_CONFIG_128},
 + /* 7 - empty */
 + {
 + memsize: 128, config:CFG_DDR_CONFIG_128}
 +};

would something like this be neater:

{ 128, CFG_DDR_CONFIG_128 }, /* 7 - empty */

?

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] UEC Driver - 83xx RGMII half duplex BUG FIX - RFC

2008-06-03 Thread Kim Phillips
On Mon, 2 Jun 2008 16:03:56 -0700
Russell McGuire [EMAIL PROTECTED] wrote:

 To work around this we need to remove the clearing of the FDX bit while in
 RGMII, RTBI, TBI, or GMII modes.
 
 The only modes that can clear the FDX bit are MII or RMII.
 
 Since I am not the author of the drivers, would the team want me to submit
 my idea on how to fix this, or would the author like to quickly add in this
 check before clearing the bit? Or alternatively before calling the
 uec_set_mac_duplex function(). Though I am a fan of making all functions
 self protecting against bad code.

Can you submit a patch for an on-list review?

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] MPC8315E-RDB kernel serial console baudrate

2008-05-28 Thread Kim Phillips
On Wed, 28 May 2008 21:33:33 +0200
Leon Woestenberg [EMAIL PROTECTED] wrote:

 I have tried to run the vanilla upstream 2.6.25.4 kernel on my

..

 U-Boot 1.3.0-rc2 (Nov 19 2007 - 16:37:36) MPC83XX
 
 CPU:   e300c3, MPC8315E, Rev: 10 at 400 MHz, CSB:  133 MHz
 Board: Freescale MPC8315ERDB Rev 0.0
 
 Is this u-boot too old, i.e. might there be a compatibility issue
 involved between u-boot / dtb / kernel?

yes

 What would be the best approach from here?
 (I plan to measure the actual bitrate using a scope tomorrow).

start by updating u-boot

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 2/2] add MPC8343 based board mvBlueLYNX-M7

2008-05-27 Thread Kim Phillips
On Tue, 27 May 2008 10:44:32 +0200
Andre Schwarz [EMAIL PROTECTED] wrote:

 Do you want me to re-send a complete set of patches for the M7-Board
 against the latest version ?
 
I'd appreciate it since I seem to have lost the board config file.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH-v3] Big white-space cleanup.

2008-05-21 Thread Kim Phillips
On Wed, 21 May 2008 15:34:32 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 In message [EMAIL PROTECTED] you wrote:
 
  Unfortunately several wrong fixes or false positives still remains,
  but this patch definitely reduce the number of wrong indentations.
 
 Could you please point out the remaining problems?  If  I  understand
 what's  gone  wrong  I probably can fix my procedure and spin another
 patch?
 

although the existing patch should definitely stay committed, here are a
couple of examples of things I'd expect it to have fixed:

board/freescale/mpc8349itx/mpc8349itx.c:120
board/freescale/mpc8349itx/mpc8349itx.c:275
board/freescale/mpc8349emds/pci.c:56

Also, although less authoritative (esp. for u-boot),
linux-2.6/scripts/checkpatch.pl output is always interesting on these
types of patches.

I'll be sure to get some by hand - I understand these expressions
sometimes get too unwieldly too fast.

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH-v3] Big white-space cleanup.

2008-05-21 Thread Kim Phillips
On Wed, 21 May 2008 10:25:20 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 The whole code in the cpu/ixp/npe/ is going to be replaced soon  by
 the  new driver code that has gone into mainline Linux recently. Thak
 makes me think we should not spend too much effort on this. Basicly I
 see three options:
 
 1) Be lazy and leave as is now
 2) Go forward and clean up all white space issues in the /npe/ code
 3) Revert the white-space cleanup patches as far as /npe/ code is
affected.
 
 Short discussion on IRC showed preference for 1).
 
 If you agree, that's what we'll do.
 
hmm, I can see reasons to leave npe alone wrt whitespace changes.

this may not be the most efficient way of doing it (I don't see
git-revert having a --path option), but this will do #3 above:

git-format-patch --stdout --relative=cpu/ixp/npe HEAD^ | patch -d cpu/ixp/npe 
-R -p 1 ; git add cpu/ixp/npe; git commit --amend

I'd add something like 'npe exempt' in the commit message.

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH][resubmit] MPC85xx, MPC83xx: Add/Fix UPM configuration support

2008-05-20 Thread Kim Phillips
On Tue, 20 May 2008 10:32:24 +0300
David Saada [EMAIL PROTECTED] wrote:

 
  Can we merge the register definition files and have a single upm_config
  for mpc8[356]xx instead of duplicating it as being done here?
 
 Not sure that I'm the right person for this job, as I don't have an 86xx 
 based board.

86xx can be tested by others; point here is to avoid code duplication.

 Also, I'm not sure this would be a good idea: There are some
 differences between the registers in these 3 families, and the header
 file would look pretty ugly (many ifdefs). Even the UPM code won't be

the registers have a common base, and all the registers this patch uses
are the same on all of them.  I don't see why they can't be unified
into a common include/asm-ppc/immap_8xxx.h file.

 the same: for instance, the mpc85xx code scans the BR registers (which
 are adjacent) to determine the UPM machine, while in the 83xx we need
 to scan the lbus banks (and extract the BRs from there). This means
 that we will also have many ifdefs there as well.

afaict, the code this patch introduces for 85xx is functionally
identical to that of 83xx:

  * Program a UPM with the code supplied in the table.
  *
  * The 'dummy' variable is used to increment the MAD. 'dummy' is
@@ -133,7 +13,7 @@
  *
  * The value can be extracted from the base address bits of the
  * Base Register (BR) associated with the specific UPM.  To find
- * that BR, we need to scan all 8 BRs until we find the one that
+ * that BR, we need to scan all BRs until we find the one that
  * has its MSEL bits matching the UPM we want.  Once we know the
  * right BR, we can extract the base address bits from it.
  *
@@ -147,20 +27,20 @@
  */
 void upmconfig (uint upm, uint *table, uint size)
 {
-   volatile immap_t *immap = (immap_t *) CFG_IMMR;
-   volatile lbus83xx_t *lbus = immap-lbus;
+volatile ccsr_lbc_t *lbc = (void *)(CFG_MPC85xx_LBC_ADDR);
volatile uchar *dummy = NULL;
-   const u32 msel = (upm + 4)  BR_MSEL_SHIFT;/* What
the MSEL field in BRn should be */
-   volatile u32 *mxmr = lbus-mamr + upm; /* Pointer to
mamr, mbmr, or mcmr */
+   const u32 msel = (upm + 4)  BRx_MS_SHIFT; /* What the MSEL
field in BRn should be */
+   volatile uint *mxmr = lbc-mamr + upm; /* Pointer to mamr,
mbmr, or mcmr */
+   volatile uint *brx = lbc-br0; /* Pointer to BRx */
uint i;
 
/* Scan all the banks to determine the base address of the
device */ for (i = 0; i  8; i++) {
-   if ((lbus-bank[i].br  BR_MSEL) == msel) {
-   dummy = (uchar *) (lbus-bank[i].br  BR_BA);
+   if ((*brx  BRx_MS_MSK) == msel) {
+   dummy = (uchar *) (*brx  BRx_BA_MSK);
break;
}
+   brx += 2; /* Skip to next BRx */
}
 
if (!dummy) {
@@ -169,10 +49,10 @@
}
 
/* Set the OP field in the MxMR to write and the MAD field
to 00 */
-   *mxmr = (*mxmr  0xCFC0) | 0x1000;
+   *mxmr = (*mxmr  0xCFC0) | MxMR_OP_WARR;
 
for (i = 0; i  size; i++) {
-   lbus-mdr = table[i];
+   lbc-mdr = table[i];
__asm__ __volatile__ (sync);
*dummy; /* Write the value to memory and
increment MAD */ __asm__ __volatile__ (sync);

btw, I don't see arch/powerpc/sysdev/fsl_lbc.c functionally differing
among the families either.

am I missing something?

 David.

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] Big white-space cleanup.

2008-05-20 Thread Kim Phillips
On Tue, 20 May 2008 16:50:44 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 This commit gets rid of a huge amount of silly white-space issues.
 Especially, all sequences of SPACEs followed by TAB characters get
 removed (unless they appear in print statements).
 
 Also remove all embedded vim: and vi: statements which hide
 indentation problems.
 
 Signed-off-by: Wolfgang Denk [EMAIL PROTECTED]
 ---
Wolfgang, are you not working on your current head?:

Applying Big white-space cleanup.
error: patch failed: common/usb.c:892
error: common/usb.c: patch does not apply
Patch failed at 0001.
When you have resolved this problem run git-am --resolved.
If you would prefer to skip this patch, instead run git-am --skip.

plus, it looks like the patch is replacing tabs with spaces (see, e.g.,
pib_init() changes in board/freescale/mpc8349emds/pci.c).

also,

  tools/easylogo/linux_logo.tga|  Bin 19244 - 19663 bytes

I don't think you meant to cleanup whitespace in binary files. :)

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH-v3] Big white-space cleanup.

2008-05-20 Thread Kim Phillips
On Wed, 21 May 2008 01:10:42 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 The latest version of this patch is available for download here:
 
 ftp://ftp.denx.de/pub/tmp/big-white-space-cleanup.patch-v.3.gz
 
 I also checked in this patch in the master branch of the
 u-boot-testing repository, so you can just pull from there for
 easier testing.

minor nit in comparison, but:

Applying Big white-space cleanup.
.dotest/patch:20954: trailing whitespace.
{   
.dotest/patch:20984: trailing whitespace.
   
.dotest/patch:21019: trailing whitespace.
ixQMgrAqmIfQueAccRegAddr[i] = 
.dotest/patch:21152: trailing whitespace.
 BOOL speed100, 
.dotest/patch:21153: trailing whitespace.
 BOOL fullDuplex, 

it looks like it's in u-boot-testing that way, too - maybe you can
amend it?  other than that things look clean(er):

Acked-by: Kim Phillips [EMAIL PROTECTED]

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH][resubmit] MPC85xx, MPC83xx: Add/Fix UPM configuration support

2008-05-19 Thread Kim Phillips
On Mon, 19 May 2008 19:05:04 +0300
David Saada [EMAIL PROTECTED] wrote:

 Add support for UPM configuration on the 85xx platform.
 In addition, on the MPC83xx, remove MPC834x precompiler condition, in order 
 to support all MPC83xx processors.
 
 Signed-off-by: David Saada [EMAIL PROTECTED]
 
 cpu/mpc83xx/cpu.c |5 
 cpu/mpc85xx/cpu.c |   66 
 ++
 include/mpc85xx.h |7 -
 3 files changed, 72 insertions(+), 6 deletions(-)

Can we merge the register definition files and have a single upm_config
for mpc8[356]xx instead of duplicating it as being done here?

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] NAND read/write.jffs2 fix

2008-05-14 Thread Kim Phillips
On Tue, 13 May 2008 15:41:06 -0500
Scott Wood [EMAIL PROTECTED] wrote:

 
   
   static int nand_dump(nand_info_t *nand, ulong off, int only_oob)
   {
  @@ -73,9 +74,9 @@ static int nand_dump(nand_info_t *nand, ulong off, int 
  only_oob)
  while (i--) {
  if (!only_oob) {
  printf( \t%02x %02x %02x %02x %02x %02x %02x %02x
  - %02x %02x %02x %02x %02x %02x %02x 
  %02x\n,
  -   p[0], p[1], p[2], p[3], p[4], p[5], 
  p[6], p[7],
  -   p[8], p[9], p[10], p[11], p[12], p[13], 
  p[14], p[15]);
  + %02x %02x %02x %02x %02x %02x %02x %02x\n,
  +   p[0], p[1], p[2], p[3], p[4], p[5], p[6], p[7],
  +   p[8], p[9], p[10], p[11], p[12], p[13], p[14], 
  p[15]);
 
 Still exceeds 80 characters on the last line.

or use lib_generic/display_options.c:print_buffer()

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] build u-boot error: No rule to make target `.depend', needed by `_depend'.

2008-05-14 Thread Kim Phillips
On Thu, 15 May 2008 01:49:50 +0800
Steven Woody [EMAIL PROTECTED] wrote:

 Dear Denk,
 
 1.2.0 is come frome eldk-4.1 iso file.  Downloading independent u-boot
 seems require `git' tool which I get problem with because my office
 network gets many limits.  Is there a http way to download u-boot?

git clone http://git.denx.de/u-boot.git

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] MPC85xx: Change traps.c to not reference non-addressable memory

2008-05-14 Thread Kim Phillips
On Wed, 14 May 2008 13:10:04 -0500
[EMAIL PROTECTED] wrote:

 From: Becky Bruce [EMAIL PROTECTED]
 
 Currently, END_OF_RAM is used by the trap code to determine if
 we should attempt to access the stack pointer or not. However,
 on systems with a lot of RAM, only a subset of the RAM is
 guaranteed to be mapped in and accessible.  Change END_OF_RAM
 to use get_effective_memsize() instead of using the raw ram
 size out of the bd.
 
 Signed-off-by: Becky Bruce [EMAIL PROTECTED]
 ---

what, no love for 4xx, 83xx, and 74xx_7xx?

can also be made a single patch for WD to apply directly.

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] MPC85xx: Change traps.c to not reference non-addressable memory

2008-05-14 Thread Kim Phillips
On Wed, 14 May 2008 15:31:43 -0500
Becky Bruce [EMAIL PROTECTED] wrote:

 
 On May 14, 2008, at 1:57 PM, Kim Phillips wrote:
  On Wed, 14 May 2008 13:10:04 -0500
  [EMAIL PROTECTED] wrote:
 
  From: Becky Bruce [EMAIL PROTECTED]
 
  Currently, END_OF_RAM is used by the trap code to determine if
  we should attempt to access the stack pointer or not. However,
  on systems with a lot of RAM, only a subset of the RAM is
  guaranteed to be mapped in and accessible.  Change END_OF_RAM
  to use get_effective_memsize() instead of using the raw ram
  size out of the bd.
 
  Signed-off-by: Becky Bruce [EMAIL PROTECTED]
  ---
 
  what, no love for 4xx, 83xx, and 74xx_7xx?
 
 The other platforms currently set a hard limit that should still be  
 valid - I just changed the ones that were reading memsize directly  
 out of the bd.
 
sure, 5xx, 82xx and 8xx are hardcoded, but 74xx_7xx (albeit only
AMIGAONE), all 83xx, and all 4xx use the bd just like in your current
patchseries.  I'm just trying to maintain some level of consistency.

  can also be made a single patch for WD to apply directly.
 
 
 Sure, I just split it up in case there was some reason we shouldn't  
 do it for one of them.

ah, I thought you wanted individual maintainers to pick them up
individually.  Either way works, really.

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Proposal for u-boot-nand-flash custodian change

2008-05-13 Thread Kim Phillips
On Tue, 13 May 2008 09:25:22 -0600
Grant Likely [EMAIL PROTECTED] wrote:

   Therefore I officially propose Scott Wood as new u-boot-nand-flash 
  custodian.
   I would really like to get some feedback on this. Any comments or other
   suggestions?
 
 Acked-by: Grant Likely [EMAIL PROTECTED]

Acked-by: Kim Phillips [EMAIL PROTECTED]

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Using uec.c and uec_phy.c

2008-05-12 Thread Kim Phillips
On Sat, 10 May 2008 04:24:08 -0700 (PDT)
klibusel [EMAIL PROTECTED] wrote:

 Hi,
 
 I am using the polled uboot ethernet driver on
 MPC8360erdk. I have trouble sending and receiving
 ethernet packets. I  have configured the interface as
 a generic MII and it detects the linking coming up.
 However when the QUICC engine reads TXBD and indicates
 TX is complete, nothing comes on the wire. This I
 confirmed using ethereal. Also on the RX side if
 connected to a PC I get RX error. I have made no
 changes to the orginial code. The interface is a
 BCM5481 on the board.
 
 Any pointer will be appreciated.

you don't mention what version u-boot you're using; can you let us know
if the problem exists on Wolfgang's top of git tree (currently
v1.3.3-rc3-51-g65dcfa7)?

Kim

-
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 2/2] add MPC8343 based board mvBlueLYNX-M7

2008-05-08 Thread Kim Phillips
On Wed, 7 May 2008 10:36:56 -0500
Kim Phillips [EMAIL PROTECTED] wrote:

 On Wed, 07 May 2008 12:01:48 +0200
 André Schwarz [EMAIL PROTECTED] wrote:
 
  I'll re-send the 23rd try after my vacation at end of may.
  If the merge window will be closed at that date it's obviously bad luck.
 
 I can make the changes, André, thanks!

hmm..made the below changes, but I don't know how to recover from:

fpga.c:55: error: ‘Altera_EP2C20_SIZE’ undeclared here (not in a function)

it seems this never built? also, we are missing a config file in the
latest submission.  One more thing: please submit patches in an order
that will not break future builds during a git-bisect session, i.e.,
since I believe a single patch is less than the ML limit of 40KB, make
it a single patch, please.

Thanks,

Kim

diff --git a/board/mvblm7/Makefile b/board/mvblm7/Makefile
index 84cd14a..42e0243 100644
--- a/board/mvblm7/Makefile
+++ b/board/mvblm7/Makefile
@@ -24,8 +24,11 @@ include $(TOPDIR)/config.mk
 
 LIB= $(obj)lib$(BOARD).a
 
-COBJS  := $(BOARD).o pci.o fpga.o
+COBJS-y:= $(BOARD).o
+COBJS-$(CONFIG_PCI) += pci.o
+COBJS-$(CONFIG_FPGA) += fpga.o
 
+COBJS  := $(COBJS-y)
 SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
 OBJS   := $(addprefix $(obj),$(COBJS))
 SOBJS  := $(addprefix $(obj),$(SOBJS))
diff --git a/board/mvblm7/fpga.c b/board/mvblm7/fpga.c
index 5eaa037..a60af01 100644
--- a/board/mvblm7/fpga.c
+++ b/board/mvblm7/fpga.c
@@ -32,8 +32,6 @@
 #include fpga.h
 #include mvblm7.h
 
-#ifdef CONFIG_FPGA
-
 #ifdef FPGA_DEBUG
 #define fpga_debug(fmt, args...)  printf(%s: fmt, __func__, ##args)
 #else
@@ -83,8 +81,8 @@ int fpga_config_fn(int assert, int flush, int cookie)
 {
volatile immap_t *im = (volatile immap_t *)CFG_IMMR;
volatile gpio83xx_t *gpio = (volatile gpio83xx_t *)im-gpio[0];
-
u32 dvo = gpio-dat;
+
fpga_debug(SET config : %s\n, assert ? low : high);
if (assert)
dvo |= FPGA_CONFIG;
@@ -134,8 +132,8 @@ int fpga_clk_fn(int assert_clk, int flush, int cookie)
 {
volatile immap_t *im = (volatile immap_t *)CFG_IMMR;
volatile gpio83xx_t *gpio = (volatile gpio83xx_t *)im-gpio[0];
-
u32 dvo = gpio-dat;
+
fpga_debug(CLOCK %s\n, assert_clk ? high : low);
if (assert_clk)
dvo |= FPGA_CCLK;
@@ -188,4 +186,3 @@ int fpga_wr_fn(void *buf, size_t len, int flush, int cookie)
 
return FPGA_SUCCESS;
 }
-#endif
diff --git a/board/mvblm7/mvblm7.c b/board/mvblm7/mvblm7.c
index 8f8e299..7e9b71f 100644
--- a/board/mvblm7/mvblm7.c
+++ b/board/mvblm7/mvblm7.c
@@ -43,15 +43,14 @@ int fixed_sdram(void)
u32 ddr_size_log2;
 
msize = CFG_DDR_SIZE;
-   for (ddr_size = msize  20, ddr_size_log2 = 0;
-   (ddr_size  1);
-   ddr_size = ddr_size1, ddr_size_log2++) {
+   for (ddr_size = msize  20, ddr_size_log2 = 0; ddr_size  1;
+   ddr_size = ddr_size  1, ddr_size_log2++)
if (ddr_size  1)
return -1;
-   }
+
im-sysconf.ddrlaw[0].bar = ((CFG_DDR_SDRAM_BASE12)  0xf);
im-sysconf.ddrlaw[0].ar = LAWAR_EN | ((ddr_size_log2 - 1) 
-   LAWAR_SIZE);
+  LAWAR_SIZE);
 
im-ddr.csbnds[0].csbnds = CFG_DDR_CS0_BNDS;
im-ddr.cs_config[0] = CFG_DDR_CS0_CONFIG;
@@ -131,5 +130,4 @@ void ft_board_setup(void *blob, bd_t *bd)
ft_pci_setup(blob, bd);
 #endif
 }
-
 #endif
diff --git a/board/mvblm7/pci.c b/board/mvblm7/pci.c
index b6453d0..72123c8 100644
--- a/board/mvblm7/pci.c
+++ b/board/mvblm7/pci.c
@@ -28,11 +28,14 @@
 #include libfdt.h
 #endif
 #include pci.h
+#include fpga.h
 #include mpc83xx.h
 #include mvblm7.h
 
 DECLARE_GLOBAL_DATA_PTR;
 
+extern int mvblm7_init_fpga(void);
+
 /* System RAM mapped to PCI space */
 #define CONFIG_PCI_SYS_MEM_BUS CFG_SDRAM_BASE
 #define CONFIG_PCI_SYS_MEM_PHYSCFG_SDRAM_BASE

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] Please pull u-boot-mpc83xx.git

2008-05-08 Thread Kim Phillips
Wolfgang Denk,

please pull a system performance tweak:

The following changes since commit 908261f3fdb418091d8c60bfbd7eb85e5869b579:
  Wolfgang Denk (1):
Merge branch 'master' of git+ssh://10.10.0.7/home/wd/git/u-boot/master

are available in the git repository at:

  git://git.denx.de/u-boot-mpc83xx.git master

Lee Nipper (1):
  mpc83xx: system performance settings for MPC8349EMDS.

 include/configs/MPC8349EMDS.h |   10 ++
 1 files changed, 10 insertions(+), 0 deletions(-)

Thanks,

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 2/2] add MPC8343 based board mvBlueLYNX-M7

2008-05-07 Thread Kim Phillips
On Wed, 07 May 2008 12:01:48 +0200
André Schwarz [EMAIL PROTECTED] wrote:

 I'll re-send the 23rd try after my vacation at end of may.
 If the merge window will be closed at that date it's obviously bad luck.

I can make the changes, André, thanks!

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] add config options for VSC8601 RGMII PHY - 2nd try

2008-05-05 Thread Kim Phillips
On Sun, 04 May 2008 12:19:32 +0200
André Schwarz [EMAIL PROTECTED] wrote:

 Thunderbird is configured according to to the kernel-doc page.
 Obviously there's something going wrong during copy+paste ...

according to this:

http://mbligh.org/linuxdocs/Email/Clients/Thunderbird

you might be able to get away with mailing the patch as an attachment.

Can you do this and try sending me the mvblm7 patch again?  I'll tell
you if it applies.

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Adding a new board to the u-boot project

2008-05-05 Thread Kim Phillips
On Mon, 5 May 2008 11:56:57 -0700
Ronald Madrid [EMAIL PROTECTED] wrote:

 
 I'm developing a board based on the MPC8313 and want to have my code included 
 in the u-boot project, but have a few questions.
 
great!

 I have created in u-boot-1.3.2 my own folder in the board directory and all 
 appropriate config files.  I have also edited the Makefile on the top level.

sounds good, you'll need to add a MAINTAINERS entry, a MAKEALL entry,
and preferably add a doc/README.myboard file.

 Everything builds for my board's u-boot, but here are my questions.  I have 
 to create a new folder in the nand_spl/board folder as my board supports NAND 
 flash.  Is that ok?  Also, to add NAND support I have to edit the 
 cpu/mpc83xx/start.S

yes

 file.  Is this allowed?

depends on the bits themselves; please send your patch for review on
this list.

 After I get my code working is there a specific person that I forward the 
 patches to?

the 83xx specific bits (including the general board support bits)
should probably go through me, and Stefan Roese can pick up your
NAND-specific bits, unless Wolfgang beats us to it.

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] mpc83xx: bump default loadaddr over fdtaddr, to 0x500000

2008-04-24 Thread Kim Phillips
this seems as a good compromise between human memory, typing,
and last but not least, to accommodate for current and future kernel bloat.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 include/configs/MPC8313ERDB.h |2 +-
 include/configs/MPC8315ERDB.h |2 +-
 include/configs/MPC8323ERDB.h |2 +-
 include/configs/MPC832XEMDS.h |2 +-
 include/configs/MPC8349EMDS.h |2 +-
 include/configs/MPC8349ITX.h  |2 +-
 include/configs/MPC8360EMDS.h |2 +-
 include/configs/MPC837XEMDS.h |2 +-
 include/configs/MPC837XERDB.h |2 +-
 include/configs/sbc8349.h |2 +-
 10 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/include/configs/MPC8313ERDB.h b/include/configs/MPC8313ERDB.h
index 9576fa5..6eec240 100644
--- a/include/configs/MPC8313ERDB.h
+++ b/include/configs/MPC8313ERDB.h
@@ -543,7 +543,7 @@
 #define CONFIG_UBOOTPATH   u-boot.bin  /* U-Boot image on TFTP server 
*/
 #define CONFIG_FDTFILE mpc8313erdb.dtb
 
-#define CONFIG_LOADADDR20  /* default location for tftp 
and bootm */
+#define CONFIG_LOADADDR50  /* default location for tftp 
and bootm */
 #define CONFIG_BOOTDELAY   -1  /* -1 disables auto-boot */
 #define CONFIG_BAUDRATE115200
 
diff --git a/include/configs/MPC8315ERDB.h b/include/configs/MPC8315ERDB.h
index 432fb31..e0a887c 100644
--- a/include/configs/MPC8315ERDB.h
+++ b/include/configs/MPC8315ERDB.h
@@ -531,7 +531,7 @@
 
 #define CONFIG_BAUDRATE 115200
 
-#define CONFIG_LOADADDR 20 /* default location for tftp and bootm */
+#define CONFIG_LOADADDR 50 /* default location for tftp and bootm */
 
 #define CONFIG_BOOTDELAY 6 /* -1 disables auto-boot */
 #undef CONFIG_BOOTARGS /* the boot command will set bootargs */
diff --git a/include/configs/MPC8323ERDB.h b/include/configs/MPC8323ERDB.h
index 92d7aa4..32f57ac 100644
--- a/include/configs/MPC8323ERDB.h
+++ b/include/configs/MPC8323ERDB.h
@@ -568,7 +568,7 @@
 #define CONFIG_UBOOTPATH   u-boot.bin  /* U-Boot image on TFTP server 
*/
 #define CONFIG_FDTFILE mpc832x_rdb.dtb
 
-#define CONFIG_LOADADDR20  /* default location for tftp 
and bootm */
+#define CONFIG_LOADADDR50  /* default location for tftp 
and bootm */
 #define CONFIG_BOOTDELAY   -1  /* -1 disables auto-boot */
 #define CONFIG_BAUDRATE115200
 
diff --git a/include/configs/MPC832XEMDS.h b/include/configs/MPC832XEMDS.h
index be2ab45..ddefa5e 100644
--- a/include/configs/MPC832XEMDS.h
+++ b/include/configs/MPC832XEMDS.h
@@ -572,7 +572,7 @@
 
 #define CONFIG_BAUDRATE115200
 
-#define CONFIG_LOADADDR20  /* default location for tftp and bootm 
*/
+#define CONFIG_LOADADDR50  /* default location for tftp and bootm 
*/
 
 #define CONFIG_BOOTDELAY 6 /* -1 disables auto-boot */
 #undef  CONFIG_BOOTARGS/* the boot command will set bootargs */
diff --git a/include/configs/MPC8349EMDS.h b/include/configs/MPC8349EMDS.h
index 07fefec..364ffac 100644
--- a/include/configs/MPC8349EMDS.h
+++ b/include/configs/MPC8349EMDS.h
@@ -709,7 +709,7 @@
 #define CONFIG_GATEWAYIP   192.168.1.1
 #define CONFIG_NETMASK 255.255.255.0
 
-#define CONFIG_LOADADDR20  /* default location for tftp 
and bootm */
+#define CONFIG_LOADADDR50  /* default location for tftp 
and bootm */
 
 #define CONFIG_BOOTDELAY   6   /* -1 disables auto-boot */
 #undef  CONFIG_BOOTARGS/* the boot command will set 
bootargs */
diff --git a/include/configs/MPC8349ITX.h b/include/configs/MPC8349ITX.h
index 6b8b74d..be8850a 100644
--- a/include/configs/MPC8349ITX.h
+++ b/include/configs/MPC8349ITX.h
@@ -474,7 +474,7 @@ boards, we say we have two, but don't display a message if 
we find only one. */
 #define CFG_PROMPT_HUSH_PS2  
 
 #define CFG_LOAD_ADDR  0x200   /* default load address */
-#define CONFIG_LOADADDR20  /* default location for tftp and bootm 
*/
+#define CONFIG_LOADADDR50  /* default location for tftp and bootm 
*/
 
 #ifdef CONFIG_MPC8349ITX
 #define CFG_PROMPT MPC8349E-mITX/* Monitor Command Prompt */
diff --git a/include/configs/MPC8360EMDS.h b/include/configs/MPC8360EMDS.h
index 46451c4..983575e 100644
--- a/include/configs/MPC8360EMDS.h
+++ b/include/configs/MPC8360EMDS.h
@@ -605,7 +605,7 @@
 
 #define CONFIG_BAUDRATE 115200
 
-#define CONFIG_LOADADDR 20 /* default location for tftp and bootm */
+#define CONFIG_LOADADDR 50 /* default location for tftp and bootm */
 
 #define CONFIG_BOOTDELAY 6 /* -1 disables auto-boot */
 #undef CONFIG_BOOTARGS /* the boot command will set bootargs */
diff --git a/include/configs/MPC837XEMDS.h b/include/configs/MPC837XEMDS.h
index 7fc0f7e..e92493a 100644
--- a/include/configs/MPC837XEMDS.h
+++ b/include/configs/MPC837XEMDS.h
@@ -595,7 +595,7 @@
 
 #define CONFIG_BAUDRATE 115200

Re: [U-Boot-Users] [PATCH] ppc: Revert patch 70431e8a that used _start instead of CFG_MONITOR_BASE

2008-04-22 Thread Kim Phillips
On Tue, 22 Apr 2008 07:49:57 +0200
Joakim Tjernlund [EMAIL PROTECTED] wrote:

  Currently WD's top of tree renders 83xx kaput.  When I revert this
  revert plus Joakim's original 70431e8 commit, things are back to
  normal.  I'm not going to pretend I know to fix it up correctly, so
  does anyone have a problem with me sending two revert patches until
  relocation is properly and comprehensively fixed?
 
 Strange, I got a 8321 and it worked for me. Maybe a toolchain issue? Perhaps
 I got something extra in my board port, dunno what though.
 
I just reconfirmed it using WD's TOT w/gcc version 4.1.2 20070925 (Red
Hat 4.1.2-27).

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] ppc: Revert patch 70431e8a that used _start instead of CFG_MONITOR_BASE

2008-04-21 Thread Kim Phillips
On Thu, 10 Apr 2008 19:45:02 -0500
Kim Phillips [EMAIL PROTECTED] wrote:

 On Wed,  9 Apr 2008 12:56:42 +0200
 Stefan Roese [EMAIL PROTECTED] wrote:
 
  The patch 70431e8a7393b6b793f77957f95b999fc9a269b8 (Make MPC83xx one step
  closer to full relocation.) doesn't use CFG_MONITOR_BASE anymore. But
  on 4xx systems _start currently cannot be used for this calculation.
  So revert back to the original version for now.
  
  Signed-off-by: Stefan Roese [EMAIL PROTECTED]
 
 Acked-by: Kim Phillips [EMAIL PROTECTED]
 
hmm..I seem to have false-acked this due to a bug in a build script of
mine.

Currently WD's top of tree renders 83xx kaput.  When I revert this
revert plus Joakim's original 70431e8 commit, things are back to
normal.  I'm not going to pretend I know to fix it up correctly, so
does anyone have a problem with me sending two revert patches until
relocation is properly and comprehensively fixed?

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] lib_ppc: Revert Make MPC83xx one step closer to full relocation.

2008-04-21 Thread Kim Phillips
This reverts commit 70431e8a7393b6b793f77957f95b999fc9a269b8 which has
proven problematic getting right from the start at least on 83xx and
4xx.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 cpu/mpc83xx/start.S |   11 ---
 lib_ppc/board.c |1 -
 2 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/cpu/mpc83xx/start.S b/cpu/mpc83xx/start.S
index fdf9d35..309eb30 100644
--- a/cpu/mpc83xx/start.S
+++ b/cpu/mpc83xx/start.S
@@ -172,11 +172,8 @@ boot_warm: /* time t 5 */
/* there and deflate the flash size back to minimal size  */
/**/
bl map_flash_by_law1
-
-   GET_GOT /* initialize GOT access*/
-   lwz r4, GOT(_start)
-   addi r4, r4, -EXC_OFF_SYS_RESET
-
+   lis r4, (CFG_MONITOR_BASE)@h
+   ori r4, r4, (CFG_MONITOR_BASE)@l
addi r5, r4, in_flash - _start + EXC_OFF_SYS_RESET
mtlr r5
blr
@@ -875,8 +872,8 @@ relocate_code:
mr  r10, r5 /* Save copy of Destination Address */
 
mr  r3,  r5 /* Destination Address */
-   lwz r4, GOT(_start)
-   addir4, r4, -EXC_OFF_SYS_RESET
+   lis r4, [EMAIL PROTECTED]   /* Source  Address */
+   ori r4, r4, [EMAIL PROTECTED]
lwz r5, GOT(__init_end)
sub r5, r5, r4
li  r6, CFG_CACHELINE_SIZE  /* Cache Line Size */
diff --git a/lib_ppc/board.c b/lib_ppc/board.c
index b2bc4eb..1b8a872 100644
--- a/lib_ppc/board.c
+++ b/lib_ppc/board.c
@@ -124,7 +124,6 @@ DECLARE_GLOBAL_DATA_PTR;
 #define CFG_MEM_TOP_HIDE   0
 #endif
 
-extern ulong _start;
 extern ulong __init_end;
 extern ulong _end;
 ulong monitor_flash_len;
-- 
1.5.5


-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] fix system config overwrite @ MPC834x

2008-04-18 Thread Kim Phillips
On Fri, 18 Apr 2008 10:48:51 +0200
Andre Schwarz [EMAIL PROTECTED] wrote:

 Kim,
 
 doing a git-pull gives Already up-to-date.
 The patch is produced with git-diff --patch-with-stat 
 cpu/mpc83xx/cpu_init.c

using git-commit and git-format-patch is recommended, but that's
probably not the problem here.  Have you followed the instructions for
Thunderbird in linux-2.6/Documentation/email-clients.txt?  You can test
by using git-am on patches you email yourself.

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] What methods of software authentication does U-Boot support?

2008-04-18 Thread Kim Phillips
On Fri, 18 Apr 2008 12:01:38 -0500
[EMAIL PROTECTED] wrote:

 Goal:
 
 U-Boot will run only software that has been
 authenticated to be from the system's producer.  
 
 --- A Potential Authentication Method ---
 
 The producer of the system generates a cryptographic
 [private-key, public-key] pair, storing the public-key
 on the same media as U-Boot (i.e. NOR flash; perhaps
 as a read-only environment variable) on all systems
 and keeping the private-key hidden at a secure site.
 A hash of the software is generated, encrypted
 with the private key and shipped with the software.
 
 U-Boot reads the private-key encrypted hash and decodes
 it with its public-key.  U-Boot loads the software and
 generates the hash.  If both hashes match, the software
 is authenticated and U-Boot executes the authenticated
 software.
 
 --- Comment ---
 
 U-Boot obviously supports loading and verification of
 the generated hash, but I haven't been able to locate
 public-key cryptographic or other authentication support
 in U-Boot.  Perhaps, it is available as a loadable
 (stand-alone) module?
 
 Any comments or suggestions?
 
this patch taps into openssl:

http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/23977

or you might want to reuse some of linux' crypto library code.

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 1/2] add MPC8343 based board mvBlueLYNX-M7 aka mvblm7

2008-04-11 Thread Kim Phillips
On Fri, 11 Apr 2008 22:02:38 +0200
André Schwarz [EMAIL PROTECTED] wrote:

 Kim,
 
 please review this resubmitted patch with changes as you have requested :
 
 - coding style issues
 - Makefile
 - Maintainer
 - common mpc83xx PCI code
 - added README.mvblm7
 
Hello André, thanks, looks generally good, but..

 +fpga_debug(%s:%d: Initialize FPGA interface (relocation offset = 
 0x%.8lx)\n,

unfortunately, both messages are linewrapped:

Applying add MPC8343 based board mvBlueLYNX-M7 aka mvblm7
fatal: corrupt patch at line 8
Patch failed at 0001.
When you have resolved this problem run git-am --resolved.
If you would prefer to skip this patch, instead run git-am --skip.

Applying add MPC8343 based board mvBlueLYNX-M7 aka mvblm7
.dotest/patch:10: trailing whitespace.
setenv bootdtb bootm \${kernel_boot} \${mv_initrd_addr_ram} 
fatal: corrupt patch at line 11
Patch failed at 0001.
When you have resolved this problem run git-am --resolved.
If you would prefer to skip this patch, instead run git-am --skip.

please reference linux-2.6/Documentation/email-clients.txt to help
resolve this.

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] System hangs while starting: get clone of u-boot-at91.git - part 2 of 2

2008-04-11 Thread Kim Phillips
On Fri, 11 Apr 2008 16:20:00 -0500
[EMAIL PROTECTED] wrote:

 I did a clone of the same git repository without any problems earlier
 today, except it wouldn't build properly.
 
 Any suggestions about what is wrong with the clone attempt and what can
 be done to fix it?
 
I'd have said use the git: protocol, but that doesn't seem to help
either:

Initialized empty Git repository in /home/kim/git/hmm.git/.git/
git.denx.de[0: 0.0.0.1]: errno=Success
fatal: unable to connect a socket (Success)
fetch-pack from 'git://git.denx.de/u-boot.git' failed.

maybe this has something to do with denx.de's new git server.  I'll fwd
this to root.

Kim

p.s. I suspect if you update your git, it'll error out without hanging.

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] System hangs while starting: get clone of u-boot-at91.git - part 2 of 2

2008-04-11 Thread Kim Phillips
On Fri, 11 Apr 2008 23:40:37 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 ... or simply work, at least with git protocol.

ok, it works when I s/git.denx.de/www.denx.de/, using the git
protocol.  This might have something to do with our firewall though.

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] Please pull u-boot-mpc83xx.git - small fixes

2008-04-11 Thread Kim Phillips
The following changes since commit 950a392464e616b4590bc4501be46e2d7d162dea:
  Wolfgang Denk (1):
Revert merge of git://www.denx.de/git/u-boot-arm, commit 62479b18:

are available in the git repository at:

  git://www.denx.de/git/u-boot-mpc83xx.git

Dave Liu (2):
  mpc83xx: Fix the SATA clock setting of 837x targets
  mpc83xx: Fix the bug of serdes initialization

Jean-Christophe PLAGNIOL-VILLARD (1):
  mpc837xerdb: Fix warning: implicit declaration of function 
'fdt_fixup_dr_usb'

Lee Nipper (1):
  mpc83xx: Update DIMM data bus width test to support 40-bit width

 board/freescale/mpc837xemds/mpc837xemds.c |2 +-
 board/freescale/mpc837xerdb/mpc837xerdb.c |2 +-
 cpu/mpc83xx/spd_sdram.c   |4 ++--
 include/configs/MPC837XEMDS.h |2 +-
 include/configs/MPC837XERDB.h |2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

Thanks,

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] ppc: Revert patch 70431e8a that used _start instead of CFG_MONITOR_BASE

2008-04-10 Thread Kim Phillips
On Wed,  9 Apr 2008 12:56:42 +0200
Stefan Roese [EMAIL PROTECTED] wrote:

 The patch 70431e8a7393b6b793f77957f95b999fc9a269b8 (Make MPC83xx one step
 closer to full relocation.) doesn't use CFG_MONITOR_BASE anymore. But
 on 4xx systems _start currently cannot be used for this calculation.
 So revert back to the original version for now.
 
 Signed-off-by: Stefan Roese [EMAIL PROTECTED]

Acked-by: Kim Phillips [EMAIL PROTECTED]

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] mpc837xerdb: Fix warning: implicit declaration of function 'fdt_fixup_dr_usb'

2008-04-02 Thread Kim Phillips
On Wed,  2 Apr 2008 13:41:21 +0200
Jean-Christophe PLAGNIOL-VILLARD [EMAIL PROTECTED] wrote:

 Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD [EMAIL PROTECTED]
 
Acked-by: Kim Phillips [EMAIL PROTECTED]

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] Add Vitesse 8601 support to TSEC driver

2008-04-01 Thread Kim Phillips
On Tue, 01 Apr 2008 15:33:10 +0200
Andre Schwarz [EMAIL PROTECTED] wrote:

 I would expect something like
 
 if ( priv-flags  TSEC_REDUCED )
 regs-ecntrl |= ECNTRL_RPM;

yeah, that register should be r/o..

 @Kim : Did you ever run a MPC834x with a RGMII  PHY ? Is it known to work ?

834x?  I haven't, but I also haven't a reason why it would not work :)

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] mpc83xx: Fix the bug of serdes initialization

2008-03-31 Thread Kim Phillips
On Mon, 31 Mar 2008 17:05:12 +0800
Dave Liu [EMAIL PROTECTED] wrote:

 Currently the serdes will not be initializated due to the
 partid's error.
 
 Signed-off-by: Dave Liu [EMAIL PROTECTED]
 ---
  board/freescale/mpc837xemds/mpc837xemds.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

Acked-by: Kim Phillips [EMAIL PROTECTED]

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] Add Vitesse 8601 support to TSEC driver

2008-03-31 Thread Kim Phillips
On Mon, 31 Mar 2008 10:01:34 -0400
Ben Warren [EMAIL PROTECTED] wrote:

 Tor Krill wrote:
  Add phy_info for Vitesse VSC8601.
  Add config option, CFG_VSC8601_SKEWFIX, to enable RGMII skew timing 
  compensation.
 
  Signed-off-by: Tor Krill [EMAIL PROTECTED]

 Acked-by: Ben Warren [EMAIL PROTECTED]

I don't have a Vitesse 8601, so technically I can't ack it, but I can:

Reviewed-by: Kim Phillips [EMAIL PROTECTED]

minor nit: it would be nice if the following:

  +#ifdef CFG_VSC8601_SKEWFIX
  +   
  {MIIM_VSC8601_EPHY_CON,MIIM_VSC8601_EPHY_CON_INIT_SKEW,NULL},
  +#endif

were made to have spaces after commas, and flow onto a separate
line so as to not be a 96 char line..

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 1/3] QE IO: Add initial data to pin configuration

2008-03-31 Thread Kim Phillips
On Mon, 31 Mar 2008 15:13:15 +0300
David Saada [EMAIL PROTECTED] wrote:

 On the MPC83xx  MPC85xx architectures that have QE, add initial data to
 the pin configuration table (qe_iop_conf_tab). 
 This is relevant for GPIO pins defined as output. One can setup a value
 of -1 to leave the value unchanged.
 In addition, add I/O pin read  write functions.
 
 Signed-off-by: David Saada [EMAIL PROTECTED]
 
 cpu/mpc83xx/cpu_init.c |7 ++---

where's the --- line?

 cpu/mpc83xx/qe_io.c|   59
 -

this patch is seriously line wrapped..and therefore unapplicable.

 cpu/mpc85xx/cpu_init.c |7 ++---
 cpu/mpc85xx/qe_io.c|   58
 ++--
 include/ioports.h  |8 ++
 5 files changed, 118 insertions(+), 21 deletions(-)
 
 --- a/include/ioports.h   2008-03-28 01:49:12.0 +0200
 +++ b/include/ioports.h   2008-03-30 16:11:09.514274000 +0300
 @@ -60,6 +60,14 @@ typedef struct {
   int dir;
   int open_drain;
   int assign;
 + int data;
  } qe_iop_conf_t;
  
  #define QE_IOP_TAB_END   (-1)
 +
 +void qe_config_iopin(u8 port, u8 pin, int dir, int open_drain, int
 assign, 
 +  int data);
 +void qe_read_iopin(u8 port, u8 pin, int *data);
 +void qe_write_iopin(u8 port, u8 pin, int data);
 +
 +
 --- a/cpu/mpc85xx/cpu_init.c  2008-03-28 01:49:12.0 +0200
 +++ b/cpu/mpc85xx/cpu_init.c  2008-03-30 15:42:32.913152000 +0300
 @@ -39,15 +39,13 @@ DECLARE_GLOBAL_DATA_PTR;
  
  #ifdef CONFIG_QE
  extern qe_iop_conf_t qe_iop_conf_tab[];
 -extern void qe_config_iopin(u8 port, u8 pin, int dir,
 - int open_drain, int assign);
  extern void qe_init(uint qe_base);
  extern void qe_reset(void);
  
  static void config_qe_ioports(void)
  {
   u8  port, pin;
 - int dir, open_drain, assign;
 + int dir, open_drain, assign, data;
   int i;
  
   for (i = 0; qe_iop_conf_tab[i].assign != QE_IOP_TAB_END; i++) {
 @@ -56,7 +54,8 @@ static void config_qe_ioports(void)
   dir = qe_iop_conf_tab[i].dir;
   open_drain  = qe_iop_conf_tab[i].open_drain;
   assign  = qe_iop_conf_tab[i].assign;
 - qe_config_iopin(port, pin, dir, open_drain, assign);
 + data= qe_iop_conf_tab[i].data;
 + qe_config_iopin(port, pin, dir, open_drain, assign,
 data);
   }
  }
  #endif
 --- a/cpu/mpc85xx/qe_io.c 2008-03-28 01:49:12.0 +0200
 +++ b/cpu/mpc85xx/qe_io.c 2008-03-30 15:44:56.961332000 +0300
 @@ -27,16 +27,30 @@
  
  #if defined(CONFIG_QE)
  #define  NUM_OF_PINS 32
 -void qe_config_iopin(u8 port, u8 pin, int dir, int open_drain, int
 assign)
 +void qe_config_iopin(u8 port, u8 pin, int dir, int open_drain, int
 assign, 
 +  int data)
  {
   u32 pin_2bit_mask;
   u32 pin_2bit_dir;
   u32 pin_2bit_assign;
   u32 pin_1bit_mask;
   u32 tmp_val;
 - volatile ccsr_gur_t *gur = (void *)(CFG_MPC85xx_GUTS_ADDR);
 - volatile par_io_t   *par_io = (volatile par_io_t *)
 - (gur-qe_par_io);
 + ccsr_gur_t  *gur = (void *)(CFG_MPC85xx_GUTS_ADDR);
 + par_io_t*par_io = (par_io_t *) (gur-qe_par_io);
 +
 + /* Calculate pin location for 1bit mask */
 + pin_1bit_mask = (u32)(1  (NUM_OF_PINS - (pin+1)));

drop the innermost parens, add spaces around +.  Is the cast really
needed?

 + /* Setup the data */
 + if ((data != -1)  /* Don't leave unchanged */ 
 + (assign == 0) /* GPIO */
 + (dir  1)) {/* Has output */

alignment

 + tmp_val = in_be32(par_io[port].cpdat);
 + if (data)
 + out_be32(par_io[port].cpdat, pin_1bit_mask |
 tmp_val);
 + else
 + out_be32(par_io[port].cpdat, ~pin_1bit_mask 
 tmp_val);
 + }

how about something like:

if (data)
tmp_val |= pin_1bit_mask;
else
tmp_val = ~pin_1bit_mask;
out_be32(.., tmp_val);

  
   /* Caculate pin location and 2bit mask and dir */
   pin_2bit_mask = (u32)(0x3 
 (NUM_OF_PINS-(pin%(NUM_OF_PINS/2)+1)*2));
 @@ -55,9 +69,6 @@ void qe_config_iopin(u8 port, u8 pin, in
   out_be32(par_io[port].cpdir1, pin_2bit_dir | tmp_val);
   }
  
 - /* Calculate pin location for 1bit mask */
 - pin_1bit_mask = (u32)(1  (NUM_OF_PINS - (pin+1)));
 -
   /* Setup the open drain */
   tmp_val = in_be32(par_io[port].cpodr);
   if (open_drain)
 @@ -82,4 +93,37 @@ void qe_config_iopin(u8 port, u8 pin, in
   }
  }
  
 +void qe_read_iopin(u8 port, u8 pin, int *data)
 +{
 + u32 pin_1bit_mask;
 

[U-Boot-Users] [PATCH 1/3] mpc83xx: unreinvent mem_clk

2008-03-28 Thread Kim Phillips
delete ddr_clk and use mem_clk instead.  Rename other ddr_*_clk to
mem_*_clk for consistency's sake.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 cpu/mpc83xx/spd_sdram.c   |   10 +-
 cpu/mpc83xx/speed.c   |   16 
 include/asm-ppc/global_data.h |3 +--
 3 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/cpu/mpc83xx/spd_sdram.c b/cpu/mpc83xx/spd_sdram.c
index 0acca47..c9151d7 100644
--- a/cpu/mpc83xx/spd_sdram.c
+++ b/cpu/mpc83xx/spd_sdram.c
@@ -78,12 +78,12 @@ extern int dma_xfer(void *dest, uint count, void *src);
 int
 picos_to_clk(int picos)
 {
-   unsigned int ddr_bus_clk;
+   unsigned int mem_bus_clk;
int clks;
 
-   ddr_bus_clk = gd-ddr_clk  1;
-   clks = picos / (10 / (ddr_bus_clk / 1000));
-   if (picos % (10 / (ddr_bus_clk / 1000)) != 0)
+   mem_bus_clk = gd-mem_clk  1;
+   clks = picos / (10 / (mem_bus_clk / 1000));
+   if (picos % (10 / (mem_bus_clk / 1000)) != 0)
clks++;
 
return clks;
@@ -313,7 +313,7 @@ long int spd_sdram()
 
debug(DDR:Module maximum data rate is: %dMhz\n, max_data_rate);
 
-   ddrc_clk = gd-ddr_clk / 100;
+   ddrc_clk = gd-mem_clk / 100;
effective_data_rate = 0;
 
if (max_data_rate = 390  max_data_rate  460) { /* it is DDR 400 */
diff --git a/cpu/mpc83xx/speed.c b/cpu/mpc83xx/speed.c
index f598699..16145dd 100644
--- a/cpu/mpc83xx/speed.c
+++ b/cpu/mpc83xx/speed.c
@@ -122,9 +122,9 @@ int get_clocks(void)
u32 enc_clk;
u32 lbiu_clk;
u32 lclk_clk;
-   u32 ddr_clk;
+   u32 mem_clk;
 #if defined(CONFIG_MPC8360)
-   u32 ddr_sec_clk;
+   u32 mem_sec_clk;
 #endif
 #if defined(CONFIG_MPC8360) || defined(CONFIG_MPC832X)
u32 qepmf;
@@ -400,11 +400,11 @@ int get_clocks(void)
return -12;
}
 
-   ddr_clk = csb_clk *
+   mem_clk = csb_clk *
  (1 + ((im-reset.rcwl  HRCWL_DDRCM)  HRCWL_DDRCM_SHIFT));
corepll = (im-reset.rcwl  HRCWL_COREPLL)  HRCWL_COREPLL_SHIFT;
 #if defined(CONFIG_MPC8360)
-   ddr_sec_clk = csb_clk * (1 +
+   mem_sec_clk = csb_clk * (1 +
   ((im-reset.rcwl  HRCWL_LBIUCM)  HRCWL_LBIUCM_SHIFT));
 #endif
 
@@ -466,9 +466,9 @@ int get_clocks(void)
gd-enc_clk = enc_clk;
gd-lbiu_clk = lbiu_clk;
gd-lclk_clk = lclk_clk;
-   gd-ddr_clk = ddr_clk;
+   gd-mem_clk = mem_clk;
 #if defined(CONFIG_MPC8360)
-   gd-ddr_sec_clk = ddr_sec_clk;
+   gd-mem_sec_clk = mem_sec_clk;
 #endif
 #if defined(CONFIG_MPC8360) || defined(CONFIG_MPC832X)
gd-qe_clk = qe_clk;
@@ -508,9 +508,9 @@ int do_clocks (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
 #endif
printf(  Local Bus Controller:%4d MHz\n, gd-lbiu_clk / 100);
printf(  Local Bus:   %4d MHz\n, gd-lclk_clk / 100);
-   printf(  DDR: %4d MHz\n, gd-ddr_clk / 100);
+   printf(  DDR: %4d MHz\n, gd-mem_clk / 100);
 #if defined(CONFIG_MPC8360)
-   printf(  DDR Secondary:   %4d MHz\n, gd-ddr_sec_clk / 100);
+   printf(  DDR Secondary:   %4d MHz\n, gd-mem_sec_clk / 100);
 #endif
printf(  SEC: %4d MHz\n, gd-enc_clk / 100);
printf(  I2C1:%4d MHz\n, gd-i2c1_clk / 100);
diff --git a/include/asm-ppc/global_data.h b/include/asm-ppc/global_data.h
index b43dba3..9ccf7d6 100644
--- a/include/asm-ppc/global_data.h
+++ b/include/asm-ppc/global_data.h
@@ -74,7 +74,6 @@ typedef   struct  global_data {
u32 enc_clk;
u32 lbiu_clk;
u32 lclk_clk;
-   u32 ddr_clk;
u32 pci_clk;
 #if defined(CONFIG_MPC837X)
u32 pciexp1_clk;
@@ -84,7 +83,7 @@ typedef   struct  global_data {
u32 sata_clk;
 #endif
 #if defined(CONFIG_MPC8360)
-   u32  ddr_sec_clk;
+   u32  mem_sec_clk;
 #endif /* CONFIG_MPC8360 */
 #endif
 #if defined(CONFIG_MPC83XX) || defined(CONFIG_MPC85xx) || 
defined(CONFIG_MPC86xx)
-- 
1.5.4.4.481.g5075


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH 2/3] mpc83xx: display ddr frequency in board_add_ram_info banner

2008-03-28 Thread Kim Phillips
Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 cpu/mpc83xx/spd_sdram.c |   11 +++
 1 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/cpu/mpc83xx/spd_sdram.c b/cpu/mpc83xx/spd_sdram.c
index c9151d7..97ac7bb 100644
--- a/cpu/mpc83xx/spd_sdram.c
+++ b/cpu/mpc83xx/spd_sdram.c
@@ -34,10 +34,13 @@
 #include asm/mmu.h
 #include spd_sdram.h
 
+DECLARE_GLOBAL_DATA_PTR;
+
 void board_add_ram_info(int use_default)
 {
volatile immap_t *immap = (immap_t *) CFG_IMMR;
volatile ddr83xx_t *ddr = immap-ddr;
+   char buf[32];
 
printf( (DDR%d, ((ddr-sdram_cfg  SDRAM_CFG_SDRAM_TYPE_MASK)
SDRAM_CFG_SDRAM_TYPE_SHIFT) - 1);
@@ -48,9 +51,11 @@ void board_add_ram_info(int use_default)
puts(, 64-bit);
 
if (ddr-sdram_cfg  SDRAM_CFG_ECC_EN)
-   puts(, ECC on));
+   puts(, ECC on);
else
-   puts(, ECC off));
+   puts(, ECC off);
+
+   printf(, %s MHz), strmhz(buf, gd-mem_clk));
 
 #if defined(CFG_LB_SDRAM)  defined(CFG_LBC_SDRAM_SIZE)
puts(\nSDRAM: );
@@ -60,8 +65,6 @@ void board_add_ram_info(int use_default)
 
 #ifdef CONFIG_SPD_EEPROM
 
-DECLARE_GLOBAL_DATA_PTR;
-
 #if defined(CONFIG_DDR_ECC)  !defined(CONFIG_ECC_INIT_VIA_DDRC)
 extern void dma_init(void);
 extern uint dma_check(void);
-- 
1.5.4.4.481.g5075


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH 3/3] mpc83xx: cleanup System Part and Revision ID Register (SPRIDR) code

2008-03-28 Thread Kim Phillips
in the spirit of commit 1ced121600b2060ab2ff9f0fddd9421fd70a0dc6,
85xx's Update SVR numbers to expand support, simplify SPRIDR processing
and processor ID display.  Add REVID_{MAJ,MIN}OR macros to make
REVID dependent code simpler.  Also added PARTID_NO_E and IS_E_PROCESSOR
convenience macros.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 board/freescale/mpc8360emds/mpc8360emds.c |   11 +--
 board/freescale/mpc837xerdb/mpc837xerdb.c |   15 +--
 cpu/mpc83xx/cpu.c |  169 -
 include/mpc83xx.h |   91 +---
 4 files changed, 84 insertions(+), 202 deletions(-)

diff --git a/board/freescale/mpc8360emds/mpc8360emds.c 
b/board/freescale/mpc8360emds/mpc8360emds.c
index d90cdb3..2119320 100644
--- a/board/freescale/mpc8360emds/mpc8360emds.c
+++ b/board/freescale/mpc8360emds/mpc8360emds.c
@@ -98,11 +98,8 @@ int board_early_init_f(void)
/* Enable flash write */
bcsr[0xa] = ~0x04;
 
-   /* Disable G1TXCLK, G2TXCLK h/w buffers (rev.2 h/w bug workaround) */
-   if (immr-sysconf.spridr == SPR_8360_REV20 ||
-   immr-sysconf.spridr == SPR_8360E_REV20 ||
-   immr-sysconf.spridr == SPR_8360_REV21 ||
-   immr-sysconf.spridr == SPR_8360E_REV21)
+   /* Disable G1TXCLK, G2TXCLK h/w buffers (rev.2.x h/w bug workaround) */
+   if (REVID_MAJOR(immr-sysconf.spridr) == 2)
bcsr[0xe] = 0x30;
 
/* Enable second UART */
@@ -308,8 +305,8 @@ void ft_board_setup(void *blob, bd_t *bd)
 * if on mpc8360ea rev. 2.1,
 * change both ucc phy-connection-types from rgmii-id to rgmii-rxid
 */
-   if (immr-sysconf.spridr == SPR_8360_REV21 ||
-   immr-sysconf.spridr == SPR_8360E_REV21) {
+   if ((REVID_MAJOR(immr-sysconf.spridr) == 2) 
+   (REVID_MINOR(immr-sysconf.spridr) == 1)) {
int nodeoffset;
const char *prop;
int path;
diff --git a/board/freescale/mpc837xerdb/mpc837xerdb.c 
b/board/freescale/mpc837xerdb/mpc837xerdb.c
index 83fb60d..e054f4e 100644
--- a/board/freescale/mpc837xerdb/mpc837xerdb.c
+++ b/board/freescale/mpc837xerdb/mpc837xerdb.c
@@ -140,24 +140,21 @@ int board_early_init_f(void)
u32 spridr = in_be32(immr-sysconf.spridr);
 
/* we check only part num, and don't look for CPU revisions */
-   switch (spridr  16) {
-   case SPR_8379E_REV10  16:
-   case SPR_8379_REV10  16:
+   switch (PARTID_NO_E(spridr)) {
+   case SPR_8377:
fsl_setup_serdes(CONFIG_FSL_SERDES1, FSL_SERDES_PROTO_SATA,
 FSL_SERDES_CLK_100, FSL_SERDES_VDD_1V);
-   fsl_setup_serdes(CONFIG_FSL_SERDES2, FSL_SERDES_PROTO_SATA,
+   fsl_setup_serdes(CONFIG_FSL_SERDES2, FSL_SERDES_PROTO_PEX,
 FSL_SERDES_CLK_100, FSL_SERDES_VDD_1V);
break;
-   case SPR_8378E_REV10  16:
-   case SPR_8378_REV10  16:
+   case SPR_8378:
fsl_setup_serdes(CONFIG_FSL_SERDES1, FSL_SERDES_PROTO_PEX,
 FSL_SERDES_CLK_100, FSL_SERDES_VDD_1V);
break;
-   case SPR_8377E_REV10  16:
-   case SPR_8377_REV10  16:
+   case SPR_8379:
fsl_setup_serdes(CONFIG_FSL_SERDES1, FSL_SERDES_PROTO_SATA,
 FSL_SERDES_CLK_100, FSL_SERDES_VDD_1V);
-   fsl_setup_serdes(CONFIG_FSL_SERDES2, FSL_SERDES_PROTO_PEX,
+   fsl_setup_serdes(CONFIG_FSL_SERDES2, FSL_SERDES_PROTO_SATA,
 FSL_SERDES_CLK_100, FSL_SERDES_VDD_1V);
break;
default:
diff --git a/cpu/mpc83xx/cpu.c b/cpu/mpc83xx/cpu.c
index bff3cef..c878268 100644
--- a/cpu/mpc83xx/cpu.c
+++ b/cpu/mpc83xx/cpu.c
@@ -35,6 +35,32 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
+struct cpu_type {
+   char name[15];
+   u32 partid;
+};
+
+#define CPU_TYPE_ENTRY(x) {#x, SPR_##x}
+
+struct cpu_type cpu_type_list [] = {
+   CPU_TYPE_ENTRY(8311),
+   CPU_TYPE_ENTRY(8313),
+   CPU_TYPE_ENTRY(8314),
+   CPU_TYPE_ENTRY(8315),
+   CPU_TYPE_ENTRY(8321),
+   CPU_TYPE_ENTRY(8323),
+   CPU_TYPE_ENTRY(8343),
+   CPU_TYPE_ENTRY(8347_TBGA_),
+   CPU_TYPE_ENTRY(8347_PBGA_),
+   CPU_TYPE_ENTRY(8349),
+   CPU_TYPE_ENTRY(8358_TBGA_),
+   CPU_TYPE_ENTRY(8358_PBGA_),
+   CPU_TYPE_ENTRY(8360),
+   CPU_TYPE_ENTRY(8377),
+   CPU_TYPE_ENTRY(8378),
+   CPU_TYPE_ENTRY(8379),
+};
+
 int checkcpu(void)
 {
volatile immap_t *immr;
@@ -42,6 +68,7 @@ int checkcpu(void)
u32 pvr = get_pvr();
u32 spridr;
char buf[32];
+   int i;
 
immr = (immap_t *)CFG_IMMR;
 
@@ -69,130 +96,26 @@ int checkcpu(void)
}
 
spridr = immr-sysconf.spridr;
-   switch(spridr) {
-   case SPR_8349E_REV10:
-   case SPR_8349E_REV11:
-   case SPR_8349E_REV31:
-   puts(MPC8349E

[U-Boot-Users] [PATCH] mpc83xx: enable the SATA interface on mpc8315 rdb and mpc837x rdb boards

2008-03-28 Thread Kim Phillips
Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 include/configs/MPC8315ERDB.h |   23 +++
 include/configs/MPC837XERDB.h |   23 +++
 2 files changed, 46 insertions(+), 0 deletions(-)

diff --git a/include/configs/MPC8315ERDB.h b/include/configs/MPC8315ERDB.h
index af78726..432fb31 100644
--- a/include/configs/MPC8315ERDB.h
+++ b/include/configs/MPC8315ERDB.h
@@ -349,6 +349,29 @@
 #define CONFIG_ETHPRIMEeTSEC1
 
 /*
+ * SATA
+ */
+#define CONFIG_LIBATA
+#define CONFIG_FSL_SATA
+
+#define CFG_SATA_MAX_DEVICE2
+#define CONFIG_SATA1
+#define CFG_SATA1_OFFSET   0x18000
+#define CFG_SATA1  (CFG_IMMR + CFG_SATA1_OFFSET)
+#define CFG_SATA1_FLAGSFLAGS_DMA
+#define CONFIG_SATA2
+#define CFG_SATA2_OFFSET   0x19000
+#define CFG_SATA2  (CFG_IMMR + CFG_SATA2_OFFSET)
+#define CFG_SATA2_FLAGSFLAGS_DMA
+
+#ifdef CONFIG_FSL_SATA
+#define CONFIG_LBA48
+#define CONFIG_CMD_SATA
+#define CONFIG_DOS_PARTITION
+#define CONFIG_CMD_EXT2
+#endif
+
+/*
  * Environment
  */
 #ifndef CFG_RAMBOOT
diff --git a/include/configs/MPC837XERDB.h b/include/configs/MPC837XERDB.h
index 90812e9..eaac525 100644
--- a/include/configs/MPC837XERDB.h
+++ b/include/configs/MPC837XERDB.h
@@ -416,6 +416,29 @@
 #endif
 
 /*
+ * SATA
+ */
+#define CONFIG_LIBATA
+#define CONFIG_FSL_SATA
+
+#define CFG_SATA_MAX_DEVICE2
+#define CONFIG_SATA1
+#define CFG_SATA1_OFFSET   0x18000
+#define CFG_SATA1  (CFG_IMMR + CFG_SATA1_OFFSET)
+#define CFG_SATA1_FLAGSFLAGS_DMA
+#define CONFIG_SATA2
+#define CFG_SATA2_OFFSET   0x19000
+#define CFG_SATA2  (CFG_IMMR + CFG_SATA2_OFFSET)
+#define CFG_SATA2_FLAGSFLAGS_DMA
+
+#ifdef CONFIG_FSL_SATA
+#define CONFIG_LBA48
+#define CONFIG_CMD_SATA
+#define CONFIG_DOS_PARTITION
+#define CONFIG_CMD_EXT2
+#endif
+
+/*
  * Environment
  */
 #ifndef CFG_RAMBOOT
-- 
1.5.5.rc0.16.g02b00


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 3/3] mpc83xx: cleanup System Part and Revision ID Register (SPRIDR) code

2008-03-28 Thread Kim Phillips
On Fri, 28 Mar 2008 21:01:30 +0100
Joakim Tjernlund [EMAIL PROTECTED] wrote:

 BTW, what happened to relocation stuff Grant was doing? It is
 still disabled, I never had any problems so perhaps time to
 turn it on again? it might even make u-boot smaller once all the
 old manuel relocation code has been properly disabled.
 
beats me.  I'm all for it, but I think we're still supposed to support
a wide range of toolchain versions (the reason Grant's stuff got
reverted).

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] Make MPC83xx one step closer to full relocation.

2008-03-28 Thread Kim Phillips
On Fri, 28 Mar 2008 15:41:25 +0100
Joakim Tjernlund [EMAIL PROTECTED] wrote:

 Remove a few absolute references to CFG_MONITOR_BASE for ppc/mpc83xx
 and use GOT relative reference.
 ---
  cpu/mpc83xx/start.S |   11 +++
  lib_ppc/board.c |3 ++-
  2 files changed, 9 insertions(+), 5 deletions(-)

Joakim,

this patch is missing your signoff, and doesn't apply cleanly (even
adds whitespace).

btw, shouldn't it fix up the other ppc start.S files?

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] Please pull u-boot-mpc83xx.git

2008-03-28 Thread Kim Phillips
Wolfgang D.,

please pull SATA configuration patches for 8315 and 837x boards, a
cleanup of the 83xx SPRIDR code, and some 8323 rdb fixes:

The following changes since commit 74d1e66d22dac91388bc538b2fe19f735edc5b82:
  Bartlomiej Sieka (1):
Fix host tool build breakage, take two

are available in the git repository at:

  git://www.denx.de/git/u-boot-mpc83xx.git master

Dave Liu (2):
  mpc83xx: initialize serdes for MPC837xEMDS boards
  mpc83xx: enable the SATA interface on mpc837xemds board

Kim Phillips (4):
  mpc83xx: enable the SATA interface on mpc8315 rdb and mpc837x rdb boards
  mpc83xx: unreinvent mem_clk
  mpc83xx: display ddr frequency in board_add_ram_info banner
  mpc83xx: cleanup System Part and Revision ID Register (SPRIDR) code

Michael Barkowski (2):
  mpc8323erdb: define CONFIG_PCI_SKIP_HOST_BRIDGE
  mpc8323erdb: fix EEPROM page size and get MAC from EEPROM

 board/freescale/mpc8323erdb/mpc8323erdb.c |   34 ++
 board/freescale/mpc8360emds/mpc8360emds.c |   11 +--
 board/freescale/mpc837xemds/mpc837xemds.c |   30 +
 board/freescale/mpc837xerdb/mpc837xerdb.c |   15 +--
 cpu/mpc83xx/cpu.c |  166 -
 cpu/mpc83xx/spd_sdram.c   |   21 ++--
 cpu/mpc83xx/speed.c   |   16 ++--
 include/asm-ppc/global_data.h |3 +-
 include/configs/MPC8315ERDB.h |   23 
 include/configs/MPC8323ERDB.h |   12 ++-
 include/configs/MPC837XEMDS.h |   28 +
 include/configs/MPC837XERDB.h |   23 
 include/mpc83xx.h |   91 +---
 lib_ppc/board.c   |2 +-
 14 files changed, 250 insertions(+), 225 deletions(-)

Thanks,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] Make MPC83xx one step closer to full relocation.

2008-03-28 Thread Kim Phillips
On Sat, 29 Mar 2008 00:47:01 +0100
Joakim Tjernlund [EMAIL PROTECTED] wrote:

  -Original Message-
  From: Kim Phillips [mailto:[EMAIL PROTECTED]
  Sent: den 29 mars 2008 00:33
  To: Joakim Tjernlund
  Cc: u-boot-users@lists.sourceforge.net
  Subject: Re: [PATCH] Make MPC83xx one step closer to full relocation.
  
  On Fri, 28 Mar 2008 15:41:25 +0100
  Joakim Tjernlund [EMAIL PROTECTED] wrote:
  
   Remove a few absolute references to CFG_MONITOR_BASE for ppc/mpc83xx
   and use GOT relative reference.
   ---
cpu/mpc83xx/start.S |   11 +++
lib_ppc/board.c |3 ++-
2 files changed, 9 insertions(+), 5 deletions(-)
  
  Joakim,
  
  this patch is missing your signoff, and doesn't apply cleanly (even
  adds whitespace).
 
 It doesn't apply because you have been sitting on it for too long :)
 I sent long ago and you said would apply it in the next merge window,
 but seems to have forgotten about it. Sorry for the whitespace. I am still
 on the last u-boot release so I am afraid I can't do much about it ATM.
 I hope you can clean up the WS.
 
 Signed-off-by: Joakim Tjernlund [EMAIL PROTECTED]

ok, I can add it now.

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Please pull u-boot-mpc83xx.git

2008-03-28 Thread Kim Phillips
On Fri, 28 Mar 2008 18:49:59 -0500
Kim Phillips [EMAIL PROTECTED] wrote:

 Wolfgang D.,
 
 please pull SATA configuration patches for 8315 and 837x boards, a
 cleanup of the 83xx SPRIDR code, and some 8323 rdb fixes:
 
ok, I've pushed Joakim's relocation fixes patch.  New summary looks
like:

Dave Liu (2):
  mpc83xx: initialize serdes for MPC837xEMDS boards
  mpc83xx: enable the SATA interface on mpc837xemds board

Joakim Tjernlund (1):
  Make MPC83xx one step closer to full relocation.

Kim Phillips (4):
  mpc83xx: enable the SATA interface on mpc8315 rdb and mpc837x rdb boards
  mpc83xx: unreinvent mem_clk
  mpc83xx: display ddr frequency in board_add_ram_info banner
  mpc83xx: cleanup System Part and Revision ID Register (SPRIDR) code

Michael Barkowski (2):
  mpc8323erdb: define CONFIG_PCI_SKIP_HOST_BRIDGE
  mpc8323erdb: fix EEPROM page size and get MAC from EEPROM

 board/freescale/mpc8323erdb/mpc8323erdb.c |   34 ++
 board/freescale/mpc8360emds/mpc8360emds.c |   11 +--
 board/freescale/mpc837xemds/mpc837xemds.c |   30 +
 board/freescale/mpc837xerdb/mpc837xerdb.c |   15 +--
 cpu/mpc83xx/cpu.c |  166 -
 cpu/mpc83xx/spd_sdram.c   |   21 ++--
 cpu/mpc83xx/speed.c   |   16 ++--
 cpu/mpc83xx/start.S   |   11 ++-
 include/asm-ppc/global_data.h |3 +-
 include/configs/MPC8315ERDB.h |   23 
 include/configs/MPC8323ERDB.h |   12 ++-
 include/configs/MPC837XEMDS.h |   28 +
 include/configs/MPC837XERDB.h |   23 
 include/mpc83xx.h |   91 +---
 lib_ppc/board.c   |5 +-
 15 files changed, 259 insertions(+), 230 deletions(-)

Thanks,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] board config u-boot

2008-03-27 Thread Kim Phillips
On Thu, 27 Mar 2008 18:28:57 +0100
karim sahli [EMAIL PROTECTED] wrote:

 hi!
 i want to add a new board config on my u-boot 1.2.0
 how can i do?
 (my board is based on the kb9202 board)
 thank you
 
see U-Boot Porting Guide in the README.

(and upgrade from 1.2.0)

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 6/7] 83xx/fdt_support: let user specifiy FSL USB Dual-Role controller role

2008-03-25 Thread Kim Phillips
On Tue, 25 Mar 2008 19:06:34 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:

 On Tue, Mar 25, 2008 at 10:42:11AM -0500, Kim Phillips wrote:
 [...]
but the QE based 83xx SoCs don't use this fixup.  So while
CONFIG_HAS_FSL_DR_USB gives you more control, I doubt saving 200 bytes
justifies the new config either.  Either way really, but I tend to be
against any new CONFIG_s.
   
   Hm... maybe then it's better to place fdt_fixup_usb_dr function
   inside the fdt_support.h, marking it with static inline -- it's used
   just once per board anyway..? Then neither space wasted nor the new
   configs introduced.
  
  I'd like to see that.  Do you want to send a patch?
 
 Sure, here it is.
 
 - - - -
 From: Anton Vorontsov [EMAIL PROTECTED]
 Subject: mpc83xx/fdt_support: let user specifiy FSL USB Dual-Role controller 
 role
 
 Linux understands host (default), peripheral and otg (broken).
 Though, U-Boot doesn't restrict dr_mode variable to these values (think
 of renames in future).
 
 Signed-off-by: Anton Vorontsov [EMAIL PROTECTED]
 ---
  board/freescale/mpc837xerdb/mpc837xerdb.c |2 ++
  include/fdt_support.h |   24 
  2 files changed, 26 insertions(+), 0 deletions(-)
 
 diff --git a/board/freescale/mpc837xerdb/mpc837xerdb.c 
 b/board/freescale/mpc837xerdb/mpc837xerdb.c
 index d091538..70f52da 100644
 --- a/board/freescale/mpc837xerdb/mpc837xerdb.c
 +++ b/board/freescale/mpc837xerdb/mpc837xerdb.c
 @@ -13,6 +13,7 @@
   */
  
  #include common.h
 +#include fdt_support.h
  #include i2c.h
  #include asm/io.h
  #include asm/fsl_serdes.h
 @@ -197,5 +198,6 @@ void ft_board_setup(void *blob, bd_t *bd)
   ft_pci_setup(blob, bd);
  #endif
   ft_cpu_setup(blob, bd);
 + fdt_fixup_dr_usb(blob);
  }
  #endif /* CONFIG_OF_BOARD_SETUP */
 diff --git a/include/fdt_support.h b/include/fdt_support.h
 index 7836f28..933ef41 100644
 --- a/include/fdt_support.h
 +++ b/include/fdt_support.h
 @@ -27,6 +27,7 @@
  #ifdef CONFIG_OF_LIBFDT
  
  #include fdt.h
 +#include libfdt.h
  
  int fdt_chosen(void *fdt, ulong initrd_start, ulong initrd_end, int force);
  void do_fixup_by_path(void *fdt, const char *path, const char *prop,
 @@ -64,5 +65,28 @@ void ft_cpu_setup(void *blob, bd_t *bd);
  void ft_pci_setup(void *blob, bd_t *bd);
  #endif
  
 +static inline void fdt_fixup_dr_usb(void *fdt)
 +{
 + char *mode;
 + const char *compat = fsl-usb2-dr;
 + const char *prop = dr_mode;
 + int node_offset;
 + int err;
 +
 + mode = getenv(usb_dr_mode);
 + if (!mode)
 + return;
 +
 + node_offset = fdt_node_offset_by_compatible(fdt, 0, compat);
 + if (node_offset  0)
 + printf(WARNING: could not find compatible node %s: %s.\n,
 + compat, fdt_strerror(node_offset));
 +
 + err = fdt_setprop(fdt, node_offset, prop, mode, strlen(mode) + 1);
 + if (err  0)
 + printf(WARNING: could not set %s for %s: %s.\n,
 +prop, compat, fdt_strerror(err));
 +}
 +
  #endif /* ifdef CONFIG_OF_LIBFDT */
  #endif /* ifndef __FDT_SUPPORT_H */

sigh yeah, that's too much code for a header file though.

I think your original patch was the more prudent approach, it follows
existing practice and it fits with the assumption that there'll be a
usb driver one day :).  I'll put it in my queue and apply it unless
anyone has further comments.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] DDR-II @ MPC8343A

2008-03-25 Thread Kim Phillips
On Tue, 25 Mar 2008 18:36:47 +0100
Andre Schwarz [EMAIL PROTECTED] wrote:

 All,
 
 has anyone set up the DDR-II controller on a Freescale MPC8343A without ?

without what?

 As far as I can see there are only DDR-I boards from Freescale.
 
The MDS can do ddr2:

U-Boot 1.3.2-00097-gbc508d1 (Mar 25 2008 - 10:47:17) MPC83XX

Reset Status: Software Hard, External/Internal Soft, External/Internal Hard

CPU:   e300c1, MPC8349E, Rev: 30 at 528 MHz, CSB:  264 MHz
Board: Freescale MPC8349EMDS
I2C:   ready
SPI:   ready
DRAM:  256 MB (DDR2, 64-bit, ECC on)
...

 Are there any patches not yet posted to handle this ?
 
should work, only diff I see is that the 8343 is 32-bit only.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] DDR-II @ MPC8343A

2008-03-25 Thread Kim Phillips
On Tue, 25 Mar 2008 20:41:41 +0100
Andre Schwarz [EMAIL PROTECTED] wrote:

 Kim Phillips schrieb:
  On Tue, 25 Mar 2008 18:36:47 +0100
  Andre Schwarz [EMAIL PROTECTED] wrote:
  As far as I can see there are only DDR-I boards from Freescale.
 
  
  The MDS can do ddr2:
 
  U-Boot 1.3.2-00097-gbc508d1 (Mar 25 2008 - 10:47:17) MPC83XX
 
  Reset Status: Software Hard, External/Internal Soft, External/Internal Hard
 
  CPU:   e300c1, MPC8349E, Rev: 30 at 528 MHz, CSB:  264 MHz
  Board: Freescale MPC8349EMDS
  I2C:   ready
  SPI:   ready
  DRAM:  256 MB (DDR2, 64-bit, ECC on)
  ...
 

 Hmmm ... I get :
 
 U-Boot 1.3.2-00075-gc5f497a-dirty (Mar 25 2008 - 19:55:00) MPC83XX
 
 Reset Status: Check Stop, External/Internal Soft, External/Internal Hard
 
 CPU:   e300c1, MPC8343, Rev: 31 at 399.999 MHz, CSB:  266 MHz
 Board: Matrix Vision mvBlueLYNX-M7
 I2C:   ready
 SPI:   ready
 DRAM:  cs0_bnds = 0x000f
 cs0_config = 0x80844102
 DDR:bar=0x
 DDR:ar=0x801b
 256 MB
 
 
 It's a 32-Bit DDR-II without ECC.
 2 Chips are soldered to CS0, i.e. no EPROM present.
 
 Since I expect something like (DDR2, 32-Bit, ECC off) and nothing 
 happens there is obviously something wrong with my memory setup.
 Is this reasonable ?

sounds like memory isn't being configured correctly.  Board specific?

 Or am I simply missing a board specific board_add_ram_info ?

you can do that if you really, really, really want to see the (DDR2..
string, but it's optional.

 Am I assuming right that your info string (DDR2, 64-bit, ECC on) comes 
 from spd_sdram  ?

yes.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] DDR-II @ MPC8343A

2008-03-25 Thread Kim Phillips
On Tue, 25 Mar 2008 21:25:45 +0100
Andre Schwarz [EMAIL PROTECTED] wrote:

 I've seen things like this in mpc8349emds code :
 
 #if (CFG_DDR_SIZE != 256)
 #warning Currenly any ddr size other than 256 is not supported
 #endif
 
 Is this reasonable ? Why ?
 
that's in the fixed sdram init code, which could be considered
reasonably unreasonable by some I suppose, since we can't guarantee the
fixed settings will work with whatever memory you decide to use with
it.  It should work with the memory that comes with the board, though.

btw, the spd code is being used by default on the 834x mds, not this
fixed_sdram stuff - it's mostly there for board porting completeness.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] DDR-II @ MPC8343A

2008-03-25 Thread Kim Phillips
On Tue, 25 Mar 2008 21:54:53 +0100
Andre Schwarz [EMAIL PROTECTED] wrote:

 Kim Phillips schrieb:
  btw, the spd code is being used by default on the 834x mds, not this
  fixed_sdram stuff - it's mostly there for board porting completeness.
 

 But it is known to work, isn't it ?

I haven't tested it recently.  Patches welcome if it doesn't.

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 6/7] 83xx/fdt_support: let user specifiy FSL USB Dual-Role controller role

2008-03-25 Thread Kim Phillips
On Fri, 14 Mar 2008 23:20:18 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:

 Linux understands host (default), peripheral and otg (broken).
 Though, U-Boot doesn't restrict dr_mode variable to these values (think
 of renames in future).
 
 Signed-off-by: Anton Vorontsov [EMAIL PROTECTED]
 ---
  board/freescale/mpc837xerdb/mpc837xerdb.c |1 +
  common/fdt_support.c  |   25 +
  include/configs/MPC837XERDB.h |1 +
  include/fdt_support.h |6 ++
  4 files changed, 33 insertions(+), 0 deletions(-)
 
applied, thanks.

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] Please pull u-boot-mpc83xx.git

2008-03-25 Thread Kim Phillips
Wolfgang,

please pull support for uploading firmware for the Vitesse VSC7385 (an
ethernet switch on some 83xx boards), support for serdes
initialization, and misc fixes:

The following changes since commit aa6f6d171a1f9f46ee4f03ad6acb97a6bfb71855:
  Wolfgang Denk (1):
Coding Style cleanyp; update CHANGELOG

are available in the git repository at:

  git://www.denx.de/git/u-boot-mpc83xx.git master

Anton Vorontsov (18):
  83xx: nand support for MPC837XRDB boards
  83xx: split COBJS onto separate lines
  83xx: serdes setup routines
  83xx: initialize serdes for MPC837XRDB boards
  83xx: define CONFIG_OF_STDOUT_VIA_ALIAS for the MPC837XERDB boards
  uec: add support for gbit mii status readings
  uec: add support for RGMII_RXID interface mode
  uec: add support for Broadcom BCM5481 Gigabit PHY
  mpc83xx: MPC8360E-RDK: use RGMII_RXID interface mode
  mpc83xx: MPC8360E-RDK: add support for NAND
  mpc83xx: MPC8360E-RDK: configure pario pins for AD7843 and FHCI
  mpc83xx: MPC8360E-RDK: rework ddr setup, enable ecc
  mpc83xx: MPC8360E-RDK: add dhcp command
  mpc83xx: MPC8360E-RDK: define CONFIG_OF_STDOUT_VIA_ALIAS
  mpc83xx: MPC8360E-RDK: use 33.3(3)MHz CLKIN/SYS_CLK
  mpc83xx: add fsl,soc and fsl,immr compatible fixups
  tsec: fix link detection for the RTL8211B PHY
  83xx/fdt_support: let user specifiy FSL USB Dual-Role controller role

Jerry Van Baren (1):
  Enable CONFIG_FLASH_SHOW_PROGRESS on the MPC8360EMDS.

Joe D'Abbraccio (1):
  Modified the DDR SDRAM clock control register to delay MCK/MCK_B 3/4 clock

Michael Barkowski (3):
  mpc8323erdb: use readable DDR config macros
  mpc8323erdb: Improve the system performance
  mpc8323erdb: remove RTC and add EEPROM

Scott Wood (1):
  mpc83xx: Set PCI I/O bus-address base to zero.

Timur Tabi (2):
  NET: Add Vitesse VSC7385 firmware uploading
  83xx: Add Vitesse VSC7385 firmware uploading

 board/freescale/mpc8313erdb/mpc8313erdb.c |   21 
 board/freescale/mpc8349itx/mpc8349itx.c   |   15 +++-
 board/freescale/mpc8360erdk/Makefile  |4 +-
 board/freescale/mpc8360erdk/mpc8360erdk.c |   17 
 board/freescale/mpc8360erdk/nand.c|   72 ++
 board/freescale/mpc837xerdb/mpc837xerdb.c |   65 -
 common/fdt_support.c  |   25 +
 cpu/mpc83xx/Makefile  |   15 +++-
 cpu/mpc83xx/cpu_init.c|6 +
 cpu/mpc83xx/fdt.c |   12 ++-
 cpu/mpc83xx/pci.c |2 -
 cpu/mpc83xx/qe_io.c   |3 -
 cpu/mpc83xx/serdes.c  |  145 +
 drivers/net/Makefile  |1 +
 drivers/net/tsec.c|5 +-
 drivers/net/vsc7385.c |  101 
 drivers/qe/uec.c  |1 +
 drivers/qe/uec.h  |1 +
 drivers/qe/uec_phy.c  |   69 --
 drivers/qe/uec_phy.h  |5 +
 include/asm-ppc/fsl_serdes.h  |   21 
 include/configs/MPC8313ERDB.h |   88 --
 include/configs/MPC8315ERDB.h |2 +-
 include/configs/MPC8323ERDB.h |   64 +++--
 include/configs/MPC832XEMDS.h |2 +-
 include/configs/MPC8349ITX.h  |   35 +--
 include/configs/MPC8360EMDS.h |3 +-
 include/configs/MPC8360ERDK.h |   88 ++
 include/configs/MPC837XEMDS.h |2 +-
 include/configs/MPC837XERDB.h |  113 +--
 include/fdt_support.h |6 +
 include/mpc83xx.h |4 +-
 include/vsc7385.h |   13 +++
 33 files changed, 894 insertions(+), 132 deletions(-)
 create mode 100644 board/freescale/mpc8360erdk/nand.c
 create mode 100644 cpu/mpc83xx/serdes.c
 create mode 100644 drivers/net/vsc7385.c
 create mode 100644 include/asm-ppc/fsl_serdes.h
 create mode 100644 include/vsc7385.h

Thanks,

Kim

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 6/7] 83xx/fdt_support: let user specifiy FSL USB Dual-Role controller role

2008-03-24 Thread Kim Phillips
On Mon, 24 Mar 2008 17:44:19 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:

 On Wed, Mar 19, 2008 at 08:35:58PM -0500, Kim Phillips wrote:
  On Fri, 14 Mar 2008 23:20:18 +0300
  Anton Vorontsov [EMAIL PROTECTED] wrote:
  
  
   diff --git a/include/fdt_support.h b/include/fdt_support.h
   index 7836f28..c10de8a 100644
   --- a/include/fdt_support.h
   +++ b/include/fdt_support.h
   @@ -50,6 +50,12 @@ int fdt_find_and_setprop(void *fdt, const char *node, 
   const char *prop,
  const void *val, int len, int create);
void fdt_fixup_qe_firmware(void *fdt);

   +#ifdef CONFIG_HAS_FSL_DR_USB
   +void fdt_fixup_dr_usb(void *blob, bd_t *bd);
   +#else
   +static inline void fdt_fixup_dr_usb(void *blob, bd_t *bd) {}
   +#endif /* CONFIG_HAS_FSL_DR_USB */
   +
  
  this looks like a prime candidate for a weak function (which would also
  eliminate the need for the new CONFIG_HAS_FSL_DR_USB introduced here).
 
 Are you sure it's supposed to work like this?.. I don't see the linker
 dropping unused weak symbols with the patch below.

with something like the following I get 24 bytes more text in
fdt_support.o, and 200 total bytes added 83xx-wide:

diff --git a/board/freescale/mpc837xerdb/mpc837xerdb.c 
b/board/freescale/mpc837xerdb/mpc837xerdb.c
index a3b20b8..e8d130c 100644
--- a/board/freescale/mpc837xerdb/mpc837xerdb.c
+++ b/board/freescale/mpc837xerdb/mpc837xerdb.c
@@ -13,6 +13,7 @@
  */
 
 #include common.h
+#include fdt_support.h
 #include i2c.h
 #include asm/io.h
 #include spd_sdram.h
@@ -153,6 +154,7 @@ int misc_init_r(void)
 }
 
 #if defined(CONFIG_OF_BOARD_SETUP)
+extern void fdt_fixup_dr_usb(void *);
 
 void ft_board_setup(void *blob, bd_t *bd)
 {
@@ -160,5 +162,6 @@ void ft_board_setup(void *blob, bd_t *bd)
ft_pci_setup(blob, bd);
 #endif
ft_cpu_setup(blob, bd);
+   fdt_fixup_dr_usb(blob);
 }
 #endif /* CONFIG_OF_BOARD_SETUP */
diff --git a/common/fdt_support.c b/common/fdt_support.c
index 69eb667..355b741 100644
--- a/common/fdt_support.c
+++ b/common/fdt_support.c
@@ -615,3 +615,10 @@ void fdt_fixup_ethernet(void *fdt, bd_t *bd)
}
 }
 #endif
+
+void __fdt_fixup_dr_usb(void *fdt)
+{
+   /* add your own */
+}
+void fdt_fixup_dr_usb(void *fdt) __attribute__((weak,
+   alias(__fdt_fixup_dr_usb)));
diff --git a/cpu/mpc83xx/fdt.c b/cpu/mpc83xx/fdt.c
index 6f55932..3224286 100644
--- a/cpu/mpc83xx/fdt.c
+++ b/cpu/mpc83xx/fdt.c
@@ -68,4 +68,27 @@ void ft_cpu_setup(void *blob, bd_t *bd)
 
fdt_fixup_memory(blob, (u64)bd-bi_memstart, (u64)bd-bi_memsize);
 }
+
+void fdt_fixup_dr_usb(void *fdt)
+{
+   char *mode;
+   const char *compat = fsl-usb2-dr;
+   const char *prop = dr_mode;
+   int node_offset;
+   int err;
+
+   mode = getenv(usb_dr_mode);
+   if (!mode)
+   return;
+
+   node_offset = fdt_node_offset_by_compatible(fdt, 0, compat);
+   if (node_offset  0)
+   printf(WARNING: could not find compatible node %s: %s.\n,
+   compat, fdt_strerror(node_offset));
+
+   err = fdt_setprop(fdt, node_offset, prop, mode, strlen(mode) + 1);
+   if (err  0)
+   printf(WARNING: could not set %s for %s: %s.\n,
+  prop, compat, fdt_strerror(err));
+}
 #endif /* CONFIG_OF_LIBFDT */

but the QE based 83xx SoCs don't use this fixup.  So while
CONFIG_HAS_FSL_DR_USB gives you more control, I doubt saving 200 bytes
justifies the new config either.  Either way really, but I tend to be
against any new CONFIG_s.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 2/2] mpc83xx: MPC837XERDB: add support for PCI-Express

2008-03-24 Thread Kim Phillips
On Mon, 24 Mar 2008 21:42:37 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:

 +/* PCIE address map */
 +#define CFG_PCIE1_BASE   0xA000
 +#define CFG_PCIE1_MEM_BASE   CFG_PCIE1_BASE
 +#define CFG_PCIE1_MEM_PHYS   CFG_PCIE1_MEM_BASE
 +#define CFG_PCIE1_MEM_SIZE   0x1000
 +#define CFG_PCIE1_CFG_BASE   (CFG_PCIE1_MEM_BASE + CFG_PCIE1_MEM_SIZE)
 +#define CFG_PCIE1_CFG_SIZE   0x0100

I'm going to wait until I see how the kernel 83xx PCIE patches do
before I apply this -- afaik, it's going to make the PCIE config
mapping a 4K window (with translation being changed at run time)
instead of a 16MB one.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 01/11] uec: add support for gbit mii status readings

2008-03-24 Thread Kim Phillips
On Mon, 24 Mar 2008 20:46:24 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:

 Signed-off-by: Anton Vorontsov [EMAIL PROTECTED]
 ---
  drivers/qe/uec_phy.c |   28 +++-
  drivers/qe/uec_phy.h |5 +
  2 files changed, 24 insertions(+), 9 deletions(-)

applied 1-11.

Thanks,

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 1/1] Modified the DDR SDRAM clock control register to delay MCK/MCK_B 3/4 clock

2008-03-24 Thread Kim Phillips
On Mon, 24 Mar 2008 13:00:59 -0400
Joe D'Abbraccio Joe.D'[EMAIL PROTECTED] wrote:

 From: Joe D'Abbraccio [EMAIL PROTECTED]
 
 With the original value of 1/2 clock cycle delay, the system ran relatively
 stable except when we run benchmarks that are intensive users of memory.
 When I run samba connected disk with a HDBENCH test, the system locks-up
 or reboots sporadically.
 
 Signed-off by: Joe D'Abbraccio Joe.D'[EMAIL PROTECTED]
 ---
  include/configs/MPC8349ITX.h |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

applied.

Thanks,

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH v2 2/9] ATA: Add the libata support

2008-03-20 Thread Kim Phillips
On Thu, 20 Mar 2008 18:19:52 +0800
Dave Liu [EMAIL PROTECTED] wrote:

 diff --git a/drivers/block/libata.c b/drivers/block/libata.c
 new file mode 100644
 index 000..aed5be3
 --- /dev/null
 +++ b/drivers/block/libata.c
 @@ -0,0 +1,158 @@
 +/*
 + * Copyright (C) 2008 Freescale Semiconductor, Inc.
 + *   Dave Liu [EMAIL PROTECTED]
 + *
 + * 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
 + *
 + * with the reference to libata of kernel

If you're going to copy code directly from the kernel, the kernel
files' copyrights need to be copied here, too.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH v2 6/9] mpc83xx: serdes setup routines

2008-03-20 Thread Kim Phillips
On Thu, 20 Mar 2008 18:26:36 +0800
Dave Liu [EMAIL PROTECTED] wrote:

 From: Anton Vorontsov [EMAIL PROTECTED]
 
 This patch adds few routines to configure serdes on 837x targets.
 
 Signed-off-by: Anton Vorontsov [EMAIL PROTECTED]
 ---

sounds like Anton is re-doing this patch.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 3/7] 83xx: serdes setup routines

2008-03-19 Thread Kim Phillips
On Tue, 18 Mar 2008 16:52:29 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:

 On Tue, Mar 18, 2008 at 11:28:20AM +0100, Jean-Christophe PLAGNIOL-VILLARD 
 wrote:
  On 23:19 Fri 14 Mar , Anton Vorontsov wrote:
   This patch adds few routines to configure serdes on 837x targets.
   
   Signed-off-by: Anton Vorontsov [EMAIL PROTECTED]
   ---
cpu/mpc83xx/Makefile |1 +
cpu/mpc83xx/serdes.c |  156 
   ++
include/asm-ppc/fsl_serdes.h |   25 +++
3 files changed, 182 insertions(+), 0 deletions(-)
create mode 100644 cpu/mpc83xx/serdes.c
create mode 100644 include/asm-ppc/fsl_serdes.h
   
   diff --git a/cpu/mpc83xx/Makefile b/cpu/mpc83xx/Makefile
   index 7ad7e3e..6b6b027 100644
   --- a/cpu/mpc83xx/Makefile
   +++ b/cpu/mpc83xx/Makefile
   @@ -39,6 +39,7 @@ COBJS-y += ecc.o
COBJS-$(CONFIG_QE) += qe_io.o
COBJS-$(CONFIG_83XX_GENERIC_PCI) += pci.o
COBJS-$(CONFIG_OF_LIBFDT) += fdt.o
   +COBJS-$(CONFIG_FSL_SERDES) += serdes.o
  Please keep it sort
 
 Keeping cobjs in alphabetical order sounds weird to me, plus as
 far as I can see, current Makefile tried to keep things in this
 (sane) order:
 
 start.S
 COBJS += core (traps, cpu)
 COBJS += less important code (speed, sdram, ..)
 COBJS += auxiliary/not so important (pci, fdt, )
 
 [Almost] the same order in which the code will be executed.
 
 Though, in this case I should place SERDES above FDT, near the PCI.
 
 
 Anyhow, firstly I'd like to know what Kim is thinking about all
 this?

sorry, I was out for a while.

I personally don't have a problem with COBJS not in alpha order.

Alpha order is something Wolfgang has brought up in the past, seemingly
more appropriately, i.e. for things like board names in the main
Makefile and MAKEALL.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 1/7] 83xx: nand support for MPC837XRDB boards

2008-03-19 Thread Kim Phillips
On Tue, 18 Mar 2008 17:10:55 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:

 
   -/* VSC7385 Gigabit Switch support */
   +/*
   + * VSC7385 Gigabit Switch support
   + */
   +#define CFG_VSC7385_BASE 0xF000
  
  What does this have to do with NAND?
 
 Nothing. I'm expanding VSC7385 comment to better separate it from
 the NAND code. Most lengthy parts of the RDB config are using
 
 /*
  *
  */
 
 as a logical separator. So did I.

that's fine, it just makes patch review easier and less susceptible to
comments like this if you keep your patches on-subject.

Also, this patch doesn't apply cleanly to the top of
u-boot-mpc83xx.git, because of the VSC7385 patches I applied last week.
Can you rebase your patches to the top of the mpc83xx git tree?  I just
updated it to Wolfgang's top-of-tree.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 6/7] 83xx/fdt_support: let user specifiy FSL USB Dual-Role controller role

2008-03-19 Thread Kim Phillips
On Fri, 14 Mar 2008 23:20:18 +0300
Anton Vorontsov [EMAIL PROTECTED] wrote:


 diff --git a/include/fdt_support.h b/include/fdt_support.h
 index 7836f28..c10de8a 100644
 --- a/include/fdt_support.h
 +++ b/include/fdt_support.h
 @@ -50,6 +50,12 @@ int fdt_find_and_setprop(void *fdt, const char *node, 
 const char *prop,
const void *val, int len, int create);
  void fdt_fixup_qe_firmware(void *fdt);
  
 +#ifdef CONFIG_HAS_FSL_DR_USB
 +void fdt_fixup_dr_usb(void *blob, bd_t *bd);
 +#else
 +static inline void fdt_fixup_dr_usb(void *blob, bd_t *bd) {}
 +#endif /* CONFIG_HAS_FSL_DR_USB */
 +

this looks like a prime candidate for a weak function (which would also
eliminate the need for the new CONFIG_HAS_FSL_DR_USB introduced here).

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 2/2] 83xx: Add Vitesse VSC7385 firmware uploading

2008-03-12 Thread Kim Phillips
On Fri,  8 Feb 2008 13:15:55 -0600
Timur Tabi [EMAIL PROTECTED] wrote:

 Update the MPC8349E-mITX, MPC8313E-RDB, and MPC837XE-RDB board files to upload
 the Vitesse VSC7385 firmware.  Changed CONFIG_VSC7385 to CONFIG_VSC7385_ENET.
 Cleaned up the board header files to make selecting the VSC7385 easier to
 control.
 
 Signed-off-by: Timur Tabi [EMAIL PROTECTED]
 ---
 
 This patch is for U-Boot 1.3.3
 
  board/freescale/mpc8313erdb/mpc8313erdb.c |   21 +++
  board/freescale/mpc8349itx/mpc8349itx.c   |   15 +-
  board/freescale/mpc837xerdb/mpc837xerdb.c |   27 +++--
  include/configs/MPC8313ERDB.h |   88 
 +++--
  include/configs/MPC8349ITX.h  |   33 ---
  include/configs/MPC837XERDB.h |   83 +++-
  6 files changed, 196 insertions(+), 71 deletions(-)

applied.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH 1/2] NET: Add Vitesse VSC7385 firmware uploading

2008-03-12 Thread Kim Phillips
On Mon, 18 Feb 2008 22:06:53 -0500
Ben Warren [EMAIL PROTECTED] wrote:

 Acked-by: Ben Warren [EMAIL PROTECTED]
 
 Kim - would you mind pulling this into your tree?  Best to keep it with 
 part 2.

done.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] mpc83xx: make dtb basename file references equal those of linux

2008-03-07 Thread Kim Phillips
the dts file basenames were updated in linux - this helps avoid
inadvertently loading any old dtbs laying around.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
 include/configs/MPC832XEMDS.h |2 +-
 include/configs/MPC8360EMDS.h |2 +-
 include/configs/MPC837XEMDS.h |2 +-
 include/configs/MPC837XERDB.h |2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/configs/MPC832XEMDS.h b/include/configs/MPC832XEMDS.h
index 702b073..f32c4f7 100644
--- a/include/configs/MPC832XEMDS.h
+++ b/include/configs/MPC832XEMDS.h
@@ -583,7 +583,7 @@
ramdiskaddr=100\0 \
ramdiskfile=ramfs.83xx\0  \
fdtaddr=40\0  \
-   fdtfile=mpc832xemds.dtb\0 \
+   fdtfile=mpc832x_mds.dtb\0 \

 
 #define CONFIG_NFSBOOTCOMMAND  \
diff --git a/include/configs/MPC8360EMDS.h b/include/configs/MPC8360EMDS.h
index eff9fba..c8dcbc6 100644
--- a/include/configs/MPC8360EMDS.h
+++ b/include/configs/MPC8360EMDS.h
@@ -615,7 +615,7 @@
ramdiskaddr=100\0 \
ramdiskfile=ramfs.83xx\0  \
fdtaddr=40\0  \
-   fdtfile=mpc8360emds.dtb\0 \
+   fdtfile=mpc836x_mds.dtb\0 \

 
 #define CONFIG_NFSBOOTCOMMAND  \
diff --git a/include/configs/MPC837XEMDS.h b/include/configs/MPC837XEMDS.h
index ef2..5586533 100644
--- a/include/configs/MPC837XEMDS.h
+++ b/include/configs/MPC837XEMDS.h
@@ -578,7 +578,7 @@
ramdiskaddr=100\0 \
ramdiskfile=ramfs.83xx\0  \
fdtaddr=40\0  \
-   fdtfile=mpc837xemds.dtb\0 \
+   fdtfile=mpc8379_mds.dtb\0 \

 
 #define CONFIG_NFSBOOTCOMMAND  \
diff --git a/include/configs/MPC837XERDB.h b/include/configs/MPC837XERDB.h
index 2da4f29..1964946 100644
--- a/include/configs/MPC837XERDB.h
+++ b/include/configs/MPC837XERDB.h
@@ -545,7 +545,7 @@
 #define CONFIG_RAMDISKFILE rootfs.ext2.gz.uboot
 #define CONFIG_BOOTFILEuImage
 #define CONFIG_UBOOTPATH   u-boot.bin  /* U-Boot image on TFTP server 
*/
-#define CONFIG_FDTFILE mpc837x_rdb.dtb
+#define CONFIG_FDTFILE mpc8379_rdb.dtb
 
 #define CONFIG_LOADADDR20  /* default location for tftp 
and bootm */
 #define CONFIG_BOOTDELAY   -1  /* -1 disables auto-boot */
-- 
1.5.4.3.450.gb92176


-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] Add flash programming counter]

2008-03-07 Thread Kim Phillips
On Fri, 07 Mar 2008 17:33:34 +0100
Wolfgang Denk [EMAIL PROTECTED] wrote:

 In message [EMAIL PROTECTED] you wrote:
 
  There is no way to add an end marker without using a new line (or use 
  '\r' which is really unacceptable, and more justifiably so).  Since we 
  are settling in on having 50 (?probably want to use a #define) dots 
  unless there are fewer than 50 units to be written, I suppose we can 
  live without a end marker.  Sigh, the end marker was the fun part.

I thought it safe to assume a terminal is 80 characters wide..(80 -
strlen(Writing to Flash) - strlen(done)) == 60?

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] please pull mpc83xx

2008-03-07 Thread Kim Phillips
Wolfgang,

Please pull mpc83xx; it has a patch you missed that was posted here:

http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/37339

and the dtb rename patch I just posted:

The following changes since commit bd4458cb47abecabd406b1210457be96c69fc49d:
  Dave Liu (1):
837xEMDS: Improve the system performance

are available in the git repository at:

  git://www.denx.de/git/u-boot-mpc83xx.git

Kim Phillips (2):
  net: uec_phy: actually increment the timeout counter
  mpc83xx: make dtb basename file references equal those of linux

 drivers/qe/uec_phy.c  |3 ++-
 include/configs/MPC832XEMDS.h |2 +-
 include/configs/MPC8360EMDS.h |2 +-
 include/configs/MPC837XEMDS.h |2 +-
 include/configs/MPC837XERDB.h |2 +-
 5 files changed, 6 insertions(+), 5 deletions(-)

Thanks,

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] mpc83xx: make dtb basename file references equal those of linux

2008-03-07 Thread Kim Phillips
On Sat, 08 Mar 2008 02:03:02 +0100
Wolfgang Denk [EMAIL PROTECTED] wrote:

 In message [EMAIL PROTECTED] you wrote:
 
  but these are the filenames in linux' device tree directory -
  arch/powerpc/boot/dts, and I'm assuming people do something sane like
  dtc -I $file.dts -O $file.dtb.
 
 Argh... what a mess.
 
indeed.

  are you suggesting we rename u-boot's config files too?
 
 Well, my personal preference is to have the official board  name  and
 the  related  file names match as closely as possible, probabl;y just
 using lowercase letters for the files.
 
 I  don't  think  it's  exactly  intuitive  to  have  a  board   named
 mpc832xemds and the related files use mpc832x_mds.
 
 That looks awfully broken to me.

afaik, you can supposedly buy an mds board with an mpc8321,
mpc8321e, mpc8323, or an mpc8323e (the trailing 'e' denotes whether the
crypto engine has its fuse intact).  It's just the mpc8323e that is
(probably by far) the most common.

 At the momnt, the U-Boot code looks OK, so if you do it differently in
 Linux, you should fix it there.
 
 Just my $ 0.02 (well, I think I  should  say  $  0.03  given  current
 exchange rates ;-).

bah, what can you buy with a cent these days? ;)

linux board support files are consistent /within/ linux, and u-boot
board support files are intra-u-boot consistent; it's just this
particular file (the device tree) needs to be inter-linux and u-boot
consistent (for interoperability purposes).  Since linux does currently
host the device tree files, we should probably follow their naming
scheme, unless you want to discuss hosting device tree files in
u-boot ;).

fyi, the board file was changed in the linux git tree commit
be156bed9ebfe365c6d95f715eae3529cf694fcb

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] 837xEMDS: Improve the system performance

2008-03-04 Thread Kim Phillips
On Tue, 04 Mar 2008 17:17:27 +0100
Wolfgang Denk [EMAIL PROTECTED] wrote:

 In message [EMAIL PROTECTED] you wrote:
  1. Make the CSB bus pipeline depth as 4, and enable
 the repeat mode;
  2. Raise the eTSEC emergency priority;
  3. Use the highest IP blocks clock.
  
  Signed-off-by: Dave Liu [EMAIL PROTECTED]
 
 Is this supposed to go into the 1.3.2 release?
 
 Directly or through Kim's 83xx custodian repo?
 
 Kim?

Hi Wolfgang,

I don't know what version Dave intended here either; if he wants it in
1.3.2 I'll apply and ask you to pull.

Dave?

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH] Fix mpc8360emds board hang on second network operation

2008-02-27 Thread Kim Phillips
On Tue, 26 Feb 2008 21:32:30 -0500
Jerry Van Baren [EMAIL PROTECTED] wrote:

 The changeset that causes my problems is ee62ed32
 
 The original code calls init_phy(dev) followed by phy_change(dev) *once*
 during PHY initialization.  The part of changeset that appears to cause
 my problems is calling phy_change(dev) every time uec_init() is called.

phy_change is there so that u-boot will renegotiate the link in case
its state has changed since the last networking command (this was the
original complaint that warranted the ee62ed32 patch, iirc).

 On my mpc8360emds board, without this change the second command that does
 a network operation hangs the board (requires a hard reset to recover).
 Example failure sequence is two ping commands: the first works, the
 second hangs the board.
 
 This also speeds up the network initialization and doesn't cause a packet
 error on the first TFTP packet.
 
 Signed-off-by: Gerald Van Baren [EMAIL PROTECTED]
 ---
 
 Kim:
 
 Does this make any sense?  Any thoughts why it isn't causing problems
 for you?  It makes all the difference in the world for me.
 
 One other interesting sequence difference with patch ee62ed32 is that
 the ethernet MAC address is set on every uec_init() call.  The following
 code was inside the if (uec-the_first_run == 0) conditional, now it
 is outside the conditional and thus run on every uec_init() call.
 Functionally, it doesn't seem to matter for me either way, but seems
 unnecessary.
 
 +   /* Set up the MAC address */
 +   if (dev-enetaddr[0]  0x01) {
 +   printf(%s: MacAddress is multcast address\n,
 +__FUNCTION__);
 +   return -1;
 +   }
 +   uec_set_mac_address(uec, dev-enetaddr);
 

this was done in order to be able to re-set eth[1]addr between
networking commands, and have the new eth[1]addr value actually
programmed on the interface.

I've tested 100BT, and experience the same as you.  The reason u-boot
doesn't recover is that i is not incremented in genmii_update_link(),
but adding that doesn't resolve the problem.  I believe the PHY status
is being incorrectly determined somehow - these PHYs are really
finicky :/.

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] net: uec_phy: handle 88e1111 rev.B2 erratum 5.6

2008-02-27 Thread Kim Phillips
erratum 5.6 states the autoneg completion bit is functional only if the
autoneg bit is asserted.

This fixes any secondarily-issued networking commands on non-gigabit
links on the mpc8360 mds board.

Signed-off-by: Kim Phillips [EMAIL PROTECTED]
---
gvb, please test, and give us your blessings if this fixes your second
ping failing issue.

To go through biggerbadderben, for 1.3.2.

 drivers/qe/uec_phy.c |8 
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/drivers/qe/uec_phy.c b/drivers/qe/uec_phy.c
index c549b6b..cf9921b 100644
--- a/drivers/qe/uec_phy.c
+++ b/drivers/qe/uec_phy.c
@@ -574,6 +574,7 @@ void marvell_phy_interface_mode (struct eth_device *dev,
 {
uec_private_t *uec = (uec_private_t *) dev-priv;
struct uec_mii_info *mii_info;
+   u16 status;
 
if (!uec-mii_info) {
printf (%s: the PHY not intialized\n, __FUNCTION__);
@@ -609,6 +610,13 @@ void marvell_phy_interface_mode (struct eth_device *dev,
phy_write (mii_info, 0x00, 0x8100);
udelay (100);
}
+
+   /* handle 88e rev.B2 erratum 5.6 */
+   if (mii_info-autoneg) {
+   status = phy_read (mii_info, PHY_BMCR);
+   phy_write (mii_info, PHY_BMCR, status | PHY_BMCR_AUTON);
+   }
+   /* now the B2 will correctly report autoneg completion status */
 }
 
 void change_phy_interface_mode (struct eth_device *dev, enet_interface_e mode)
-- 
1.5.4.23.gef5b9


-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Problems with new ethernet init on mpc8360

2008-02-26 Thread Kim Phillips
On Mon, 25 Feb 2008 22:55:37 -0500
Jerry Van Baren [EMAIL PROTECTED] wrote:

 Hi Kim,
 
 I have a mpc8360emds eval board.  The ee62ed patch (below) speeds up the 
 start up (good), but seems to cause the board to hang on the *second* 
 ethernet operation.
 
 I can cause this by doing two pings in a row: the first ping initializes 
 the network and completes successfully.  The second ping hangs the 
 board: it never completes.  I cannot abort it with ^C.  The only way out 
 is to reset the board.
 
 The same thing happens with two back-to-back tftp operations or a ping 
 followed by a tftp operation.
 
 If I back out the ee62ed patch (below), it fixes the problem.
 
 I tried backing out some simple portions of the patch (added delays back 
 in) with no success.  I've given up chasing this for tonight.  Any 
 inspiration you can generate would be appreciated, I'll chase it some 
 more tomorrow.

I verified rc2 on rev. 1.1 and 2.1 boards pings more than once.

can you provide more details about which u-boot version, what board rev
you're on, what kind of connection you're using, speed, etc.?

Kim

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


  1   2   >