On Sun, 19 Nov 2017 12:46:40 -0800 Matt Ranostay <matt.ranostay@xxxxxxxxxxxx> wrote: > On Fri, Oct 27, 2017 at 12:45 PM, Peter Meerwald-Stadler > <pmeerw@xxxxxxxxxx> wrote: > > Table 4 of the datasheet specifies the mode control, these are not > > individual bits; add multi LED mode > > > > Add multi-LED mode and fix MODE_MASK (3 bits wide, not 2) > > Yeah this much cleaner than the macros I had defined > > Acked-by: Matt Ranostay <matt.ranostay@xxxxxxxxxxxx> > Applied. > > > > Signed-off-by: Peter Meerwald-Stadler <pmeerw@xxxxxxxxxx> > > Cc: Matt Ranostay <matt.ranostay@xxxxxxxxxxxx> > > --- > > drivers/iio/health/max30102.c | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c > > index 6ca6182..13d5312 100644 > > --- a/drivers/iio/health/max30102.c > > +++ b/drivers/iio/health/max30102.c > > @@ -60,9 +60,10 @@ > > #define MAX30102_REG_FIFO_CONFIG_AFULL BIT(0) > > > > #define MAX30102_REG_MODE_CONFIG 0x09 > > -#define MAX30102_REG_MODE_CONFIG_MODE_SPO2_EN BIT(0) > > -#define MAX30102_REG_MODE_CONFIG_MODE_HR_EN BIT(1) > > -#define MAX30102_REG_MODE_CONFIG_MODE_MASK 0x03 > > +#define MAX30102_REG_MODE_CONFIG_MODE_HR 0x02 /* red LED */ > > +#define MAX30102_REG_MODE_CONFIG_MODE_HR_SPO2 0x03 /* red + IR LED */ > > +#define MAX30102_REG_MODE_CONFIG_MODE_MULTI 0x07 /* multi-LED mode */ > > +#define MAX30102_REG_MODE_CONFIG_MODE_MASK GENMASK(2, 0) > > #define MAX30102_REG_MODE_CONFIG_PWR BIT(7) > > > > #define MAX30102_REG_SPO2_CONFIG 0x0a > > @@ -287,11 +288,10 @@ static int max30102_chip_init(struct max30102_data *data) > > if (ret) > > return ret; > > > > - /* enable SPO2 mode */ > > + /* enable HR + SPO2 mode */ > > ret = regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG, > > MAX30102_REG_MODE_CONFIG_MODE_MASK, > > - MAX30102_REG_MODE_CONFIG_MODE_HR_EN | > > - MAX30102_REG_MODE_CONFIG_MODE_SPO2_EN); > > + MAX30102_REG_MODE_CONFIG_MODE_HR_SPO2); > > if (ret) > > return ret; > > > > -- > > 2.7.4 > > > -- > 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 -- 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
