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

Stefan Roese sr at denx.de
Tue Feb 28 10:24:13 CET 2006


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




More information about the U-Boot mailing list