[U-Boot-Users] [patch 1/1] DNP1110

[U-Boot-Users] [patch 1/1] DNP1110

[Rolf Offermanns]

Hi,
I ported the dnp1110 related changes from the current armboot cvs to 
current u-boot cvs.

init_critical works, flash works, but unfortunately smc91111 does not 
work. The code is identical to the armboot code and I was not able to 
find the problem. Maybe one of you can help out and give me a hint.
I marked the problem in smc91111.c, grep for "FIXME".

Thanks,
-Rolf
-- 
Rolf Offermanns <roffermanns@sysgo.de>
Sysgo AG
Am Pfaffenstein 14                    phone: +49 6136 9948-0
D-55270 Klein-Winternheim / Germany   fax  : +49 6136 9948-10
http://www.sysgo.de                   http://www.elinos.com

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: dnp1110.diff
Url: http://lists.denx.de/pipermail/u-boot/attachments/20030321/db5eb96f/attachment.txt 

[U-Boot-Users] [patch 1/1] DNP1110

[Robert Schwebel]

Hi Rolf, 

On Fri, Mar 21, 2003 at 11:14:34AM +0100, Rolf Offermanns wrote:
> I ported the dnp1110 related changes from the current armboot cvs to
> current u-boot cvs.

I suppose we should fix the incorrect memory map for ARM, which was
different in ARMboot than in u-boot. I have a patch for pxa, but it has
to be reworked as Wolfgang had some objections. 

> init_critical works, flash works, but unfortunately smc91111 does not
> work. The code is identical to the armboot code and I was not able to
> find the problem. Maybe one of you can help out and give me a hint.  I
> marked the problem in smc91111.c, grep for "FIXME".

I've also seen that problem on my PXA board - the driver starts working
when you switch on the debug code and add a random number of debug()
calls right before the SMC_SELECT_BANK( 2 ); line.

Unfortunately, I have no idea what the reason of this problem may be,
looks like some timing problem. 

Robert
-- 
 Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
   Braunschweiger Str. 79,  31134 Hildesheim, Germany
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
    Phone: +49-5121-28619-0 |  Fax: +49-5121-28619-4

[U-Boot-Users] [patch 1/1] DNP1110

[Marius Groeger]

Hi Robert,

On Fri, 21 Mar 2003, Robert Schwebel wrote:

> I suppose we should fix the incorrect memory map for ARM, which was
> different in ARMboot than in u-boot. I have a patch for pxa, but it has
> to be reworked as Wolfgang had some objections.

Which one of those patches was that exactly? I'm not quite sure what
your mean by "incorrect memory map for ARM", since this is usually a
board specific issue, so I'd like to check that patch of yours first.

Thanks,
Marius

-----------------------------------------------------------------------------
Marius Groeger           SYSGO Real-Time Solutions AG       mgroeger at sysgo.de
Software Engineering     Embedded and Real-Time Software    www.sysgo.de
Voice: +49-6136-9948-0   Am Pfaffenstein 14                 www.osek.de
FAX:   +49-6136-9948-10  55270 Klein-Winternheim, Germany   www.elinos.com

[U-Boot-Users] [patch 1/1] DNP1110

[Robert Schwebel]

On Fri, Mar 21, 2003 at 12:29:51PM +0100, Marius Groeger wrote:
> Which one of those patches was that exactly? I'm not quite sure what
> your mean by "incorrect memory map for ARM", since this is usually a
> board specific issue, so I'd like to check that patch of yours first.

ARMboot had another memory layout, compare it to that one described in
the u-boot README. Especially the stack was _above_ the relocated
bootloader, not below. 

Robert
-- 
 Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
   Braunschweiger Str. 79,  31134 Hildesheim, Germany
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
    Phone: +49-5121-28619-0 |  Fax: +49-5121-28619-4

[U-Boot-Users] [patch 1/1] DNP1110

[Wolfgang Denk]

Hi Marius,

in message <Pine.LNX.4.52.0303211228070.1040@mag.devdep.sysgo.de> you wrote:
> 
> On Fri, 21 Mar 2003, Robert Schwebel wrote:
> 
> > I suppose we should fix the incorrect memory map for ARM, which was
> > different in ARMboot than in u-boot. I have a patch for pxa, but it has
> > to be reworked as Wolfgang had some objections.
> 
> Which one of those patches was that exactly? I'm not quite sure what
> your mean by "incorrect memory map for ARM", since this is usually a
> board specific issue, so I'd like to check that patch of yours first.

I think Robert refers to the fact that the memory map  as  documented
in the "Memory Management" section of the README is not valid for ARM
(malloc area etc are not below but above the U-Boot code, etc.).

There are other issues as well, as the fact that the code gets copied
to a fixed address instead of being relocated to the end of RAM.


I think we will fix this in our next ARM based project.

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
Ninety-Ninety Rule of Project Schedules:
        The first ninety percent of the task takes ninety percent of
the time, and the last ten percent takes the other ninety percent.