[U-Boot] [PATCH] mtd: spi: Replace ad-hoc default implementation with spi_mem_op

Jagan Teki jagan at amarulasolutions.com
Fri Apr 12 06:14:44 UTC 2019


On Fri, Apr 12, 2019 at 11:39 AM Vignesh Raghavendra <vigneshr at ti.com> wrote:
>
>
>
> On 12/04/19 10:46 AM, Jagan Teki wrote:
>
> >>>>
> >>>> On 02/04/19 6:22 AM, Marek Vasut wrote:
> >>>>> Replace the ad-hoc erase operation implementation with a simple
> >>>>> spi_mem_op implementation of the same functionality. This is a minor
> >>>>> optimization and removal of the ad-hoc code.
> >>>>>
> >>>>> This however also changes the behavior of the execution of the erase
> >>>>> opcode from two separate transfer requests to the SPI NOR driver to
> >>>>> one transfer request to the SPI NOR driver. The former was how U-Boot
> >>>>> behaved before the SPI NOR framework was imported and the later was
> >>>>> introduced by the SPI NOR framework. The former is more optimal, so
> >>>>> keep that.
> >>>>>
> >>>>> Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> >>>>> Cc: Horatiu Vultur <horatiu.vultur at microchip.com>
> >>>>> Cc: Jagan Teki <jagan at openedev.com>
> >>>>> Cc: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> >>>>> Cc: Stefan Roese <sr at denx.de>
> >>>>> Cc: Vignesh R <vigneshr at ti.com>
> >>>>> ---
> >>>>
> >>>> Marek, thanks for the fix!
> >>>>
> >>>> Jagan,
> >>>>
> >>>> Could we get this merged before v2019.04 release as this should fix regression
> >>>> wrt erase reported by Ashish.
> >>>
> >>>
> >>> Yes, follow patch reverts back to old method of erase implementation.
> >>> Tested-by: Ashish Kumar <Ashish.kumar at nxp.com>
> >>
> >> I guess this never made it into the release, so the SPI NOR erase is
> >> broken in v2019.04 ?
> >
> > I usually review on top of patchwork, that seems not assigned to me.
> > Vignesh will this break? I can see the erase seems fine on my microzed
> > with last week testing.
> >
>
> This issue affects SPI controllers that try to interpret spi_xfer()
> calls back into cmd+addr+data format instead of just sending dout over
> the bus as is.
> So, it does not all SPI controllers. From a quick look fsl-qspi,
> renesas_rpc_spi and mtk_qspi maybe affected.
>
> This is a valid fix. Would be great if this patch is part of early pull
> request

Thanks, re-assigned from patchwork and applied.


More information about the U-Boot mailing list