regmap: rbtree: Fix register default look-up in sync
authorLars-Peter Clausen <lars@metafoo.de>
Fri, 23 Mar 2012 10:04:57 +0000 (11:04 +0100)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Sun, 1 Apr 2012 10:47:13 +0000 (11:47 +0100)
The code currently passes the register offset in the current block to
regcache_lookup_reg. This works fine as long as there is only one block and with
base register of 0, but in all other cases it will look-up the default for a
wrong register, which can cause unnecessary register writes. This patch fixes
it by passing the actual register number to regcache_lookup_reg.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: <stable@vger.kernel.org>
drivers/base/regmap/regcache-rbtree.c

index 5157fa04c2f0e04e6120878a97764d4375d52769..fb14a6343d4f52c780d186a31a3a39313dac02be 100644 (file)
@@ -396,7 +396,7 @@ static int regcache_rbtree_sync(struct regmap *map, unsigned int min,
                                                           map->cache_word_size);
 
                        /* Is this the hardware default?  If so skip. */
-                       ret = regcache_lookup_reg(map, i);
+                       ret = regcache_lookup_reg(map, regtmp);
                        if (ret >= 0 && val == map->reg_defaults[ret].def)
                                continue;