[U-Boot-Users] CVS tree broken for MPC8245-based boards

[U-Boot-Users] CVS tree broken for MPC8245-based boards

[Matthew Karger]

Hello,

It appears that only the first part of Marc Leeman's patch submitted on 11
May, 2004 (subject: [patch] master read fix mpc8245) was applied, resulting
in the current CVS tree being broken for MPC8245-based boards.  The portion
of the patch that was applied requires a new register address to be defined
in include/mpc8245.h, which was done in the second part of the patch.  Gleb
Natapov also raised this issue on 12 July, 2004, but received no reply from
the list.  I don't mean to sound redundant, but is the second part of the
patch going to be applied soon? 

Thanks,
Matthew

[U-Boot-Users] CVS tree broken for MPC8245-based boards

[Marc Leeman]

> Natapov also raised this issue on 12 July, 2004, but received no reply from

Sorry, missed the mail.

> the list.  I don't mean to sound redundant, but is the second part of the
> patch going to be applied soon? 


--- u-boot-1.1.1-orig/include/mpc824x.h 2004-07-16 11:46:01.000000000
+0200
+++ u-boot-1.1.1/include/mpc824x.h      2004-07-16 15:13:39.000000000
+0200
@@ -297,6 +297,7 @@
 #define PBESR          0x800000c7  /* PCI Bus Error Status Register */
 #define PBEAR          0x800000c8  /* Processor/PCI Bus Error Status Register */
 #define AMBOR          0x800000e0  /* Address Map B Options Register */
+#define PCMBCR          0x800000e1  /* PCI/Memeory Buffer Configuration */
 #define MCCR1          0x800000f0  /* Memory Control Configuration Register 1 */
 #define MCCR2          0x800000f4  /* Memory Control Configuration Register 2 */
 #define MCCR3          0x800000f8  /* Memory Control Configuration Register 3 */


-- 
  Marc Leeman
  Hardware R&D Engineer
  
  Barco Control Rooms
  Noordlaan 5, Industriezone, B-8520 Kuurne (BE)
  Tel. +32 56 368 428
  http://www.barcocontrolrooms.com
  mailto:marc.leeman at barco.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.denx.de/pipermail/u-boot/attachments/20040722/7def0a76/attachment.pgp 

[U-Boot-Users] CVS tree broken for MPC8245-based boards

[Gleb Natapov]

Hello,

On Thu, Jul 22, 2004 at 09:25:14AM +0200, Marc Leeman wrote:
> > Natapov also raised this issue on 12 July, 2004, but received no reply from
> 
> Sorry, missed the mail.

There was one more question in that mail, but if you missed it let me
repeat:
Where register 0x800000e1 is documented? :)

TIA.

--
			Gleb.

[U-Boot-Users] CVS tree broken for MPC8245-based boards

[Marc Leeman]

> Where register 0x800000e1 is documented? :)

Page 4-51, section 4.11 of the MPC8245 Integrated Processor User's
Manual.

A slightly more verbose bugreport can be downloaded from:
http://scorpius.homelinux.org/~marc/downloads/masterwrite.pdf

It seems to be a known problem inside Motorola.

-- 
  Marc Leeman
  Hardware R&D Engineer
  
  Barco Control Rooms
  Noordlaan 5, Industriezone, B-8520 Kuurne (BE)
  Tel. +32 56 368 428
  http://www.barcocontrolrooms.com
  mailto:marc.leeman at barco.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.denx.de/pipermail/u-boot/attachments/20040722/dcce5e2c/attachment.pgp 

[U-Boot-Users] CVS tree broken for MPC8245-based boards

[Gleb Natapov]

On Thu, Jul 22, 2004 at 10:48:42AM +0200, Marc Leeman wrote:
> > Where register 0x800000e1 is documented? :)
> 
> Page 4-51, section 4.11 of the MPC8245 Integrated Processor User's
> Manual.
> 
Thank you! It is not mentioned in revision 1 of this manual, but it is
present in rev 2.

--
			Gleb.

[U-Boot-Users] CVS tree broken for MPC8245-based boards

[Wolfgang Denk]

In message <200407212305.i6LN5DRs042238@ns5.arlut.utexas.edu> you wrote:
> 
> It appears that only the first part of Marc Leeman's patch submitted on 11
> May, 2004 (subject: [patch] master read fix mpc8245) was applied, resulting
> in the current CVS tree being broken for MPC8245-based boards.  The portion
> of the patch that was applied requires a new register address to be defined
> in include/mpc8245.h, which was done in the second part of the patch.  Gleb
> Natapov also raised this issue on 12 July, 2004, but received no reply from
> the list.  I don't mean to sound redundant, but is the second part of the
> patch going to be applied soon? 

Just for the record: done.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
I wrote my name at the top of the page. I wrote down  the  number  of
the  question  ``1''.  After much reflection I put a bracket round it
thus ``(1)''. But thereafter I could not think of anything  connected
with it that was either relevant or true.
                        - Sir Winston Churchill _My Early Life_ ch. 2