mirror of
https://github.com/openwrt/openwrt.git
synced 2024-12-22 15:02:32 +00:00
8299d1f057
Rebased RPi foundation patches on linux 5.10.59, removed applied and reverted patches, wireless patches and defconfig patches. bcm2708: boot tested on RPi B+ v1.2 bcm2709: boot tested on RPi 4B v1.1 4G bcm2711: boot tested on RPi 4B v1.1 4G Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
51 lines
1.7 KiB
Diff
51 lines
1.7 KiB
Diff
From 54545710d40fd00e3c727603a446500b1b9b3d88 Mon Sep 17 00:00:00 2001
|
|
From: Phil Elwell <phil@raspberrypi.com>
|
|
Date: Thu, 28 Jan 2021 11:30:04 +0000
|
|
Subject: [PATCH] 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(+)
|
|
|
|
--- a/drivers/spi/spi-bcm2835.c
|
|
+++ b/drivers/spi/spi-bcm2835.c
|
|
@@ -1092,6 +1092,16 @@ static int bcm2835_spi_transfer_one(stru
|
|
unsigned long hz_per_byte, byte_limit;
|
|
u32 cs = bs->prepare_cs[spi->chip_select];
|
|
|
|
+ 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;
|
|
|