Hi, On Tue, Jul 25, 2023 at 4:48 AM Vijaya Krishna Nivarthi <quic_vnivarth@xxxxxxxxxxx> wrote: > > During FIFO/DMA modes dynamic switching, only corresponding interrupts are > enabled. However its possible that FIFO related interrupt status registers > get set during DMA mode. For example WR_FIFO_EMPTY bit is set during DMA > TX. > > Ignore such status bits so that they don't trip unwanted operations. > > Suggested-by: Douglas Anderson <dianders@xxxxxxxxxxxx> > Signed-off-by: Vijaya Krishna Nivarthi <quic_vnivarth@xxxxxxxxxxx> > --- > drivers/spi/spi-qcom-qspi.c | 3 +++ > 1 file changed, 3 insertions(+) Things are pretty broken without this fix, so it definitely needs a Fixes tag so it can find its way to mainline as soon as possible: Fixes: b5762d95607e ("spi: spi-qcom-qspi: Add DMA mode support") Looks good to me and also works fixes the problem for me. Thus I'm happy with any of these tags: Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> Tested-by: Douglas Anderson <dianders@xxxxxxxxxxxx>