Patchwork [U-Boot] sf:eon: Add support for EN25S64

login
register
mail settings
Submitter Priyanka Jain
Date Sept. 5, 2013, 7:19 a.m.
Message ID <1378365573-32662-1-git-send-email-Priyanka.Jain@freescale.com>
Download mbox | patch
Permalink /patch/272805/
State Superseded
Delegated to: Jagannadha Sutradharudu Teki
Headers show

Comments

Priyanka Jain - Sept. 5, 2013, 7:19 a.m.
Add support for EON EN25S64 SPI flash memory
Features: 64Mb size, 1.8V, 4KB sector

Signed-off-by: Priyanka Jain <Priyanka.Jain@freescale.com>
---
 drivers/mtd/spi/eon.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
Jagannadha Sutradharudu Teki - Sept. 7, 2013, 6:49 a.m.
Hi,

On Thu, Sep 5, 2013 at 12:49 PM, Priyanka Jain
<Priyanka.Jain@freescale.com> wrote:
> Add support for EON EN25S64 SPI flash memory
> Features: 64Mb size, 1.8V, 4KB sector
>
> Signed-off-by: Priyanka Jain <Priyanka.Jain@freescale.com>
> ---
>  drivers/mtd/spi/eon.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mtd/spi/eon.c b/drivers/mtd/spi/eon.c
> index 25cfc12..45a0052 100644
> --- a/drivers/mtd/spi/eon.c
> +++ b/drivers/mtd/spi/eon.c
> @@ -27,6 +27,11 @@ static const struct eon_spi_flash_params eon_spi_flash_table[] = {
>                 .nr_sectors = 4096,
>                 .name = "EN25Q128",
>         },
> +       {
> +               .idcode1 = 0x3817,
> +               .nr_sectors = 4096,
> +               .name = "EN25S64",
> +       },
>  };
>
>  struct spi_flash *spi_flash_probe_eon(struct spi_slave *spi, u8 *idcode)
> --
> 1.7.4.1

Thanks for you patch.
Can you test your hw with this branch
http://git.denx.de/?p=u-boot/u-boot-spi.git;a=shortlog;h=refs/heads/master-test

Sooner this will merge into master, new updated sf layer.
Let me know for any issues.
Jain Priyanka-B32167 - Sept. 11, 2013, 6:33 a.m.
Hello Jagan,

Reply in-lined.

Regards
Priyanka

> -----Original Message-----
> From: Jagan Teki [mailto:jagannadh.teki@gmail.com]
> Sent: Saturday, September 07, 2013 12:19 PM
> To: Jain Priyanka-B32167
> Cc: u-boot@lists.denx.de
> Subject: Re: [PATCH] sf:eon: Add support for EN25S64
> 
> Hi,
> 
> On Thu, Sep 5, 2013 at 12:49 PM, Priyanka Jain
> <Priyanka.Jain@freescale.com> wrote:
> > Add support for EON EN25S64 SPI flash memory
> > Features: 64Mb size, 1.8V, 4KB sector
> >
> > Signed-off-by: Priyanka Jain <Priyanka.Jain@freescale.com>
> > ---
> >  drivers/mtd/spi/eon.c |    5 +++++
> >  1 files changed, 5 insertions(+), 0 deletions(-)
> >
> > diff --git a/drivers/mtd/spi/eon.c b/drivers/mtd/spi/eon.c index
> > 25cfc12..45a0052 100644
> > --- a/drivers/mtd/spi/eon.c
> > +++ b/drivers/mtd/spi/eon.c
> > @@ -27,6 +27,11 @@ static const struct eon_spi_flash_params
> eon_spi_flash_table[] = {
> >                 .nr_sectors = 4096,
> >                 .name = "EN25Q128",
> >         },
> > +       {
> > +               .idcode1 = 0x3817,
> > +               .nr_sectors = 4096,
> > +               .name = "EN25S64",
> > +       },
> >  };
> >
> >  struct spi_flash *spi_flash_probe_eon(struct spi_slave *spi, u8
> > *idcode)
> > --
> > 1.7.4.1
> 
> Thanks for you patch.
> Can you test your hw with this branch
> http://git.denx.de/?p=u-boot/u-boot-
> spi.git;a=shortlog;h=refs/heads/master-test
> 
> Sooner this will merge into master, new updated sf layer.
> Let me know for any issues.
> 
[Jain Priyanka-B32167] I tested this patch on development board whose base code patch has not yet been send on the mail-list. Work is in progress on that.
So, it will be difficult to test this patch on u-boot-spi.git. But I can rebase the patch to u-boot-spi.git and send again if required. Please let me know.
> --
> Thanks,
> Jagan.
> --------
> Jagannadha Sutradharudu Teki,
> E: Jagannadh.teki@gmail.com, P: +91-9676773388 Engineer - System software
> hacker U-boot - SPI Custodian and Zynq APSOC Android - BSP/Kernel
> Ln: http://www.linkedin.com/in/jaganteki
Jagannadha Sutradharudu Teki - Sept. 11, 2013, 6:38 a.m.
On Wed, Sep 11, 2013 at 12:03 PM, Jain Priyanka-B32167
<B32167@freescale.com> wrote:
> Hello Jagan,
>
> Reply in-lined.
>
> Regards
> Priyanka
>
>> -----Original Message-----
>> From: Jagan Teki [mailto:jagannadh.teki@gmail.com]
>> Sent: Saturday, September 07, 2013 12:19 PM
>> To: Jain Priyanka-B32167
>> Cc: u-boot@lists.denx.de
>> Subject: Re: [PATCH] sf:eon: Add support for EN25S64
>>
>> Hi,
>>
>> On Thu, Sep 5, 2013 at 12:49 PM, Priyanka Jain
>> <Priyanka.Jain@freescale.com> wrote:
>> > Add support for EON EN25S64 SPI flash memory
>> > Features: 64Mb size, 1.8V, 4KB sector
>> >
>> > Signed-off-by: Priyanka Jain <Priyanka.Jain@freescale.com>
>> > ---
>> >  drivers/mtd/spi/eon.c |    5 +++++
>> >  1 files changed, 5 insertions(+), 0 deletions(-)
>> >
>> > diff --git a/drivers/mtd/spi/eon.c b/drivers/mtd/spi/eon.c index
>> > 25cfc12..45a0052 100644
>> > --- a/drivers/mtd/spi/eon.c
>> > +++ b/drivers/mtd/spi/eon.c
>> > @@ -27,6 +27,11 @@ static const struct eon_spi_flash_params
>> eon_spi_flash_table[] = {
>> >                 .nr_sectors = 4096,
>> >                 .name = "EN25Q128",
>> >         },
>> > +       {
>> > +               .idcode1 = 0x3817,
>> > +               .nr_sectors = 4096,
>> > +               .name = "EN25S64",
>> > +       },
>> >  };
>> >
>> >  struct spi_flash *spi_flash_probe_eon(struct spi_slave *spi, u8
>> > *idcode)
>> > --
>> > 1.7.4.1
>>
>> Thanks for you patch.
>> Can you test your hw with this branch
>> http://git.denx.de/?p=u-boot/u-boot-
>> spi.git;a=shortlog;h=refs/heads/master-test
>>
>> Sooner this will merge into master, new updated sf layer.
>> Let me know for any issues.
>>
> [Jain Priyanka-B32167] I tested this patch on development board whose base code patch has not yet been send on the mail-list. Work is in progress on that.
> So, it will be difficult to test this patch on u-boot-spi.git. But I can rebase the patch to u-boot-spi.git and send again if required. Please let me know.

Please add your patch manually on u-boot-spi.git and test and let me
know the result so-that I will tell you.

Patch

diff --git a/drivers/mtd/spi/eon.c b/drivers/mtd/spi/eon.c
index 25cfc12..45a0052 100644
--- a/drivers/mtd/spi/eon.c
+++ b/drivers/mtd/spi/eon.c
@@ -27,6 +27,11 @@  static const struct eon_spi_flash_params eon_spi_flash_table[] = {
 		.nr_sectors = 4096,
 		.name = "EN25Q128",
 	},
+	{
+		.idcode1 = 0x3817,
+		.nr_sectors = 4096,
+		.name = "EN25S64",
+	},
 };
 
 struct spi_flash *spi_flash_probe_eon(struct spi_slave *spi, u8 *idcode)