Re: [PATCH] spi: stm32_qspi: Fix short data write operation

2021-06-18 Thread Patrice CHOTARD
On 5/25/21 6:02 PM, Patrice CHOTARD wrote: > Hi Daniil > > On 5/24/21 2:53 PM, Daniil Stas wrote: >> On Mon, 24 May 2021 09:40:05 +0200 >> Patrice CHOTARD wrote: >> >>> Hi Daniil >>> >>> On 5/24/21 12:24 AM, Daniil Stas wrote: TCF flag only means that all data was sent to FIFO. To check

Re: [PATCH] spi: stm32_qspi: Fix short data write operation

2021-06-01 Thread Patrick DELAUNAY
Hi, On 5/24/21 12:24 AM, Daniil Stas wrote: TCF flag only means that all data was sent to FIFO. To check if the data was sent out of FIFO we should also wait for the BUSY flag to be cleared. Otherwise there is a race condition which can lead to inability to write short (one byte long) data.

Re: [PATCH] spi: stm32_qspi: Fix short data write operation

2021-05-25 Thread Patrice CHOTARD
Hi Daniil On 5/24/21 2:53 PM, Daniil Stas wrote: > On Mon, 24 May 2021 09:40:05 +0200 > Patrice CHOTARD wrote: > >> Hi Daniil >> >> On 5/24/21 12:24 AM, Daniil Stas wrote: >>> TCF flag only means that all data was sent to FIFO. To check if the >>> data was sent out of FIFO we should also wait

Re: [PATCH] spi: stm32_qspi: Fix short data write operation

2021-05-24 Thread Daniil Stas
On Mon, 24 May 2021 09:40:05 +0200 Patrice CHOTARD wrote: > Hi Daniil > > On 5/24/21 12:24 AM, Daniil Stas wrote: > > TCF flag only means that all data was sent to FIFO. To check if the > > data was sent out of FIFO we should also wait for the BUSY flag to > > be cleared. Otherwise there is a

Re: [PATCH] spi: stm32_qspi: Fix short data write operation

2021-05-24 Thread Patrice CHOTARD
Hi Daniil On 5/24/21 12:24 AM, Daniil Stas wrote: > TCF flag only means that all data was sent to FIFO. To check if the > data was sent out of FIFO we should also wait for the BUSY flag to be > cleared. Otherwise there is a race condition which can lead to > inability to write short (one byte

[PATCH] spi: stm32_qspi: Fix short data write operation

2021-05-23 Thread Daniil Stas
TCF flag only means that all data was sent to FIFO. To check if the data was sent out of FIFO we should also wait for the BUSY flag to be cleared. Otherwise there is a race condition which can lead to inability to write short (one byte long) data. Signed-off-by: Daniil Stas Cc: Patrick Delaunay