diff options
author | Nikita Yushchenko <nikita.yoush@cogentembedded.com> | 2016-09-22 12:02:25 +0300 |
---|---|---|
committer | Huang, Tao <huangtao@rock-chips.com> | 2016-09-23 10:05:03 +0800 |
commit | 756d9ad54889396db40c1bca2ece8b9c07f0888d (patch) | |
tree | d8f3cd8887d0f85700bbfc6b1be21d23097075a8 /drivers | |
parent | 60a514699d9c3a99392d1325dff79b112f48ddaf (diff) |
UPSTREAM: regmap: fix deadlock on _regmap_raw_write() error path
Commit 815806e39bf6 ("regmap: drop cache if the bus transfer error")
added a call to regcache_drop_region() to error path in
_regmap_raw_write(). However that path runs with regmap lock taken,
and regcache_drop_region() tries to re-take it, causing a deadlock.
Fix that by calling map->cache_ops->drop() directly.
Change-Id: I55c6d3ed490c47e8b3f5ca774d051a700f707b6e
Signed-off-by: Nikita Yushchenko <nikita.yoush@cogentembedded.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Huang, Tao <huangtao@rock-chips.com>
(cherry picked from git.kernel.org broonie/regmap.git for-next
commit f0aa1ce6259eb65f53f969b3250c1d0aac84f30b)
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/base/regmap/regmap.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c index bb216c9d3c09..d94831a45b74 100644 --- a/drivers/base/regmap/regmap.c +++ b/drivers/base/regmap/regmap.c @@ -1373,7 +1373,11 @@ int _regmap_raw_write(struct regmap *map, unsigned int reg, kfree(buf); } else if (ret != 0 && !map->cache_bypass && map->format.parse_val) { - regcache_drop_region(map, reg, reg + 1); + /* regcache_drop_region() takes lock that we already have, + * thus call map->cache_ops->drop() directly + */ + if (map->cache_ops && map->cache_ops->drop) + map->cache_ops->drop(map, reg, reg + 1); } trace_regmap_hw_write_done(map, reg, val_len / map->format.val_bytes); |