regmap: Add missing cache_only checks
authorCharles Keepax <ckeepax@opensource.cirrus.com>
Thu, 1 Jun 2023 10:10:36 +0000 (11:10 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 1 Jun 2023 11:32:53 +0000 (12:32 +0100)
commit99e8dd39f34333d745e6c220be5d166e85214e6c
treed42eb7bac014213e3a19312b5d1b7b715093786c
parent02534c8e967b51940ae7c0cd99befe216f1c2c8d
regmap: Add missing cache_only checks

The current behaviour around cache_only is slightly inconsistent,
most paths will only check cache_only if cache_bypass is false,
and will return -EBUSY if a read attempts to go to the hardware
whilst cache_only is true. However, a couple of paths will not check
cache_only at all.  The most notable of these being regmap_raw_read
which will check cache_only in the case it processes the transaction
one register at a time, but not in the case it handles them as a
block. In the typical case a device has been put into cache_only
whilst powered down this can cause physical reads to happen whilst the
device is unavailable.

Add a check in regmap_raw_read and move the check in regmap_noinc_read,
adding a check for cache_bypass, such that all paths are covered and
consistent.

Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20230601101036.1499612-2-ckeepax@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/base/regmap/regmap.c