[U-Boot] [PATCH v1 2/4] spi: fsl_qspi: Add 4bytes address support

Yao Yuan yao.yuan at nxp.com
Wed Oct 26 10:01:28 CEST 2016


On Wed, Oct 26, 2016 at 3:39 PM, Jagan Teki wrote:
> On Wed, Sep 28, 2016 at 4:39 PM, Yuan Yao <yao.yuan at freescale.com> wrote:
> > From: Yuan Yao <yao.yuan at nxp.com>
> >
> > The QSPI support the direct 4bytes address command for flash
> > read/write/erase.
> > And the address can cover the whole QSPI memory space.
> >
> > Signed-off-by: Yuan Yao <yao.yuan at nxp.com>
> > ---
> >  drivers/spi/fsl_qspi.c | 8 ++++++--
> >  1 file changed, 6 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/spi/fsl_qspi.c b/drivers/spi/fsl_qspi.c index
> > 2144fca..119b782 100644
> > --- a/drivers/spi/fsl_qspi.c
> > +++ b/drivers/spi/fsl_qspi.c
> > @@ -25,7 +25,8 @@ DECLARE_GLOBAL_DATA_PTR;
> >  #define TX_BUFFER_SIZE         0x40
> >  #endif
> >
> > -#define OFFSET_BITS_MASK       GENMASK(23, 0)
> > +#define OFFSET_BITS_MASK       ((FSL_QSPI_FLASH_SIZE  > SZ_16M) ? \
> > +                                       GENMASK(27, 0) :  GENMASK(23,
> > +0))
> >
> >  #define FLASH_STATUS_WEL       0x02
> >
> > @@ -760,7 +761,10 @@ int qspi_xfer(struct fsl_qspi_priv *priv, unsigned int
> bitlen,
> >         if (dout) {
> >                 if (flags & SPI_XFER_BEGIN) {
> >                         priv->cur_seqid = *(u8 *)dout;
> > -                       memcpy(&txbuf, dout, 4);
> > +                       if (FSL_QSPI_FLASH_SIZE  > SZ_16M)
> > +                               memcpy(&txbuf, dout + 1, 4);
> > +                       else
> > +                               memcpy(&txbuf, dout, 4);
> 
> This driver has lot of flash hacks, and I am completely unhappy about this
> please try to write new driver on flash side (mtd/spi).
> 
Hi Jagan,

Yes, I think so, I also think I need rewrite this driver now.
In fact, I'm working on this. But need more time, So I have to send those patch set to support 16M+ support.

Thanks for your support.


More information about the U-Boot mailing list