diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2019-03-13 22:00:34 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2019-03-14 15:52:33 +0000 |
| commit | 6d85028134d3f4f946924e2f9f0aaff47d9de840 (patch) | |
| tree | ee5eb5f00462fe027f4a032e9c03d4901772f6da | |
| parent | 8fcb830a00f0980ffe38d223cdd9a4d2d24da476 (diff) | |
| download | linux-6d85028134d3f4f946924e2f9f0aaff47d9de840.tar.bz2 | |
spi: fix SPI_BPW_RANGE_MASK() regression
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>
| -rw-r--r-- | include/linux/spi/spi.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h index b27386450089..a0975cf76cf6 100644 --- a/include/linux/spi/spi.h +++ b/include/linux/spi/spi.h @@ -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; |