[U-Boot-Users] [PATCH] CFI driver AMD Command Set Top boot geometry reversal, etc. [Updated]

[Stefan Roese <sr@denx.de>]

Hi Tolunay,

On Friday 10 November 2006 00:46, Tolunay Orkun wrote:
> This patch (replaces patch submitted via DNX#2006110942000016):
>
>    * Adds support for AMD command set Top Boot flash geometry reversal
>    * Adds support for reading JEDEC Manufacturer ID and Device ID
>    * Adds support for displaying command set, manufacturer id and
>      device ids (flinfo)
>    * Makes flinfo output to be consistent when CFG_FLASH_EMPTY_INFO defined
>    * Removes outdated change history (refer to git log instead)

I tied you patch one a board with a ST M29W160ET FLASH, that couldn't be 
handled by the "old" CFI driver, because of the geometry reversal. I had 
hoped your patch would have solved this issue. Unfortunately not:

=> fli

Bank # 1: CFI conformant FLASH (16 x 16)  Size: 2 MB in 35 Sectors
  AMD Standard command set, Manufacturer ID: 0x05, Device ID: 0x56
  Erase timeout: 8192 ms, write timeout: 1 ms

  Sector Start Addresses:
  FFE00000        FFE04000        FFE06000        FFE08000        FFE10000
  FFE20000 E      FFE30000 E      FFE40000 E      FFE50000 E      FFE60000 E
  FFE70000 E      FFE80000 E      FFE90000 E      FFEA0000 E      FFEB0000 E
  FFEC0000 E      FFED0000 E      FFEE0000 E      FFEF0000 E      FFF00000 E
  FFF10000 E      FFF20000 E      FFF30000 E      FFF40000 E      FFF50000 E
  FFF60000 E      FFF70000 E      FFF80000 E      FFF90000 E      FFFA0000   
RO
  FFFB0000   RO   FFFC0000   RO   FFFD0000   RO   FFFE0000   RO   FFFF0000   
RO


As you may notice, even the ID's are not correct (0020 and 22c4 are correct) 
and the geometry is not correct (bottom instead of top).

I didn't look into this deeply and since you have the insight right now (;-)), 
perhaps you have an idea what is going wrong here.

Thanks a lot.

Best regards,
Stefan