regmap: spi: Fix potential off-by-one when calculating reserved size
authorAndy Shevchenko <andy.shevchenko@gmail.com>
Wed, 5 Jun 2024 20:53:15 +0000 (23:53 +0300)
committerMark Brown <broonie@kernel.org>
Wed, 5 Jun 2024 21:23:52 +0000 (22:23 +0100)
commitd4ea1d504d2701ba04412f98dc00d45a104c52ab
treed5e679ddefa0618238d9cdac37fc6ee1140c98fb
parent3b0d6a32115c88618794406123ef8466f0327898
regmap: spi: Fix potential off-by-one when calculating reserved size

If we ever meet a hardware that uses weird register bits and padding,
we may end up in off-by-one error since x/8 + y/8 might not be equal
to (x + y)/8 in some cases.

bits    pad   x/8+y/8 (x+y)/8
4..7    0..3    0       0 // x + y from 4 up to 7
4..7    4..7    0       1 // x + y from 8 up to 11
4..7    8..11   1       1 // x + y from 12 up to 15
8..15   0..7    1       1 // x + y from 8 up to 15
8..15   8..15   2       2 // x + y from 16 up to 23

Fix this by using (x+y)/8.

Signed-off-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Link: https://msgid.link/r/20240605205315.19132-1-andy.shevchenko@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/base/regmap/regmap-spi.c