Re: [PATCH] regmap: clarify struct regmap::max_register value some more
On Thu, 11 Jan 2024 08:34:12 +0100, Ahmad Fatoum wrote: > We already have documentation for the member that was taken from Linux, > but still managed to use it wrongly at multiple places that were > recently fixed. To prevent such issues from reoccurring in the future, > expand the documentation a bit. > > Applied, thanks! [1/1] regmap: clarify struct regmap::max_register value some more commit: 36e096eeed5cd1a5f0f8a0930cd5473d4bd5a699 Best regards, -- Sascha Hauer
Re: [PATCH] regmap: clarify struct regmap::max_register value some more
Thanks! Acked-by: Robin van der Gracht On Thu, 11 Jan 2024 08:34:12 +0100 Ahmad Fatoum wrote: > We already have documentation for the member that was taken from Linux, > but still managed to use it wrongly at multiple places that were > recently fixed. To prevent such issues from reoccurring in the future, > expand the documentation a bit. > > Suggested-by: Robin van der Gracht > Signed-off-by: Ahmad Fatoum > --- > include/linux/regmap.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/include/linux/regmap.h b/include/linux/regmap.h > index 690dc3b1dccb..9e38fbc12e16 100644 > --- a/include/linux/regmap.h > +++ b/include/linux/regmap.h > @@ -31,6 +31,9 @@ enum regmap_endian { > * data. > * > * @max_register: Optional, specifies the maximum valid register index. > + * This must be a valid register address and thus a multiple > + * of the register stride returned by regmap_get_reg_stride() > + * after registration. > * > * @read_flag_mask: Mask to be set in the top byte of the register when doing > * a read.
[PATCH] regmap: clarify struct regmap::max_register value some more
We already have documentation for the member that was taken from Linux, but still managed to use it wrongly at multiple places that were recently fixed. To prevent such issues from reoccurring in the future, expand the documentation a bit. Suggested-by: Robin van der Gracht Signed-off-by: Ahmad Fatoum --- include/linux/regmap.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/linux/regmap.h b/include/linux/regmap.h index 690dc3b1dccb..9e38fbc12e16 100644 --- a/include/linux/regmap.h +++ b/include/linux/regmap.h @@ -31,6 +31,9 @@ enum regmap_endian { * data. * * @max_register: Optional, specifies the maximum valid register index. + * This must be a valid register address and thus a multiple + * of the register stride returned by regmap_get_reg_stride() + * after registration. * * @read_flag_mask: Mask to be set in the top byte of the register when doing * a read. -- 2.39.2