Hi Marek,

This patch breaks meson_spifc:

SF: Detected gd25lq128 with page size 256 Bytes, erase size 64 KiB, total
16 MiB
meson_spifc spi@8c80: Cannot set mode (err=-19)
Failed to initialize SPI flash at 0:0 (error -19)

Best,

Da

On Wed, Jun 30, 2021 at 12:49 PM Tom Rini <tr...@konsulko.com> wrote:

> On Thu, Jun 10, 2021 at 02:00:00PM +0200, Marek Vasut wrote:
>
> > The spi_get_bus_and_cs() may be called on the same bus and chipselect
> > with different frequency or mode. This is valid usecase, but the code
> > fails to notify the controller of such a configuration change. Call
> > spi_set_speed_mode() in case bus frequency or bus mode changed to let
> > the controller update the configuration.
> >
> > The problem can easily be triggered using the sspi command:
> > => sspi 0:0@1000
> > => sspi 0:0@2000
> > Without this patch, both transfers happen at 1000 Hz. With this patch,
> > the later transfer happens correctly at 2000 Hz.
> >
> > Signed-off-by: Marek Vasut <ma...@denx.de>
> > Cc: Jagan Teki <ja...@amarulasolutions.com>
> > Cc: Patrick Delaunay <patrick.delau...@foss.st.com>
>
> Applied to u-boot/master, thanks!
>
> --
> Tom
>

Reply via email to