[U-Boot-Users] [PATCH] CFI support for Versatile & Integrator/CP boards

Stefan Roese
2006-02-28T09:24:13Z

Hi Peter,

On Monday, 27. February 2006 16:59, Peter Pearse wrote:
> The attached gzipped patch was made against the U-Boot GIT commit
> 6624b687bc2b747233090e67628df37d1c84ed17.

Thanks.

> It is a completely independent patch, containing the patches to
> drivers/cfi_flash.c
> which were in my patch 012.
>
> The patch
> - Makes flash_get_size() static.

Rejected. This function is called from other board files. I will remove this
part of the patch.

> - Changes flash_full_status_check() to test for timeout OR failure,
> rather than timeout AND failure.

I'm not sure here. Please take a look at the patch from Marcus Hall:

http://article.gmane.org/gmane.comp.boot-loaders.u-boot/18530

Could you and others (Marcus, Tolunay) please comment on what patch should be
applied here. I tend to take the patch from Marcus right now.

> - Makes find_sector() available for BOTH versions of flash_write_cfiword()

Applied.

> CHANGELOG:
> Update drivers/cfi_flash.c.
> - flash_get_size() now static.
> - flash_full_status_check() to test for timeout OR failure,
> rather than timeout AND failure.
> - find_sector() called in both versions of flash_write_cfiword()
> Patch by Peter Pearse, 27th Feb 2006
>
> I intend to re-submit the changes for the boards (Integrator/CP,
> VersatileAB & PB) separately,
> once you have completed your CFI work.

OK. Please give us a few more days and the new CFI driver should be available.

Best regards,
Stefan