[U-Boot] [PATCH] driver/mtd/spi: Default page size Spansion flash "S25FS512S" is 256b

[Ashish Kumar]

> -----Original Message-----
> From: Jagan Teki [mailto:jagannadh.teki at gmail.com]
> Sent: Wednesday, May 9, 2018 4:44 PM
> To: Ashish Kumar <ashish.kumar at nxp.com>
> Cc: U-Boot Mailing List <u-boot at lists.denx.de>
> Subject: Re: [U-Boot] [PATCH] driver/mtd/spi: Default page size Spansion flash
> "S25FS512S" is 256b
> 
> On Mon, May 7, 2018 at 4:01 PM, Ashish Kumar <Ashish.Kumar at nxp.com>
> wrote:
> > Signed-off-by: Ashish Kumar <Ashish.Kumar at nxp.com>
> > ---
> >  drivers/mtd/spi/spi_flash.c | 11 ++++++-----
> >  1 file changed, 6 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c
> > index 3d24a08..ce420cb 100644
> > --- a/drivers/mtd/spi/spi_flash.c
> > +++ b/drivers/mtd/spi/spi_flash.c
> > @@ -1046,14 +1046,15 @@ int spi_flash_scan(struct spi_flash *flash)
> >         flash->shift = (flash->dual_flash & SF_DUAL_PARALLEL_FLASH) ? 1 : 0;
> >         flash->page_size = info->page_size;
> >         /*
> > -        * The Spansion S25FL032P and S25FL064P have 256b pages, yet use the
> > -        * 0x4d00 Extended JEDEC code. The rest of the Spansion flashes with
> > -        * the 0x4d00 Extended JEDEC code have 512b pages. All of the others
> > -        * have 256b pages.
> > +        * The Spansion S25FS512S, S25FL032P and S25FL064P have 256b pages,
> > +        * yet use the 0x4d00 Extended JEDEC code. The rest of the Spansion
> > +        * flashes with the 0x4d00 Extended JEDEC code have 512b pages.
> > +        * All of the others have 256b pages.
> >          */
> >         if (JEDEC_EXT(info) == 0x4d00) {
> >                 if ((JEDEC_ID(info) != 0x0215) &&
> > -                   (JEDEC_ID(info) != 0x0216))
> > +                   (JEDEC_ID(info) != 0x0216) &&
> > +                   (JEDEC_ID(info) != 0x0220))
> 
> 0220 is JEDEC ID for S25FS512S which is 512 is it? commit message says 256b?
Yes, 0220 is for S25FS512S. The loops says all JEDEC EXT starting 0x4d00 is 512b, except 215,216 and 220.

Regards
Ashish