Re: [PATCH] regmap: introduce tables forreadable/writeable/volatile/precious checks

From: Mark Brown
Date: Tue Nov 20 2012 - 01:05:08 EST


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.

Attachment: signature.asc
Description: Digital signature