RE: [PATCH 2/5] spi: spi-nxp-fspi: change the default lut index

[Bough Chen <haibo.chen@xxxxxxx>]

> -----Original Message-----
> From: Adam Ford <aford173@xxxxxxxxx>
> Sent: 2023年12月14日 11:04
> To: Bough Chen <haibo.chen@xxxxxxx>
> Cc: broonie@xxxxxxxxxx; yogeshgaur.83@xxxxxxxxx; linux-spi@xxxxxxxxxxxxxxx;
> dl-linux-imx <linux-imx@xxxxxxx>; Han Xu <han.xu@xxxxxxx>
> Subject: Re: [PATCH 2/5] spi: spi-nxp-fspi: change the default lut index
> 
> On Wed, Dec 13, 2023 at 3:08 AM <haibo.chen@xxxxxxx> wrote:
> >
> > From: Haibo Chen <haibo.chen@xxxxxxx>
> >
> > The fspi dynamic lut use the last lut for all IPS operations, the
> > imx8ulp only supports 15 luts, so change the last lut index from
> > 31 to 15.
> >
> > Signed-off-by: Han Xu <han.xu@xxxxxxx>
> > Signed-off-by: Haibo Chen <haibo.chen@xxxxxxx>
> > ---
> >  drivers/spi/spi-nxp-fspi.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/spi/spi-nxp-fspi.c b/drivers/spi/spi-nxp-fspi.c
> > index 0feecf5ba010..9d6b4d22263c 100644
> > --- a/drivers/spi/spi-nxp-fspi.c
> > +++ b/drivers/spi/spi-nxp-fspi.c
> > @@ -64,9 +64,9 @@
> >  /*
> >   * The driver only uses one single LUT entry, that is updated on
> >   * each call of exec_op(). Index 0 is preset at boot with a basic
> > - * read operation, so let's use the last entry (31).
> > + * read operation, so let's use the last entry (15).
> >   */
> > -#define        SEQID_LUT                       31
> > +#define        SEQID_LUT                       15
> 
> What impact does this have on other SoC's with the FlexSPI with 32?

No impact, just use another LUT to send command queue.

Best Regards
Haibo Chen
> 
> adam
> >
> >  /* Registers used by the driver */
> >  #define FSPI_MCR0                      0x00
> > --
> > 2.34.1
> >
> >