| 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
| ||
|
Message-ID: <20121120084135.GC10665@mail.gnudd.com> Date: Tue, 20 Nov 2012 09:41:35 +0100 From: Davide Ciminaghi <ciminaghi@...dd.com> To: Mark Brown <broonie@...nsource.wolfsonmicro.com> Cc: gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] regmap: introduce tables for readable/writeable/volatile/precious checks On Tue, Nov 20, 2012 at 03:05:11PM +0900, Mark Brown wrote: > On Mon, Nov 19, 2012 at 02:31:53PM +0100, Davide Ciminaghi wrote: > > Many of the regmap enabled drivers implementing one or more of the > > readable, writeable, volatile and precious methods use the same code > > pattern: > > > > return ((reg >= X && reg <= Y) || (reg >= W && reg <= Z) || ...) > > > > Switch to a data driven approach, using tables to describe > > readable/writeable/volatile and precious registers ranges instead. > > The table based check can still be overridden by passing the usual function > > pointers via struct regmap_config. > > Sorry, the code looks good but I just realised we already have a feature > called ranges - the paged window support also calls the regions it uses > ranges. Can I suggest access_table or similar instead? Sorry about > that - I should've realised. ok no problem, I'll change the name and resend. Thanks and regards Davide -- 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