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.
signature.asc
Description: Digital signature
