From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Tue, 28 Nov 2006 12:41:27 +0100 (MET)
Subject: [U-Boot-Users] Can U-boot support CFI and None CFI devices on
	the same board
In-Reply-To: <200611272344.08016.rui.sousa@laposte.net>
References: <3DBBCC5C2604704D9351456CFE118AC1057F29F6@msilexch01.marvell.com>
	<200611272344.08016.rui.sousa@laposte.net>
Message-ID: <200611281241.34583.sr@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Rui,

On Monday 27 November 2006 23:44, Rui Sousa wrote:
> What I did in our local U-boot version was(*):
>
> 1. Add a cfi_ prefix to flash functions in drivers/cfi_flash.c:
> flash_init --> cfi_flash_init
> flash_erase --> cfi_flash_erase
> write_buff--> cfi_write_buff
> ...
>
> 2. Change flash base address table, in drivers/cfi_flash.c also:
> flash_info_t flash_info --> extern flash_info_t flash_info
>
> 3. Create in our board specific directory a flash.c file with all the flash
> interface functions and the declaration of flash_info:
> flash_info_t flash_info[CFG_MAX_FLASH_BANKS];
> flash_init()
> flash_erase()
> write_buff()

Hmmm. When I understood Tolunay correctly (Tolunay, please correct me if I am 
wrong here), he _does_ want to use most of the CFI drivers functions like 
erasing and programming for those non CFI compatible FLASH's too. Most of 
those devices are kind of AMD compatible, so this should be possible with not 
too many changes.

If we don't use this code, then merging those devices into this driver doesn't 
make much sense to me anymore.

Best regards,
Stefan