OSDN Git Service

spi: fix SPI_BPW_RANGE_MASK() regression
authorArnd Bergmann <arnd@arndb.de>
Wed, 13 Mar 2019 21:00:34 +0000 (22:00 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 14 Mar 2019 15:52:33 +0000 (15:52 +0000)
Geert points out that I confused the min/max arguments that are
reversed between SPI_BPW_RANGE_MASK() and GENMASK(). This time
I have verified the result of the macro after fixing the arguments.

Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Fixes: eefffb42f665 ("spi: work around clang bug in SPI_BPW_RANGE_MASK()")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Mark Brown <broonie@kernel.org>
include/linux/spi/spi.h

index b273864..a0975cf 100644 (file)
@@ -444,7 +444,7 @@ struct spi_controller {
        /* bitmask of supported bits_per_word for transfers */
        u32                     bits_per_word_mask;
 #define SPI_BPW_MASK(bits) BIT((bits) - 1)
-#define SPI_BPW_RANGE_MASK(min, max) GENMASK((min) - 1, (max) - 1)
+#define SPI_BPW_RANGE_MASK(min, max) GENMASK((max) - 1, (min) - 1)
 
        /* limits on transfer speed */
        u32                     min_speed_hz;