<br><div class="gmail_quote">On Sun, Jul 6, 2008 at 4:57 PM, Wolfgang Denk <<a href="mailto:wd@denx.de">wd@denx.de</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">In message <<a href="mailto:48644421.9000108@gmail.com">48644421.9000108@gmail.com</a>> you wrote:<br>
><br>
> I noticed that the device id for AMD flash part AM29DL800BB is wrong.<br>
><br>
> Attached patch corrects it.<br>
</div>...<br>
> -#define AM29DL800BB 0x22C8<br>
> +#define AM29DL800BB 0x22CB<br>
<br>
Hmmmm... I have but an old data sheet (Nov 2000) for this chip, which<br>
indeed says 22CB. But then, even the current Linux kernel tree has<br>
22C8.<br>
<br>
Can anybody with access to real hardware / current documentation<br>
comment?<br>
<br>
Best regards,<br>
<br>
Wolfgang Denk</blockquote><div><br>(I had a connectivity "burp" earlier when sending a reply.<br> Please excuse if double-posted)<br><br>We were using a custom driver written by Andy Lowe of Montavista.<br>He had a #define local to our flash.c implementation when I noticed<br>
the discrepancy between his local definition and the one in the u-boot<br>header.<br><br>On our hardware it works with 0x22CB but not with 0x22C8.<br><br>The current documentation from Spansion also shows it as 0x22CB.<br>
<br><a href="http://www.spansion.com/datasheets/21519c4.pdf" target="_blank">http://www.spansion.com/datasheets/21519c4.pdf</a><br><br>Cheers,<br><br>Jerry.<br><br></div></div>