regmap: cache: set max_register with reg_stride

jtpark-sec · broonie · commit d640947562ce · 2022-04-25T14:00:44.000+01:00
Current logic does not consider multi-stride cases, the max_register have to calculate with reg_stride because it is a kind of address range. Signed-off-by: Jeongtae Park <jtp.park@samsung.com> Link: https://lore.kernel.org/r/20220425114613.15934-1-jtp.park@samsung.com Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/drivers/base/regmap/regcache.c b/drivers/base/regmap/regcache.c
@@ -183,8 +183,8 @@ int regcache_init(struct regmap *map, const struct regmap_config *config)
 			return 0;
 	}
 
-	if (!map->max_register)
-		map->max_register = map->num_reg_defaults_raw;
+	if (!map->max_register && map->num_reg_defaults_raw)
+		map->max_register = (map->num_reg_defaults_raw  - 1) * map->reg_stride;
 
 	if (map->cache_ops->init) {
 		dev_dbg(map->dev, "Initializing %s cache\n",

Original file line number	Diff line number	Diff line change
`@@ -183,8 +183,8 @@ int regcache_init(struct regmap map, const struct regmap_config config)`
`183`	`183`	`return 0;`
`184`	`184`	`}`
`185`	`185`
`186`		`- if (!map->max_register)`
`187`		`- map->max_register = map->num_reg_defaults_raw;`
	`186`	`+ if (!map->max_register && map->num_reg_defaults_raw)`
	`187`	`+ map->max_register = (map->num_reg_defaults_raw - 1) * map->reg_stride;`
`188`	`188`
`189`	`189`	`if (map->cache_ops->init) {`
`190`	`190`	`dev_dbg(map->dev, "Initializing %s cache\n",`