[U-Boot] [PATCH 1/4] sf: params: Add IS25LP032 part support

[Siva Durga Prasad Paladugu]

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 at 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 at openedev.com> wrote:
> >> > Added support for IS25LP032 flash part.
> >> >
> >> > Signed-off-by: Jagan Teki <jteki at openedev.com>
> >> > Cc: Siva Durga Prasad Paladugu <sivadur at xilinx.com>
> >> > Cc: Michal Simek <michal.simek at 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.