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

[Jagan Teki]

On Thu, May 10, 2018 at 10:33 AM, Ashish Kumar <ashish.ku...@nxp.com> wrote:
>
>
>> -Original Message-
>> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
>> Sent: Wednesday, May 9, 2018 4:44 PM
>> To: Ashish Kumar <ashish.ku...@nxp.com>
>> Cc: U-Boot Mailing List <u-boot@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.ku...@nxp.com>
>> wrote:
>> > Signed-off-by: Ashish Kumar <ashish.ku...@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.

Added proper commit message and

Applied to u-boot-spi/master
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

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

[Ashish Kumar]

> -Original Message-
> From: Jagan Teki [mailto:jagannadh.t...@gmail.com]
> Sent: Wednesday, May 9, 2018 4:44 PM
> To: Ashish Kumar <ashish.ku...@nxp.com>
> Cc: U-Boot Mailing List <u-boot@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.ku...@nxp.com>
> wrote:
> > Signed-off-by: Ashish Kumar <ashish.ku...@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 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

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

[Jagan Teki]

On Mon, May 7, 2018 at 4:01 PM, Ashish Kumar  wrote:
> Signed-off-by: Ashish Kumar 
> ---
>  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?
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

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

[Ashish Kumar]

Signed-off-by: Ashish Kumar 
---
 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))
flash->page_size = 512;
}
flash->page_size <<= flash->shift;
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot