diff options
| author | Lukas Wunner <lukas@wunner.de> | 2018-11-29 16:45:24 +0100 | 
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2018-12-04 16:51:12 +0000 | 
| commit | b31a9299bca66c42583ef2e2f685369780cdf350 (patch) | |
| tree | cbf55606c5fd9e083df02bb205e87d0cc332e0e9 /include/linux/fpga/fpga-bridge.h | |
| parent | b12a084c8729ef423089bb9a5a143eed39cd94e7 (diff) | |
spi: bcm2835: Polish transfer of DMA prologue
Commit 3bd7f6589f67 ("spi: bcm2835: Overcome sglist entry length
limitation") was unfortunately merged even though submission of a
refined version was imminent.  Apply those refinements as an amendment:
* Drop no longer needed #include <asm/page.h>.  The lines requiring
  its inclusion were removed by the commit.
* Change type of tx_spillover flag from bool to unsigned int for
  consistency with dma_pending flag and pursuant to Linus' dictum:
  https://lkml.org/lkml/2017/11/21/384
* In bcm2835_rd_fifo_count() do not check for bs->rx_buf != NULL.
  The function will never be called if that's the case.
* Amend kerneldoc of bcm2835_wait_tx_fifo_empty() to prevent its use in
  situations where the function might spin forever.  (In response to a
  review comment by Stefan Wahren.)
* Sync only the cacheline containing the RX prologue back to memory,
  not the full first sglist entry.
* Use sg_dma_address() and sg_dma_len() instead of referencing the
  sglist entry members directly.  Seems to be the more common syntax in
  the tree, even for lvalues.
Signed-off-by: Lukas Wunner <lukas@wunner.de>
Cc: Frank Pavlic <f.pavlic@kunbus.de>
Cc: Martin Sperl <kernel@martin.sperl.org>
Cc: Noralf Trønnes <noralf@tronnes.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'include/linux/fpga/fpga-bridge.h')
0 files changed, 0 insertions, 0 deletions
