On Mon, Jun 10, 2013 at 10:21:58AM +0100, Srinivas KANDAGATLA wrote:
> It is common to access regmap registers at bit level, using
> regmap_update_bits or regmap_read functions, however the end user has to
> take care of a mask or shifting. This becomes overhead when such use
> cases are high. Having a common function to do this is much convenient
> and less error prone.
So this looks basically good. A couple of smallish nits below, if you
could respin I'll apply this on a branch so it can be pulled in as
dependency for the other things that use it.
> + regmap_field_init(rm_field, regmap, reg_field);
> +
> + return rm_field;
> +
> +}
> +EXPORT_SYMBOL_GPL(devm_regmap_field_alloc);
> +/**
> + * regmap_field_alloc(): Allocate and initialise a register field
Needs a blank line between the two functions.
> +#include <linux/device.h>
> +#include <linux/slab.h>
> +static inline void regmap_field_free(struct regmap_field *field)
> +{
> + kfree(field);
> +}
> +static inline void devm_regmap_field_free(struct device *dev,
> + struct regmap_field *field)
> +{
> + devm_kfree(dev, field);
> +}
Probably not worth inlining these, just put them in the code.
Attachment:
signature.asc
Description: Digital signature
