* [PATCH v2 0/2] iio: health: add MAX30102 oximeter driver support
@ 2017-01-25 7:04 Matt Ranostay
[not found] ` <20170125070442.10715-1-matt-sk+viVC6FLCDq+mSdOJa79kegs52MxvZ@public.gmane.org>
2017-01-25 7:04 ` [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
0 siblings, 2 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-25 7:04 UTC (permalink / raw)
To: linux-iio; +Cc: jic23, Matt Ranostay
Changes from v1:
* add newlines to dev_err/dev_info statements
* update documentation to 400hz samples, but actually 100hz to FIFO
* fix storage bits to a power of 2, and the required buffer processing
* fixed return code from max30102_read_measurement()
Matt Ranostay (2):
devicetree: add documentation for MAX30102 oximeter
iio: health: add MAX30102 oximeter driver support
.../devicetree/bindings/iio/health/max30102.txt | 30 ++
drivers/iio/health/Kconfig | 13 +
drivers/iio/health/Makefile | 1 +
drivers/iio/health/max30102.c | 485 +++++++++++++++++++++
4 files changed, 529 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/health/max30102.txt
create mode 100644 drivers/iio/health/max30102.c
--
2.10.2
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2 1/2] devicetree: add documentation for MAX30102 oximeter
2017-01-25 7:04 [PATCH v2 0/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
@ 2017-01-25 7:04 ` Matt Ranostay
2017-01-25 7:04 ` [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
1 sibling, 0 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-25 7:04 UTC (permalink / raw)
To: linux-iio-u79uwXL29TY76Z2rM5mHXA
Cc: jic23-DgEjT+Ai2ygdnm+yROfE0A, Matt Ranostay,
devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Signed-off-by: Matt Ranostay <matt-sk+viVC6FLCDq+mSdOJa79kegs52MxvZ@public.gmane.org>
---
.../devicetree/bindings/iio/health/max30102.txt | 30 ++++++++++++++++++++++
1 file changed, 30 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/health/max30102.txt
diff --git a/Documentation/devicetree/bindings/iio/health/max30102.txt b/Documentation/devicetree/bindings/iio/health/max30102.txt
new file mode 100644
index 000000000000..c93d1bb25597
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/health/max30102.txt
@@ -0,0 +1,30 @@
+Maxim MAX30102 heart rate and pulse oximeter sensor
+
+* https://datasheets.maximintegrated.com/en/ds/MAX30102.pdf
+
+Required properties:
+ - compatible: must be "maxim,max30102"
+ - reg: the I2C address of the sensor
+ - interrupt-parent: should be the phandle for the interrupt controller
+ - interrupts: the sole interrupt generated by the device
+
+ Refer to interrupt-controller/interrupts.txt for generic
+ interrupt client node bindings.
+
+Optional properties:
+ - maxim,red-led-current-microamp: configuration for RED LED current
+ - maxim,ir-led-current-microamp: configuration for IR LED current
+
+ Note that each step is approximately 200 microamps, ranging from 0 uA to
+ 50800 uA.
+
+Example:
+
+max30100@57 {
+ compatible = "maxim,max30102";
+ reg = <57>;
+ maxim,red-led-current-microamp = <7000>;
+ maxim,ir-led-current-microamp = <7000>;
+ interrupt-parent = <&gpio1>;
+ interrupts = <16 2>;
+};
--
2.10.2
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 1/2] devicetree: add documentation for MAX30102 oximeter
@ 2017-01-25 7:04 ` Matt Ranostay
0 siblings, 0 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-25 7:04 UTC (permalink / raw)
To: linux-iio; +Cc: jic23, Matt Ranostay, devicetree
Cc: devicetree@vger.kernel.org
Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
---
.../devicetree/bindings/iio/health/max30102.txt | 30 ++++++++++++++++++++++
1 file changed, 30 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/health/max30102.txt
diff --git a/Documentation/devicetree/bindings/iio/health/max30102.txt b/Documentation/devicetree/bindings/iio/health/max30102.txt
new file mode 100644
index 000000000000..c93d1bb25597
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/health/max30102.txt
@@ -0,0 +1,30 @@
+Maxim MAX30102 heart rate and pulse oximeter sensor
+
+* https://datasheets.maximintegrated.com/en/ds/MAX30102.pdf
+
+Required properties:
+ - compatible: must be "maxim,max30102"
+ - reg: the I2C address of the sensor
+ - interrupt-parent: should be the phandle for the interrupt controller
+ - interrupts: the sole interrupt generated by the device
+
+ Refer to interrupt-controller/interrupts.txt for generic
+ interrupt client node bindings.
+
+Optional properties:
+ - maxim,red-led-current-microamp: configuration for RED LED current
+ - maxim,ir-led-current-microamp: configuration for IR LED current
+
+ Note that each step is approximately 200 microamps, ranging from 0 uA to
+ 50800 uA.
+
+Example:
+
+max30100@57 {
+ compatible = "maxim,max30102";
+ reg = <57>;
+ maxim,red-led-current-microamp = <7000>;
+ maxim,ir-led-current-microamp = <7000>;
+ interrupt-parent = <&gpio1>;
+ interrupts = <16 2>;
+};
--
2.10.2
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support
2017-01-25 7:04 [PATCH v2 0/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
[not found] ` <20170125070442.10715-1-matt-sk+viVC6FLCDq+mSdOJa79kegs52MxvZ@public.gmane.org>
@ 2017-01-25 7:04 ` Matt Ranostay
2017-01-25 7:29 ` Peter Meerwald-Stadler
` (2 more replies)
1 sibling, 3 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-25 7:04 UTC (permalink / raw)
To: linux-iio; +Cc: jic23, Matt Ranostay
MAX30102 is an heart rate and pulse oximeter sensor that works using
two LEDS of different wavelengths, and detecting the light reflected
back.
This patchset adds support for both IR and RED LED channels which can
be processed in userspace to determine heart rate and blood oxygen
levels. The MAX30102 part isn't completely register and functional
compatible with the existing MAX30100 driver.
Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
---
drivers/iio/health/Kconfig | 13 ++
drivers/iio/health/Makefile | 1 +
drivers/iio/health/max30102.c | 485 ++++++++++++++++++++++++++++++++++++++++++
3 files changed, 499 insertions(+)
create mode 100644 drivers/iio/health/max30102.c
diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig
index c5f004a8e447..a2ecb4c94c2a 100644
--- a/drivers/iio/health/Kconfig
+++ b/drivers/iio/health/Kconfig
@@ -46,6 +46,19 @@ config MAX30100
To compile this driver as a module, choose M here: the
module will be called max30100.
+config MAX30102
+ tristate "MAX30102 heart rate and pulse oximeter sensor"
+ depends on I2C
+ select REGMAP_I2C
+ select IIO_BUFFER
+ select IIO_KFIFO_BUF
+ help
+ Say Y here to build I2C interface support for the Maxim
+ MAX30102 heart rate, and pulse oximeter sensor.
+
+ To compile this driver as a module, choose M here: the
+ module will be called max30102.
+
endmenu
endmenu
diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile
index 9955a2ae8df1..3558f9d64954 100644
--- a/drivers/iio/health/Makefile
+++ b/drivers/iio/health/Makefile
@@ -7,3 +7,4 @@
obj-$(CONFIG_AFE4403) += afe4403.o
obj-$(CONFIG_AFE4404) += afe4404.o
obj-$(CONFIG_MAX30100) += max30100.o
+obj-$(CONFIG_MAX30102) += max30102.o
diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c
new file mode 100644
index 000000000000..87df1e6d712f
--- /dev/null
+++ b/drivers/iio/health/max30102.c
@@ -0,0 +1,485 @@
+/*
+ * max30102.c - Support for MAX30102 heart rate and pulse oximeter sensor
+ *
+ * Copyright (C) 2017 Matt Ranostay <matt@ranostay.consulting>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * TODO: proximity power saving feature
+ */
+
+#include <linux/module.h>
+#include <linux/init.h>
+#include <linux/interrupt.h>
+#include <linux/delay.h>
+#include <linux/err.h>
+#include <linux/irq.h>
+#include <linux/i2c.h>
+#include <linux/mutex.h>
+#include <linux/of.h>
+#include <linux/regmap.h>
+#include <linux/iio/iio.h>
+#include <linux/iio/buffer.h>
+#include <linux/iio/kfifo_buf.h>
+
+#define MAX30102_REGMAP_NAME "max30102_regmap"
+#define MAX30102_DRV_NAME "max30102"
+
+#define MAX30102_REG_INT_STATUS 0x00
+#define MAX30102_REG_INT_STATUS_PWR_RDY BIT(0)
+#define MAX30102_REG_INT_STATUS_PROX_INT BIT(4)
+#define MAX30102_REG_INT_STATUS_ALC_OVF BIT(5)
+#define MAX30102_REG_INT_STATUS_PPG_RDY BIT(6)
+#define MAX30102_REG_INT_STATUS_FIFO_RDY BIT(7)
+
+#define MAX30102_REG_INT_ENABLE 0x02
+#define MAX30102_REG_INT_ENABLE_PROX_INT_EN BIT(4)
+#define MAX30102_REG_INT_ENABLE_ALC_OVF_EN BIT(5)
+#define MAX30102_REG_INT_ENABLE_PPG_EN BIT(6)
+#define MAX30102_REG_INT_ENABLE_FIFO_EN BIT(7)
+#define MAX30102_REG_INT_ENABLE_MASK 0xf0
+#define MAX30102_REG_INT_ENABLE_MASK_SHIFT 4
+
+#define MAX30102_REG_FIFO_WR_PTR 0x04
+#define MAX30102_REG_FIFO_OVR_CTR 0x05
+#define MAX30102_REG_FIFO_RD_PTR 0x06
+#define MAX30102_REG_FIFO_DATA 0x07
+#define MAX30102_REG_FIFO_DATA_ENTRY_LEN 6
+
+#define MAX30102_REG_FIFO_CONFIG 0x08
+#define MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES BIT(1)
+#define MAX30102_REG_FIFO_CONFIG_AVG_SHIFT 5
+#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_PWR BIT(7)
+
+#define MAX30102_REG_SPO2_CONFIG 0x0a
+#define MAX30102_REG_SPO2_CONFIG_PULSE_411_US 0x03
+#define MAX30102_REG_SPO2_CONFIG_SR_400HZ 0x03
+#define MAX30102_REG_SPO2_CONFIG_SR_MASK 0x07
+#define MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT 2
+#define MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS BIT(0)
+#define MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT 4
+
+#define MAX30102_REG_RED_LED_CONFIG 0x0c
+#define MAX30102_REG_IR_LED_CONFIG 0x0d
+
+#define MAX30102_REG_TEMP_CONFIG 0x21
+#define MAX30102_REG_TEMP_CONFIG_TEMP_EN BIT(0)
+
+#define MAX30102_REG_TEMP_INTEGER 0x1f
+#define MAX30102_REG_TEMP_FRACTION 0x20
+
+struct max30102_data {
+ struct i2c_client *client;
+ struct iio_dev *indio_dev;
+ struct mutex lock;
+ struct regmap *regmap;
+
+ u8 buffer[8];
+ __be32 processed_buffer[2]; /* 2 x 18-bit (padded to 32-bits) */
+};
+
+static const struct regmap_config max30102_regmap_config = {
+ .name = MAX30102_REGMAP_NAME,
+
+ .reg_bits = 8,
+ .val_bits = 8,
+};
+
+static const unsigned long max30102_scan_masks[] = {0x3, 0};
+
+static const struct iio_chan_spec max30102_channels[] = {
+ {
+ .type = IIO_INTENSITY,
+ .channel2 = IIO_MOD_LIGHT_RED,
+ .modified = 1,
+
+ .scan_index = 0,
+ .scan_type = {
+ .sign = 'u',
+ .shift = 14,
+ .realbits = 18,
+ .storagebits = 32,
+ .endianness = IIO_BE,
+ },
+ },
+ {
+ .type = IIO_INTENSITY,
+ .channel2 = IIO_MOD_LIGHT_IR,
+ .modified = 1,
+
+ .scan_index = 1,
+ .scan_type = {
+ .sign = 'u',
+ .shift = 14,
+ .realbits = 18,
+ .storagebits = 32,
+ .endianness = IIO_BE,
+ },
+ },
+ {
+ .type = IIO_TEMP,
+ .info_mask_separate =
+ BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
+ .scan_index = -1,
+ },
+};
+
+static int max30102_set_powermode(struct max30102_data *data, bool state)
+{
+ return regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG,
+ MAX30102_REG_MODE_CONFIG_PWR,
+ state ? 0 : MAX30102_REG_MODE_CONFIG_PWR);
+}
+
+static int max30102_buffer_postenable(struct iio_dev *indio_dev)
+{
+ struct max30102_data *data = iio_priv(indio_dev);
+
+ return max30102_set_powermode(data, true);
+}
+
+static int max30102_buffer_predisable(struct iio_dev *indio_dev)
+{
+ struct max30102_data *data = iio_priv(indio_dev);
+
+ return max30102_set_powermode(data, false);
+}
+
+static const struct iio_buffer_setup_ops max30102_buffer_setup_ops = {
+ .postenable = max30102_buffer_postenable,
+ .predisable = max30102_buffer_predisable,
+};
+
+static inline int max30102_fifo_count(struct max30102_data *data)
+{
+ unsigned int val;
+ int ret;
+
+ ret = regmap_read(data->regmap, MAX30102_REG_INT_STATUS, &val);
+ if (ret)
+ return ret;
+
+ /* FIFO has one sample slot left */
+ if (val & MAX30102_REG_INT_STATUS_FIFO_RDY)
+ return 1;
+
+ return 0;
+}
+
+static int max30102_read_measurement(struct max30102_data *data)
+{
+ int ret;
+ u8 *buffer = (u8 *) &data->buffer;
+
+ ret = i2c_smbus_read_i2c_block_data(data->client,
+ MAX30102_REG_FIFO_DATA,
+ MAX30102_REG_FIFO_DATA_ENTRY_LEN,
+ buffer);
+
+ memcpy(&data->processed_buffer[0], &buffer[0], 3);
+ memcpy(&data->processed_buffer[1], &buffer[3], 3);
+
+ return (ret == MAX30102_REG_FIFO_DATA_ENTRY_LEN) ? 0 : -EINVAL;
+}
+
+static irqreturn_t max30102_interrupt_handler(int irq, void *private)
+{
+ struct iio_dev *indio_dev = private;
+ struct max30102_data *data = iio_priv(indio_dev);
+ int ret, cnt = 0;
+
+ mutex_lock(&data->lock);
+
+ while (cnt || (cnt = max30102_fifo_count(data)) > 0) {
+ ret = max30102_read_measurement(data);
+ if (ret)
+ break;
+
+ iio_push_to_buffers(data->indio_dev, data->processed_buffer);
+ cnt--;
+ }
+
+ mutex_unlock(&data->lock);
+
+ return IRQ_HANDLED;
+}
+
+static int max30102_get_current_idx(unsigned int val, int *reg)
+{
+ /* each step is 0.200 mA */
+ *reg = val / 200;
+
+ return *reg > 0xff ? -EINVAL : 0;
+}
+
+static int max30102_led_init(struct max30102_data *data)
+{
+ struct device *dev = &data->client->dev;
+ struct device_node *np = dev->of_node;
+ unsigned int val;
+ int reg, ret;
+
+ ret = of_property_read_u32(np, "maxim,red-led-current-microamp", &val);
+ if (ret) {
+ dev_info(dev, "no red-led-current-microamp set\n");
+
+ /* Default to 7 mA RED LED */
+ val = 7000;
+ }
+
+ ret = max30102_get_current_idx(val, ®);
+ if (ret) {
+ dev_err(dev, "invalid RED LED current setting %d\n", val);
+ return ret;
+ }
+
+ ret = regmap_write(data->regmap, MAX30102_REG_RED_LED_CONFIG, reg);
+ if (ret)
+ return ret;
+
+ ret = of_property_read_u32(np, "maxim,ir-led-current-microamp", &val);
+ if (ret) {
+ dev_info(dev, "no ir-led-current-microamp set\n");
+
+ /* Default to 7 mA IR LED */
+ val = 7000;
+ }
+
+ ret = max30102_get_current_idx(val, ®);
+ if (ret) {
+ dev_err(dev, "invalid IR LED current setting %d", val);
+ return ret;
+ }
+
+ return regmap_write(data->regmap, MAX30102_REG_IR_LED_CONFIG, reg);
+}
+
+static int max30102_chip_init(struct max30102_data *data)
+{
+ int ret;
+
+ /* setup LED current settings */
+ ret = max30102_led_init(data);
+ if (ret)
+ return ret;
+
+ /* enable 18-bit HR + SPO2 readings at 400Hz */
+ ret = regmap_write(data->regmap, MAX30102_REG_SPO2_CONFIG,
+ (MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS
+ << MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT) |
+ (MAX30102_REG_SPO2_CONFIG_SR_400HZ
+ << MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT) |
+ MAX30102_REG_SPO2_CONFIG_PULSE_411_US);
+ if (ret)
+ return ret;
+
+ /* enable 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);
+ if (ret)
+ return ret;
+
+ /* average 4 samples + generate FIFO interrupt */
+ ret = regmap_write(data->regmap, MAX30102_REG_FIFO_CONFIG,
+ (MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES
+ << MAX30102_REG_FIFO_CONFIG_AVG_SHIFT) |
+ MAX30102_REG_FIFO_CONFIG_AFULL);
+ if (ret)
+ return ret;
+
+ /* enable FIFO interrupt */
+ return regmap_update_bits(data->regmap, MAX30102_REG_INT_ENABLE,
+ MAX30102_REG_INT_ENABLE_MASK,
+ MAX30102_REG_INT_ENABLE_FIFO_EN);
+}
+
+static int max30102_read_temp(struct max30102_data *data, int *val)
+{
+ int ret;
+ unsigned int reg;
+
+ ret = regmap_read(data->regmap, MAX30102_REG_TEMP_INTEGER, ®);
+ if (ret < 0)
+ return ret;
+ *val = reg << 4;
+
+ ret = regmap_read(data->regmap, MAX30102_REG_TEMP_FRACTION, ®);
+ if (ret < 0)
+ return ret;
+
+ *val |= reg & 0xf;
+ *val = sign_extend32(*val, 11);
+
+ return 0;
+}
+
+static int max30102_get_temp(struct max30102_data *data, int *val)
+{
+ int ret;
+
+ /* start acquisition */
+ ret = regmap_update_bits(data->regmap, MAX30102_REG_TEMP_CONFIG,
+ MAX30102_REG_TEMP_CONFIG_TEMP_EN,
+ MAX30102_REG_TEMP_CONFIG_TEMP_EN);
+ if (ret)
+ return ret;
+
+ msleep(35);
+
+ return max30102_read_temp(data, val);
+}
+
+static int max30102_read_raw(struct iio_dev *indio_dev,
+ struct iio_chan_spec const *chan,
+ int *val, int *val2, long mask)
+{
+ struct max30102_data *data = iio_priv(indio_dev);
+ int ret = -EINVAL;
+
+ switch (mask) {
+ case IIO_CHAN_INFO_RAW:
+ /*
+ * Temperature reading can only be acquired while engine
+ * is running
+ */
+ mutex_lock(&indio_dev->mlock);
+
+ if (!iio_buffer_enabled(indio_dev))
+ ret = -EAGAIN;
+ else {
+ ret = max30102_get_temp(data, val);
+ if (!ret)
+ ret = IIO_VAL_INT;
+ }
+
+ mutex_unlock(&indio_dev->mlock);
+ break;
+ case IIO_CHAN_INFO_SCALE:
+ *val = 1; /* 0.0625 */
+ *val2 = 16;
+ ret = IIO_VAL_FRACTIONAL;
+ break;
+ }
+
+ return ret;
+}
+
+static const struct iio_info max30102_info = {
+ .driver_module = THIS_MODULE,
+ .read_raw = max30102_read_raw,
+};
+
+static int max30102_probe(struct i2c_client *client,
+ const struct i2c_device_id *id)
+{
+ struct max30102_data *data;
+ struct iio_buffer *buffer;
+ struct iio_dev *indio_dev;
+ int ret;
+
+ indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
+ if (!indio_dev)
+ return -ENOMEM;
+
+ buffer = devm_iio_kfifo_allocate(&client->dev);
+ if (!buffer)
+ return -ENOMEM;
+
+ iio_device_attach_buffer(indio_dev, buffer);
+
+ indio_dev->name = MAX30102_DRV_NAME;
+ indio_dev->channels = max30102_channels;
+ indio_dev->info = &max30102_info;
+ indio_dev->num_channels = ARRAY_SIZE(max30102_channels);
+ indio_dev->available_scan_masks = max30102_scan_masks;
+ indio_dev->modes = (INDIO_BUFFER_SOFTWARE | INDIO_DIRECT_MODE);
+ indio_dev->setup_ops = &max30102_buffer_setup_ops;
+
+ data = iio_priv(indio_dev);
+ data->indio_dev = indio_dev;
+ data->client = client;
+
+ mutex_init(&data->lock);
+ i2c_set_clientdata(client, indio_dev);
+
+ data->regmap = devm_regmap_init_i2c(client, &max30102_regmap_config);
+ if (IS_ERR(data->regmap)) {
+ dev_err(&client->dev, "regmap initialization failed.\n");
+ return PTR_ERR(data->regmap);
+ }
+ max30102_set_powermode(data, false);
+
+ ret = max30102_chip_init(data);
+ if (ret)
+ return ret;
+
+ if (client->irq <= 0) {
+ dev_err(&client->dev, "no valid irq defined\n");
+ return -EINVAL;
+ }
+
+ ret = devm_request_threaded_irq(&client->dev, client->irq,
+ NULL, max30102_interrupt_handler,
+ IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
+ "max30102_irq", indio_dev);
+ if (ret) {
+ dev_err(&client->dev, "request irq (%d) failed\n", client->irq);
+ return ret;
+ }
+
+ return iio_device_register(indio_dev);
+}
+
+static int max30102_remove(struct i2c_client *client)
+{
+ struct iio_dev *indio_dev = i2c_get_clientdata(client);
+ struct max30102_data *data = iio_priv(indio_dev);
+
+ iio_device_unregister(indio_dev);
+ max30102_set_powermode(data, false);
+
+ return 0;
+}
+
+static const struct i2c_device_id max30102_id[] = {
+ { "max30102", 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(i2c, max30102_id);
+
+static const struct of_device_id max30102_dt_ids[] = {
+ { .compatible = "maxim,max30102" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, max30102_dt_ids);
+
+static struct i2c_driver max30102_driver = {
+ .driver = {
+ .name = MAX30102_DRV_NAME,
+ .of_match_table = of_match_ptr(max30102_dt_ids),
+ },
+ .probe = max30102_probe,
+ .remove = max30102_remove,
+ .id_table = max30102_id,
+};
+module_i2c_driver(max30102_driver);
+
+MODULE_AUTHOR("Matt Ranostay <matt@ranostay.consulting>");
+MODULE_DESCRIPTION("MAX30102 heart rate and pulse oximeter sensor");
+MODULE_LICENSE("GPL");
--
2.10.2
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support
2017-01-25 7:04 ` [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
@ 2017-01-25 7:29 ` Peter Meerwald-Stadler
2017-01-25 8:24 ` Matt Ranostay
2017-01-25 9:43 ` Matt Ranostay
2017-01-27 7:11 ` Matt Ranostay
2017-01-28 11:51 ` Jonathan Cameron
2 siblings, 2 replies; 12+ messages in thread
From: Peter Meerwald-Stadler @ 2017-01-25 7:29 UTC (permalink / raw)
To: Matt Ranostay; +Cc: linux-iio, jic23
On Tue, 24 Jan 2017, Matt Ranostay wrote:
> MAX30102 is an heart rate and pulse oximeter sensor that works using
> two LEDS of different wavelengths, and detecting the light reflected
> back.
>
> This patchset adds support for both IR and RED LED channels which can
> be processed in userspace to determine heart rate and blood oxygen
> levels. The MAX30102 part isn't completely register and functional
> compatible with the existing MAX30100 driver.
>
> Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
> ---
a changelog for v2 would be nice, earlier comments have not all been
incorporated
> drivers/iio/health/Kconfig | 13 ++
> drivers/iio/health/Makefile | 1 +
> drivers/iio/health/max30102.c | 485 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 499 insertions(+)
> create mode 100644 drivers/iio/health/max30102.c
>
> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig
> index c5f004a8e447..a2ecb4c94c2a 100644
> --- a/drivers/iio/health/Kconfig
> +++ b/drivers/iio/health/Kconfig
> @@ -46,6 +46,19 @@ config MAX30100
> To compile this driver as a module, choose M here: the
> module will be called max30100.
>
> +config MAX30102
> + tristate "MAX30102 heart rate and pulse oximeter sensor"
> + depends on I2C
> + select REGMAP_I2C
> + select IIO_BUFFER
> + select IIO_KFIFO_BUF
> + help
> + Say Y here to build I2C interface support for the Maxim
> + MAX30102 heart rate, and pulse oximeter sensor.
> +
> + To compile this driver as a module, choose M here: the
> + module will be called max30102.
> +
> endmenu
>
> endmenu
> diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile
> index 9955a2ae8df1..3558f9d64954 100644
> --- a/drivers/iio/health/Makefile
> +++ b/drivers/iio/health/Makefile
> @@ -7,3 +7,4 @@
> obj-$(CONFIG_AFE4403) += afe4403.o
> obj-$(CONFIG_AFE4404) += afe4404.o
> obj-$(CONFIG_MAX30100) += max30100.o
> +obj-$(CONFIG_MAX30102) += max30102.o
> diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c
> new file mode 100644
> index 000000000000..87df1e6d712f
> --- /dev/null
> +++ b/drivers/iio/health/max30102.c
> @@ -0,0 +1,485 @@
> +/*
> + * max30102.c - Support for MAX30102 heart rate and pulse oximeter sensor
> + *
> + * Copyright (C) 2017 Matt Ranostay <matt@ranostay.consulting>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * TODO: proximity power saving feature
> + */
> +
> +#include <linux/module.h>
> +#include <linux/init.h>
> +#include <linux/interrupt.h>
> +#include <linux/delay.h>
> +#include <linux/err.h>
> +#include <linux/irq.h>
> +#include <linux/i2c.h>
> +#include <linux/mutex.h>
> +#include <linux/of.h>
> +#include <linux/regmap.h>
> +#include <linux/iio/iio.h>
> +#include <linux/iio/buffer.h>
> +#include <linux/iio/kfifo_buf.h>
> +
> +#define MAX30102_REGMAP_NAME "max30102_regmap"
> +#define MAX30102_DRV_NAME "max30102"
> +
> +#define MAX30102_REG_INT_STATUS 0x00
> +#define MAX30102_REG_INT_STATUS_PWR_RDY BIT(0)
> +#define MAX30102_REG_INT_STATUS_PROX_INT BIT(4)
> +#define MAX30102_REG_INT_STATUS_ALC_OVF BIT(5)
> +#define MAX30102_REG_INT_STATUS_PPG_RDY BIT(6)
> +#define MAX30102_REG_INT_STATUS_FIFO_RDY BIT(7)
> +
> +#define MAX30102_REG_INT_ENABLE 0x02
> +#define MAX30102_REG_INT_ENABLE_PROX_INT_EN BIT(4)
> +#define MAX30102_REG_INT_ENABLE_ALC_OVF_EN BIT(5)
> +#define MAX30102_REG_INT_ENABLE_PPG_EN BIT(6)
> +#define MAX30102_REG_INT_ENABLE_FIFO_EN BIT(7)
> +#define MAX30102_REG_INT_ENABLE_MASK 0xf0
> +#define MAX30102_REG_INT_ENABLE_MASK_SHIFT 4
> +
> +#define MAX30102_REG_FIFO_WR_PTR 0x04
> +#define MAX30102_REG_FIFO_OVR_CTR 0x05
> +#define MAX30102_REG_FIFO_RD_PTR 0x06
> +#define MAX30102_REG_FIFO_DATA 0x07
> +#define MAX30102_REG_FIFO_DATA_ENTRY_LEN 6
still 6?
> +
> +#define MAX30102_REG_FIFO_CONFIG 0x08
> +#define MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES BIT(1)
> +#define MAX30102_REG_FIFO_CONFIG_AVG_SHIFT 5
> +#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_PWR BIT(7)
> +
> +#define MAX30102_REG_SPO2_CONFIG 0x0a
> +#define MAX30102_REG_SPO2_CONFIG_PULSE_411_US 0x03
> +#define MAX30102_REG_SPO2_CONFIG_SR_400HZ 0x03
> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK 0x07
> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT 2
> +#define MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS BIT(0)
> +#define MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT 4
> +
> +#define MAX30102_REG_RED_LED_CONFIG 0x0c
> +#define MAX30102_REG_IR_LED_CONFIG 0x0d
> +
> +#define MAX30102_REG_TEMP_CONFIG 0x21
> +#define MAX30102_REG_TEMP_CONFIG_TEMP_EN BIT(0)
> +
> +#define MAX30102_REG_TEMP_INTEGER 0x1f
> +#define MAX30102_REG_TEMP_FRACTION 0x20
order of regs
> +
> +struct max30102_data {
> + struct i2c_client *client;
> + struct iio_dev *indio_dev;
> + struct mutex lock;
> + struct regmap *regmap;
> +
> + u8 buffer[8];
> + __be32 processed_buffer[2]; /* 2 x 18-bit (padded to 32-bits) */
> +};
> +
> +static const struct regmap_config max30102_regmap_config = {
> + .name = MAX30102_REGMAP_NAME,
> +
> + .reg_bits = 8,
> + .val_bits = 8,
> +};
> +
> +static const unsigned long max30102_scan_masks[] = {0x3, 0};
> +
> +static const struct iio_chan_spec max30102_channels[] = {
> + {
> + .type = IIO_INTENSITY,
> + .channel2 = IIO_MOD_LIGHT_RED,
> + .modified = 1,
> +
> + .scan_index = 0,
> + .scan_type = {
> + .sign = 'u',
> + .shift = 14,
> + .realbits = 18,
> + .storagebits = 32,
> + .endianness = IIO_BE,
> + },
> + },
> + {
> + .type = IIO_INTENSITY,
> + .channel2 = IIO_MOD_LIGHT_IR,
> + .modified = 1,
> +
> + .scan_index = 1,
> + .scan_type = {
> + .sign = 'u',
> + .shift = 14,
> + .realbits = 18,
> + .storagebits = 32,
> + .endianness = IIO_BE,
> + },
> + },
> + {
> + .type = IIO_TEMP,
> + .info_mask_separate =
> + BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
> + .scan_index = -1,
> + },
> +};
> +
> +static int max30102_set_powermode(struct max30102_data *data, bool state)
> +{
> + return regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG,
> + MAX30102_REG_MODE_CONFIG_PWR,
> + state ? 0 : MAX30102_REG_MODE_CONFIG_PWR);
> +}
> +
> +static int max30102_buffer_postenable(struct iio_dev *indio_dev)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + return max30102_set_powermode(data, true);
> +}
> +
> +static int max30102_buffer_predisable(struct iio_dev *indio_dev)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + return max30102_set_powermode(data, false);
> +}
> +
> +static const struct iio_buffer_setup_ops max30102_buffer_setup_ops = {
> + .postenable = max30102_buffer_postenable,
> + .predisable = max30102_buffer_predisable,
> +};
> +
> +static inline int max30102_fifo_count(struct max30102_data *data)
> +{
> + unsigned int val;
> + int ret;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_INT_STATUS, &val);
> + if (ret)
> + return ret;
> +
> + /* FIFO has one sample slot left */
> + if (val & MAX30102_REG_INT_STATUS_FIFO_RDY)
> + return 1;
> +
> + return 0;
> +}
> +
> +static int max30102_read_measurement(struct max30102_data *data)
> +{
> + int ret;
> + u8 *buffer = (u8 *) &data->buffer;
> +
> + ret = i2c_smbus_read_i2c_block_data(data->client,
> + MAX30102_REG_FIFO_DATA,
> + MAX30102_REG_FIFO_DATA_ENTRY_LEN,
> + buffer);
> +
> + memcpy(&data->processed_buffer[0], &buffer[0], 3);
> + memcpy(&data->processed_buffer[1], &buffer[3], 3);
> +
> + return (ret == MAX30102_REG_FIFO_DATA_ENTRY_LEN) ? 0 : -EINVAL;
> +}
> +
> +static irqreturn_t max30102_interrupt_handler(int irq, void *private)
> +{
> + struct iio_dev *indio_dev = private;
> + struct max30102_data *data = iio_priv(indio_dev);
> + int ret, cnt = 0;
> +
> + mutex_lock(&data->lock);
> +
> + while (cnt || (cnt = max30102_fifo_count(data)) > 0) {
> + ret = max30102_read_measurement(data);
> + if (ret)
> + break;
> +
> + iio_push_to_buffers(data->indio_dev, data->processed_buffer);
> + cnt--;
> + }
> +
> + mutex_unlock(&data->lock);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static int max30102_get_current_idx(unsigned int val, int *reg)
> +{
> + /* each step is 0.200 mA */
> + *reg = val / 200;
> +
> + return *reg > 0xff ? -EINVAL : 0;
> +}
> +
> +static int max30102_led_init(struct max30102_data *data)
> +{
> + struct device *dev = &data->client->dev;
> + struct device_node *np = dev->of_node;
> + unsigned int val;
> + int reg, ret;
> +
> + ret = of_property_read_u32(np, "maxim,red-led-current-microamp", &val);
> + if (ret) {
> + dev_info(dev, "no red-led-current-microamp set\n");
> +
> + /* Default to 7 mA RED LED */
> + val = 7000;
> + }
> +
> + ret = max30102_get_current_idx(val, ®);
> + if (ret) {
> + dev_err(dev, "invalid RED LED current setting %d\n", val);
> + return ret;
> + }
> +
> + ret = regmap_write(data->regmap, MAX30102_REG_RED_LED_CONFIG, reg);
> + if (ret)
> + return ret;
> +
> + ret = of_property_read_u32(np, "maxim,ir-led-current-microamp", &val);
> + if (ret) {
> + dev_info(dev, "no ir-led-current-microamp set\n");
> +
> + /* Default to 7 mA IR LED */
> + val = 7000;
> + }
> +
> + ret = max30102_get_current_idx(val, ®);
> + if (ret) {
> + dev_err(dev, "invalid IR LED current setting %d", val);
\n
> + return ret;
> + }
> +
> + return regmap_write(data->regmap, MAX30102_REG_IR_LED_CONFIG, reg);
> +}
> +
> +static int max30102_chip_init(struct max30102_data *data)
> +{
> + int ret;
> +
> + /* setup LED current settings */
> + ret = max30102_led_init(data);
> + if (ret)
> + return ret;
> +
> + /* enable 18-bit HR + SPO2 readings at 400Hz */
> + ret = regmap_write(data->regmap, MAX30102_REG_SPO2_CONFIG,
> + (MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS
> + << MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT) |
> + (MAX30102_REG_SPO2_CONFIG_SR_400HZ
> + << MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT) |
> + MAX30102_REG_SPO2_CONFIG_PULSE_411_US);
> + if (ret)
> + return ret;
> +
> + /* enable 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);
> + if (ret)
> + return ret;
> +
> + /* average 4 samples + generate FIFO interrupt */
> + ret = regmap_write(data->regmap, MAX30102_REG_FIFO_CONFIG,
> + (MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES
> + << MAX30102_REG_FIFO_CONFIG_AVG_SHIFT) |
> + MAX30102_REG_FIFO_CONFIG_AFULL);
> + if (ret)
> + return ret;
> +
> + /* enable FIFO interrupt */
> + return regmap_update_bits(data->regmap, MAX30102_REG_INT_ENABLE,
> + MAX30102_REG_INT_ENABLE_MASK,
> + MAX30102_REG_INT_ENABLE_FIFO_EN);
> +}
> +
> +static int max30102_read_temp(struct max30102_data *data, int *val)
> +{
> + int ret;
> + unsigned int reg;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_INTEGER, ®);
> + if (ret < 0)
> + return ret;
> + *val = reg << 4;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_FRACTION, ®);
> + if (ret < 0)
> + return ret;
> +
> + *val |= reg & 0xf;
> + *val = sign_extend32(*val, 11);
> +
> + return 0;
> +}
> +
> +static int max30102_get_temp(struct max30102_data *data, int *val)
> +{
> + int ret;
> +
> + /* start acquisition */
> + ret = regmap_update_bits(data->regmap, MAX30102_REG_TEMP_CONFIG,
> + MAX30102_REG_TEMP_CONFIG_TEMP_EN,
> + MAX30102_REG_TEMP_CONFIG_TEMP_EN);
> + if (ret)
> + return ret;
> +
> + msleep(35);
> +
> + return max30102_read_temp(data, val);
> +}
> +
> +static int max30102_read_raw(struct iio_dev *indio_dev,
> + struct iio_chan_spec const *chan,
> + int *val, int *val2, long mask)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> + int ret = -EINVAL;
> +
> + switch (mask) {
> + case IIO_CHAN_INFO_RAW:
> + /*
> + * Temperature reading can only be acquired while engine
> + * is running
> + */
> + mutex_lock(&indio_dev->mlock);
> +
> + if (!iio_buffer_enabled(indio_dev))
> + ret = -EAGAIN;
> + else {
> + ret = max30102_get_temp(data, val);
> + if (!ret)
> + ret = IIO_VAL_INT;
> + }
> +
> + mutex_unlock(&indio_dev->mlock);
> + break;
> + case IIO_CHAN_INFO_SCALE:
> + *val = 1; /* 0.0625 */
> + *val2 = 16;
> + ret = IIO_VAL_FRACTIONAL;
> + break;
> + }
> +
> + return ret;
> +}
> +
> +static const struct iio_info max30102_info = {
> + .driver_module = THIS_MODULE,
> + .read_raw = max30102_read_raw,
> +};
> +
> +static int max30102_probe(struct i2c_client *client,
> + const struct i2c_device_id *id)
> +{
> + struct max30102_data *data;
> + struct iio_buffer *buffer;
> + struct iio_dev *indio_dev;
> + int ret;
> +
> + indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
> + if (!indio_dev)
> + return -ENOMEM;
> +
> + buffer = devm_iio_kfifo_allocate(&client->dev);
> + if (!buffer)
> + return -ENOMEM;
> +
> + iio_device_attach_buffer(indio_dev, buffer);
> +
> + indio_dev->name = MAX30102_DRV_NAME;
> + indio_dev->channels = max30102_channels;
> + indio_dev->info = &max30102_info;
> + indio_dev->num_channels = ARRAY_SIZE(max30102_channels);
> + indio_dev->available_scan_masks = max30102_scan_masks;
> + indio_dev->modes = (INDIO_BUFFER_SOFTWARE | INDIO_DIRECT_MODE);
> + indio_dev->setup_ops = &max30102_buffer_setup_ops;
> +
> + data = iio_priv(indio_dev);
> + data->indio_dev = indio_dev;
> + data->client = client;
> +
> + mutex_init(&data->lock);
> + i2c_set_clientdata(client, indio_dev);
> +
> + data->regmap = devm_regmap_init_i2c(client, &max30102_regmap_config);
> + if (IS_ERR(data->regmap)) {
> + dev_err(&client->dev, "regmap initialization failed.\n");
maybe without .
> + return PTR_ERR(data->regmap);
> + }
> + max30102_set_powermode(data, false);
> +
> + ret = max30102_chip_init(data);
> + if (ret)
> + return ret;
> +
> + if (client->irq <= 0) {
> + dev_err(&client->dev, "no valid irq defined\n");
> + return -EINVAL;
> + }
> +
> + ret = devm_request_threaded_irq(&client->dev, client->irq,
> + NULL, max30102_interrupt_handler,
> + IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
> + "max30102_irq", indio_dev);
> + if (ret) {
> + dev_err(&client->dev, "request irq (%d) failed\n", client->irq);
> + return ret;
> + }
> +
> + return iio_device_register(indio_dev);
> +}
> +
> +static int max30102_remove(struct i2c_client *client)
> +{
> + struct iio_dev *indio_dev = i2c_get_clientdata(client);
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + iio_device_unregister(indio_dev);
> + max30102_set_powermode(data, false);
> +
> + return 0;
> +}
> +
> +static const struct i2c_device_id max30102_id[] = {
> + { "max30102", 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(i2c, max30102_id);
> +
> +static const struct of_device_id max30102_dt_ids[] = {
> + { .compatible = "maxim,max30102" },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, max30102_dt_ids);
> +
> +static struct i2c_driver max30102_driver = {
> + .driver = {
> + .name = MAX30102_DRV_NAME,
> + .of_match_table = of_match_ptr(max30102_dt_ids),
> + },
> + .probe = max30102_probe,
> + .remove = max30102_remove,
> + .id_table = max30102_id,
> +};
> +module_i2c_driver(max30102_driver);
> +
> +MODULE_AUTHOR("Matt Ranostay <matt@ranostay.consulting>");
> +MODULE_DESCRIPTION("MAX30102 heart rate and pulse oximeter sensor");
> +MODULE_LICENSE("GPL");
>
--
Peter Meerwald-Stadler
+43-664-2444418 (mobile)
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support
2017-01-25 7:29 ` Peter Meerwald-Stadler
@ 2017-01-25 8:24 ` Matt Ranostay
2017-01-25 9:43 ` Matt Ranostay
1 sibling, 0 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-25 8:24 UTC (permalink / raw)
To: Peter Meerwald-Stadler; +Cc: linux-iio, Jonathan Cameron
On Tue, Jan 24, 2017 at 11:29 PM, Peter Meerwald-Stadler
<pmeerw@pmeerw.net> wrote:
> On Tue, 24 Jan 2017, Matt Ranostay wrote:
>
>> MAX30102 is an heart rate and pulse oximeter sensor that works using
>> two LEDS of different wavelengths, and detecting the light reflected
>> back.
>>
>> This patchset adds support for both IR and RED LED channels which can
>> be processed in userspace to determine heart rate and blood oxygen
>> levels. The MAX30102 part isn't completely register and functional
>> compatible with the existing MAX30100 driver.
>>
>> Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
>> ---
>
> a changelog for v2 would be nice, earlier comments have not all been
> incorporated
Yeah I have the bad habit of having the cover letter named the same of
one patchsets.
But comments below noted for v3.
>
>> drivers/iio/health/Kconfig | 13 ++
>> drivers/iio/health/Makefile | 1 +
>> drivers/iio/health/max30102.c | 485 ++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 499 insertions(+)
>> create mode 100644 drivers/iio/health/max30102.c
>>
>> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig
>> index c5f004a8e447..a2ecb4c94c2a 100644
>> --- a/drivers/iio/health/Kconfig
>> +++ b/drivers/iio/health/Kconfig
>> @@ -46,6 +46,19 @@ config MAX30100
>> To compile this driver as a module, choose M here: the
>> module will be called max30100.
>>
>> +config MAX30102
>> + tristate "MAX30102 heart rate and pulse oximeter sensor"
>> + depends on I2C
>> + select REGMAP_I2C
>> + select IIO_BUFFER
>> + select IIO_KFIFO_BUF
>> + help
>> + Say Y here to build I2C interface support for the Maxim
>> + MAX30102 heart rate, and pulse oximeter sensor.
>> +
>> + To compile this driver as a module, choose M here: the
>> + module will be called max30102.
>> +
>> endmenu
>>
>> endmenu
>> diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile
>> index 9955a2ae8df1..3558f9d64954 100644
>> --- a/drivers/iio/health/Makefile
>> +++ b/drivers/iio/health/Makefile
>> @@ -7,3 +7,4 @@
>> obj-$(CONFIG_AFE4403) += afe4403.o
>> obj-$(CONFIG_AFE4404) += afe4404.o
>> obj-$(CONFIG_MAX30100) += max30100.o
>> +obj-$(CONFIG_MAX30102) += max30102.o
>> diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c
>> new file mode 100644
>> index 000000000000..87df1e6d712f
>> --- /dev/null
>> +++ b/drivers/iio/health/max30102.c
>> @@ -0,0 +1,485 @@
>> +/*
>> + * max30102.c - Support for MAX30102 heart rate and pulse oximeter sensor
>> + *
>> + * Copyright (C) 2017 Matt Ranostay <matt@ranostay.consulting>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * TODO: proximity power saving feature
>> + */
>> +
>> +#include <linux/module.h>
>> +#include <linux/init.h>
>> +#include <linux/interrupt.h>
>> +#include <linux/delay.h>
>> +#include <linux/err.h>
>> +#include <linux/irq.h>
>> +#include <linux/i2c.h>
>> +#include <linux/mutex.h>
>> +#include <linux/of.h>
>> +#include <linux/regmap.h>
>> +#include <linux/iio/iio.h>
>> +#include <linux/iio/buffer.h>
>> +#include <linux/iio/kfifo_buf.h>
>> +
>> +#define MAX30102_REGMAP_NAME "max30102_regmap"
>> +#define MAX30102_DRV_NAME "max30102"
>> +
>> +#define MAX30102_REG_INT_STATUS 0x00
>> +#define MAX30102_REG_INT_STATUS_PWR_RDY BIT(0)
>> +#define MAX30102_REG_INT_STATUS_PROX_INT BIT(4)
>> +#define MAX30102_REG_INT_STATUS_ALC_OVF BIT(5)
>> +#define MAX30102_REG_INT_STATUS_PPG_RDY BIT(6)
>> +#define MAX30102_REG_INT_STATUS_FIFO_RDY BIT(7)
>> +
>> +#define MAX30102_REG_INT_ENABLE 0x02
>> +#define MAX30102_REG_INT_ENABLE_PROX_INT_EN BIT(4)
>> +#define MAX30102_REG_INT_ENABLE_ALC_OVF_EN BIT(5)
>> +#define MAX30102_REG_INT_ENABLE_PPG_EN BIT(6)
>> +#define MAX30102_REG_INT_ENABLE_FIFO_EN BIT(7)
>> +#define MAX30102_REG_INT_ENABLE_MASK 0xf0
>> +#define MAX30102_REG_INT_ENABLE_MASK_SHIFT 4
>> +
>> +#define MAX30102_REG_FIFO_WR_PTR 0x04
>> +#define MAX30102_REG_FIFO_OVR_CTR 0x05
>> +#define MAX30102_REG_FIFO_RD_PTR 0x06
>> +#define MAX30102_REG_FIFO_DATA 0x07
>> +#define MAX30102_REG_FIFO_DATA_ENTRY_LEN 6
>
> still 6?
>
>> +
>> +#define MAX30102_REG_FIFO_CONFIG 0x08
>> +#define MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES BIT(1)
>> +#define MAX30102_REG_FIFO_CONFIG_AVG_SHIFT 5
>> +#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_PWR BIT(7)
>> +
>> +#define MAX30102_REG_SPO2_CONFIG 0x0a
>> +#define MAX30102_REG_SPO2_CONFIG_PULSE_411_US 0x03
>> +#define MAX30102_REG_SPO2_CONFIG_SR_400HZ 0x03
>> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK 0x07
>> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT 2
>> +#define MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS BIT(0)
>> +#define MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT 4
>> +
>> +#define MAX30102_REG_RED_LED_CONFIG 0x0c
>> +#define MAX30102_REG_IR_LED_CONFIG 0x0d
>> +
>> +#define MAX30102_REG_TEMP_CONFIG 0x21
>> +#define MAX30102_REG_TEMP_CONFIG_TEMP_EN BIT(0)
>> +
>> +#define MAX30102_REG_TEMP_INTEGER 0x1f
>> +#define MAX30102_REG_TEMP_FRACTION 0x20
>
> order of regs
>
>> +
>> +struct max30102_data {
>> + struct i2c_client *client;
>> + struct iio_dev *indio_dev;
>> + struct mutex lock;
>> + struct regmap *regmap;
>> +
>> + u8 buffer[8];
>> + __be32 processed_buffer[2]; /* 2 x 18-bit (padded to 32-bits) */
>> +};
>> +
>> +static const struct regmap_config max30102_regmap_config = {
>> + .name = MAX30102_REGMAP_NAME,
>> +
>> + .reg_bits = 8,
>> + .val_bits = 8,
>> +};
>> +
>> +static const unsigned long max30102_scan_masks[] = {0x3, 0};
>> +
>> +static const struct iio_chan_spec max30102_channels[] = {
>> + {
>> + .type = IIO_INTENSITY,
>> + .channel2 = IIO_MOD_LIGHT_RED,
>> + .modified = 1,
>> +
>> + .scan_index = 0,
>> + .scan_type = {
>> + .sign = 'u',
>> + .shift = 14,
>> + .realbits = 18,
>> + .storagebits = 32,
>> + .endianness = IIO_BE,
>> + },
>> + },
>> + {
>> + .type = IIO_INTENSITY,
>> + .channel2 = IIO_MOD_LIGHT_IR,
>> + .modified = 1,
>> +
>> + .scan_index = 1,
>> + .scan_type = {
>> + .sign = 'u',
>> + .shift = 14,
>> + .realbits = 18,
>> + .storagebits = 32,
>> + .endianness = IIO_BE,
>> + },
>> + },
>> + {
>> + .type = IIO_TEMP,
>> + .info_mask_separate =
>> + BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
>> + .scan_index = -1,
>> + },
>> +};
>> +
>> +static int max30102_set_powermode(struct max30102_data *data, bool state)
>> +{
>> + return regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG,
>> + MAX30102_REG_MODE_CONFIG_PWR,
>> + state ? 0 : MAX30102_REG_MODE_CONFIG_PWR);
>> +}
>> +
>> +static int max30102_buffer_postenable(struct iio_dev *indio_dev)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + return max30102_set_powermode(data, true);
>> +}
>> +
>> +static int max30102_buffer_predisable(struct iio_dev *indio_dev)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + return max30102_set_powermode(data, false);
>> +}
>> +
>> +static const struct iio_buffer_setup_ops max30102_buffer_setup_ops = {
>> + .postenable = max30102_buffer_postenable,
>> + .predisable = max30102_buffer_predisable,
>> +};
>> +
>> +static inline int max30102_fifo_count(struct max30102_data *data)
>> +{
>> + unsigned int val;
>> + int ret;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_INT_STATUS, &val);
>> + if (ret)
>> + return ret;
>> +
>> + /* FIFO has one sample slot left */
>> + if (val & MAX30102_REG_INT_STATUS_FIFO_RDY)
>> + return 1;
>> +
>> + return 0;
>> +}
>> +
>> +static int max30102_read_measurement(struct max30102_data *data)
>> +{
>> + int ret;
>> + u8 *buffer = (u8 *) &data->buffer;
>> +
>> + ret = i2c_smbus_read_i2c_block_data(data->client,
>> + MAX30102_REG_FIFO_DATA,
>> + MAX30102_REG_FIFO_DATA_ENTRY_LEN,
>> + buffer);
>> +
>> + memcpy(&data->processed_buffer[0], &buffer[0], 3);
>> + memcpy(&data->processed_buffer[1], &buffer[3], 3);
>> +
>> + return (ret == MAX30102_REG_FIFO_DATA_ENTRY_LEN) ? 0 : -EINVAL;
>> +}
>> +
>> +static irqreturn_t max30102_interrupt_handler(int irq, void *private)
>> +{
>> + struct iio_dev *indio_dev = private;
>> + struct max30102_data *data = iio_priv(indio_dev);
>> + int ret, cnt = 0;
>> +
>> + mutex_lock(&data->lock);
>> +
>> + while (cnt || (cnt = max30102_fifo_count(data)) > 0) {
>> + ret = max30102_read_measurement(data);
>> + if (ret)
>> + break;
>> +
>> + iio_push_to_buffers(data->indio_dev, data->processed_buffer);
>> + cnt--;
>> + }
>> +
>> + mutex_unlock(&data->lock);
>> +
>> + return IRQ_HANDLED;
>> +}
>> +
>> +static int max30102_get_current_idx(unsigned int val, int *reg)
>> +{
>> + /* each step is 0.200 mA */
>> + *reg = val / 200;
>> +
>> + return *reg > 0xff ? -EINVAL : 0;
>> +}
>> +
>> +static int max30102_led_init(struct max30102_data *data)
>> +{
>> + struct device *dev = &data->client->dev;
>> + struct device_node *np = dev->of_node;
>> + unsigned int val;
>> + int reg, ret;
>> +
>> + ret = of_property_read_u32(np, "maxim,red-led-current-microamp", &val);
>> + if (ret) {
>> + dev_info(dev, "no red-led-current-microamp set\n");
>> +
>> + /* Default to 7 mA RED LED */
>> + val = 7000;
>> + }
>> +
>> + ret = max30102_get_current_idx(val, ®);
>> + if (ret) {
>> + dev_err(dev, "invalid RED LED current setting %d\n", val);
>> + return ret;
>> + }
>> +
>> + ret = regmap_write(data->regmap, MAX30102_REG_RED_LED_CONFIG, reg);
>> + if (ret)
>> + return ret;
>> +
>> + ret = of_property_read_u32(np, "maxim,ir-led-current-microamp", &val);
>> + if (ret) {
>> + dev_info(dev, "no ir-led-current-microamp set\n");
>> +
>> + /* Default to 7 mA IR LED */
>> + val = 7000;
>> + }
>> +
>> + ret = max30102_get_current_idx(val, ®);
>> + if (ret) {
>> + dev_err(dev, "invalid IR LED current setting %d", val);
>
> \n
>
>> + return ret;
>> + }
>> +
>> + return regmap_write(data->regmap, MAX30102_REG_IR_LED_CONFIG, reg);
>> +}
>> +
>> +static int max30102_chip_init(struct max30102_data *data)
>> +{
>> + int ret;
>> +
>> + /* setup LED current settings */
>> + ret = max30102_led_init(data);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable 18-bit HR + SPO2 readings at 400Hz */
>> + ret = regmap_write(data->regmap, MAX30102_REG_SPO2_CONFIG,
>> + (MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS
>> + << MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT) |
>> + (MAX30102_REG_SPO2_CONFIG_SR_400HZ
>> + << MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT) |
>> + MAX30102_REG_SPO2_CONFIG_PULSE_411_US);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable 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);
>> + if (ret)
>> + return ret;
>> +
>> + /* average 4 samples + generate FIFO interrupt */
>> + ret = regmap_write(data->regmap, MAX30102_REG_FIFO_CONFIG,
>> + (MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES
>> + << MAX30102_REG_FIFO_CONFIG_AVG_SHIFT) |
>> + MAX30102_REG_FIFO_CONFIG_AFULL);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable FIFO interrupt */
>> + return regmap_update_bits(data->regmap, MAX30102_REG_INT_ENABLE,
>> + MAX30102_REG_INT_ENABLE_MASK,
>> + MAX30102_REG_INT_ENABLE_FIFO_EN);
>> +}
>> +
>> +static int max30102_read_temp(struct max30102_data *data, int *val)
>> +{
>> + int ret;
>> + unsigned int reg;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_INTEGER, ®);
>> + if (ret < 0)
>> + return ret;
>> + *val = reg << 4;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_FRACTION, ®);
>> + if (ret < 0)
>> + return ret;
>> +
>> + *val |= reg & 0xf;
>> + *val = sign_extend32(*val, 11);
>> +
>> + return 0;
>> +}
>> +
>> +static int max30102_get_temp(struct max30102_data *data, int *val)
>> +{
>> + int ret;
>> +
>> + /* start acquisition */
>> + ret = regmap_update_bits(data->regmap, MAX30102_REG_TEMP_CONFIG,
>> + MAX30102_REG_TEMP_CONFIG_TEMP_EN,
>> + MAX30102_REG_TEMP_CONFIG_TEMP_EN);
>> + if (ret)
>> + return ret;
>> +
>> + msleep(35);
>> +
>> + return max30102_read_temp(data, val);
>> +}
>> +
>> +static int max30102_read_raw(struct iio_dev *indio_dev,
>> + struct iio_chan_spec const *chan,
>> + int *val, int *val2, long mask)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> + int ret = -EINVAL;
>> +
>> + switch (mask) {
>> + case IIO_CHAN_INFO_RAW:
>> + /*
>> + * Temperature reading can only be acquired while engine
>> + * is running
>> + */
>> + mutex_lock(&indio_dev->mlock);
>> +
>> + if (!iio_buffer_enabled(indio_dev))
>> + ret = -EAGAIN;
>> + else {
>> + ret = max30102_get_temp(data, val);
>> + if (!ret)
>> + ret = IIO_VAL_INT;
>> + }
>> +
>> + mutex_unlock(&indio_dev->mlock);
>> + break;
>> + case IIO_CHAN_INFO_SCALE:
>> + *val = 1; /* 0.0625 */
>> + *val2 = 16;
>> + ret = IIO_VAL_FRACTIONAL;
>> + break;
>> + }
>> +
>> + return ret;
>> +}
>> +
>> +static const struct iio_info max30102_info = {
>> + .driver_module = THIS_MODULE,
>> + .read_raw = max30102_read_raw,
>> +};
>> +
>> +static int max30102_probe(struct i2c_client *client,
>> + const struct i2c_device_id *id)
>> +{
>> + struct max30102_data *data;
>> + struct iio_buffer *buffer;
>> + struct iio_dev *indio_dev;
>> + int ret;
>> +
>> + indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
>> + if (!indio_dev)
>> + return -ENOMEM;
>> +
>> + buffer = devm_iio_kfifo_allocate(&client->dev);
>> + if (!buffer)
>> + return -ENOMEM;
>> +
>> + iio_device_attach_buffer(indio_dev, buffer);
>> +
>> + indio_dev->name = MAX30102_DRV_NAME;
>> + indio_dev->channels = max30102_channels;
>> + indio_dev->info = &max30102_info;
>> + indio_dev->num_channels = ARRAY_SIZE(max30102_channels);
>> + indio_dev->available_scan_masks = max30102_scan_masks;
>> + indio_dev->modes = (INDIO_BUFFER_SOFTWARE | INDIO_DIRECT_MODE);
>> + indio_dev->setup_ops = &max30102_buffer_setup_ops;
>> +
>> + data = iio_priv(indio_dev);
>> + data->indio_dev = indio_dev;
>> + data->client = client;
>> +
>> + mutex_init(&data->lock);
>> + i2c_set_clientdata(client, indio_dev);
>> +
>> + data->regmap = devm_regmap_init_i2c(client, &max30102_regmap_config);
>> + if (IS_ERR(data->regmap)) {
>> + dev_err(&client->dev, "regmap initialization failed.\n");
>
> maybe without .
>
>> + return PTR_ERR(data->regmap);
>> + }
>> + max30102_set_powermode(data, false);
>> +
>> + ret = max30102_chip_init(data);
>> + if (ret)
>> + return ret;
>> +
>> + if (client->irq <= 0) {
>> + dev_err(&client->dev, "no valid irq defined\n");
>> + return -EINVAL;
>> + }
>> +
>> + ret = devm_request_threaded_irq(&client->dev, client->irq,
>> + NULL, max30102_interrupt_handler,
>> + IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
>> + "max30102_irq", indio_dev);
>> + if (ret) {
>> + dev_err(&client->dev, "request irq (%d) failed\n", client->irq);
>> + return ret;
>> + }
>> +
>> + return iio_device_register(indio_dev);
>> +}
>> +
>> +static int max30102_remove(struct i2c_client *client)
>> +{
>> + struct iio_dev *indio_dev = i2c_get_clientdata(client);
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + iio_device_unregister(indio_dev);
>> + max30102_set_powermode(data, false);
>> +
>> + return 0;
>> +}
>> +
>> +static const struct i2c_device_id max30102_id[] = {
>> + { "max30102", 0 },
>> + {}
>> +};
>> +MODULE_DEVICE_TABLE(i2c, max30102_id);
>> +
>> +static const struct of_device_id max30102_dt_ids[] = {
>> + { .compatible = "maxim,max30102" },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(of, max30102_dt_ids);
>> +
>> +static struct i2c_driver max30102_driver = {
>> + .driver = {
>> + .name = MAX30102_DRV_NAME,
>> + .of_match_table = of_match_ptr(max30102_dt_ids),
>> + },
>> + .probe = max30102_probe,
>> + .remove = max30102_remove,
>> + .id_table = max30102_id,
>> +};
>> +module_i2c_driver(max30102_driver);
>> +
>> +MODULE_AUTHOR("Matt Ranostay <matt@ranostay.consulting>");
>> +MODULE_DESCRIPTION("MAX30102 heart rate and pulse oximeter sensor");
>> +MODULE_LICENSE("GPL");
>>
>
> --
>
> Peter Meerwald-Stadler
> +43-664-2444418 (mobile)
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support
2017-01-25 7:29 ` Peter Meerwald-Stadler
2017-01-25 8:24 ` Matt Ranostay
@ 2017-01-25 9:43 ` Matt Ranostay
1 sibling, 0 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-25 9:43 UTC (permalink / raw)
To: Peter Meerwald-Stadler; +Cc: linux-iio, Jonathan Cameron
On Tue, Jan 24, 2017 at 11:29 PM, Peter Meerwald-Stadler
<pmeerw@pmeerw.net> wrote:
> On Tue, 24 Jan 2017, Matt Ranostay wrote:
>
>> MAX30102 is an heart rate and pulse oximeter sensor that works using
>> two LEDS of different wavelengths, and detecting the light reflected
>> back.
>>
>> This patchset adds support for both IR and RED LED channels which can
>> be processed in userspace to determine heart rate and blood oxygen
>> levels. The MAX30102 part isn't completely register and functional
>> compatible with the existing MAX30100 driver.
>>
>> Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
>> ---
>
> a changelog for v2 would be nice, earlier comments have not all been
> incorporated
>
>> drivers/iio/health/Kconfig | 13 ++
>> drivers/iio/health/Makefile | 1 +
>> drivers/iio/health/max30102.c | 485 ++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 499 insertions(+)
>> create mode 100644 drivers/iio/health/max30102.c
>>
>> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig
>> index c5f004a8e447..a2ecb4c94c2a 100644
>> --- a/drivers/iio/health/Kconfig
>> +++ b/drivers/iio/health/Kconfig
>> @@ -46,6 +46,19 @@ config MAX30100
>> To compile this driver as a module, choose M here: the
>> module will be called max30100.
>>
>> +config MAX30102
>> + tristate "MAX30102 heart rate and pulse oximeter sensor"
>> + depends on I2C
>> + select REGMAP_I2C
>> + select IIO_BUFFER
>> + select IIO_KFIFO_BUF
>> + help
>> + Say Y here to build I2C interface support for the Maxim
>> + MAX30102 heart rate, and pulse oximeter sensor.
>> +
>> + To compile this driver as a module, choose M here: the
>> + module will be called max30102.
>> +
>> endmenu
>>
>> endmenu
>> diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile
>> index 9955a2ae8df1..3558f9d64954 100644
>> --- a/drivers/iio/health/Makefile
>> +++ b/drivers/iio/health/Makefile
>> @@ -7,3 +7,4 @@
>> obj-$(CONFIG_AFE4403) += afe4403.o
>> obj-$(CONFIG_AFE4404) += afe4404.o
>> obj-$(CONFIG_MAX30100) += max30100.o
>> +obj-$(CONFIG_MAX30102) += max30102.o
>> diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c
>> new file mode 100644
>> index 000000000000..87df1e6d712f
>> --- /dev/null
>> +++ b/drivers/iio/health/max30102.c
>> @@ -0,0 +1,485 @@
>> +/*
>> + * max30102.c - Support for MAX30102 heart rate and pulse oximeter sensor
>> + *
>> + * Copyright (C) 2017 Matt Ranostay <matt@ranostay.consulting>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * TODO: proximity power saving feature
>> + */
>> +
>> +#include <linux/module.h>
>> +#include <linux/init.h>
>> +#include <linux/interrupt.h>
>> +#include <linux/delay.h>
>> +#include <linux/err.h>
>> +#include <linux/irq.h>
>> +#include <linux/i2c.h>
>> +#include <linux/mutex.h>
>> +#include <linux/of.h>
>> +#include <linux/regmap.h>
>> +#include <linux/iio/iio.h>
>> +#include <linux/iio/buffer.h>
>> +#include <linux/iio/kfifo_buf.h>
>> +
>> +#define MAX30102_REGMAP_NAME "max30102_regmap"
>> +#define MAX30102_DRV_NAME "max30102"
>> +
>> +#define MAX30102_REG_INT_STATUS 0x00
>> +#define MAX30102_REG_INT_STATUS_PWR_RDY BIT(0)
>> +#define MAX30102_REG_INT_STATUS_PROX_INT BIT(4)
>> +#define MAX30102_REG_INT_STATUS_ALC_OVF BIT(5)
>> +#define MAX30102_REG_INT_STATUS_PPG_RDY BIT(6)
>> +#define MAX30102_REG_INT_STATUS_FIFO_RDY BIT(7)
>> +
>> +#define MAX30102_REG_INT_ENABLE 0x02
>> +#define MAX30102_REG_INT_ENABLE_PROX_INT_EN BIT(4)
>> +#define MAX30102_REG_INT_ENABLE_ALC_OVF_EN BIT(5)
>> +#define MAX30102_REG_INT_ENABLE_PPG_EN BIT(6)
>> +#define MAX30102_REG_INT_ENABLE_FIFO_EN BIT(7)
>> +#define MAX30102_REG_INT_ENABLE_MASK 0xf0
>> +#define MAX30102_REG_INT_ENABLE_MASK_SHIFT 4
>> +
>> +#define MAX30102_REG_FIFO_WR_PTR 0x04
>> +#define MAX30102_REG_FIFO_OVR_CTR 0x05
>> +#define MAX30102_REG_FIFO_RD_PTR 0x06
>> +#define MAX30102_REG_FIFO_DATA 0x07
>> +#define MAX30102_REG_FIFO_DATA_ENTRY_LEN 6
>
> still 6?
>
Weirdly after testing even wiht datasheet sample code the same data
comes both 6 and 7 bytes reads.
>> +
>> +#define MAX30102_REG_FIFO_CONFIG 0x08
>> +#define MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES BIT(1)
>> +#define MAX30102_REG_FIFO_CONFIG_AVG_SHIFT 5
>> +#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_PWR BIT(7)
>> +
>> +#define MAX30102_REG_SPO2_CONFIG 0x0a
>> +#define MAX30102_REG_SPO2_CONFIG_PULSE_411_US 0x03
>> +#define MAX30102_REG_SPO2_CONFIG_SR_400HZ 0x03
>> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK 0x07
>> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT 2
>> +#define MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS BIT(0)
>> +#define MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT 4
>> +
>> +#define MAX30102_REG_RED_LED_CONFIG 0x0c
>> +#define MAX30102_REG_IR_LED_CONFIG 0x0d
>> +
>> +#define MAX30102_REG_TEMP_CONFIG 0x21
>> +#define MAX30102_REG_TEMP_CONFIG_TEMP_EN BIT(0)
>> +
>> +#define MAX30102_REG_TEMP_INTEGER 0x1f
>> +#define MAX30102_REG_TEMP_FRACTION 0x20
>
> order of regs
>
>> +
>> +struct max30102_data {
>> + struct i2c_client *client;
>> + struct iio_dev *indio_dev;
>> + struct mutex lock;
>> + struct regmap *regmap;
>> +
>> + u8 buffer[8];
>> + __be32 processed_buffer[2]; /* 2 x 18-bit (padded to 32-bits) */
>> +};
>> +
>> +static const struct regmap_config max30102_regmap_config = {
>> + .name = MAX30102_REGMAP_NAME,
>> +
>> + .reg_bits = 8,
>> + .val_bits = 8,
>> +};
>> +
>> +static const unsigned long max30102_scan_masks[] = {0x3, 0};
>> +
>> +static const struct iio_chan_spec max30102_channels[] = {
>> + {
>> + .type = IIO_INTENSITY,
>> + .channel2 = IIO_MOD_LIGHT_RED,
>> + .modified = 1,
>> +
>> + .scan_index = 0,
>> + .scan_type = {
>> + .sign = 'u',
>> + .shift = 14,
>> + .realbits = 18,
>> + .storagebits = 32,
>> + .endianness = IIO_BE,
>> + },
>> + },
>> + {
>> + .type = IIO_INTENSITY,
>> + .channel2 = IIO_MOD_LIGHT_IR,
>> + .modified = 1,
>> +
>> + .scan_index = 1,
>> + .scan_type = {
>> + .sign = 'u',
>> + .shift = 14,
>> + .realbits = 18,
>> + .storagebits = 32,
>> + .endianness = IIO_BE,
>> + },
>> + },
>> + {
>> + .type = IIO_TEMP,
>> + .info_mask_separate =
>> + BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
>> + .scan_index = -1,
>> + },
>> +};
>> +
>> +static int max30102_set_powermode(struct max30102_data *data, bool state)
>> +{
>> + return regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG,
>> + MAX30102_REG_MODE_CONFIG_PWR,
>> + state ? 0 : MAX30102_REG_MODE_CONFIG_PWR);
>> +}
>> +
>> +static int max30102_buffer_postenable(struct iio_dev *indio_dev)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + return max30102_set_powermode(data, true);
>> +}
>> +
>> +static int max30102_buffer_predisable(struct iio_dev *indio_dev)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + return max30102_set_powermode(data, false);
>> +}
>> +
>> +static const struct iio_buffer_setup_ops max30102_buffer_setup_ops = {
>> + .postenable = max30102_buffer_postenable,
>> + .predisable = max30102_buffer_predisable,
>> +};
>> +
>> +static inline int max30102_fifo_count(struct max30102_data *data)
>> +{
>> + unsigned int val;
>> + int ret;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_INT_STATUS, &val);
>> + if (ret)
>> + return ret;
>> +
>> + /* FIFO has one sample slot left */
>> + if (val & MAX30102_REG_INT_STATUS_FIFO_RDY)
>> + return 1;
>> +
>> + return 0;
>> +}
>> +
>> +static int max30102_read_measurement(struct max30102_data *data)
>> +{
>> + int ret;
>> + u8 *buffer = (u8 *) &data->buffer;
>> +
>> + ret = i2c_smbus_read_i2c_block_data(data->client,
>> + MAX30102_REG_FIFO_DATA,
>> + MAX30102_REG_FIFO_DATA_ENTRY_LEN,
>> + buffer);
>> +
>> + memcpy(&data->processed_buffer[0], &buffer[0], 3);
>> + memcpy(&data->processed_buffer[1], &buffer[3], 3);
>> +
>> + return (ret == MAX30102_REG_FIFO_DATA_ENTRY_LEN) ? 0 : -EINVAL;
>> +}
>> +
>> +static irqreturn_t max30102_interrupt_handler(int irq, void *private)
>> +{
>> + struct iio_dev *indio_dev = private;
>> + struct max30102_data *data = iio_priv(indio_dev);
>> + int ret, cnt = 0;
>> +
>> + mutex_lock(&data->lock);
>> +
>> + while (cnt || (cnt = max30102_fifo_count(data)) > 0) {
>> + ret = max30102_read_measurement(data);
>> + if (ret)
>> + break;
>> +
>> + iio_push_to_buffers(data->indio_dev, data->processed_buffer);
>> + cnt--;
>> + }
>> +
>> + mutex_unlock(&data->lock);
>> +
>> + return IRQ_HANDLED;
>> +}
>> +
>> +static int max30102_get_current_idx(unsigned int val, int *reg)
>> +{
>> + /* each step is 0.200 mA */
>> + *reg = val / 200;
>> +
>> + return *reg > 0xff ? -EINVAL : 0;
>> +}
>> +
>> +static int max30102_led_init(struct max30102_data *data)
>> +{
>> + struct device *dev = &data->client->dev;
>> + struct device_node *np = dev->of_node;
>> + unsigned int val;
>> + int reg, ret;
>> +
>> + ret = of_property_read_u32(np, "maxim,red-led-current-microamp", &val);
>> + if (ret) {
>> + dev_info(dev, "no red-led-current-microamp set\n");
>> +
>> + /* Default to 7 mA RED LED */
>> + val = 7000;
>> + }
>> +
>> + ret = max30102_get_current_idx(val, ®);
>> + if (ret) {
>> + dev_err(dev, "invalid RED LED current setting %d\n", val);
>> + return ret;
>> + }
>> +
>> + ret = regmap_write(data->regmap, MAX30102_REG_RED_LED_CONFIG, reg);
>> + if (ret)
>> + return ret;
>> +
>> + ret = of_property_read_u32(np, "maxim,ir-led-current-microamp", &val);
>> + if (ret) {
>> + dev_info(dev, "no ir-led-current-microamp set\n");
>> +
>> + /* Default to 7 mA IR LED */
>> + val = 7000;
>> + }
>> +
>> + ret = max30102_get_current_idx(val, ®);
>> + if (ret) {
>> + dev_err(dev, "invalid IR LED current setting %d", val);
>
> \n
>
>> + return ret;
>> + }
>> +
>> + return regmap_write(data->regmap, MAX30102_REG_IR_LED_CONFIG, reg);
>> +}
>> +
>> +static int max30102_chip_init(struct max30102_data *data)
>> +{
>> + int ret;
>> +
>> + /* setup LED current settings */
>> + ret = max30102_led_init(data);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable 18-bit HR + SPO2 readings at 400Hz */
>> + ret = regmap_write(data->regmap, MAX30102_REG_SPO2_CONFIG,
>> + (MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS
>> + << MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT) |
>> + (MAX30102_REG_SPO2_CONFIG_SR_400HZ
>> + << MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT) |
>> + MAX30102_REG_SPO2_CONFIG_PULSE_411_US);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable 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);
>> + if (ret)
>> + return ret;
>> +
>> + /* average 4 samples + generate FIFO interrupt */
>> + ret = regmap_write(data->regmap, MAX30102_REG_FIFO_CONFIG,
>> + (MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES
>> + << MAX30102_REG_FIFO_CONFIG_AVG_SHIFT) |
>> + MAX30102_REG_FIFO_CONFIG_AFULL);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable FIFO interrupt */
>> + return regmap_update_bits(data->regmap, MAX30102_REG_INT_ENABLE,
>> + MAX30102_REG_INT_ENABLE_MASK,
>> + MAX30102_REG_INT_ENABLE_FIFO_EN);
>> +}
>> +
>> +static int max30102_read_temp(struct max30102_data *data, int *val)
>> +{
>> + int ret;
>> + unsigned int reg;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_INTEGER, ®);
>> + if (ret < 0)
>> + return ret;
>> + *val = reg << 4;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_FRACTION, ®);
>> + if (ret < 0)
>> + return ret;
>> +
>> + *val |= reg & 0xf;
>> + *val = sign_extend32(*val, 11);
>> +
>> + return 0;
>> +}
>> +
>> +static int max30102_get_temp(struct max30102_data *data, int *val)
>> +{
>> + int ret;
>> +
>> + /* start acquisition */
>> + ret = regmap_update_bits(data->regmap, MAX30102_REG_TEMP_CONFIG,
>> + MAX30102_REG_TEMP_CONFIG_TEMP_EN,
>> + MAX30102_REG_TEMP_CONFIG_TEMP_EN);
>> + if (ret)
>> + return ret;
>> +
>> + msleep(35);
>> +
>> + return max30102_read_temp(data, val);
>> +}
>> +
>> +static int max30102_read_raw(struct iio_dev *indio_dev,
>> + struct iio_chan_spec const *chan,
>> + int *val, int *val2, long mask)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> + int ret = -EINVAL;
>> +
>> + switch (mask) {
>> + case IIO_CHAN_INFO_RAW:
>> + /*
>> + * Temperature reading can only be acquired while engine
>> + * is running
>> + */
>> + mutex_lock(&indio_dev->mlock);
>> +
>> + if (!iio_buffer_enabled(indio_dev))
>> + ret = -EAGAIN;
>> + else {
>> + ret = max30102_get_temp(data, val);
>> + if (!ret)
>> + ret = IIO_VAL_INT;
>> + }
>> +
>> + mutex_unlock(&indio_dev->mlock);
>> + break;
>> + case IIO_CHAN_INFO_SCALE:
>> + *val = 1; /* 0.0625 */
>> + *val2 = 16;
>> + ret = IIO_VAL_FRACTIONAL;
>> + break;
>> + }
>> +
>> + return ret;
>> +}
>> +
>> +static const struct iio_info max30102_info = {
>> + .driver_module = THIS_MODULE,
>> + .read_raw = max30102_read_raw,
>> +};
>> +
>> +static int max30102_probe(struct i2c_client *client,
>> + const struct i2c_device_id *id)
>> +{
>> + struct max30102_data *data;
>> + struct iio_buffer *buffer;
>> + struct iio_dev *indio_dev;
>> + int ret;
>> +
>> + indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
>> + if (!indio_dev)
>> + return -ENOMEM;
>> +
>> + buffer = devm_iio_kfifo_allocate(&client->dev);
>> + if (!buffer)
>> + return -ENOMEM;
>> +
>> + iio_device_attach_buffer(indio_dev, buffer);
>> +
>> + indio_dev->name = MAX30102_DRV_NAME;
>> + indio_dev->channels = max30102_channels;
>> + indio_dev->info = &max30102_info;
>> + indio_dev->num_channels = ARRAY_SIZE(max30102_channels);
>> + indio_dev->available_scan_masks = max30102_scan_masks;
>> + indio_dev->modes = (INDIO_BUFFER_SOFTWARE | INDIO_DIRECT_MODE);
>> + indio_dev->setup_ops = &max30102_buffer_setup_ops;
>> +
>> + data = iio_priv(indio_dev);
>> + data->indio_dev = indio_dev;
>> + data->client = client;
>> +
>> + mutex_init(&data->lock);
>> + i2c_set_clientdata(client, indio_dev);
>> +
>> + data->regmap = devm_regmap_init_i2c(client, &max30102_regmap_config);
>> + if (IS_ERR(data->regmap)) {
>> + dev_err(&client->dev, "regmap initialization failed.\n");
>
> maybe without .
>
>> + return PTR_ERR(data->regmap);
>> + }
>> + max30102_set_powermode(data, false);
>> +
>> + ret = max30102_chip_init(data);
>> + if (ret)
>> + return ret;
>> +
>> + if (client->irq <= 0) {
>> + dev_err(&client->dev, "no valid irq defined\n");
>> + return -EINVAL;
>> + }
>> +
>> + ret = devm_request_threaded_irq(&client->dev, client->irq,
>> + NULL, max30102_interrupt_handler,
>> + IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
>> + "max30102_irq", indio_dev);
>> + if (ret) {
>> + dev_err(&client->dev, "request irq (%d) failed\n", client->irq);
>> + return ret;
>> + }
>> +
>> + return iio_device_register(indio_dev);
>> +}
>> +
>> +static int max30102_remove(struct i2c_client *client)
>> +{
>> + struct iio_dev *indio_dev = i2c_get_clientdata(client);
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + iio_device_unregister(indio_dev);
>> + max30102_set_powermode(data, false);
>> +
>> + return 0;
>> +}
>> +
>> +static const struct i2c_device_id max30102_id[] = {
>> + { "max30102", 0 },
>> + {}
>> +};
>> +MODULE_DEVICE_TABLE(i2c, max30102_id);
>> +
>> +static const struct of_device_id max30102_dt_ids[] = {
>> + { .compatible = "maxim,max30102" },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(of, max30102_dt_ids);
>> +
>> +static struct i2c_driver max30102_driver = {
>> + .driver = {
>> + .name = MAX30102_DRV_NAME,
>> + .of_match_table = of_match_ptr(max30102_dt_ids),
>> + },
>> + .probe = max30102_probe,
>> + .remove = max30102_remove,
>> + .id_table = max30102_id,
>> +};
>> +module_i2c_driver(max30102_driver);
>> +
>> +MODULE_AUTHOR("Matt Ranostay <matt@ranostay.consulting>");
>> +MODULE_DESCRIPTION("MAX30102 heart rate and pulse oximeter sensor");
>> +MODULE_LICENSE("GPL");
>>
>
> --
>
> Peter Meerwald-Stadler
> +43-664-2444418 (mobile)
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support
2017-01-25 7:04 ` [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
2017-01-25 7:29 ` Peter Meerwald-Stadler
@ 2017-01-27 7:11 ` Matt Ranostay
2017-01-28 11:51 ` Jonathan Cameron
2 siblings, 0 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-27 7:11 UTC (permalink / raw)
To: linux-iio; +Cc: Jonathan Cameron, Matt Ranostay
On Tue, Jan 24, 2017 at 11:04 PM, Matt Ranostay
<matt@ranostay.consulting> wrote:
> MAX30102 is an heart rate and pulse oximeter sensor that works using
> two LEDS of different wavelengths, and detecting the light reflected
> back.
>
> This patchset adds support for both IR and RED LED channels which can
> be processed in userspace to determine heart rate and blood oxygen
> levels. The MAX30102 part isn't completely register and functional
> compatible with the existing MAX30100 driver.
>
> Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
> ---
> drivers/iio/health/Kconfig | 13 ++
> drivers/iio/health/Makefile | 1 +
> drivers/iio/health/max30102.c | 485 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 499 insertions(+)
> create mode 100644 drivers/iio/health/max30102.c
>
> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig
> index c5f004a8e447..a2ecb4c94c2a 100644
> --- a/drivers/iio/health/Kconfig
> +++ b/drivers/iio/health/Kconfig
> @@ -46,6 +46,19 @@ config MAX30100
> To compile this driver as a module, choose M here: the
> module will be called max30100.
>
> +config MAX30102
> + tristate "MAX30102 heart rate and pulse oximeter sensor"
> + depends on I2C
> + select REGMAP_I2C
> + select IIO_BUFFER
> + select IIO_KFIFO_BUF
> + help
> + Say Y here to build I2C interface support for the Maxim
> + MAX30102 heart rate, and pulse oximeter sensor.
> +
> + To compile this driver as a module, choose M here: the
> + module will be called max30102.
> +
> endmenu
>
> endmenu
> diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile
> index 9955a2ae8df1..3558f9d64954 100644
> --- a/drivers/iio/health/Makefile
> +++ b/drivers/iio/health/Makefile
> @@ -7,3 +7,4 @@
> obj-$(CONFIG_AFE4403) += afe4403.o
> obj-$(CONFIG_AFE4404) += afe4404.o
> obj-$(CONFIG_MAX30100) += max30100.o
> +obj-$(CONFIG_MAX30102) += max30102.o
> diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c
> new file mode 100644
> index 000000000000..87df1e6d712f
> --- /dev/null
> +++ b/drivers/iio/health/max30102.c
> @@ -0,0 +1,485 @@
> +/*
> + * max30102.c - Support for MAX30102 heart rate and pulse oximeter sensor
> + *
> + * Copyright (C) 2017 Matt Ranostay <matt@ranostay.consulting>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * TODO: proximity power saving feature
> + */
> +
> +#include <linux/module.h>
> +#include <linux/init.h>
> +#include <linux/interrupt.h>
> +#include <linux/delay.h>
> +#include <linux/err.h>
> +#include <linux/irq.h>
> +#include <linux/i2c.h>
> +#include <linux/mutex.h>
> +#include <linux/of.h>
> +#include <linux/regmap.h>
> +#include <linux/iio/iio.h>
> +#include <linux/iio/buffer.h>
> +#include <linux/iio/kfifo_buf.h>
> +
> +#define MAX30102_REGMAP_NAME "max30102_regmap"
> +#define MAX30102_DRV_NAME "max30102"
> +
> +#define MAX30102_REG_INT_STATUS 0x00
> +#define MAX30102_REG_INT_STATUS_PWR_RDY BIT(0)
> +#define MAX30102_REG_INT_STATUS_PROX_INT BIT(4)
> +#define MAX30102_REG_INT_STATUS_ALC_OVF BIT(5)
> +#define MAX30102_REG_INT_STATUS_PPG_RDY BIT(6)
> +#define MAX30102_REG_INT_STATUS_FIFO_RDY BIT(7)
> +
> +#define MAX30102_REG_INT_ENABLE 0x02
> +#define MAX30102_REG_INT_ENABLE_PROX_INT_EN BIT(4)
> +#define MAX30102_REG_INT_ENABLE_ALC_OVF_EN BIT(5)
> +#define MAX30102_REG_INT_ENABLE_PPG_EN BIT(6)
> +#define MAX30102_REG_INT_ENABLE_FIFO_EN BIT(7)
> +#define MAX30102_REG_INT_ENABLE_MASK 0xf0
> +#define MAX30102_REG_INT_ENABLE_MASK_SHIFT 4
> +
> +#define MAX30102_REG_FIFO_WR_PTR 0x04
> +#define MAX30102_REG_FIFO_OVR_CTR 0x05
> +#define MAX30102_REG_FIFO_RD_PTR 0x06
> +#define MAX30102_REG_FIFO_DATA 0x07
> +#define MAX30102_REG_FIFO_DATA_ENTRY_LEN 6
> +
> +#define MAX30102_REG_FIFO_CONFIG 0x08
> +#define MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES BIT(1)
> +#define MAX30102_REG_FIFO_CONFIG_AVG_SHIFT 5
> +#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_PWR BIT(7)
> +
> +#define MAX30102_REG_SPO2_CONFIG 0x0a
> +#define MAX30102_REG_SPO2_CONFIG_PULSE_411_US 0x03
> +#define MAX30102_REG_SPO2_CONFIG_SR_400HZ 0x03
> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK 0x07
> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT 2
> +#define MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS BIT(0)
> +#define MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT 4
This shift value is off by one... noticed when using the SPo2 samples
application
> +
> +#define MAX30102_REG_RED_LED_CONFIG 0x0c
> +#define MAX30102_REG_IR_LED_CONFIG 0x0d
> +
> +#define MAX30102_REG_TEMP_CONFIG 0x21
> +#define MAX30102_REG_TEMP_CONFIG_TEMP_EN BIT(0)
> +
> +#define MAX30102_REG_TEMP_INTEGER 0x1f
> +#define MAX30102_REG_TEMP_FRACTION 0x20
> +
> +struct max30102_data {
> + struct i2c_client *client;
> + struct iio_dev *indio_dev;
> + struct mutex lock;
> + struct regmap *regmap;
> +
> + u8 buffer[8];
> + __be32 processed_buffer[2]; /* 2 x 18-bit (padded to 32-bits) */
> +};
> +
> +static const struct regmap_config max30102_regmap_config = {
> + .name = MAX30102_REGMAP_NAME,
> +
> + .reg_bits = 8,
> + .val_bits = 8,
> +};
> +
> +static const unsigned long max30102_scan_masks[] = {0x3, 0};
> +
> +static const struct iio_chan_spec max30102_channels[] = {
> + {
> + .type = IIO_INTENSITY,
> + .channel2 = IIO_MOD_LIGHT_RED,
> + .modified = 1,
> +
> + .scan_index = 0,
> + .scan_type = {
> + .sign = 'u',
> + .shift = 14,
> + .realbits = 18,
> + .storagebits = 32,
> + .endianness = IIO_BE,
> + },
> + },
> + {
> + .type = IIO_INTENSITY,
> + .channel2 = IIO_MOD_LIGHT_IR,
> + .modified = 1,
> +
> + .scan_index = 1,
> + .scan_type = {
> + .sign = 'u',
> + .shift = 14,
> + .realbits = 18,
> + .storagebits = 32,
> + .endianness = IIO_BE,
> + },
> + },
> + {
> + .type = IIO_TEMP,
> + .info_mask_separate =
> + BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
> + .scan_index = -1,
> + },
> +};
> +
> +static int max30102_set_powermode(struct max30102_data *data, bool state)
> +{
> + return regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG,
> + MAX30102_REG_MODE_CONFIG_PWR,
> + state ? 0 : MAX30102_REG_MODE_CONFIG_PWR);
> +}
> +
> +static int max30102_buffer_postenable(struct iio_dev *indio_dev)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + return max30102_set_powermode(data, true);
> +}
> +
> +static int max30102_buffer_predisable(struct iio_dev *indio_dev)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + return max30102_set_powermode(data, false);
> +}
> +
> +static const struct iio_buffer_setup_ops max30102_buffer_setup_ops = {
> + .postenable = max30102_buffer_postenable,
> + .predisable = max30102_buffer_predisable,
> +};
> +
> +static inline int max30102_fifo_count(struct max30102_data *data)
> +{
> + unsigned int val;
> + int ret;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_INT_STATUS, &val);
> + if (ret)
> + return ret;
> +
> + /* FIFO has one sample slot left */
> + if (val & MAX30102_REG_INT_STATUS_FIFO_RDY)
> + return 1;
> +
> + return 0;
> +}
> +
> +static int max30102_read_measurement(struct max30102_data *data)
> +{
> + int ret;
> + u8 *buffer = (u8 *) &data->buffer;
> +
> + ret = i2c_smbus_read_i2c_block_data(data->client,
> + MAX30102_REG_FIFO_DATA,
> + MAX30102_REG_FIFO_DATA_ENTRY_LEN,
> + buffer);
> +
> + memcpy(&data->processed_buffer[0], &buffer[0], 3);
> + memcpy(&data->processed_buffer[1], &buffer[3], 3);
> +
> + return (ret == MAX30102_REG_FIFO_DATA_ENTRY_LEN) ? 0 : -EINVAL;
> +}
> +
> +static irqreturn_t max30102_interrupt_handler(int irq, void *private)
> +{
> + struct iio_dev *indio_dev = private;
> + struct max30102_data *data = iio_priv(indio_dev);
> + int ret, cnt = 0;
> +
> + mutex_lock(&data->lock);
> +
> + while (cnt || (cnt = max30102_fifo_count(data)) > 0) {
> + ret = max30102_read_measurement(data);
> + if (ret)
> + break;
> +
> + iio_push_to_buffers(data->indio_dev, data->processed_buffer);
> + cnt--;
> + }
> +
> + mutex_unlock(&data->lock);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static int max30102_get_current_idx(unsigned int val, int *reg)
> +{
> + /* each step is 0.200 mA */
> + *reg = val / 200;
> +
> + return *reg > 0xff ? -EINVAL : 0;
> +}
> +
> +static int max30102_led_init(struct max30102_data *data)
> +{
> + struct device *dev = &data->client->dev;
> + struct device_node *np = dev->of_node;
> + unsigned int val;
> + int reg, ret;
> +
> + ret = of_property_read_u32(np, "maxim,red-led-current-microamp", &val);
> + if (ret) {
> + dev_info(dev, "no red-led-current-microamp set\n");
> +
> + /* Default to 7 mA RED LED */
> + val = 7000;
> + }
> +
> + ret = max30102_get_current_idx(val, ®);
> + if (ret) {
> + dev_err(dev, "invalid RED LED current setting %d\n", val);
> + return ret;
> + }
> +
> + ret = regmap_write(data->regmap, MAX30102_REG_RED_LED_CONFIG, reg);
> + if (ret)
> + return ret;
> +
> + ret = of_property_read_u32(np, "maxim,ir-led-current-microamp", &val);
> + if (ret) {
> + dev_info(dev, "no ir-led-current-microamp set\n");
> +
> + /* Default to 7 mA IR LED */
> + val = 7000;
> + }
> +
> + ret = max30102_get_current_idx(val, ®);
> + if (ret) {
> + dev_err(dev, "invalid IR LED current setting %d", val);
> + return ret;
> + }
> +
> + return regmap_write(data->regmap, MAX30102_REG_IR_LED_CONFIG, reg);
> +}
> +
> +static int max30102_chip_init(struct max30102_data *data)
> +{
> + int ret;
> +
> + /* setup LED current settings */
> + ret = max30102_led_init(data);
> + if (ret)
> + return ret;
> +
> + /* enable 18-bit HR + SPO2 readings at 400Hz */
> + ret = regmap_write(data->regmap, MAX30102_REG_SPO2_CONFIG,
> + (MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS
> + << MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT) |
> + (MAX30102_REG_SPO2_CONFIG_SR_400HZ
> + << MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT) |
> + MAX30102_REG_SPO2_CONFIG_PULSE_411_US);
> + if (ret)
> + return ret;
> +
> + /* enable 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);
> + if (ret)
> + return ret;
> +
> + /* average 4 samples + generate FIFO interrupt */
> + ret = regmap_write(data->regmap, MAX30102_REG_FIFO_CONFIG,
> + (MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES
> + << MAX30102_REG_FIFO_CONFIG_AVG_SHIFT) |
> + MAX30102_REG_FIFO_CONFIG_AFULL);
> + if (ret)
> + return ret;
> +
> + /* enable FIFO interrupt */
> + return regmap_update_bits(data->regmap, MAX30102_REG_INT_ENABLE,
> + MAX30102_REG_INT_ENABLE_MASK,
> + MAX30102_REG_INT_ENABLE_FIFO_EN);
> +}
> +
> +static int max30102_read_temp(struct max30102_data *data, int *val)
> +{
> + int ret;
> + unsigned int reg;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_INTEGER, ®);
> + if (ret < 0)
> + return ret;
> + *val = reg << 4;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_FRACTION, ®);
> + if (ret < 0)
> + return ret;
> +
> + *val |= reg & 0xf;
> + *val = sign_extend32(*val, 11);
> +
> + return 0;
> +}
> +
> +static int max30102_get_temp(struct max30102_data *data, int *val)
> +{
> + int ret;
> +
> + /* start acquisition */
> + ret = regmap_update_bits(data->regmap, MAX30102_REG_TEMP_CONFIG,
> + MAX30102_REG_TEMP_CONFIG_TEMP_EN,
> + MAX30102_REG_TEMP_CONFIG_TEMP_EN);
> + if (ret)
> + return ret;
> +
> + msleep(35);
> +
> + return max30102_read_temp(data, val);
> +}
> +
> +static int max30102_read_raw(struct iio_dev *indio_dev,
> + struct iio_chan_spec const *chan,
> + int *val, int *val2, long mask)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> + int ret = -EINVAL;
> +
> + switch (mask) {
> + case IIO_CHAN_INFO_RAW:
> + /*
> + * Temperature reading can only be acquired while engine
> + * is running
> + */
> + mutex_lock(&indio_dev->mlock);
> +
> + if (!iio_buffer_enabled(indio_dev))
> + ret = -EAGAIN;
> + else {
> + ret = max30102_get_temp(data, val);
> + if (!ret)
> + ret = IIO_VAL_INT;
> + }
> +
> + mutex_unlock(&indio_dev->mlock);
> + break;
> + case IIO_CHAN_INFO_SCALE:
> + *val = 1; /* 0.0625 */
> + *val2 = 16;
> + ret = IIO_VAL_FRACTIONAL;
> + break;
> + }
> +
> + return ret;
> +}
> +
> +static const struct iio_info max30102_info = {
> + .driver_module = THIS_MODULE,
> + .read_raw = max30102_read_raw,
> +};
> +
> +static int max30102_probe(struct i2c_client *client,
> + const struct i2c_device_id *id)
> +{
> + struct max30102_data *data;
> + struct iio_buffer *buffer;
> + struct iio_dev *indio_dev;
> + int ret;
> +
> + indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
> + if (!indio_dev)
> + return -ENOMEM;
> +
> + buffer = devm_iio_kfifo_allocate(&client->dev);
> + if (!buffer)
> + return -ENOMEM;
> +
> + iio_device_attach_buffer(indio_dev, buffer);
> +
> + indio_dev->name = MAX30102_DRV_NAME;
> + indio_dev->channels = max30102_channels;
> + indio_dev->info = &max30102_info;
> + indio_dev->num_channels = ARRAY_SIZE(max30102_channels);
> + indio_dev->available_scan_masks = max30102_scan_masks;
> + indio_dev->modes = (INDIO_BUFFER_SOFTWARE | INDIO_DIRECT_MODE);
> + indio_dev->setup_ops = &max30102_buffer_setup_ops;
> +
> + data = iio_priv(indio_dev);
> + data->indio_dev = indio_dev;
> + data->client = client;
> +
> + mutex_init(&data->lock);
> + i2c_set_clientdata(client, indio_dev);
> +
> + data->regmap = devm_regmap_init_i2c(client, &max30102_regmap_config);
> + if (IS_ERR(data->regmap)) {
> + dev_err(&client->dev, "regmap initialization failed.\n");
> + return PTR_ERR(data->regmap);
> + }
> + max30102_set_powermode(data, false);
> +
> + ret = max30102_chip_init(data);
> + if (ret)
> + return ret;
> +
> + if (client->irq <= 0) {
> + dev_err(&client->dev, "no valid irq defined\n");
> + return -EINVAL;
> + }
> +
> + ret = devm_request_threaded_irq(&client->dev, client->irq,
> + NULL, max30102_interrupt_handler,
> + IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
> + "max30102_irq", indio_dev);
> + if (ret) {
> + dev_err(&client->dev, "request irq (%d) failed\n", client->irq);
> + return ret;
> + }
> +
> + return iio_device_register(indio_dev);
> +}
> +
> +static int max30102_remove(struct i2c_client *client)
> +{
> + struct iio_dev *indio_dev = i2c_get_clientdata(client);
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + iio_device_unregister(indio_dev);
> + max30102_set_powermode(data, false);
> +
> + return 0;
> +}
> +
> +static const struct i2c_device_id max30102_id[] = {
> + { "max30102", 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(i2c, max30102_id);
> +
> +static const struct of_device_id max30102_dt_ids[] = {
> + { .compatible = "maxim,max30102" },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, max30102_dt_ids);
> +
> +static struct i2c_driver max30102_driver = {
> + .driver = {
> + .name = MAX30102_DRV_NAME,
> + .of_match_table = of_match_ptr(max30102_dt_ids),
> + },
> + .probe = max30102_probe,
> + .remove = max30102_remove,
> + .id_table = max30102_id,
> +};
> +module_i2c_driver(max30102_driver);
> +
> +MODULE_AUTHOR("Matt Ranostay <matt@ranostay.consulting>");
> +MODULE_DESCRIPTION("MAX30102 heart rate and pulse oximeter sensor");
> +MODULE_LICENSE("GPL");
> --
> 2.10.2
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] devicetree: add documentation for MAX30102 oximeter
2017-01-25 7:04 ` Matt Ranostay
@ 2017-01-27 22:36 ` Rob Herring
-1 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2017-01-27 22:36 UTC (permalink / raw)
To: Matt Ranostay
Cc: linux-iio-u79uwXL29TY76Z2rM5mHXA, jic23-DgEjT+Ai2ygdnm+yROfE0A,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Tue, Jan 24, 2017 at 11:04:41PM -0800, Matt Ranostay wrote:
> Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> Signed-off-by: Matt Ranostay <matt-sk+viVC6FLCDq+mSdOJa79kegs52MxvZ@public.gmane.org>
> ---
> .../devicetree/bindings/iio/health/max30102.txt | 30 ++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/health/max30102.txt
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] devicetree: add documentation for MAX30102 oximeter
@ 2017-01-27 22:36 ` Rob Herring
0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2017-01-27 22:36 UTC (permalink / raw)
To: Matt Ranostay; +Cc: linux-iio, jic23, devicetree
On Tue, Jan 24, 2017 at 11:04:41PM -0800, Matt Ranostay wrote:
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
> ---
> .../devicetree/bindings/iio/health/max30102.txt | 30 ++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/health/max30102.txt
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support
2017-01-25 7:04 ` [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
2017-01-25 7:29 ` Peter Meerwald-Stadler
2017-01-27 7:11 ` Matt Ranostay
@ 2017-01-28 11:51 ` Jonathan Cameron
2017-01-29 4:04 ` Matt Ranostay
2 siblings, 1 reply; 12+ messages in thread
From: Jonathan Cameron @ 2017-01-28 11:51 UTC (permalink / raw)
To: Matt Ranostay, linux-iio
On 25/01/17 07:04, Matt Ranostay wrote:
> MAX30102 is an heart rate and pulse oximeter sensor that works using
> two LEDS of different wavelengths, and detecting the light reflected
> back.
>
> This patchset adds support for both IR and RED LED channels which can
> be processed in userspace to determine heart rate and blood oxygen
> levels. The MAX30102 part isn't completely register and functional
> compatible with the existing MAX30100 driver.
>
> Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
Really minor comment inline from me. Would guess V3 will do the job
on this one ;)
Jonathan
> ---
> drivers/iio/health/Kconfig | 13 ++
> drivers/iio/health/Makefile | 1 +
> drivers/iio/health/max30102.c | 485 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 499 insertions(+)
> create mode 100644 drivers/iio/health/max30102.c
>
> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig
> index c5f004a8e447..a2ecb4c94c2a 100644
> --- a/drivers/iio/health/Kconfig
> +++ b/drivers/iio/health/Kconfig
> @@ -46,6 +46,19 @@ config MAX30100
> To compile this driver as a module, choose M here: the
> module will be called max30100.
>
> +config MAX30102
> + tristate "MAX30102 heart rate and pulse oximeter sensor"
> + depends on I2C
> + select REGMAP_I2C
> + select IIO_BUFFER
> + select IIO_KFIFO_BUF
> + help
> + Say Y here to build I2C interface support for the Maxim
> + MAX30102 heart rate, and pulse oximeter sensor.
> +
> + To compile this driver as a module, choose M here: the
> + module will be called max30102.
> +
> endmenu
>
> endmenu
> diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile
> index 9955a2ae8df1..3558f9d64954 100644
> --- a/drivers/iio/health/Makefile
> +++ b/drivers/iio/health/Makefile
> @@ -7,3 +7,4 @@
> obj-$(CONFIG_AFE4403) += afe4403.o
> obj-$(CONFIG_AFE4404) += afe4404.o
> obj-$(CONFIG_MAX30100) += max30100.o
> +obj-$(CONFIG_MAX30102) += max30102.o
> diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c
> new file mode 100644
> index 000000000000..87df1e6d712f
> --- /dev/null
> +++ b/drivers/iio/health/max30102.c
> @@ -0,0 +1,485 @@
> +/*
> + * max30102.c - Support for MAX30102 heart rate and pulse oximeter sensor
> + *
> + * Copyright (C) 2017 Matt Ranostay <matt@ranostay.consulting>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * TODO: proximity power saving feature
> + */
> +
> +#include <linux/module.h>
> +#include <linux/init.h>
> +#include <linux/interrupt.h>
> +#include <linux/delay.h>
> +#include <linux/err.h>
> +#include <linux/irq.h>
> +#include <linux/i2c.h>
> +#include <linux/mutex.h>
> +#include <linux/of.h>
> +#include <linux/regmap.h>
> +#include <linux/iio/iio.h>
> +#include <linux/iio/buffer.h>
> +#include <linux/iio/kfifo_buf.h>
> +
> +#define MAX30102_REGMAP_NAME "max30102_regmap"
> +#define MAX30102_DRV_NAME "max30102"
> +
> +#define MAX30102_REG_INT_STATUS 0x00
> +#define MAX30102_REG_INT_STATUS_PWR_RDY BIT(0)
> +#define MAX30102_REG_INT_STATUS_PROX_INT BIT(4)
> +#define MAX30102_REG_INT_STATUS_ALC_OVF BIT(5)
> +#define MAX30102_REG_INT_STATUS_PPG_RDY BIT(6)
> +#define MAX30102_REG_INT_STATUS_FIFO_RDY BIT(7)
> +
> +#define MAX30102_REG_INT_ENABLE 0x02
> +#define MAX30102_REG_INT_ENABLE_PROX_INT_EN BIT(4)
> +#define MAX30102_REG_INT_ENABLE_ALC_OVF_EN BIT(5)
> +#define MAX30102_REG_INT_ENABLE_PPG_EN BIT(6)
> +#define MAX30102_REG_INT_ENABLE_FIFO_EN BIT(7)
> +#define MAX30102_REG_INT_ENABLE_MASK 0xf0
> +#define MAX30102_REG_INT_ENABLE_MASK_SHIFT 4
> +
> +#define MAX30102_REG_FIFO_WR_PTR 0x04
> +#define MAX30102_REG_FIFO_OVR_CTR 0x05
> +#define MAX30102_REG_FIFO_RD_PTR 0x06
> +#define MAX30102_REG_FIFO_DATA 0x07
> +#define MAX30102_REG_FIFO_DATA_ENTRY_LEN 6
> +
> +#define MAX30102_REG_FIFO_CONFIG 0x08
> +#define MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES BIT(1)
> +#define MAX30102_REG_FIFO_CONFIG_AVG_SHIFT 5
> +#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_PWR BIT(7)
> +
> +#define MAX30102_REG_SPO2_CONFIG 0x0a
> +#define MAX30102_REG_SPO2_CONFIG_PULSE_411_US 0x03
> +#define MAX30102_REG_SPO2_CONFIG_SR_400HZ 0x03
> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK 0x07
> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT 2
> +#define MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS BIT(0)
> +#define MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT 4
> +
> +#define MAX30102_REG_RED_LED_CONFIG 0x0c
> +#define MAX30102_REG_IR_LED_CONFIG 0x0d
> +
> +#define MAX30102_REG_TEMP_CONFIG 0x21
> +#define MAX30102_REG_TEMP_CONFIG_TEMP_EN BIT(0)
> +
> +#define MAX30102_REG_TEMP_INTEGER 0x1f
> +#define MAX30102_REG_TEMP_FRACTION 0x20
> +
> +struct max30102_data {
> + struct i2c_client *client;
> + struct iio_dev *indio_dev;
> + struct mutex lock;
> + struct regmap *regmap;
> +
> + u8 buffer[8];
> + __be32 processed_buffer[2]; /* 2 x 18-bit (padded to 32-bits) */
> +};
> +
> +static const struct regmap_config max30102_regmap_config = {
> + .name = MAX30102_REGMAP_NAME,
> +
> + .reg_bits = 8,
> + .val_bits = 8,
> +};
> +
> +static const unsigned long max30102_scan_masks[] = {0x3, 0};
> +
> +static const struct iio_chan_spec max30102_channels[] = {
> + {
> + .type = IIO_INTENSITY,
> + .channel2 = IIO_MOD_LIGHT_RED,
> + .modified = 1,
> +
> + .scan_index = 0,
> + .scan_type = {
> + .sign = 'u',
> + .shift = 14,
> + .realbits = 18,
> + .storagebits = 32,
> + .endianness = IIO_BE,
> + },
> + },
> + {
> + .type = IIO_INTENSITY,
> + .channel2 = IIO_MOD_LIGHT_IR,
> + .modified = 1,
> +
> + .scan_index = 1,
> + .scan_type = {
> + .sign = 'u',
> + .shift = 14,
> + .realbits = 18,
> + .storagebits = 32,
> + .endianness = IIO_BE,
> + },
> + },
> + {
> + .type = IIO_TEMP,
> + .info_mask_separate =
> + BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
> + .scan_index = -1,
> + },
> +};
> +
> +static int max30102_set_powermode(struct max30102_data *data, bool state)
> +{
> + return regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG,
> + MAX30102_REG_MODE_CONFIG_PWR,
> + state ? 0 : MAX30102_REG_MODE_CONFIG_PWR);
> +}
> +
> +static int max30102_buffer_postenable(struct iio_dev *indio_dev)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + return max30102_set_powermode(data, true);
> +}
> +
> +static int max30102_buffer_predisable(struct iio_dev *indio_dev)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + return max30102_set_powermode(data, false);
> +}
> +
> +static const struct iio_buffer_setup_ops max30102_buffer_setup_ops = {
> + .postenable = max30102_buffer_postenable,
> + .predisable = max30102_buffer_predisable,
> +};
> +
> +static inline int max30102_fifo_count(struct max30102_data *data)
> +{
> + unsigned int val;
> + int ret;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_INT_STATUS, &val);
> + if (ret)
> + return ret;
> +
> + /* FIFO has one sample slot left */
> + if (val & MAX30102_REG_INT_STATUS_FIFO_RDY)
> + return 1;
> +
> + return 0;
> +}
> +
> +static int max30102_read_measurement(struct max30102_data *data)
> +{
> + int ret;
> + u8 *buffer = (u8 *) &data->buffer;
> +
> + ret = i2c_smbus_read_i2c_block_data(data->client,
> + MAX30102_REG_FIFO_DATA,
> + MAX30102_REG_FIFO_DATA_ENTRY_LEN,
> + buffer);
> +
> + memcpy(&data->processed_buffer[0], &buffer[0], 3);
> + memcpy(&data->processed_buffer[1], &buffer[3], 3);
> +
> + return (ret == MAX30102_REG_FIFO_DATA_ENTRY_LEN) ? 0 : -EINVAL;
> +}
> +
> +static irqreturn_t max30102_interrupt_handler(int irq, void *private)
> +{
> + struct iio_dev *indio_dev = private;
> + struct max30102_data *data = iio_priv(indio_dev);
> + int ret, cnt = 0;
> +
> + mutex_lock(&data->lock);
> +
> + while (cnt || (cnt = max30102_fifo_count(data)) > 0) {
> + ret = max30102_read_measurement(data);
> + if (ret)
> + break;
> +
> + iio_push_to_buffers(data->indio_dev, data->processed_buffer);
> + cnt--;
> + }
> +
> + mutex_unlock(&data->lock);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static int max30102_get_current_idx(unsigned int val, int *reg)
> +{
> + /* each step is 0.200 mA */
> + *reg = val / 200;
> +
> + return *reg > 0xff ? -EINVAL : 0;
> +}
> +
> +static int max30102_led_init(struct max30102_data *data)
> +{
> + struct device *dev = &data->client->dev;
> + struct device_node *np = dev->of_node;
> + unsigned int val;
> + int reg, ret;
> +
> + ret = of_property_read_u32(np, "maxim,red-led-current-microamp", &val);
> + if (ret) {
> + dev_info(dev, "no red-led-current-microamp set\n");
> +
> + /* Default to 7 mA RED LED */
> + val = 7000;
> + }
> +
> + ret = max30102_get_current_idx(val, ®);
> + if (ret) {
> + dev_err(dev, "invalid RED LED current setting %d\n", val);
> + return ret;
> + }
> +
> + ret = regmap_write(data->regmap, MAX30102_REG_RED_LED_CONFIG, reg);
> + if (ret)
> + return ret;
> +
> + ret = of_property_read_u32(np, "maxim,ir-led-current-microamp", &val);
> + if (ret) {
> + dev_info(dev, "no ir-led-current-microamp set\n");
> +
> + /* Default to 7 mA IR LED */
> + val = 7000;
> + }
> +
> + ret = max30102_get_current_idx(val, ®);
> + if (ret) {
> + dev_err(dev, "invalid IR LED current setting %d", val);
> + return ret;
> + }
> +
> + return regmap_write(data->regmap, MAX30102_REG_IR_LED_CONFIG, reg);
> +}
> +
> +static int max30102_chip_init(struct max30102_data *data)
> +{
> + int ret;
> +
> + /* setup LED current settings */
> + ret = max30102_led_init(data);
> + if (ret)
> + return ret;
> +
> + /* enable 18-bit HR + SPO2 readings at 400Hz */
> + ret = regmap_write(data->regmap, MAX30102_REG_SPO2_CONFIG,
> + (MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS
> + << MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT) |
> + (MAX30102_REG_SPO2_CONFIG_SR_400HZ
> + << MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT) |
> + MAX30102_REG_SPO2_CONFIG_PULSE_411_US);
> + if (ret)
> + return ret;
> +
> + /* enable 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);
> + if (ret)
> + return ret;
> +
> + /* average 4 samples + generate FIFO interrupt */
> + ret = regmap_write(data->regmap, MAX30102_REG_FIFO_CONFIG,
> + (MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES
> + << MAX30102_REG_FIFO_CONFIG_AVG_SHIFT) |
> + MAX30102_REG_FIFO_CONFIG_AFULL);
> + if (ret)
> + return ret;
> +
> + /* enable FIFO interrupt */
> + return regmap_update_bits(data->regmap, MAX30102_REG_INT_ENABLE,
> + MAX30102_REG_INT_ENABLE_MASK,
> + MAX30102_REG_INT_ENABLE_FIFO_EN);
> +}
> +
> +static int max30102_read_temp(struct max30102_data *data, int *val)
> +{
> + int ret;
> + unsigned int reg;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_INTEGER, ®);
> + if (ret < 0)
> + return ret;
> + *val = reg << 4;
> +
> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_FRACTION, ®);
> + if (ret < 0)
> + return ret;
> +
> + *val |= reg & 0xf;
> + *val = sign_extend32(*val, 11);
> +
> + return 0;
> +}
> +
> +static int max30102_get_temp(struct max30102_data *data, int *val)
> +{
> + int ret;
> +
> + /* start acquisition */
> + ret = regmap_update_bits(data->regmap, MAX30102_REG_TEMP_CONFIG,
> + MAX30102_REG_TEMP_CONFIG_TEMP_EN,
> + MAX30102_REG_TEMP_CONFIG_TEMP_EN);
> + if (ret)
> + return ret;
> +
> + msleep(35);
> +
> + return max30102_read_temp(data, val);
> +}
> +
> +static int max30102_read_raw(struct iio_dev *indio_dev,
> + struct iio_chan_spec const *chan,
> + int *val, int *val2, long mask)
> +{
> + struct max30102_data *data = iio_priv(indio_dev);
> + int ret = -EINVAL;
> +
> + switch (mask) {
> + case IIO_CHAN_INFO_RAW:
> + /*
> + * Temperature reading can only be acquired while engine
> + * is running
> + */
> + mutex_lock(&indio_dev->mlock);
> +
> + if (!iio_buffer_enabled(indio_dev))
> + ret = -EAGAIN;
Issue with this is typically userspace will just try having another go
immediately. -EBUSY would be normal way of indicating not in a state
where it can do this, but here it's kind of inverted. Not obvious!
> + else {
> + ret = max30102_get_temp(data, val);
> + if (!ret)
> + ret = IIO_VAL_INT;
> + }
> +
> + mutex_unlock(&indio_dev->mlock);
> + break;
> + case IIO_CHAN_INFO_SCALE:
> + *val = 1; /* 0.0625 */
> + *val2 = 16;
> + ret = IIO_VAL_FRACTIONAL;
> + break;
> + }
> +
> + return ret;
> +}
> +
> +static const struct iio_info max30102_info = {
> + .driver_module = THIS_MODULE,
> + .read_raw = max30102_read_raw,
> +};
> +
> +static int max30102_probe(struct i2c_client *client,
> + const struct i2c_device_id *id)
> +{
> + struct max30102_data *data;
> + struct iio_buffer *buffer;
> + struct iio_dev *indio_dev;
> + int ret;
> +
> + indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
> + if (!indio_dev)
> + return -ENOMEM;
> +
> + buffer = devm_iio_kfifo_allocate(&client->dev);
> + if (!buffer)
> + return -ENOMEM;
> +
> + iio_device_attach_buffer(indio_dev, buffer);
> +
> + indio_dev->name = MAX30102_DRV_NAME;
> + indio_dev->channels = max30102_channels;
> + indio_dev->info = &max30102_info;
> + indio_dev->num_channels = ARRAY_SIZE(max30102_channels);
> + indio_dev->available_scan_masks = max30102_scan_masks;
> + indio_dev->modes = (INDIO_BUFFER_SOFTWARE | INDIO_DIRECT_MODE);
> + indio_dev->setup_ops = &max30102_buffer_setup_ops;
> +
> + data = iio_priv(indio_dev);
> + data->indio_dev = indio_dev;
> + data->client = client;
> +
> + mutex_init(&data->lock);
> + i2c_set_clientdata(client, indio_dev);
> +
> + data->regmap = devm_regmap_init_i2c(client, &max30102_regmap_config);
> + if (IS_ERR(data->regmap)) {
> + dev_err(&client->dev, "regmap initialization failed.\n");
> + return PTR_ERR(data->regmap);
> + }
> + max30102_set_powermode(data, false);
> +
> + ret = max30102_chip_init(data);
> + if (ret)
> + return ret;
> +
> + if (client->irq <= 0) {
> + dev_err(&client->dev, "no valid irq defined\n");
> + return -EINVAL;
> + }
> +
> + ret = devm_request_threaded_irq(&client->dev, client->irq,
> + NULL, max30102_interrupt_handler,
> + IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
> + "max30102_irq", indio_dev);
> + if (ret) {
> + dev_err(&client->dev, "request irq (%d) failed\n", client->irq);
> + return ret;
> + }
> +
> + return iio_device_register(indio_dev);
> +}
> +
> +static int max30102_remove(struct i2c_client *client)
> +{
> + struct iio_dev *indio_dev = i2c_get_clientdata(client);
> + struct max30102_data *data = iio_priv(indio_dev);
> +
> + iio_device_unregister(indio_dev);
> + max30102_set_powermode(data, false);
> +
> + return 0;
> +}
> +
> +static const struct i2c_device_id max30102_id[] = {
> + { "max30102", 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(i2c, max30102_id);
> +
> +static const struct of_device_id max30102_dt_ids[] = {
> + { .compatible = "maxim,max30102" },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, max30102_dt_ids);
> +
> +static struct i2c_driver max30102_driver = {
> + .driver = {
> + .name = MAX30102_DRV_NAME,
> + .of_match_table = of_match_ptr(max30102_dt_ids),
> + },
> + .probe = max30102_probe,
> + .remove = max30102_remove,
> + .id_table = max30102_id,
> +};
> +module_i2c_driver(max30102_driver);
> +
> +MODULE_AUTHOR("Matt Ranostay <matt@ranostay.consulting>");
> +MODULE_DESCRIPTION("MAX30102 heart rate and pulse oximeter sensor");
> +MODULE_LICENSE("GPL");
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support
2017-01-28 11:51 ` Jonathan Cameron
@ 2017-01-29 4:04 ` Matt Ranostay
0 siblings, 0 replies; 12+ messages in thread
From: Matt Ranostay @ 2017-01-29 4:04 UTC (permalink / raw)
To: Jonathan Cameron; +Cc: linux-iio
On Sat, Jan 28, 2017 at 3:51 AM, Jonathan Cameron <jic23@kernel.org> wrote:
> On 25/01/17 07:04, Matt Ranostay wrote:
>> MAX30102 is an heart rate and pulse oximeter sensor that works using
>> two LEDS of different wavelengths, and detecting the light reflected
>> back.
>>
>> This patchset adds support for both IR and RED LED channels which can
>> be processed in userspace to determine heart rate and blood oxygen
>> levels. The MAX30102 part isn't completely register and functional
>> compatible with the existing MAX30100 driver.
>>
>> Signed-off-by: Matt Ranostay <matt@ranostay.consulting>
> Really minor comment inline from me. Would guess V3 will do the job
> on this one ;)
>
> Jonathan
>> ---
>> drivers/iio/health/Kconfig | 13 ++
>> drivers/iio/health/Makefile | 1 +
>> drivers/iio/health/max30102.c | 485 ++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 499 insertions(+)
>> create mode 100644 drivers/iio/health/max30102.c
>>
>> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig
>> index c5f004a8e447..a2ecb4c94c2a 100644
>> --- a/drivers/iio/health/Kconfig
>> +++ b/drivers/iio/health/Kconfig
>> @@ -46,6 +46,19 @@ config MAX30100
>> To compile this driver as a module, choose M here: the
>> module will be called max30100.
>>
>> +config MAX30102
>> + tristate "MAX30102 heart rate and pulse oximeter sensor"
>> + depends on I2C
>> + select REGMAP_I2C
>> + select IIO_BUFFER
>> + select IIO_KFIFO_BUF
>> + help
>> + Say Y here to build I2C interface support for the Maxim
>> + MAX30102 heart rate, and pulse oximeter sensor.
>> +
>> + To compile this driver as a module, choose M here: the
>> + module will be called max30102.
>> +
>> endmenu
>>
>> endmenu
>> diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile
>> index 9955a2ae8df1..3558f9d64954 100644
>> --- a/drivers/iio/health/Makefile
>> +++ b/drivers/iio/health/Makefile
>> @@ -7,3 +7,4 @@
>> obj-$(CONFIG_AFE4403) += afe4403.o
>> obj-$(CONFIG_AFE4404) += afe4404.o
>> obj-$(CONFIG_MAX30100) += max30100.o
>> +obj-$(CONFIG_MAX30102) += max30102.o
>> diff --git a/drivers/iio/health/max30102.c b/drivers/iio/health/max30102.c
>> new file mode 100644
>> index 000000000000..87df1e6d712f
>> --- /dev/null
>> +++ b/drivers/iio/health/max30102.c
>> @@ -0,0 +1,485 @@
>> +/*
>> + * max30102.c - Support for MAX30102 heart rate and pulse oximeter sensor
>> + *
>> + * Copyright (C) 2017 Matt Ranostay <matt@ranostay.consulting>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * TODO: proximity power saving feature
>> + */
>> +
>> +#include <linux/module.h>
>> +#include <linux/init.h>
>> +#include <linux/interrupt.h>
>> +#include <linux/delay.h>
>> +#include <linux/err.h>
>> +#include <linux/irq.h>
>> +#include <linux/i2c.h>
>> +#include <linux/mutex.h>
>> +#include <linux/of.h>
>> +#include <linux/regmap.h>
>> +#include <linux/iio/iio.h>
>> +#include <linux/iio/buffer.h>
>> +#include <linux/iio/kfifo_buf.h>
>> +
>> +#define MAX30102_REGMAP_NAME "max30102_regmap"
>> +#define MAX30102_DRV_NAME "max30102"
>> +
>> +#define MAX30102_REG_INT_STATUS 0x00
>> +#define MAX30102_REG_INT_STATUS_PWR_RDY BIT(0)
>> +#define MAX30102_REG_INT_STATUS_PROX_INT BIT(4)
>> +#define MAX30102_REG_INT_STATUS_ALC_OVF BIT(5)
>> +#define MAX30102_REG_INT_STATUS_PPG_RDY BIT(6)
>> +#define MAX30102_REG_INT_STATUS_FIFO_RDY BIT(7)
>> +
>> +#define MAX30102_REG_INT_ENABLE 0x02
>> +#define MAX30102_REG_INT_ENABLE_PROX_INT_EN BIT(4)
>> +#define MAX30102_REG_INT_ENABLE_ALC_OVF_EN BIT(5)
>> +#define MAX30102_REG_INT_ENABLE_PPG_EN BIT(6)
>> +#define MAX30102_REG_INT_ENABLE_FIFO_EN BIT(7)
>> +#define MAX30102_REG_INT_ENABLE_MASK 0xf0
>> +#define MAX30102_REG_INT_ENABLE_MASK_SHIFT 4
>> +
>> +#define MAX30102_REG_FIFO_WR_PTR 0x04
>> +#define MAX30102_REG_FIFO_OVR_CTR 0x05
>> +#define MAX30102_REG_FIFO_RD_PTR 0x06
>> +#define MAX30102_REG_FIFO_DATA 0x07
>> +#define MAX30102_REG_FIFO_DATA_ENTRY_LEN 6
>> +
>> +#define MAX30102_REG_FIFO_CONFIG 0x08
>> +#define MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES BIT(1)
>> +#define MAX30102_REG_FIFO_CONFIG_AVG_SHIFT 5
>> +#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_PWR BIT(7)
>> +
>> +#define MAX30102_REG_SPO2_CONFIG 0x0a
>> +#define MAX30102_REG_SPO2_CONFIG_PULSE_411_US 0x03
>> +#define MAX30102_REG_SPO2_CONFIG_SR_400HZ 0x03
>> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK 0x07
>> +#define MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT 2
>> +#define MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS BIT(0)
>> +#define MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT 4
>> +
>> +#define MAX30102_REG_RED_LED_CONFIG 0x0c
>> +#define MAX30102_REG_IR_LED_CONFIG 0x0d
>> +
>> +#define MAX30102_REG_TEMP_CONFIG 0x21
>> +#define MAX30102_REG_TEMP_CONFIG_TEMP_EN BIT(0)
>> +
>> +#define MAX30102_REG_TEMP_INTEGER 0x1f
>> +#define MAX30102_REG_TEMP_FRACTION 0x20
>> +
>> +struct max30102_data {
>> + struct i2c_client *client;
>> + struct iio_dev *indio_dev;
>> + struct mutex lock;
>> + struct regmap *regmap;
>> +
>> + u8 buffer[8];
>> + __be32 processed_buffer[2]; /* 2 x 18-bit (padded to 32-bits) */
>> +};
>> +
>> +static const struct regmap_config max30102_regmap_config = {
>> + .name = MAX30102_REGMAP_NAME,
>> +
>> + .reg_bits = 8,
>> + .val_bits = 8,
>> +};
>> +
>> +static const unsigned long max30102_scan_masks[] = {0x3, 0};
>> +
>> +static const struct iio_chan_spec max30102_channels[] = {
>> + {
>> + .type = IIO_INTENSITY,
>> + .channel2 = IIO_MOD_LIGHT_RED,
>> + .modified = 1,
>> +
>> + .scan_index = 0,
>> + .scan_type = {
>> + .sign = 'u',
>> + .shift = 14,
>> + .realbits = 18,
>> + .storagebits = 32,
>> + .endianness = IIO_BE,
>> + },
>> + },
>> + {
>> + .type = IIO_INTENSITY,
>> + .channel2 = IIO_MOD_LIGHT_IR,
>> + .modified = 1,
>> +
>> + .scan_index = 1,
>> + .scan_type = {
>> + .sign = 'u',
>> + .shift = 14,
>> + .realbits = 18,
>> + .storagebits = 32,
>> + .endianness = IIO_BE,
>> + },
>> + },
>> + {
>> + .type = IIO_TEMP,
>> + .info_mask_separate =
>> + BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
>> + .scan_index = -1,
>> + },
>> +};
>> +
>> +static int max30102_set_powermode(struct max30102_data *data, bool state)
>> +{
>> + return regmap_update_bits(data->regmap, MAX30102_REG_MODE_CONFIG,
>> + MAX30102_REG_MODE_CONFIG_PWR,
>> + state ? 0 : MAX30102_REG_MODE_CONFIG_PWR);
>> +}
>> +
>> +static int max30102_buffer_postenable(struct iio_dev *indio_dev)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + return max30102_set_powermode(data, true);
>> +}
>> +
>> +static int max30102_buffer_predisable(struct iio_dev *indio_dev)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + return max30102_set_powermode(data, false);
>> +}
>> +
>> +static const struct iio_buffer_setup_ops max30102_buffer_setup_ops = {
>> + .postenable = max30102_buffer_postenable,
>> + .predisable = max30102_buffer_predisable,
>> +};
>> +
>> +static inline int max30102_fifo_count(struct max30102_data *data)
>> +{
>> + unsigned int val;
>> + int ret;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_INT_STATUS, &val);
>> + if (ret)
>> + return ret;
>> +
>> + /* FIFO has one sample slot left */
>> + if (val & MAX30102_REG_INT_STATUS_FIFO_RDY)
>> + return 1;
>> +
>> + return 0;
>> +}
>> +
>> +static int max30102_read_measurement(struct max30102_data *data)
>> +{
>> + int ret;
>> + u8 *buffer = (u8 *) &data->buffer;
>> +
>> + ret = i2c_smbus_read_i2c_block_data(data->client,
>> + MAX30102_REG_FIFO_DATA,
>> + MAX30102_REG_FIFO_DATA_ENTRY_LEN,
>> + buffer);
>> +
>> + memcpy(&data->processed_buffer[0], &buffer[0], 3);
>> + memcpy(&data->processed_buffer[1], &buffer[3], 3);
>> +
>> + return (ret == MAX30102_REG_FIFO_DATA_ENTRY_LEN) ? 0 : -EINVAL;
>> +}
>> +
>> +static irqreturn_t max30102_interrupt_handler(int irq, void *private)
>> +{
>> + struct iio_dev *indio_dev = private;
>> + struct max30102_data *data = iio_priv(indio_dev);
>> + int ret, cnt = 0;
>> +
>> + mutex_lock(&data->lock);
>> +
>> + while (cnt || (cnt = max30102_fifo_count(data)) > 0) {
>> + ret = max30102_read_measurement(data);
>> + if (ret)
>> + break;
>> +
>> + iio_push_to_buffers(data->indio_dev, data->processed_buffer);
>> + cnt--;
>> + }
>> +
>> + mutex_unlock(&data->lock);
>> +
>> + return IRQ_HANDLED;
>> +}
>> +
>> +static int max30102_get_current_idx(unsigned int val, int *reg)
>> +{
>> + /* each step is 0.200 mA */
>> + *reg = val / 200;
>> +
>> + return *reg > 0xff ? -EINVAL : 0;
>> +}
>> +
>> +static int max30102_led_init(struct max30102_data *data)
>> +{
>> + struct device *dev = &data->client->dev;
>> + struct device_node *np = dev->of_node;
>> + unsigned int val;
>> + int reg, ret;
>> +
>> + ret = of_property_read_u32(np, "maxim,red-led-current-microamp", &val);
>> + if (ret) {
>> + dev_info(dev, "no red-led-current-microamp set\n");
>> +
>> + /* Default to 7 mA RED LED */
>> + val = 7000;
>> + }
>> +
>> + ret = max30102_get_current_idx(val, ®);
>> + if (ret) {
>> + dev_err(dev, "invalid RED LED current setting %d\n", val);
>> + return ret;
>> + }
>> +
>> + ret = regmap_write(data->regmap, MAX30102_REG_RED_LED_CONFIG, reg);
>> + if (ret)
>> + return ret;
>> +
>> + ret = of_property_read_u32(np, "maxim,ir-led-current-microamp", &val);
>> + if (ret) {
>> + dev_info(dev, "no ir-led-current-microamp set\n");
>> +
>> + /* Default to 7 mA IR LED */
>> + val = 7000;
>> + }
>> +
>> + ret = max30102_get_current_idx(val, ®);
>> + if (ret) {
>> + dev_err(dev, "invalid IR LED current setting %d", val);
>> + return ret;
>> + }
>> +
>> + return regmap_write(data->regmap, MAX30102_REG_IR_LED_CONFIG, reg);
>> +}
>> +
>> +static int max30102_chip_init(struct max30102_data *data)
>> +{
>> + int ret;
>> +
>> + /* setup LED current settings */
>> + ret = max30102_led_init(data);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable 18-bit HR + SPO2 readings at 400Hz */
>> + ret = regmap_write(data->regmap, MAX30102_REG_SPO2_CONFIG,
>> + (MAX30102_REG_SPO2_CONFIG_ADC_4096_STEPS
>> + << MAX30102_REG_SPO2_CONFIG_ADC_MASK_SHIFT) |
>> + (MAX30102_REG_SPO2_CONFIG_SR_400HZ
>> + << MAX30102_REG_SPO2_CONFIG_SR_MASK_SHIFT) |
>> + MAX30102_REG_SPO2_CONFIG_PULSE_411_US);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable 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);
>> + if (ret)
>> + return ret;
>> +
>> + /* average 4 samples + generate FIFO interrupt */
>> + ret = regmap_write(data->regmap, MAX30102_REG_FIFO_CONFIG,
>> + (MAX30102_REG_FIFO_CONFIG_AVG_4SAMPLES
>> + << MAX30102_REG_FIFO_CONFIG_AVG_SHIFT) |
>> + MAX30102_REG_FIFO_CONFIG_AFULL);
>> + if (ret)
>> + return ret;
>> +
>> + /* enable FIFO interrupt */
>> + return regmap_update_bits(data->regmap, MAX30102_REG_INT_ENABLE,
>> + MAX30102_REG_INT_ENABLE_MASK,
>> + MAX30102_REG_INT_ENABLE_FIFO_EN);
>> +}
>> +
>> +static int max30102_read_temp(struct max30102_data *data, int *val)
>> +{
>> + int ret;
>> + unsigned int reg;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_INTEGER, ®);
>> + if (ret < 0)
>> + return ret;
>> + *val = reg << 4;
>> +
>> + ret = regmap_read(data->regmap, MAX30102_REG_TEMP_FRACTION, ®);
>> + if (ret < 0)
>> + return ret;
>> +
>> + *val |= reg & 0xf;
>> + *val = sign_extend32(*val, 11);
>> +
>> + return 0;
>> +}
>> +
>> +static int max30102_get_temp(struct max30102_data *data, int *val)
>> +{
>> + int ret;
>> +
>> + /* start acquisition */
>> + ret = regmap_update_bits(data->regmap, MAX30102_REG_TEMP_CONFIG,
>> + MAX30102_REG_TEMP_CONFIG_TEMP_EN,
>> + MAX30102_REG_TEMP_CONFIG_TEMP_EN);
>> + if (ret)
>> + return ret;
>> +
>> + msleep(35);
>> +
>> + return max30102_read_temp(data, val);
>> +}
>> +
>> +static int max30102_read_raw(struct iio_dev *indio_dev,
>> + struct iio_chan_spec const *chan,
>> + int *val, int *val2, long mask)
>> +{
>> + struct max30102_data *data = iio_priv(indio_dev);
>> + int ret = -EINVAL;
>> +
>> + switch (mask) {
>> + case IIO_CHAN_INFO_RAW:
>> + /*
>> + * Temperature reading can only be acquired while engine
>> + * is running
>> + */
>> + mutex_lock(&indio_dev->mlock);
>> +
>> + if (!iio_buffer_enabled(indio_dev))
>> + ret = -EAGAIN;
> Issue with this is typically userspace will just try having another go
> immediately. -EBUSY would be normal way of indicating not in a state
> where it can do this, but here it's kind of inverted. Not obvious!
Ah good catch.. Now I see your point..
>> + else {
>> + ret = max30102_get_temp(data, val);
>> + if (!ret)
>> + ret = IIO_VAL_INT;
>> + }
>> +
>> + mutex_unlock(&indio_dev->mlock);
>> + break;
>> + case IIO_CHAN_INFO_SCALE:
>> + *val = 1; /* 0.0625 */
>> + *val2 = 16;
>> + ret = IIO_VAL_FRACTIONAL;
>> + break;
>> + }
>> +
>> + return ret;
>> +}
>> +
>> +static const struct iio_info max30102_info = {
>> + .driver_module = THIS_MODULE,
>> + .read_raw = max30102_read_raw,
>> +};
>> +
>> +static int max30102_probe(struct i2c_client *client,
>> + const struct i2c_device_id *id)
>> +{
>> + struct max30102_data *data;
>> + struct iio_buffer *buffer;
>> + struct iio_dev *indio_dev;
>> + int ret;
>> +
>> + indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
>> + if (!indio_dev)
>> + return -ENOMEM;
>> +
>> + buffer = devm_iio_kfifo_allocate(&client->dev);
>> + if (!buffer)
>> + return -ENOMEM;
>> +
>> + iio_device_attach_buffer(indio_dev, buffer);
>> +
>> + indio_dev->name = MAX30102_DRV_NAME;
>> + indio_dev->channels = max30102_channels;
>> + indio_dev->info = &max30102_info;
>> + indio_dev->num_channels = ARRAY_SIZE(max30102_channels);
>> + indio_dev->available_scan_masks = max30102_scan_masks;
>> + indio_dev->modes = (INDIO_BUFFER_SOFTWARE | INDIO_DIRECT_MODE);
>> + indio_dev->setup_ops = &max30102_buffer_setup_ops;
>> +
>> + data = iio_priv(indio_dev);
>> + data->indio_dev = indio_dev;
>> + data->client = client;
>> +
>> + mutex_init(&data->lock);
>> + i2c_set_clientdata(client, indio_dev);
>> +
>> + data->regmap = devm_regmap_init_i2c(client, &max30102_regmap_config);
>> + if (IS_ERR(data->regmap)) {
>> + dev_err(&client->dev, "regmap initialization failed.\n");
>> + return PTR_ERR(data->regmap);
>> + }
>> + max30102_set_powermode(data, false);
>> +
>> + ret = max30102_chip_init(data);
>> + if (ret)
>> + return ret;
>> +
>> + if (client->irq <= 0) {
>> + dev_err(&client->dev, "no valid irq defined\n");
>> + return -EINVAL;
>> + }
>> +
>> + ret = devm_request_threaded_irq(&client->dev, client->irq,
>> + NULL, max30102_interrupt_handler,
>> + IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
>> + "max30102_irq", indio_dev);
>> + if (ret) {
>> + dev_err(&client->dev, "request irq (%d) failed\n", client->irq);
>> + return ret;
>> + }
>> +
>> + return iio_device_register(indio_dev);
>> +}
>> +
>> +static int max30102_remove(struct i2c_client *client)
>> +{
>> + struct iio_dev *indio_dev = i2c_get_clientdata(client);
>> + struct max30102_data *data = iio_priv(indio_dev);
>> +
>> + iio_device_unregister(indio_dev);
>> + max30102_set_powermode(data, false);
>> +
>> + return 0;
>> +}
>> +
>> +static const struct i2c_device_id max30102_id[] = {
>> + { "max30102", 0 },
>> + {}
>> +};
>> +MODULE_DEVICE_TABLE(i2c, max30102_id);
>> +
>> +static const struct of_device_id max30102_dt_ids[] = {
>> + { .compatible = "maxim,max30102" },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(of, max30102_dt_ids);
>> +
>> +static struct i2c_driver max30102_driver = {
>> + .driver = {
>> + .name = MAX30102_DRV_NAME,
>> + .of_match_table = of_match_ptr(max30102_dt_ids),
>> + },
>> + .probe = max30102_probe,
>> + .remove = max30102_remove,
>> + .id_table = max30102_id,
>> +};
>> +module_i2c_driver(max30102_driver);
>> +
>> +MODULE_AUTHOR("Matt Ranostay <matt@ranostay.consulting>");
>> +MODULE_DESCRIPTION("MAX30102 heart rate and pulse oximeter sensor");
>> +MODULE_LICENSE("GPL");
>>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2017-01-29 4:31 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-25 7:04 [PATCH v2 0/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
[not found] ` <20170125070442.10715-1-matt-sk+viVC6FLCDq+mSdOJa79kegs52MxvZ@public.gmane.org>
2017-01-25 7:04 ` [PATCH v2 1/2] devicetree: add documentation for MAX30102 oximeter Matt Ranostay
2017-01-25 7:04 ` Matt Ranostay
[not found] ` <20170125070442.10715-2-matt-sk+viVC6FLCDq+mSdOJa79kegs52MxvZ@public.gmane.org>
2017-01-27 22:36 ` Rob Herring
2017-01-27 22:36 ` Rob Herring
2017-01-25 7:04 ` [PATCH v2 2/2] iio: health: add MAX30102 oximeter driver support Matt Ranostay
2017-01-25 7:29 ` Peter Meerwald-Stadler
2017-01-25 8:24 ` Matt Ranostay
2017-01-25 9:43 ` Matt Ranostay
2017-01-27 7:11 ` Matt Ranostay
2017-01-28 11:51 ` Jonathan Cameron
2017-01-29 4:04 ` Matt Ranostay
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.