* [PATCH for-4.20] spi: bcm2835: Unbreak the build of esoteric configs
@ 2018-11-29 14:14 Lukas Wunner
[not found] ` <4a53a5c59b9e454476a2313735d6353428e982cb.1543500560.git.lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Lukas Wunner @ 2018-11-29 14:14 UTC (permalink / raw)
To: Mark Brown
Cc: linux-spi-u79uwXL29TY76Z2rM5mHXA, Noralf Tronnes,
linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Frank Pavlic
Commit e82b0b382845 ("spi: bcm2835: Fix race on DMA termination") broke
the build with COMPILE_TEST=y on arches whose cmpxchg() requires 32-bit
operands (xtensa, older arm ISAs).
Unfortunately the commit was applied despite prior warning that it needs
to be respun:
https://marc.info/?l=linux-spi&m=154186019626675&w=2
Fix by changing the dma_pending flag's type from bool to unsigned int.
Signed-off-by: Lukas Wunner <lukas@wunner.de>
Fixes: e82b0b382845 ("spi: bcm2835: Fix race on DMA termination")
Cc: Frank Pavlic <f.pavlic@kunbus.de>
Cc: Martin Sperl <kernel@martin.sperl.org>
Cc: Noralf Trønnes <noralf@tronnes.org>
---
drivers/spi/spi-bcm2835.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/spi/spi-bcm2835.c b/drivers/spi/spi-bcm2835.c
index 774161b..25abf2d 100644
--- a/drivers/spi/spi-bcm2835.c
+++ b/drivers/spi/spi-bcm2835.c
@@ -88,7 +88,7 @@ struct bcm2835_spi {
u8 *rx_buf;
int tx_len;
int rx_len;
- bool dma_pending;
+ unsigned int dma_pending;
};
static inline u32 bcm2835_rd(struct bcm2835_spi *bs, unsigned reg)
--
2.19.2
_______________________________________________
linux-rpi-kernel mailing list
linux-rpi-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rpi-kernel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH for-4.20] spi: bcm2835: Unbreak the build of esoteric configs
[not found] ` <4a53a5c59b9e454476a2313735d6353428e982cb.1543500560.git.lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
@ 2018-11-29 15:31 ` Mark Brown
0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2018-11-29 15:31 UTC (permalink / raw)
To: Lukas Wunner
Cc: linux-spi-u79uwXL29TY76Z2rM5mHXA, Noralf Tronnes,
linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Frank Pavlic
[-- Attachment #1.1: Type: text/plain, Size: 449 bytes --]
On Thu, Nov 29, 2018 at 03:14:49PM +0100, Lukas Wunner wrote:
> Unfortunately the commit was applied despite prior warning that it needs
> to be respun:
> https://marc.info/?l=linux-spi&m=154186019626675&w=2
Sorry about that, I started scanning the changelog when it hit
discussion about doing a more involved fix for -next - it's good to draw
attention to reasons not to merge the patch a bit more prominently as it
helps avoid things like that.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 206 bytes --]
_______________________________________________
linux-rpi-kernel mailing list
linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
http://lists.infradead.org/mailman/listinfo/linux-rpi-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-11-29 15:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-29 14:14 [PATCH for-4.20] spi: bcm2835: Unbreak the build of esoteric configs Lukas Wunner
[not found] ` <4a53a5c59b9e454476a2313735d6353428e982cb.1543500560.git.lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2018-11-29 15:31 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).