lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 5 Sep 2011 18:27:34 -0700 From: Mark Brown <broonie@...nsource.wolfsonmicro.com> To: Lars-Peter Clausen <lars@...afoo.de> Cc: Dimitris Papastamos <dp@...nsource.wolfsonmicro.com>, linux-kernel@...r.kernel.org, Liam Girdwood <lrg@...com>, Graeme Gregory <gg@...mlogic.co.uk>, Samuel Oritz <sameo@...ux.intel.com> Subject: Re: [PATCH 6/6] regmap: Incorporate the regcache core into regmap On Mon, Sep 05, 2011 at 06:22:36PM +0200, Lars-Peter Clausen wrote: > On 09/05/2011 03:51 PM, Dimitris Papastamos wrote: > > +#ifdef CONFIG_REGCACHE > > + if (!map->cache_bypass) { > > + ret = regcache_read(map, reg, val); > > + if (!ret) { > > + mutex_unlock(&map->lock); > > + return 0; > > + } > > + } > I think this will need tighter integration. For example do we want to fallback > to a hw read when the cached read fails? And also regcache_read will return for > REGCACHE_NONE, which means we neither do a cached read nor a hw read. I think falling through to hardware is sensible - if nothing else we've got to do that for uncached registers. Though if we're in cache only mode we probably don't want to do that as the hardware may be powered off. But in general it does make sense to me. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists