[U-Boot] [PATCH v4] mtd: cfi_flash: Fix CFI flash driver for 8-bit bus support

[Jagannadha Sutradharudu Teki]

Hi Stefan,

> -----Original Message-----
> From: Stefan Roese [mailto:sr at denx.de]
> Sent: 01 March 2013 17:43
> To: Jagannadha Sutradharudu Teki
> Cc: u-boot at lists.denx.de; monstr at monstr.eu; Jagannadha Sutradharudu
> Teki; Aaron Williams
> Subject: Re: [PATCH v4] mtd: cfi_flash: Fix CFI flash driver for 8-bit bus
> support
>
> On 01.03.2013 13:05, Jagannadha Sutradharudu Teki wrote:
> > This commit is based on that patch from
> > aaron.williams at caviumnetworks.com with same commit title.
> >
> > http://patchwork.ozlabs.org/patch/140863/
> > http://lists.denx.de/pipermail/u-boot/2011-April/089606.html
> >
> > This patch corrects the addresses used when working with Spansion/AMD
> FLASH chips.
> > Addressing for 8 and 16 bits is almost identical except in the 16-bit
> > case the LSB of the address is always 0.  The confusion arose because
> > the addresses in the datasheet for 16-bit mode are word addresses but
> > this code assumed it was byte addresses.
> >
> > I have only been able to test this on our Octeon boards which use
> > either an 8-bit or 16-bit bus.  I have not tested the case where
> > there's an 8-bit part on a 16-bit bus.
> >
> > This patch also adds some delays as suggested by Spansion.
> >
> > If a part can be both 8 and 16-bits, it forces it to work in 8-bit
> > mode if an 8-bit bus is detected.
> >
> > Signed-off-by: Aaron Williams <aaron.williams at caviumnetworks.com>
> > Tested-by: Luka Perkov <uboot at lukaperkov.net>
> >
> > Apart from the pulled changes from above commiters, fixed few minor
> > code cleanups and tested on 256M29EW, 512M29EW flashes.
> >
> > Before this fix:
> > ---------------
> > Bank # 1: CFI conformant flash (8 x 8)  Size: 64 MB in 512 Sectors
> >   AMD Standard command set, Manufacturer ID: 0xFF, Device ID: 0xFF
> >   Erase timeout: 4096 ms, write timeout: 2 ms
> >   Buffer write timeout: 5 ms, buffer size: 1024 bytes
> >
> > After this fix:
> > --------------
> > Bank # 1: CFI conformant flash (8 x 8)  Size: 64 MB in 512 Sectors
> >   AMD Standard command set, Manufacturer ID: 0x89, Device ID: 0x7E2301
> >   Erase timeout: 4096 ms, write timeout: 2 ms
> >   Buffer write timeout: 5 ms, buffer size: 1024 bytes
> >
> > Signed-off-by: Jagannadha Sutradharudu Teki <jaganna at xilinx.com>
> > Tested-by: Jagannadha Sutradharudu Teki <jaganna at xilinx.com>
>
> Please collect all "s-o-b" lines together:
>
> Signed-off-by: Aaron Williams <aaron.williams at caviumnetworks.com>
> Tested-by: Luka Perkov <uboot at lukaperkov.net>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna at xilinx.com>
> Tested-by: Jagannadha Sutradharudu Teki <jaganna at xilinx.com>
>
> Please also add a "From: Aaran ..." line as first line in your commit text to
> preserve the patch authorship.

Seems like I have added the first line with Aran..
as
+ This commit is based on that patch from aaron.williams at caviumnetworks.com with same commit title

Was the above text is not sufficient, if not please suggest.

Thanks,
Jagan.

>
> Looking forward to v5... ;)
>
> Thanks,
> Stefan



This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.