Re: [PATCH 6/6 v5] regmap: Incorporate the regcache core into regmap

Mark Brown
2011-09-19T16:36:26Z

On Mon, Sep 19, 2011 at 06:28:31PM +0200, Lars-Peter Clausen wrote:

Please delete unneeded context from mails and configure your MUA to wrap
early enough to allow for at least one level of quoting.

> On 09/19/2011 06:13 PM, Mark Brown wrote:
> > On Mon, Sep 19, 2011 at 05:57:49PM +0200, Lars-Peter Clausen wrote:

> > if (!map->cache_bypass) {
> > ret = regcache_write(map, reg, val);
> > - if (!ret || map->cache_only)
> > + if (ret != 0)
> > + return ret;

> Hm... last time we said we want to fallback to hw read/write even if the cache
> operation has failed. The issue is that regcache_write will check for
> regmap_writeable, so you'll get different behaviour if caching is enabled for
> registers where regmap_writeable returns false.

I'd expect regcache_write() to silently eat such updates, though I see
it doesn't at the minute.

> >> Btw. what should happen if both cache_bypass and cache_only are set? Or is that
> >> an invalid configuration?

> > That's not sensible. Probably BUG_ON().

> BUG_ON might be a bit to strict, if we wanted to allow cache_only to be enabled
> through debugfs like it is done for ASoC right now.

If we're going to allow those to be writable (I'm not convinced that's
sane) then it seems better that the write should enforce basic sanity.