On Thu, Apr 07, 2016 at 09:17:29PM +0900, Akinobu Mita wrote: > 2016-03-22 1:00 GMT+09:00 Akinobu Mita <akinobu.mita@xxxxxxxxx>: > > if ((l & OMAP2_MCSPI_CHCONF_TURBO) && mcspi->fifo_depth == 0) > > dma_count -= es; > The actual transfer length can be smaller than xfer->len by above line. > And the last word is filled after dma transfer completion. > So this patch is broken. Could you revert remove this from your > development tree? > I don't find a way how to fix it without copy and pasting a lot of > line from spi core. Can you please send a patch doing the revert and explaining why we can't do this in the changelog? That might help someone else looking at this to figure it out in future, or at least stop them trying the same thing and running into problems.
Attachment:
signature.asc
Description: PGP signature
