diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2015-03-06 13:42:01 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-03-07 12:22:02 +0100 |
commit | 854d2f241d71f6ca08ccde30e6c7c2e403363e52 (patch) | |
tree | 28b2c40697eb3e3e316021541793ed646d177f69 /drivers/spi/spi-dw.c | |
parent | spi: dw-mid: avoid potential NULL dereference (diff) | |
download | linux-854d2f241d71f6ca08ccde30e6c7c2e403363e52.tar.xz linux-854d2f241d71f6ca08ccde30e6c7c2e403363e52.zip |
spi: dw-mid: clear BUSY flag fist and test other one
The logic of DMA completion is broken now since test_and_clear_bit() never
returns the other bit is set. It means condition are always false and we have
spi_finalize_current_transfer() called per each DMA completion which is wrong.
The patch fixes logic by clearing BUSY bit first and then check for the other
one.
Fixes: 30c8eb52cc4a (spi: dw-mid: split rx and tx callbacks when DMA)
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Cc: stable@vger.kernel.org
Diffstat (limited to 'drivers/spi/spi-dw.c')
0 files changed, 0 insertions, 0 deletions