Re: [PATCH 2/4] Input: pmic8xxx-keypad - use regmap_field for register access
From: Mark Brown
Date: Wed Oct 08 2014 - 16:04:59 EST
On Wed, Oct 08, 2014 at 11:20:58AM -0700, Stephen Boyd wrote:
> On 10/08/2014 11:13 AM, Dmitry Torokhov wrote:
> >>>Oops. struct regmap_field is opaque. It seems that the allocation
> >>>is the only way that I could have instance of it.
> >>Maybe we can add an API to allocate an array of fields?
> >Maybe we could make the structure public instead? I do not see any
> >reason for allocating something separately that has exactly the same
> >lifetime as owning structure.
The lifetime may be different to that of the register map it references,
consider MFD function devices for example.
> Srini/Mark, any reason why the regmap_field structure is opaque?
So you can't peer into it and rely on the contents. I can see it being
useful to add a bulk allocator.
Attachment:
signature.asc
Description: Digital signature