regmap: kunit: Fix marking of the range window as volatile

broonie · broonie · commit fabe32cc1eca · 2023-10-26T14:00:42.000+01:00
For some reason the regmap used for testing ranges was not including the end of the range of paged registers as volatile since it found the end by counting from the selector register rather than the base of the window. Signed-off-by: Mark Brown <broonie@kernel.org> Link: https://lore.kernel.org/r/20231023-regmap-test-window-cache-v1-1-d8a71f441968@kernel.org Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/drivers/base/regmap/regmap-kunit.c b/drivers/base/regmap/regmap-kunit.c
@@ -445,7 +445,7 @@ static struct regmap_range_cfg test_range = {
 static bool test_range_volatile(struct device *dev, unsigned int reg)
 {
 	if (reg >= test_range.window_start &&
-	    reg <= test_range.selector_reg + test_range.window_len)
+	    reg <= test_range.window_start + test_range.window_len)
 		return true;
 
 	if (reg >= test_range.range_min && reg <= test_range.range_max)

Original file line number	Diff line number	Diff line change
`@@ -445,7 +445,7 @@ static struct regmap_range_cfg test_range = {`
`445`	`445`	`static bool test_range_volatile(struct device *dev, unsigned int reg)`
`446`	`446`	`{`
`447`	`447`	`if (reg >= test_range.window_start &&`
`448`		`- reg <= test_range.selector_reg + test_range.window_len)`
	`448`	`+ reg <= test_range.window_start + test_range.window_len)`
`449`	`449`	`return true;`
`450`	`450`
`451`	`451`	`if (reg >= test_range.range_min && reg <= test_range.range_max)`