Raspberry Pi DMA fixes + cleanups
Status from this message: Applied all, thanks
Get series with b4
b4 am [email protected]Versions newest first
Changes in v2 from the cover letter
* Patch [1/6]: Rework the patch to fix a race condition which could still occur with v1, and no longer check for idleness of a channel using the ACTIVE flag in the CS register (which is unreliable), but check for a zero control block address instead.
* Patch [2/6]: If outstanding writes fail to complete on abort of a transaction, print an error message and document a possible cause in a code comment. (Stefan Wahren)
* Patch [5/6] and [6/6]: Split patch to remove dead code in two and rephrase commit subject to be less generic. (Stefan Wahren, Vinod Koul)
Patches in v2 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/6 | dmaengine: bcm2835: Fix interrupt race on RTdrivers/ | +18 | 0 replies | |
| 2/6 | dmaengine: bcm2835: Fix abort of transactionsdrivers/ | +9 | 0 replies | |
| 3/6 | dmaengine: bcm2835: Return void from abort of transactionsdrivers/ | +2 | 0 replies | |
| 4/6 | dmaengine: bcm2835: Enforce control block alignmentdrivers/ | +5 | 0 replies | |
| 5/6 | dmaengine: bcm2835: Drop gratuitous list deletiondrivers/ | +0 | 0 replies | |
| 6/6 | dmaengine: bcm2835: Drop outdated comment on supported transactionsdrivers/ | +0 | 0 replies |
Reviews trailers per patch
None of the 6 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | Stefan Wahren | Florian Kauer |
|---|---|---|
| Series | T-b | A-b |
| 1/6 | ||
| 2/6 | ||
| 3/6 | ||
| 4/6 | ||
| 5/6 | ||
| 6/6 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v2 0/6] Raspberry Pi DMA fixes + cleanups
Two fixes and four cleanups for the Raspberry Pi DMA driver, v2.
Changes since v1:
* Patch [1/6]: Rework the patch to fix a race condition which could still occur with v1, and no longer check for idleness of a channel using the ACTIVE flag in the CS register (which is unreliable), but check for a zero control block address instead.
* Patch [2/6]: If outstanding writes fail to complete on abort of a transaction, print an error message and document a possible cause in a code comment. (Stefan Wahren)
* Patch [5/6] and [6/6]: Split patch to remove dead code in two and rephrase commit subject to be less generic. (Stefan Wahren, Vinod Koul)
Thanks,
Lukas
Lukas Wunner (6):
dmaengine: bcm2835: Fix interrupt race on RT
dmaengine: bcm2835: Fix abort of transactions
dmaengine: bcm2835: Return void from abort of transactions
dmaengine: bcm2835: Enforce control block alignment
dmaengine: bcm2835: Drop gratuitous list deletion
dmaengine: bcm2835: Drop outdated comment on supported transactions
drivers/dma/bcm2835-dma.c | 92 +++++++++++++-------------------------- 1 file changed, 31 insertions(+), 61 deletions(-)
Signature · 1 line
2.20.1