Re: [PATCH v4 3/6] regmap: Add regmap_noinc_read API

From: Popa, Stefan Serban
Date: Mon Aug 06 2018 - 11:40:06 EST


On Lu, 2018-08-06 at 15:39 +0100, Mark Brown wrote:
> On Mon, Aug 06, 2018 at 03:04:44PM +0300, Stefan Popa wrote:
>
> >
> > + if (!regmap_volatile(map, reg) || !regmap_readable(map, reg))
> > {
> > + ret = -EINVAL;
> > + goto out_unlock;
> > + }
> I suggested having an explicit operation to check if a register supports
> this mode of operation when I reviewed an earlier version, I didn't
> notice a reply on that?

Hi Mark,

Sorry, I got confused about what needs to be done. Should I add a new field
to theÂregmap_config struct which will be checked during the function call?
Something likeÂcan_multi_write? How do you suggest it should be called? Is
readable_noinc_reg ok?

Thank you!
Stefan