From: Rogan Dawes <rogan@dawes.za.net>
Date: Tue, 20 Jul 2010 10:38:11 +0200
Subject: [U-Boot] Non standard CFI detection tweaks
In-Reply-To: <201007200958.33622.sr@denx.de>
References: <4C3CC5E5.1040606@dawes.za.net> <201007191533.37947.sr@denx.de>
	<4C446314.7040702@dawes.za.net> <201007200958.33622.sr@denx.de>
Message-ID: <4C456073.3000706@dawes.za.net>
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

On 2010/07/20 9:58 AM, Stefan Roese wrote:
> Hi Rogan,
>
> OK, so the width is definitely 8 bit and not 16 bit:
>
> static struct physmap_flash_data dns323_nor_flash_data = {
> 	.width		= 1,
> ...
>
> But the base address is listed here as 0xf4000000:
>
> #define DNS323_NOR_BOOT_BASE 0xf4000000

Hmmm, maybe I must try changing that, then. That might be the answer to 
all the confusion.

Although it is definitely an 8MB flash, and I thought it was supposed to 
be at the end of memory?

> Are you sure you got this right? Nevertheless, now we know for sure, that the
> NOR chip is connected as an 8bit device and the chip is operated in byte mode.
> So all tests from the U-Boot prompt should be done with ".b" extension.
>
> BTW: This code you referenced is also available in mainline kernel.org.

Yes, I eventually got my "git clone" finished.

Thanks again for your help!

Rogan