Messages in this thread | | | Subject | Re: [PATCH v5 1/5] iio: adc: ad7949: define and use bitfield names | From | Joe Perches <> | Date | Sun, 08 Aug 2021 09:51:50 -0700 |
| |
On Sat, 2021-08-07 at 21:56 -0400, Liam Beguin wrote: > Replace raw configuration register values by using FIELD_PREP and > defines to improve readability. [] > diff --git a/drivers/iio/adc/ad7949.c b/drivers/iio/adc/ad7949.c [] +#define AD7949_CFG_BIT_INCC GENMASK(12, 10)
I think the naming is a bit confusing as it appears as if these bitfield ranges are single bits.
> +/* REF: reference/buffer selection */ > +#define AD7949_CFG_BIT_REF GENMASK(5, 3) [] > +/* SEQ: channel sequencer. Allows for scanning channels */ > +#define AD7949_CFG_BIT_SEQ GENMASK(2, 1) >
| |