public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] [PATCH] fix incorrect bus clk of 83xx
@ 2006-07-27  9:27 Liu Dave-r63238
  2006-07-27 21:28 ` [U-Boot-Users] Invalid MPC8349EMDS OR0 setting? David Hawkins
  0 siblings, 1 reply; 7+ messages in thread
From: Liu Dave-r63238 @ 2006-07-27  9:27 UTC (permalink / raw)
  To: u-boot

All,

The gd->bus_clk of 83xx should be csb_clk, not libu_clk,
If it don't be changed, will make the timer incorrect.

CHANGELOG:
* fix incorrect gd->bus_clk, it should be csb_clk
  Patch by Dave Liu, 21 July 2006


diff --git a/cpu/mpc83xx/speed.c b/cpu/mpc83xx/speed.c
index ad6b3f6..bc689c1 100644
--- a/cpu/mpc83xx/speed.c
+++ b/cpu/mpc83xx/speed.c
@@ -333,7 +333,7 @@ #endif /* (CFG_HRCW_HIGH | RCWH_PCIHOST)
        gd->pci_clk    = pci_sync_in;

        gd->cpu_clk = gd->core_clk;
-       gd->bus_clk = gd->lbiu_clk;
+       gd->bus_clk = gd->csb_clk;
        return 0;
 }

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [U-Boot-Users] Invalid MPC8349EMDS OR0 setting?
  2006-07-27  9:27 [U-Boot-Users] [PATCH] fix incorrect bus clk of 83xx Liu Dave-r63238
@ 2006-07-27 21:28 ` David Hawkins
  2006-07-28  5:38   ` Liu Dave-r63238
                     ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: David Hawkins @ 2006-07-27 21:28 UTC (permalink / raw)
  To: u-boot


Hey Dave,

While looking through the U-Boot source for the
MPC8349E-MDS board, I noticed the following:

u-boot/include/configs/MPC8349EMDS.h

CFG_OR0_PRELIM 0xFF806FF7

The '6' in the middle of this word corresponds
to bits 17 and 18, but in the reference manual
these bits are marked as reserved, and at processor
reset are 00b.

Since you and Ben have been working on this
board a while, I was wondering if there was
anything special about these particular bits,
eg. needed in silicon 1.0, but not the 1.1
on the MDS board.

Is this something I should try setting to 0,
test on the MDS board, and then submit a patch
for?

Also, do you know if the 2.0 silicon can
be dropped into the MDS board without change?

Thanks!
Dave

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] Invalid MPC8349EMDS OR0 setting?
  2006-07-27 21:28 ` [U-Boot-Users] Invalid MPC8349EMDS OR0 setting? David Hawkins
@ 2006-07-28  5:38   ` Liu Dave-r63238
  2006-07-28 10:23   ` Liu Dave-r63238
  2006-07-28 14:02   ` Ben Warren
  2 siblings, 0 replies; 7+ messages in thread
From: Liu Dave-r63238 @ 2006-07-28  5:38 UTC (permalink / raw)
  To: u-boot

> 
> Hey Dave,
> 
> While looking through the U-Boot source for the
> MPC8349E-MDS board, I noticed the following:
> 
> u-boot/include/configs/MPC8349EMDS.h
> 
> CFG_OR0_PRELIM 0xFF806FF7
> 
> The '6' in the middle of this word corresponds
> to bits 17 and 18, but in the reference manual
> these bits are marked as reserved, and at processor
> reset are 00b.

Please refer to the latest reference manual about the two bits.
The bit 17 and 18 should be XAM. See the section 10.3.1.2.1
> 
> Since you and Ben have been working on this
> board a while, I was wondering if there was
> anything special about these particular bits,
> eg. needed in silicon 1.0, but not the 1.1
> on the MDS board.
> 
> Is this something I should try setting to 0,
> test on the MDS board, and then submit a patch
> for?
> 
> Also, do you know if the 2.0 silicon can
> be dropped into the MDS board without change?
> 
I don't recommend you put the 2.0 silicon to orginal MDS board.
Please contact with Freescale FAE or sales to get more information
about it.

-Dave

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] Invalid MPC8349EMDS OR0 setting?
  2006-07-27 21:28 ` [U-Boot-Users] Invalid MPC8349EMDS OR0 setting? David Hawkins
  2006-07-28  5:38   ` Liu Dave-r63238
@ 2006-07-28 10:23   ` Liu Dave-r63238
  2006-07-28 18:26     ` David Hawkins
  2006-07-28 14:02   ` Ben Warren
  2 siblings, 1 reply; 7+ messages in thread
From: Liu Dave-r63238 @ 2006-07-28 10:23 UTC (permalink / raw)
  To: u-boot

Please refer to the latest reference manual about the two bits. 
The bit 17 and 18 should be XAM. See the section 10.3.1.2.1

I don't recommend you put the 2.0 silicon to orginal MDS board directly.
Please contact with Freescale FAE or sales to get more information about
it.

> 
> Hey Dave,
> 
> While looking through the U-Boot source for the
> MPC8349E-MDS board, I noticed the following:
> 
> u-boot/include/configs/MPC8349EMDS.h
> 
> CFG_OR0_PRELIM 0xFF806FF7
> 
> The '6' in the middle of this word corresponds
> to bits 17 and 18, but in the reference manual
> these bits are marked as reserved, and at processor
> reset are 00b.
> 
> Since you and Ben have been working on this
> board a while, I was wondering if there was
> anything special about these particular bits,
> eg. needed in silicon 1.0, but not the 1.1
> on the MDS board.
> 
> Is this something I should try setting to 0,
> test on the MDS board, and then submit a patch
> for?
> 
> Also, do you know if the 2.0 silicon can
> be dropped into the MDS board without change?
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] Invalid MPC8349EMDS OR0 setting?
  2006-07-27 21:28 ` [U-Boot-Users] Invalid MPC8349EMDS OR0 setting? David Hawkins
  2006-07-28  5:38   ` Liu Dave-r63238
  2006-07-28 10:23   ` Liu Dave-r63238
@ 2006-07-28 14:02   ` Ben Warren
  2006-07-28 18:41     ` David Hawkins
  2 siblings, 1 reply; 7+ messages in thread
From: Ben Warren @ 2006-07-28 14:02 UTC (permalink / raw)
  To: u-boot

Hi Dave,

I just used this blindly...  The boot flash is the only peripheral on CS0 and I certainly haven't seen any problems here, but it's probably best to follow the manual.

FYI - Don't hold your breath for rev2.0 silicon.  My understanding is that the next release will be early 2007 with rev3.0 silicon (renamed rev A).  Check with your rep... The next rev is supposed to be pin compatible, but with additional pins being used for more chip selects and DDR2 support.  If we have to spin our board I'm going to be royally pissed!

B-)
David Hawkins <dwh@ovro.caltech.edu> wrote: 
Hey Dave,

While looking through the U-Boot source for the
MPC8349E-MDS board, I noticed the following:

u-boot/include/configs/MPC8349EMDS.h

CFG_OR0_PRELIM 0xFF806FF7

The '6' in the middle of this word corresponds
to bits 17 and 18, but in the reference manual
these bits are marked as reserved, and at processor
reset are 00b.

Since you and Ben have been working on this
board a while, I was wondering if there was
anything special about these particular bits,
eg. needed in silicon 1.0, but not the 1.1
on the MDS board.

Is this something I should try setting to 0,
test on the MDS board, and then submit a patch
for?

Also, do you know if the 2.0 silicon can
be dropped into the MDS board without change?

Thanks!
Dave


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.denx.de/pipermail/u-boot/attachments/20060728/53e3a488/attachment.htm 

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] Invalid MPC8349EMDS OR0 setting?
  2006-07-28 10:23   ` Liu Dave-r63238
@ 2006-07-28 18:26     ` David Hawkins
  0 siblings, 0 replies; 7+ messages in thread
From: David Hawkins @ 2006-07-28 18:26 UTC (permalink / raw)
  To: u-boot

Hi Dave,

> Please refer to the latest reference manual about the two bits. 
> The bit 17 and 18 should be XAM. See the section 10.3.1.2.1

Yikes, thats subtle. The OR tables for GPCM lists those bits
as reserved, and reset and U-Boot is using the GPCM to access
flash ... so I would have interpreted 'Figure 10-3 Option
Registers (ORn) in GPCM' to mean those bits were reserved.

Thanks for the heads-up.

> I don't recommend you put the 2.0 silicon to orginal MDS board directly.
> Please contact with Freescale FAE or sales to get more information about
> it.

Ok, I will thanks.

Dave

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] Invalid MPC8349EMDS OR0 setting?
  2006-07-28 14:02   ` Ben Warren
@ 2006-07-28 18:41     ` David Hawkins
  0 siblings, 0 replies; 7+ messages in thread
From: David Hawkins @ 2006-07-28 18:41 UTC (permalink / raw)
  To: u-boot

Hi Ben,

> I just used this blindly...  The boot flash is the only peripheral on 
> CS0 and I certainly haven't seen any problems here, but it's probably 
> best to follow the manual.

Yeah, but if you look at Dave's response, the section of the manual
Dave referred to does indeed reference those bits as XAM, but then
the GPCM OR table (and the other OR tables) refer to those bits as
reserved! Not exactly crystal clear.

> FYI - Don't hold your breath for rev2.0 silicon.  My understanding is 
> that the next release will be early 2007 with rev3.0 silicon (renamed 
> rev A).  Check with your rep... The next rev is supposed to be pin 
> compatible, but with additional pins being used for more chip selects 
> and DDR2 support.  If we have to spin our board I'm going to be royally 
> pissed!

Ohhh ... I've sent a request to my board subcontractor to start
talking to FAEs.

I'll be building prototypes in Nov/Dec. However, I had wanted to
use the Rev 2.0 silicon since it enables DDR2 and fixes the
external DMA errata (DMA_REQ/ACK/DONE not implemented).
Strangely though the errata rev 9 on the web site has
removed this errata listing (it was present in the rev 8
errata), and have even removed mention of the silicon 2.0.
Of course they might have just renamed the silicon 2.0
to MPC8349EA, but then there is no errata for that
silicon. Even more confusing is that the part numbering
scheme PDF lists parts ending in A as being silicon 3.0!

http://www.freescale.com/files/netcomm/doc/prod_num_scheme/MPC834XEFAMPNS.pdf

What silicon were you able to get for your boards?

Cheers
Dave

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2006-07-28 18:41 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-07-27  9:27 [U-Boot-Users] [PATCH] fix incorrect bus clk of 83xx Liu Dave-r63238
2006-07-27 21:28 ` [U-Boot-Users] Invalid MPC8349EMDS OR0 setting? David Hawkins
2006-07-28  5:38   ` Liu Dave-r63238
2006-07-28 10:23   ` Liu Dave-r63238
2006-07-28 18:26     ` David Hawkins
2006-07-28 14:02   ` Ben Warren
2006-07-28 18:41     ` David Hawkins

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox