> Use GENMASK to define the integration time bitmask. Nack, should be GENMASK(6, 4) I think > Signed-off-by: Hartmut Knaack <knaack.h@xxxxxx> > --- > drivers/iio/light/cm3323.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/light/cm3323.c b/drivers/iio/light/cm3323.c > index 6bb75e7..4f362d5 100644 > --- a/drivers/iio/light/cm3323.c > +++ b/drivers/iio/light/cm3323.c > @@ -29,7 +29,7 @@ > > #define CM3323_CONF_SD_BIT BIT(0) /* sensor disable */ > #define CM3323_CONF_AF_BIT BIT(1) /* auto/manual force mode */ > -#define CM3323_CONF_IT_MASK (BIT(4) | BIT(5) | BIT(6)) > +#define CM3323_CONF_IT_MASK GENMASK(4, 6) > #define CM3323_CONF_IT_SHIFT 4 > > #define CM3323_INT_TIME_AVAILABLE "0.04 0.08 0.16 0.32 0.64 1.28" > -- Peter Meerwald +43-664-2444418 (mobile) -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
