DNP1110

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 <[email protected]>
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 

Hi Rolf,

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.

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

Hi Robert,

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

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

Hi Marius,

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.