All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support
@ 2015-09-04 13:03 Jagan Teki
  2015-09-04 13:03 ` [U-Boot] [PATCH 2/4] sf: params: Add IS25LP064 " Jagan Teki
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Jagan Teki @ 2015-09-04 13:03 UTC (permalink / raw)
  To: u-boot

Added support for IS25LP032 flash part.

Signed-off-by: Jagan Teki <jteki@openedev.com>
Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Cc: Michal Simek <michal.simek@xilinx.com>
---
 drivers/mtd/spi/sf_params.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
index 4a4a3af..184ba9c 100644
--- a/drivers/mtd/spi/sf_params.c
+++ b/drivers/mtd/spi/sf_params.c
@@ -34,6 +34,9 @@ const struct spi_flash_params spi_flash_params_table[] = {
 	{"GD25Q64B",	   0xc84017, 0x0,	64 * 1024,   128, RD_NORM,		    SECT_4K},
 	{"GD25LQ32",	   0xc86016, 0x0,	64 * 1024,    64, RD_NORM,		    SECT_4K},
 #endif
+#ifdef CONFIG_SPI_FLASH_ISSI		/* ISSI */
+	{"IS25LP032",	   0x9d6016, 0x0,	64 * 1024,    64, RD_NORM,			  0},
+#endif
 #ifdef CONFIG_SPI_FLASH_MACRONIX	/* MACRONIX */
 	{"MX25L2006E",	   0xc22012, 0x0,	64 * 1024,     4, RD_NORM,			  0},
 	{"MX25L4005",	   0xc22013, 0x0,	64 * 1024,     8, RD_NORM,			  0},
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH 2/4] sf: params: Add IS25LP064 part support
  2015-09-04 13:03 [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
@ 2015-09-04 13:03 ` Jagan Teki
  2015-09-04 13:03 ` [U-Boot] [PATCH 3/4] sf: params: Add IS25LP128 " Jagan Teki
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 8+ messages in thread
From: Jagan Teki @ 2015-09-04 13:03 UTC (permalink / raw)
  To: u-boot

Added support for IS25LP064 flash part.

Signed-off-by: Jagan Teki <jteki@openedev.com>
Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Cc: Michal Simek <michal.simek@xilinx.com>
---
 drivers/mtd/spi/sf_params.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
index 184ba9c..c3c38ee 100644
--- a/drivers/mtd/spi/sf_params.c
+++ b/drivers/mtd/spi/sf_params.c
@@ -36,6 +36,7 @@ const struct spi_flash_params spi_flash_params_table[] = {
 #endif
 #ifdef CONFIG_SPI_FLASH_ISSI		/* ISSI */
 	{"IS25LP032",	   0x9d6016, 0x0,	64 * 1024,    64, RD_NORM,			  0},
+	{"IS25LP064",	   0x9d6017, 0x0,	64 * 1024,   128, RD_NORM,			  0},
 #endif
 #ifdef CONFIG_SPI_FLASH_MACRONIX	/* MACRONIX */
 	{"MX25L2006E",	   0xc22012, 0x0,	64 * 1024,     4, RD_NORM,			  0},
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH 3/4] sf: params: Add IS25LP128 part support
  2015-09-04 13:03 [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
  2015-09-04 13:03 ` [U-Boot] [PATCH 2/4] sf: params: Add IS25LP064 " Jagan Teki
@ 2015-09-04 13:03 ` Jagan Teki
  2015-09-04 13:03 ` [U-Boot] [PATCH 4/4] zynq-common: Enable ISSI SPI-NOR flash support Jagan Teki
  2015-09-08  9:21 ` [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
  3 siblings, 0 replies; 8+ messages in thread
From: Jagan Teki @ 2015-09-04 13:03 UTC (permalink / raw)
  To: u-boot

Added support for IS25LP128 flash part.

Signed-off-by: Jagan Teki <jteki@openedev.com>
Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Cc: Michal Simek <michal.simek@xilinx.com>
---
 drivers/mtd/spi/sf_params.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
index c3c38ee..bb731ea 100644
--- a/drivers/mtd/spi/sf_params.c
+++ b/drivers/mtd/spi/sf_params.c
@@ -37,6 +37,7 @@ const struct spi_flash_params spi_flash_params_table[] = {
 #ifdef CONFIG_SPI_FLASH_ISSI		/* ISSI */
 	{"IS25LP032",	   0x9d6016, 0x0,	64 * 1024,    64, RD_NORM,			  0},
 	{"IS25LP064",	   0x9d6017, 0x0,	64 * 1024,   128, RD_NORM,			  0},
+	{"IS25LP128",	   0x9d6018, 0x0,	64 * 1024,   256, RD_NORM,			  0},
 #endif
 #ifdef CONFIG_SPI_FLASH_MACRONIX	/* MACRONIX */
 	{"MX25L2006E",	   0xc22012, 0x0,	64 * 1024,     4, RD_NORM,			  0},
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH 4/4] zynq-common: Enable ISSI SPI-NOR flash support
  2015-09-04 13:03 [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
  2015-09-04 13:03 ` [U-Boot] [PATCH 2/4] sf: params: Add IS25LP064 " Jagan Teki
  2015-09-04 13:03 ` [U-Boot] [PATCH 3/4] sf: params: Add IS25LP128 " Jagan Teki
@ 2015-09-04 13:03 ` Jagan Teki
  2015-09-08  9:21 ` [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
  3 siblings, 0 replies; 8+ messages in thread
From: Jagan Teki @ 2015-09-04 13:03 UTC (permalink / raw)
  To: u-boot

This patch enabled CONFIG_SPI_FLASH_ISSI for qspi on zynq boards.

Signed-off-by: Jagan Teki <jteki@openedev.com>
Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Cc: Michal Simek <michal.simek@xilinx.com>
---
 include/configs/zynq-common.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h
index f6dd03b..35a3a0a 100644
--- a/include/configs/zynq-common.h
+++ b/include/configs/zynq-common.h
@@ -75,6 +75,7 @@
 /* QSPI */
 #ifdef CONFIG_ZYNQ_QSPI
 # define CONFIG_SF_DEFAULT_SPEED	30000000
+# define CONFIG_SPI_FLASH_ISSI
 # define CONFIG_SPI_FLASH_SPANSION
 # define CONFIG_SPI_FLASH_STMICRO
 # define CONFIG_SPI_FLASH_WINBOND
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support
  2015-09-04 13:03 [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
                   ` (2 preceding siblings ...)
  2015-09-04 13:03 ` [U-Boot] [PATCH 4/4] zynq-common: Enable ISSI SPI-NOR flash support Jagan Teki
@ 2015-09-08  9:21 ` Jagan Teki
  2015-09-14  6:12   ` Siva Durga Prasad Paladugu
  3 siblings, 1 reply; 8+ messages in thread
From: Jagan Teki @ 2015-09-08  9:21 UTC (permalink / raw)
  To: u-boot

Hi Siva,

On 4 September 2015 at 18:33, Jagan Teki <jteki@openedev.com> wrote:
> Added support for IS25LP032 flash part.
>
> Signed-off-by: Jagan Teki <jteki@openedev.com>
> Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
> Cc: Michal Simek <michal.simek@xilinx.com>
> ---

Can you test this parts?

>  drivers/mtd/spi/sf_params.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
> index 4a4a3af..184ba9c 100644
> --- a/drivers/mtd/spi/sf_params.c
> +++ b/drivers/mtd/spi/sf_params.c
> @@ -34,6 +34,9 @@ const struct spi_flash_params spi_flash_params_table[] = {
>         {"GD25Q64B",       0xc84017, 0x0,       64 * 1024,   128, RD_NORM,                  SECT_4K},
>         {"GD25LQ32",       0xc86016, 0x0,       64 * 1024,    64, RD_NORM,                  SECT_4K},
>  #endif
> +#ifdef CONFIG_SPI_FLASH_ISSI           /* ISSI */
> +       {"IS25LP032",      0x9d6016, 0x0,       64 * 1024,    64, RD_NORM,                        0},
> +#endif
>  #ifdef CONFIG_SPI_FLASH_MACRONIX       /* MACRONIX */
>         {"MX25L2006E",     0xc22012, 0x0,       64 * 1024,     4, RD_NORM,                        0},
>         {"MX25L4005",      0xc22013, 0x0,       64 * 1024,     8, RD_NORM,                        0},
> --
> 1.9.1
>

thanks!
-- 
Jagan | openedev.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support
  2015-09-08  9:21 ` [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
@ 2015-09-14  6:12   ` Siva Durga Prasad Paladugu
  2015-09-14  6:40     ` Jagan Teki
  0 siblings, 1 reply; 8+ messages in thread
From: Siva Durga Prasad Paladugu @ 2015-09-14  6:12 UTC (permalink / raw)
  To: u-boot

Hi Jagan,

> -----Original Message-----
> From: Jagan Teki [mailto:jteki at openedev.com]
> Sent: Tuesday, September 08, 2015 2:52 PM
> To: u-boot at lists.denx.de
> Cc: Jagan Teki; Siva Durga Prasad Paladugu; Michal Simek
> Subject: Re: [PATCH 1/4] sf: params: Add IS25LP032 part support
> 
> Hi Siva,
> 
> On 4 September 2015 at 18:33, Jagan Teki <jteki@openedev.com> wrote:
> > Added support for IS25LP032 flash part.
> >
> > Signed-off-by: Jagan Teki <jteki@openedev.com>
> > Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
> > Cc: Michal Simek <michal.simek@xilinx.com>
> > ---
> 
> Can you test this parts?

I can't test this as these parts are not mounted on any of our boards now, but I can say that 
this won't work straight away and needs another patch I will be sending you soon. 


Regards,
Siva

> 
> >  drivers/mtd/spi/sf_params.c | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
> > index 4a4a3af..184ba9c 100644
> > --- a/drivers/mtd/spi/sf_params.c
> > +++ b/drivers/mtd/spi/sf_params.c
> > @@ -34,6 +34,9 @@ const struct spi_flash_params
> spi_flash_params_table[] = {
> >         {"GD25Q64B",       0xc84017, 0x0,       64 * 1024,   128, RD_NORM,
> SECT_4K},
> >         {"GD25LQ32",       0xc86016, 0x0,       64 * 1024,    64, RD_NORM,
> SECT_4K},
> >  #endif
> > +#ifdef CONFIG_SPI_FLASH_ISSI           /* ISSI */
> > +       {"IS25LP032",      0x9d6016, 0x0,       64 * 1024,    64, RD_NORM,
> 0},
> > +#endif
> >  #ifdef CONFIG_SPI_FLASH_MACRONIX       /* MACRONIX */
> >         {"MX25L2006E",     0xc22012, 0x0,       64 * 1024,     4, RD_NORM,
> 0},
> >         {"MX25L4005",      0xc22013, 0x0,       64 * 1024,     8, RD_NORM,
> 0},
> > --
> > 1.9.1
> >
> 
> thanks!
> --
> Jagan | openedev.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support
  2015-09-14  6:12   ` Siva Durga Prasad Paladugu
@ 2015-09-14  6:40     ` Jagan Teki
  2015-09-14  6:51       ` Siva Durga Prasad Paladugu
  0 siblings, 1 reply; 8+ messages in thread
From: Jagan Teki @ 2015-09-14  6:40 UTC (permalink / raw)
  To: u-boot

Hi Siva,

On 14 September 2015 at 11:42, Siva Durga Prasad Paladugu
<siva.durga.paladugu@xilinx.com> wrote:
> Hi Jagan,
>
>> -----Original Message-----
>> From: Jagan Teki [mailto:jteki at openedev.com]
>> Sent: Tuesday, September 08, 2015 2:52 PM
>> To: u-boot at lists.denx.de
>> Cc: Jagan Teki; Siva Durga Prasad Paladugu; Michal Simek
>> Subject: Re: [PATCH 1/4] sf: params: Add IS25LP032 part support
>>
>> Hi Siva,
>>
>> On 4 September 2015 at 18:33, Jagan Teki <jteki@openedev.com> wrote:
>> > Added support for IS25LP032 flash part.
>> >
>> > Signed-off-by: Jagan Teki <jteki@openedev.com>
>> > Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
>> > Cc: Michal Simek <michal.simek@xilinx.com>
>> > ---
>>
>> Can you test this parts?
>
> I can't test this as these parts are not mounted on any of our boards now, but I can say that
> this won't work straight away and needs another patch I will be sending you soon.

Does this means ISSI must require to set quad, bcz these patches I
never enable quad on fast read and page program.

>
>>
>> >  drivers/mtd/spi/sf_params.c | 3 +++
>> >  1 file changed, 3 insertions(+)
>> >
>> > diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
>> > index 4a4a3af..184ba9c 100644
>> > --- a/drivers/mtd/spi/sf_params.c
>> > +++ b/drivers/mtd/spi/sf_params.c
>> > @@ -34,6 +34,9 @@ const struct spi_flash_params
>> spi_flash_params_table[] = {
>> >         {"GD25Q64B",       0xc84017, 0x0,       64 * 1024,   128, RD_NORM,
>> SECT_4K},
>> >         {"GD25LQ32",       0xc86016, 0x0,       64 * 1024,    64, RD_NORM,
>> SECT_4K},
>> >  #endif
>> > +#ifdef CONFIG_SPI_FLASH_ISSI           /* ISSI */
>> > +       {"IS25LP032",      0x9d6016, 0x0,       64 * 1024,    64, RD_NORM,
>> 0},
>> > +#endif
>> >  #ifdef CONFIG_SPI_FLASH_MACRONIX       /* MACRONIX */
>> >         {"MX25L2006E",     0xc22012, 0x0,       64 * 1024,     4, RD_NORM,
>> 0},
>> >         {"MX25L4005",      0xc22013, 0x0,       64 * 1024,     8, RD_NORM,
>> 0},
>> > --
>> > 1.9.1
>> >
>>
>> thanks!
>> --
>> Jagan | openedev.



-- 
Jagan | openedev.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support
  2015-09-14  6:40     ` Jagan Teki
@ 2015-09-14  6:51       ` Siva Durga Prasad Paladugu
  0 siblings, 0 replies; 8+ messages in thread
From: Siva Durga Prasad Paladugu @ 2015-09-14  6:51 UTC (permalink / raw)
  To: u-boot

Hi Jagan,

> -----Original Message-----
> From: Jagan Teki [mailto:jteki at openedev.com]
> Sent: Monday, September 14, 2015 12:11 PM
> To: Siva Durga Prasad Paladugu
> Cc: u-boot at lists.denx.de; Michal Simek
> Subject: Re: [PATCH 1/4] sf: params: Add IS25LP032 part support
>
> Hi Siva,
>
> On 14 September 2015 at 11:42, Siva Durga Prasad Paladugu
> <siva.durga.paladugu@xilinx.com> wrote:
> > Hi Jagan,
> >
> >> -----Original Message-----
> >> From: Jagan Teki [mailto:jteki at openedev.com]
> >> Sent: Tuesday, September 08, 2015 2:52 PM
> >> To: u-boot at lists.denx.de
> >> Cc: Jagan Teki; Siva Durga Prasad Paladugu; Michal Simek
> >> Subject: Re: [PATCH 1/4] sf: params: Add IS25LP032 part support
> >>
> >> Hi Siva,
> >>
> >> On 4 September 2015 at 18:33, Jagan Teki <jteki@openedev.com> wrote:
> >> > Added support for IS25LP032 flash part.
> >> >
> >> > Signed-off-by: Jagan Teki <jteki@openedev.com>
> >> > Cc: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
> >> > Cc: Michal Simek <michal.simek@xilinx.com>
> >> > ---
> >>
> >> Can you test this parts?
> >
> > I can't test this as these parts are not mounted on any of our boards
> > now, but I can say that this won't work straight away and needs another
> patch I will be sending you soon.
>
> Does this means ISSI must require to set quad, bcz these patches I never
> enable quad on fast read and page program.

Yeah, I already sent patch to set qeb, this is just for the quad commands to work.
Any way I just realized that in your patches , you didn?t enabled quad then its fine.
If you enable quad then only the quad enable patch is required, otherwise its fine.

Regards,
Siva

>
> >
> >>
> >> >  drivers/mtd/spi/sf_params.c | 3 +++
> >> >  1 file changed, 3 insertions(+)
> >> >
> >> > diff --git a/drivers/mtd/spi/sf_params.c
> >> > b/drivers/mtd/spi/sf_params.c index 4a4a3af..184ba9c 100644
> >> > --- a/drivers/mtd/spi/sf_params.c
> >> > +++ b/drivers/mtd/spi/sf_params.c
> >> > @@ -34,6 +34,9 @@ const struct spi_flash_params
> >> spi_flash_params_table[] = {
> >> >         {"GD25Q64B",       0xc84017, 0x0,       64 * 1024,   128, RD_NORM,
> >> SECT_4K},
> >> >         {"GD25LQ32",       0xc86016, 0x0,       64 * 1024,    64, RD_NORM,
> >> SECT_4K},
> >> >  #endif
> >> > +#ifdef CONFIG_SPI_FLASH_ISSI           /* ISSI */
> >> > +       {"IS25LP032",      0x9d6016, 0x0,       64 * 1024,    64, RD_NORM,
> >> 0},
> >> > +#endif
> >> >  #ifdef CONFIG_SPI_FLASH_MACRONIX       /* MACRONIX */
> >> >         {"MX25L2006E",     0xc22012, 0x0,       64 * 1024,     4, RD_NORM,
> >> 0},
> >> >         {"MX25L4005",      0xc22013, 0x0,       64 * 1024,     8, RD_NORM,
> >> 0},
> >> > --
> >> > 1.9.1
> >> >
> >>
> >> thanks!
> >> --
> >> Jagan | openedev.
>
>
>
> --
> Jagan | openedev.


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.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-09-14  6:51 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-04 13:03 [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
2015-09-04 13:03 ` [U-Boot] [PATCH 2/4] sf: params: Add IS25LP064 " Jagan Teki
2015-09-04 13:03 ` [U-Boot] [PATCH 3/4] sf: params: Add IS25LP128 " Jagan Teki
2015-09-04 13:03 ` [U-Boot] [PATCH 4/4] zynq-common: Enable ISSI SPI-NOR flash support Jagan Teki
2015-09-08  9:21 ` [U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support Jagan Teki
2015-09-14  6:12   ` Siva Durga Prasad Paladugu
2015-09-14  6:40     ` Jagan Teki
2015-09-14  6:51       ` Siva Durga Prasad Paladugu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.