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

Stefan Roese
2006-11-10T11:16:12Z

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