linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).