Unverified Commit 2b308e71 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Mark Brown
Browse files

spi: Replace if-else-if by bitops and multiplications



Instead of if-else-if, simply call roundup_pow_of_two(BITS_PER_BYTES()).
Note, there is no division assumed as compiler may optimize it away.

Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20230710154932.68377-4-andriy.shevchenko@linux.intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 440c4733
Loading
Loading
Loading
Loading
+1 −7
Original line number Diff line number Diff line
@@ -3643,13 +3643,7 @@ int spi_split_transfers_maxwords(struct spi_controller *ctlr,
		size_t maxsize;
		int ret;

		if (xfer->bits_per_word <= 8)
			maxsize = maxwords;
		else if (xfer->bits_per_word <= 16)
			maxsize = 2 * maxwords;
		else
			maxsize = 4 * maxwords;

		maxsize = maxwords * roundup_pow_of_two(BITS_TO_BYTES(xfer->bits_per_word));
		if (xfer->len > maxsize) {
			ret = __spi_split_transfer_maxsize(ctlr, msg, &xfer,
							   maxsize, gfp);