mirror of
https://github.com/Ysurac/openmptcprouter.git
synced 2025-03-09 15:40:20 +00:00
55 lines
1.8 KiB
Diff
55 lines
1.8 KiB
Diff
From 9c7f25237330b70cdcf6b28c66f35e063101f852 Mon Sep 17 00:00:00 2001
|
|
From: Phil Elwell <phil@raspberrypi.com>
|
|
Date: Thu, 28 Jan 2021 11:30:04 +0000
|
|
Subject: [PATCH 314/726] spi: bcm2835: Workaround/fix for zero-length
|
|
transfers
|
|
|
|
A relatively recent commit ([1]) contained optimisation for the PIO
|
|
SPI FIFO-filling functions. The commit message includes the phrase
|
|
"[t]he blind and counted loops are always called with nonzero count".
|
|
This is technically true, but it is still possible for count to become
|
|
zero before the loop is entered - if tfr->len is zero. Moving the loop
|
|
exit condition to the end of the loop saves a few cycles, but results
|
|
in a near-infinite loop should the revised count be zero on entry.
|
|
|
|
Strangely, zero-lengthed transfers aren't filtered by the SPI framework
|
|
and, even more strangely, the Python3 spidev library is triggering them
|
|
for no obvious reason.
|
|
|
|
Avoid the problem completely by bailing out of the main transfer
|
|
function early if trf->len is zero, although there may be a case for
|
|
moving the mitigation into the framework.
|
|
|
|
See: https://github.com/raspberrypi/linux/issues/4100
|
|
|
|
Signed-off-by: Phil Elwell <phil@raspberrypi.com>
|
|
|
|
[1] 26751de25d25 ("spi: bcm2835: Micro-optimise FIFO loops")
|
|
---
|
|
drivers/spi/spi-bcm2835.c | 10 ++++++++++
|
|
1 file changed, 10 insertions(+)
|
|
|
|
diff --git a/drivers/spi/spi-bcm2835.c b/drivers/spi/spi-bcm2835.c
|
|
index 747e03228c48..7bc7eab92759 100644
|
|
--- a/drivers/spi/spi-bcm2835.c
|
|
+++ b/drivers/spi/spi-bcm2835.c
|
|
@@ -1055,6 +1055,16 @@ static int bcm2835_spi_transfer_one(struct spi_controller *ctlr,
|
|
unsigned long hz_per_byte, byte_limit;
|
|
u32 cs = slv->prepare_cs;
|
|
|
|
+ if (unlikely(!tfr->len)) {
|
|
+ static int warned;
|
|
+
|
|
+ if (!warned)
|
|
+ dev_warn(&spi->dev,
|
|
+ "zero-length SPI transfer ignored\n");
|
|
+ warned = 1;
|
|
+ return 0;
|
|
+ }
|
|
+
|
|
/* set clock */
|
|
spi_hz = tfr->speed_hz;
|
|
|
|
--
|
|
2.33.1
|
|
|