Commit 1c79771a authored by Kevin Cernekee's avatar Kevin Cernekee Committed by Mark Brown

regmap: Use regcache_mark_dirty() to indicate power loss or reset

Existing regmap users call regcache_mark_dirty() as part of the
suspend/resume sequence, to tell regcache that non-default values need to
be resynced post-resume.  Add an internal "no_sync_defaults" regmap flag
to remember this state, so that regcache_sync() can differentiate between
these two cases:

1) HW was reset, so any cache values that match map->reg_defaults can be
safely skipped.  On some chips there are a lot of registers in the
reg_defaults list, so this optimization speeds things up quite a bit.

2) HW was not reset (maybe it was just clock-gated), so if we cached
any writes, they should be sent to the hardware regardless of whether
they match the HW default.  Currently this will write out all values in
the regcache, since we don't maintain per-register dirty bits.
Suggested-by: default avatarMark Brown <broonie@kernel.org>
Signed-off-by: default avatarKevin Cernekee <cernekee@chromium.org>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 3969fa08
...@@ -131,7 +131,10 @@ struct regmap { ...@@ -131,7 +131,10 @@ struct regmap {
struct reg_default *reg_defaults; struct reg_default *reg_defaults;
const void *reg_defaults_raw; const void *reg_defaults_raw;
void *cache; void *cache;
/* if set, the cache contains newer data than the HW */
u32 cache_dirty; u32 cache_dirty;
/* if set, the HW registers are known to match map->reg_defaults */
bool no_sync_defaults;
struct reg_default *patch; struct reg_default *patch;
int patch_regs; int patch_regs;
......
...@@ -254,6 +254,10 @@ static bool regcache_reg_needs_sync(struct regmap *map, unsigned int reg, ...@@ -254,6 +254,10 @@ static bool regcache_reg_needs_sync(struct regmap *map, unsigned int reg,
{ {
int ret; int ret;
/* If we don't know the chip just got reset, then sync everything. */
if (!map->no_sync_defaults)
return true;
/* Is this the hardware default? If so skip. */ /* Is this the hardware default? If so skip. */
ret = regcache_lookup_reg(map, reg); ret = regcache_lookup_reg(map, reg);
if (ret >= 0 && val == map->reg_defaults[ret].def) if (ret >= 0 && val == map->reg_defaults[ret].def)
...@@ -352,6 +356,7 @@ int regcache_sync(struct regmap *map) ...@@ -352,6 +356,7 @@ int regcache_sync(struct regmap *map)
/* Restore the bypass state */ /* Restore the bypass state */
map->async = false; map->async = false;
map->cache_bypass = bypass; map->cache_bypass = bypass;
map->no_sync_defaults = false;
map->unlock(map->lock_arg); map->unlock(map->lock_arg);
regmap_async_complete(map); regmap_async_complete(map);
...@@ -407,6 +412,7 @@ int regcache_sync_region(struct regmap *map, unsigned int min, ...@@ -407,6 +412,7 @@ int regcache_sync_region(struct regmap *map, unsigned int min,
/* Restore the bypass state */ /* Restore the bypass state */
map->cache_bypass = bypass; map->cache_bypass = bypass;
map->async = false; map->async = false;
map->no_sync_defaults = false;
map->unlock(map->lock_arg); map->unlock(map->lock_arg);
regmap_async_complete(map); regmap_async_complete(map);
...@@ -471,18 +477,23 @@ void regcache_cache_only(struct regmap *map, bool enable) ...@@ -471,18 +477,23 @@ void regcache_cache_only(struct regmap *map, bool enable)
EXPORT_SYMBOL_GPL(regcache_cache_only); EXPORT_SYMBOL_GPL(regcache_cache_only);
/** /**
* regcache_mark_dirty: Mark the register cache as dirty * regcache_mark_dirty: Indicate that HW registers were reset to default values
* *
* @map: map to mark * @map: map to mark
* *
* Mark the register cache as dirty, for example due to the device * Inform regcache that the device has been powered down or reset, so that
* having been powered down for suspend. If the cache is not marked * on resume, regcache_sync() knows to write out all non-default values
* as dirty then the cache sync will be suppressed. * stored in the cache.
*
* If this function is not called, regcache_sync() will assume that
* the hardware state still matches the cache state, modulo any writes that
* happened when cache_only was true.
*/ */
void regcache_mark_dirty(struct regmap *map) void regcache_mark_dirty(struct regmap *map)
{ {
map->lock(map->lock_arg); map->lock(map->lock_arg);
map->cache_dirty = true; map->cache_dirty = true;
map->no_sync_defaults = true;
map->unlock(map->lock_arg); map->unlock(map->lock_arg);
} }
EXPORT_SYMBOL_GPL(regcache_mark_dirty); EXPORT_SYMBOL_GPL(regcache_mark_dirty);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment