From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753009AbcBFTJY (ORCPT ); Sat, 6 Feb 2016 14:09:24 -0500 Received: from bear.ext.ti.com ([192.94.94.41]:56014 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751923AbcBFTJW (ORCPT ); Sat, 6 Feb 2016 14:09:22 -0500 Subject: Re: [PATCH v5 4/4] iio: health: Add driver for the TI AFE4403 heart monitor To: Jonathan Cameron , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald References: <1454435446-20586-1-git-send-email-afd@ti.com> <1454435446-20586-5-git-send-email-afd@ti.com> <56B64353.4060601@kernel.org> CC: , , , From: "Andrew F. Davis" Message-ID: <56B644C2.9060301@ti.com> Date: Sat, 6 Feb 2016 13:08:50 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <56B64353.4060601@kernel.org> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/06/2016 01:02 PM, Jonathan Cameron wrote: > On 02/02/16 17:50, Andrew F. Davis wrote: >> Add driver for the TI AFE4403 heart rate monitor and pulse oximeter. >> This device detects reflected LED light fluctuations and presents an ADC >> value to the user space for further signal processing. >> >> Data sheet located here: >> http://www.ti.com/product/AFE4403/datasheet >> >> Signed-off-by: Andrew F. Davis > Same issue in probe unwinding of errors + you still have that > unusually ___cacheline_aligned data on the stack which I'm unconvinced works > (really not sure either way!) > Anyhow. 1 thing to far for me to feel confident about just fixing it up so > please reroll just this patch one last time. > > Jonathan Hmm, that's odd, it's fixed in my local branch, I must have pushed the wrong patch here, I'll push the new one here in a minute. Thanks, Andrew >> --- >> drivers/iio/health/Kconfig | 12 + >> drivers/iio/health/Makefile | 1 + >> drivers/iio/health/afe4403.c | 700 +++++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 713 insertions(+) >> create mode 100644 drivers/iio/health/afe4403.c >> >> diff --git a/drivers/iio/health/Kconfig b/drivers/iio/health/Kconfig >> index 632a14b..f0c1977 100644 >> --- a/drivers/iio/health/Kconfig >> +++ b/drivers/iio/health/Kconfig >> @@ -7,6 +7,18 @@ menu "Health Sensors" >> >> menu "Heart Rate Monitors" >> >> +config AFE4403 >> + tristate "TI AFE4403 Heart Rate Monitor" >> + depends on SPI_MASTER >> + select IIO_BUFFER >> + select IIO_TRIGGERED_BUFFER >> + help >> + Say yes to choose the Texas Instruments AFE4403 >> + heart rate monitor and low-cost pulse oximeter. >> + >> + To compile this driver as a module, choose M here: the >> + module will be called afe4403. >> + >> config AFE4404 >> tristate "TI AFE4404 heart rate and pulse oximeter sensor" >> depends on I2C >> diff --git a/drivers/iio/health/Makefile b/drivers/iio/health/Makefile >> index b37c0d5..9955a2a 100644 >> --- a/drivers/iio/health/Makefile >> +++ b/drivers/iio/health/Makefile >> @@ -4,5 +4,6 @@ >> >> # When adding new entries keep the list in alphabetical order >> >> +obj-$(CONFIG_AFE4403) += afe4403.o >> obj-$(CONFIG_AFE4404) += afe4404.o >> obj-$(CONFIG_MAX30100) += max30100.o >> diff --git a/drivers/iio/health/afe4403.c b/drivers/iio/health/afe4403.c >> new file mode 100644 >> index 0000000..d69acda >> --- /dev/null >> +++ b/drivers/iio/health/afe4403.c >> @@ -0,0 +1,700 @@ >> +/* >> + * AFE4403 Heart Rate Monitors and Low-Cost Pulse Oximeters >> + * >> + * Copyright (C) 2015 Texas Instruments Incorporated - http://www.ti.com/ >> + * Andrew F. Davis >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 as >> + * published by the Free Software Foundation. >> + * >> + * 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. >> + */ >> + >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> + >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> + >> +#include "afe440x.h" >> + >> +#define AFE4403_DRIVER_NAME "afe4403" >> + >> +/* AFE4403 Registers */ >> +#define AFE4403_TIAGAIN 0x20 >> +#define AFE4403_TIA_AMB_GAIN 0x21 >> + >> +/* AFE4403 GAIN register fields */ >> +#define AFE4403_TIAGAIN_RES_MASK GENMASK(2, 0) >> +#define AFE4403_TIAGAIN_RES_SHIFT 0 >> +#define AFE4403_TIAGAIN_CAP_MASK GENMASK(7, 3) >> +#define AFE4403_TIAGAIN_CAP_SHIFT 3 >> + >> +/* AFE4403 LEDCNTRL register fields */ >> +#define AFE440X_LEDCNTRL_LED1_MASK GENMASK(15, 8) >> +#define AFE440X_LEDCNTRL_LED1_SHIFT 8 >> +#define AFE440X_LEDCNTRL_LED2_MASK GENMASK(7, 0) >> +#define AFE440X_LEDCNTRL_LED2_SHIFT 0 >> +#define AFE440X_LEDCNTRL_LED_RANGE_MASK GENMASK(17, 16) >> +#define AFE440X_LEDCNTRL_LED_RANGE_SHIFT 16 >> + >> +/* AFE4403 CONTROL2 register fields */ >> +#define AFE440X_CONTROL2_PWR_DWN_TX BIT(2) >> +#define AFE440X_CONTROL2_EN_SLOW_DIAG BIT(8) >> +#define AFE440X_CONTROL2_DIAG_OUT_TRI BIT(10) >> +#define AFE440X_CONTROL2_TX_BRDG_MOD BIT(11) >> +#define AFE440X_CONTROL2_TX_REF_MASK GENMASK(18, 17) >> +#define AFE440X_CONTROL2_TX_REF_SHIFT 17 >> + >> +/* AFE4404 NULL fields */ >> +#define NULL_MASK 0 >> +#define NULL_SHIFT 0 >> + >> +/* AFE4403 LEDCNTRL values */ >> +#define AFE440X_LEDCNTRL_RANGE_TX_HALF 0x1 >> +#define AFE440X_LEDCNTRL_RANGE_TX_FULL 0x2 >> +#define AFE440X_LEDCNTRL_RANGE_TX_OFF 0x3 >> + >> +/* AFE4403 CONTROL2 values */ >> +#define AFE440X_CONTROL2_TX_REF_025 0x0 >> +#define AFE440X_CONTROL2_TX_REF_050 0x1 >> +#define AFE440X_CONTROL2_TX_REF_100 0x2 >> +#define AFE440X_CONTROL2_TX_REF_075 0x3 >> + >> +/* AFE4403 CONTROL3 values */ >> +#define AFE440X_CONTROL3_CLK_DIV_2 0x0 >> +#define AFE440X_CONTROL3_CLK_DIV_4 0x2 >> +#define AFE440X_CONTROL3_CLK_DIV_6 0x3 >> +#define AFE440X_CONTROL3_CLK_DIV_8 0x4 >> +#define AFE440X_CONTROL3_CLK_DIV_12 0x5 >> +#define AFE440X_CONTROL3_CLK_DIV_1 0x7 >> + >> +/* AFE4403 TIAGAIN_CAP values */ >> +#define AFE4403_TIAGAIN_CAP_5_P 0x0 >> +#define AFE4403_TIAGAIN_CAP_10_P 0x1 >> +#define AFE4403_TIAGAIN_CAP_20_P 0x2 >> +#define AFE4403_TIAGAIN_CAP_30_P 0x3 >> +#define AFE4403_TIAGAIN_CAP_55_P 0x8 >> +#define AFE4403_TIAGAIN_CAP_155_P 0x10 >> + >> +/* AFE4403 TIAGAIN_RES values */ >> +#define AFE4403_TIAGAIN_RES_500_K 0x0 >> +#define AFE4403_TIAGAIN_RES_250_K 0x1 >> +#define AFE4403_TIAGAIN_RES_100_K 0x2 >> +#define AFE4403_TIAGAIN_RES_50_K 0x3 >> +#define AFE4403_TIAGAIN_RES_25_K 0x4 >> +#define AFE4403_TIAGAIN_RES_10_K 0x5 >> +#define AFE4403_TIAGAIN_RES_1_M 0x6 >> +#define AFE4403_TIAGAIN_RES_NONE 0x7 >> + >> +/** >> + * struct afe4403_data >> + * @dev - Device structure >> + * @spi - SPI device handle >> + * @regmap - Register map of the device >> + * @regulator - Pointer to the regulator for the IC >> + * @trig - IIO trigger for this device >> + * @irq - ADC_RDY line interrupt number >> + */ >> +struct afe4403_data { >> + struct device *dev; >> + struct spi_device *spi; >> + struct regmap *regmap; >> + struct regulator *regulator; >> + struct iio_trigger *trig; >> + int irq; >> +}; >> + >> +enum afe4403_chan_id { >> + LED1, >> + ALED1, >> + LED2, >> + ALED2, >> + LED1_ALED1, >> + LED2_ALED2, >> + ILED1, >> + ILED2, >> +}; >> + >> +static const struct afe440x_reg_info afe4403_reg_info[] = { >> + [LED1] = AFE440X_REG_INFO(AFE440X_LED1VAL, 0, NULL), >> + [ALED1] = AFE440X_REG_INFO(AFE440X_ALED1VAL, 0, NULL), >> + [LED2] = AFE440X_REG_INFO(AFE440X_LED2VAL, 0, NULL), >> + [ALED2] = AFE440X_REG_INFO(AFE440X_ALED2VAL, 0, NULL), >> + [LED1_ALED1] = AFE440X_REG_INFO(AFE440X_LED1_ALED1VAL, 0, NULL), >> + [LED2_ALED2] = AFE440X_REG_INFO(AFE440X_LED2_ALED2VAL, 0, NULL), >> + [ILED1] = AFE440X_REG_INFO(AFE440X_LEDCNTRL, 0, AFE440X_LEDCNTRL_LED1), >> + [ILED2] = AFE440X_REG_INFO(AFE440X_LEDCNTRL, 0, AFE440X_LEDCNTRL_LED2), >> +}; >> + >> +static const struct iio_chan_spec afe4403_channels[] = { >> + /* ADC values */ >> + AFE440X_INTENSITY_CHAN(LED1, "led1", 0), >> + AFE440X_INTENSITY_CHAN(ALED1, "led1_ambient", 0), >> + AFE440X_INTENSITY_CHAN(LED2, "led2", 0), >> + AFE440X_INTENSITY_CHAN(ALED2, "led2_ambient", 0), >> + AFE440X_INTENSITY_CHAN(LED1_ALED1, "led1-led1_ambient", 0), >> + AFE440X_INTENSITY_CHAN(LED2_ALED2, "led2-led2_ambient", 0), >> + /* LED current */ >> + AFE440X_CURRENT_CHAN(ILED1, "led1"), >> + AFE440X_CURRENT_CHAN(ILED2, "led2"), >> +}; >> + >> +static const struct afe440x_val_table afe4403_res_table[] = { >> + { 500000 }, { 250000 }, { 100000 }, { 50000 }, >> + { 25000 }, { 10000 }, { 1000000 }, { 0 }, >> +}; >> +AFE440X_TABLE_ATTR(tia_resistance_available, afe4403_res_table); >> + >> +static const struct afe440x_val_table afe4403_cap_table[] = { >> + { 0, 5000 }, { 0, 10000 }, { 0, 20000 }, { 0, 25000 }, >> + { 0, 30000 }, { 0, 35000 }, { 0, 45000 }, { 0, 50000 }, >> + { 0, 55000 }, { 0, 60000 }, { 0, 70000 }, { 0, 75000 }, >> + { 0, 80000 }, { 0, 85000 }, { 0, 95000 }, { 0, 100000 }, >> + { 0, 155000 }, { 0, 160000 }, { 0, 170000 }, { 0, 175000 }, >> + { 0, 180000 }, { 0, 185000 }, { 0, 195000 }, { 0, 200000 }, >> + { 0, 205000 }, { 0, 210000 }, { 0, 220000 }, { 0, 225000 }, >> + { 0, 230000 }, { 0, 235000 }, { 0, 245000 }, { 0, 250000 }, >> +}; >> +AFE440X_TABLE_ATTR(tia_capacitance_available, afe4403_cap_table); >> + >> +static ssize_t afe440x_show_register(struct device *dev, >> + struct device_attribute *attr, >> + char *buf) >> +{ >> + struct iio_dev *indio_dev = dev_to_iio_dev(dev); >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + struct afe440x_attr *afe440x_attr = to_afe440x_attr(attr); >> + unsigned int reg_val, type; >> + int vals[2]; >> + int ret, val_len; >> + >> + ret = regmap_read(afe->regmap, afe440x_attr->reg, ®_val); >> + if (ret) >> + return ret; >> + >> + reg_val &= afe440x_attr->mask; >> + reg_val >>= afe440x_attr->shift; >> + >> + switch (afe440x_attr->type) { >> + case SIMPLE: >> + type = IIO_VAL_INT; >> + val_len = 1; >> + vals[0] = reg_val; >> + break; >> + case RESISTANCE: >> + case CAPACITANCE: >> + type = IIO_VAL_INT_PLUS_MICRO; >> + val_len = 2; >> + if (reg_val < afe440x_attr->table_size) { >> + vals[0] = afe440x_attr->val_table[reg_val].integer; >> + vals[1] = afe440x_attr->val_table[reg_val].fract; >> + break; >> + } >> + return -EINVAL; >> + default: >> + return -EINVAL; >> + } >> + >> + return iio_format_value(buf, type, val_len, vals); >> +} >> + >> +static ssize_t afe440x_store_register(struct device *dev, >> + struct device_attribute *attr, >> + const char *buf, size_t count) >> +{ >> + struct iio_dev *indio_dev = dev_to_iio_dev(dev); >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + struct afe440x_attr *afe440x_attr = to_afe440x_attr(attr); >> + int val, integer, fract, ret; >> + >> + ret = iio_str_to_fixpoint(buf, 100000, &integer, &fract); >> + if (ret) >> + return ret; >> + >> + switch (afe440x_attr->type) { >> + case SIMPLE: >> + val = integer; >> + break; >> + case RESISTANCE: >> + case CAPACITANCE: >> + for (val = 0; val < afe440x_attr->table_size; val++) >> + if (afe440x_attr->val_table[val].integer == integer && >> + afe440x_attr->val_table[val].fract == fract) >> + break; >> + if (val == afe440x_attr->table_size) >> + return -EINVAL; >> + break; >> + default: >> + return -EINVAL; >> + } >> + >> + ret = regmap_update_bits(afe->regmap, afe440x_attr->reg, >> + afe440x_attr->mask, >> + (val << afe440x_attr->shift)); >> + if (ret) >> + return ret; >> + >> + return count; >> +} >> + >> +static AFE440X_ATTR(tia_separate_en, AFE4403_TIAGAIN, AFE440X_TIAGAIN_ENSEPGAIN, SIMPLE, NULL, 0); >> + >> +static AFE440X_ATTR(tia_resistance1, AFE4403_TIAGAIN, AFE4403_TIAGAIN_RES, RESISTANCE, afe4403_res_table, ARRAY_SIZE(afe4403_res_table)); >> +static AFE440X_ATTR(tia_capacitance1, AFE4403_TIAGAIN, AFE4403_TIAGAIN_CAP, CAPACITANCE, afe4403_cap_table, ARRAY_SIZE(afe4403_cap_table)); >> + >> +static AFE440X_ATTR(tia_resistance2, AFE4403_TIA_AMB_GAIN, AFE4403_TIAGAIN_RES, RESISTANCE, afe4403_res_table, ARRAY_SIZE(afe4403_res_table)); >> +static AFE440X_ATTR(tia_capacitance2, AFE4403_TIA_AMB_GAIN, AFE4403_TIAGAIN_RES, CAPACITANCE, afe4403_cap_table, ARRAY_SIZE(afe4403_cap_table)); >> + >> +static struct attribute *afe440x_attributes[] = { >> + &afe440x_attr_tia_separate_en.dev_attr.attr, >> + &afe440x_attr_tia_resistance1.dev_attr.attr, >> + &afe440x_attr_tia_capacitance1.dev_attr.attr, >> + &afe440x_attr_tia_resistance2.dev_attr.attr, >> + &afe440x_attr_tia_capacitance2.dev_attr.attr, >> + &dev_attr_tia_resistance_available.attr, >> + &dev_attr_tia_capacitance_available.attr, >> + NULL >> +}; >> + >> +static const struct attribute_group afe440x_attribute_group = { >> + .attrs = afe440x_attributes >> +}; >> + >> +static int afe4403_read(struct afe4403_data *afe, unsigned int reg, u32 *val) >> +{ >> + u8 tx[4] = {AFE440X_CONTROL0, 0x0, 0x0, AFE440X_CONTROL0_READ}; >> + u8 rx[3]; >> + int ret; >> + >> + /* Enable reading from the device */ >> + ret = spi_write_then_read(afe->spi, tx, 4, NULL, 0); >> + if (ret) >> + goto out; >> + >> + ret = spi_write_then_read(afe->spi, ®, 1, rx, 3); >> + if (ret) >> + goto out; >> + >> + *val = (rx[0] << 16) | >> + (rx[1] << 8) | >> + (rx[2]); >> + >> + /* Disable reading from the device */ >> + tx[3] = AFE440X_CONTROL0_WRITE; >> + ret = spi_write_then_read(afe->spi, tx, 4, NULL, 0); >> + if (ret) >> + goto out; >> + >> +out: >> + >> + return ret; >> +} >> + >> +static int afe4403_read_raw(struct iio_dev *indio_dev, >> + struct iio_chan_spec const *chan, >> + int *val, int *val2, long mask) >> +{ >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + const struct afe440x_reg_info reg_info = afe4403_reg_info[chan->address]; >> + int ret; >> + >> + switch (chan->type) { >> + case IIO_INTENSITY: >> + switch (mask) { >> + case IIO_CHAN_INFO_RAW: >> + ret = afe4403_read(afe, reg_info.reg, val); >> + if (ret) >> + return ret; >> + return IIO_VAL_INT; >> + case IIO_CHAN_INFO_OFFSET: >> + ret = regmap_read(afe->regmap, reg_info.offreg, >> + val); >> + if (ret) >> + return ret; >> + *val &= reg_info.mask; >> + *val >>= reg_info.shift; >> + return IIO_VAL_INT; >> + } >> + break; >> + case IIO_CURRENT: >> + switch (mask) { >> + case IIO_CHAN_INFO_RAW: >> + ret = regmap_read(afe->regmap, reg_info.reg, val); >> + if (ret) >> + return ret; >> + *val &= reg_info.mask; >> + *val >>= reg_info.shift; >> + return IIO_VAL_INT; >> + case IIO_CHAN_INFO_SCALE: >> + *val = 0; >> + *val2 = 800000; >> + return IIO_VAL_INT_PLUS_MICRO; >> + } >> + break; >> + default: >> + break; >> + } >> + >> + return -EINVAL; >> +} >> + >> +static int afe4403_write_raw(struct iio_dev *indio_dev, >> + struct iio_chan_spec const *chan, >> + int val, int val2, long mask) >> +{ >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + const struct afe440x_reg_info reg_info = afe4403_reg_info[chan->address]; >> + >> + switch (chan->type) { >> + case IIO_INTENSITY: >> + switch (mask) { >> + case IIO_CHAN_INFO_OFFSET: >> + return regmap_update_bits(afe->regmap, >> + reg_info.offreg, >> + reg_info.mask, >> + (val << reg_info.shift)); >> + } >> + break; >> + case IIO_CURRENT: >> + switch (mask) { >> + case IIO_CHAN_INFO_RAW: >> + return regmap_update_bits(afe->regmap, >> + reg_info.reg, >> + reg_info.mask, >> + (val << reg_info.shift)); >> + } >> + break; >> + default: >> + break; >> + } >> + >> + return -EINVAL; >> +} >> + >> +static const struct iio_info afe4403_iio_info = { >> + .attrs = &afe440x_attribute_group, >> + .read_raw = afe4403_read_raw, >> + .write_raw = afe4403_write_raw, >> + .driver_module = THIS_MODULE, >> +}; >> + >> +static irqreturn_t afe4403_trigger_handler(int irq, void *private) >> +{ >> + struct iio_poll_func *pf = private; >> + struct iio_dev *indio_dev = pf->indio_dev; >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + int ret, bit, i = 0; >> + s32 buffer[8]; >> + u8 tx[4] ____cacheline_aligned = {AFE440X_CONTROL0, 0x0, 0x0, >> + AFE440X_CONTROL0_READ}; > I thought you were going to remove this on the basis that neither of > of was convinced if is safe. >> + u8 rx[3]; >> + >> + /* Enable reading from the device */ >> + ret = spi_write(afe->spi, tx, 4); >> + if (ret) >> + goto err; >> + >> + for_each_set_bit(bit, indio_dev->active_scan_mask, >> + indio_dev->masklength) { >> + ret = spi_write_then_read(afe->spi, >> + &afe4403_reg_info[bit].reg, 1, >> + rx, 3); >> + if (ret) >> + goto err; >> + >> + buffer[i++] = (rx[0] << 16) | >> + (rx[1] << 8) | >> + (rx[2]); >> + } >> + >> + /* Disable reading from the device */ >> + tx[3] = AFE440X_CONTROL0_WRITE; >> + ret = spi_write(afe->spi, tx, 4); >> + if (ret) >> + goto err; >> + >> + iio_push_to_buffers_with_timestamp(indio_dev, buffer, pf->timestamp); >> +err: >> + iio_trigger_notify_done(indio_dev->trig); >> + >> + return IRQ_HANDLED; >> +} >> + >> +static const struct iio_trigger_ops afe4403_trigger_ops = { >> + .owner = THIS_MODULE, >> +}; >> + >> +#define AFE4403_TIMING_PAIRS \ >> + { AFE440X_LED2STC, 0x000050 }, \ >> + { AFE440X_LED2ENDC, 0x0003e7 }, \ >> + { AFE440X_LED1LEDSTC, 0x0007d0 }, \ >> + { AFE440X_LED1LEDENDC, 0x000bb7 }, \ >> + { AFE440X_ALED2STC, 0x000438 }, \ >> + { AFE440X_ALED2ENDC, 0x0007cf }, \ >> + { AFE440X_LED1STC, 0x000820 }, \ >> + { AFE440X_LED1ENDC, 0x000bb7 }, \ >> + { AFE440X_LED2LEDSTC, 0x000000 }, \ >> + { AFE440X_LED2LEDENDC, 0x0003e7 }, \ >> + { AFE440X_ALED1STC, 0x000c08 }, \ >> + { AFE440X_ALED1ENDC, 0x000f9f }, \ >> + { AFE440X_LED2CONVST, 0x0003ef }, \ >> + { AFE440X_LED2CONVEND, 0x0007cf }, \ >> + { AFE440X_ALED2CONVST, 0x0007d7 }, \ >> + { AFE440X_ALED2CONVEND, 0x000bb7 }, \ >> + { AFE440X_LED1CONVST, 0x000bbf }, \ >> + { AFE440X_LED1CONVEND, 0x009c3f }, \ >> + { AFE440X_ALED1CONVST, 0x000fa7 }, \ >> + { AFE440X_ALED1CONVEND, 0x001387 }, \ >> + { AFE440X_ADCRSTSTCT0, 0x0003e8 }, \ >> + { AFE440X_ADCRSTENDCT0, 0x0003eb }, \ >> + { AFE440X_ADCRSTSTCT1, 0x0007d0 }, \ >> + { AFE440X_ADCRSTENDCT1, 0x0007d3 }, \ >> + { AFE440X_ADCRSTSTCT2, 0x000bb8 }, \ >> + { AFE440X_ADCRSTENDCT2, 0x000bbb }, \ >> + { AFE440X_ADCRSTSTCT3, 0x000fa0 }, \ >> + { AFE440X_ADCRSTENDCT3, 0x000fa3 }, \ >> + { AFE440X_PRPCOUNT, 0x009c3f }, \ >> + { AFE440X_PDNCYCLESTC, 0x001518 }, \ >> + { AFE440X_PDNCYCLEENDC, 0x00991f } >> + >> +static const struct reg_sequence afe4403_reg_sequences[] = { >> + AFE4403_TIMING_PAIRS, >> + { AFE440X_CONTROL1, AFE440X_CONTROL1_TIMEREN | 0x000007}, >> + { AFE4403_TIA_AMB_GAIN, AFE4403_TIAGAIN_RES_1_M }, >> + { AFE440X_LEDCNTRL, (0x14 << AFE440X_LEDCNTRL_LED1_SHIFT) | >> + (0x14 << AFE440X_LEDCNTRL_LED2_SHIFT) }, >> + { AFE440X_CONTROL2, AFE440X_CONTROL2_TX_REF_050 << >> + AFE440X_CONTROL2_TX_REF_SHIFT }, >> +}; >> + >> +static const struct regmap_range afe4403_yes_ranges[] = { >> + regmap_reg_range(AFE440X_LED2VAL, AFE440X_LED1_ALED1VAL), >> +}; >> + >> +static const struct regmap_access_table afe4403_volatile_table = { >> + .yes_ranges = afe4403_yes_ranges, >> + .n_yes_ranges = ARRAY_SIZE(afe4403_yes_ranges), >> +}; >> + >> +static const struct regmap_config afe4403_regmap_config = { >> + .reg_bits = 8, >> + .val_bits = 24, >> + >> + .max_register = AFE440X_PDNCYCLEENDC, >> + .cache_type = REGCACHE_RBTREE, >> + .volatile_table = &afe4403_volatile_table, >> +}; >> + >> +#ifdef CONFIG_OF >> +static const struct of_device_id afe4403_of_match[] = { >> + { .compatible = "ti,afe4403", }, >> + { /* sentinel */ } >> +}; >> +MODULE_DEVICE_TABLE(of, afe4403_of_match); >> +#endif >> + >> +static int afe4403_suspend(struct device *dev) >> +{ >> + struct iio_dev *indio_dev = dev_to_iio_dev(dev); >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + int ret; >> + >> + ret = regmap_update_bits(afe->regmap, AFE440X_CONTROL2, >> + AFE440X_CONTROL2_PDN_AFE, >> + AFE440X_CONTROL2_PDN_AFE); >> + if (ret) >> + return ret; >> + >> + ret = regulator_disable(afe->regulator); >> + if (ret) { >> + dev_err(dev, "Unable to disable regulator\n"); >> + return ret; >> + } >> + >> + return 0; >> +} >> + >> +static int afe4403_resume(struct device *dev) >> +{ >> + struct iio_dev *indio_dev = dev_to_iio_dev(dev); >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + int ret; >> + >> + ret = regulator_enable(afe->regulator); >> + if (ret) { >> + dev_err(dev, "Unable to enable regulator\n"); >> + return ret; >> + } >> + >> + ret = regmap_update_bits(afe->regmap, AFE440X_CONTROL2, >> + AFE440X_CONTROL2_PDN_AFE, 0); >> + if (ret) >> + return ret; >> + >> + return 0; >> +} >> + >> +static SIMPLE_DEV_PM_OPS(afe4403_pm_ops, afe4403_suspend, afe4403_resume); >> + >> +static int afe4403_probe(struct spi_device *spi) >> +{ >> + struct iio_dev *indio_dev; >> + struct afe4403_data *afe; >> + int ret; >> + >> + indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*afe)); >> + if (!indio_dev) >> + return -ENOMEM; >> + >> + afe = iio_priv(indio_dev); >> + spi_set_drvdata(spi, indio_dev); >> + >> + afe->dev = &spi->dev; >> + afe->spi = spi; >> + afe->irq = spi->irq; >> + >> + afe->regmap = devm_regmap_init_spi(spi, &afe4403_regmap_config); >> + if (IS_ERR(afe->regmap)) { >> + dev_err(afe->dev, "Unable to allocate register map\n"); >> + return PTR_ERR(afe->regmap); >> + } >> + >> + afe->regulator = devm_regulator_get(afe->dev, "tx_sup"); >> + if (IS_ERR(afe->regulator)) { >> + dev_err(afe->dev, "Unable to get regulator\n"); >> + return PTR_ERR(afe->regulator); >> + } >> + ret = regulator_enable(afe->regulator); >> + if (ret) { >> + dev_err(afe->dev, "Unable to enable regulator\n"); >> + return ret; >> + } >> + >> + ret = regmap_write(afe->regmap, AFE440X_CONTROL0, >> + AFE440X_CONTROL0_SW_RESET); >> + if (ret) { >> + dev_err(afe->dev, "Unable to reset device\n"); >> + return ret; >> + } >> + >> + ret = regmap_multi_reg_write(afe->regmap, afe4403_reg_sequences, >> + ARRAY_SIZE(afe4403_reg_sequences)); >> + if (ret) { >> + dev_err(afe->dev, "Unable to set register defaults\n"); >> + return ret; >> + } >> + >> + indio_dev->modes = INDIO_DIRECT_MODE; >> + indio_dev->dev.parent = afe->dev; >> + indio_dev->channels = afe4403_channels; >> + indio_dev->num_channels = ARRAY_SIZE(afe4403_channels); >> + indio_dev->name = AFE4403_DRIVER_NAME; >> + indio_dev->info = &afe4403_iio_info; >> + >> + if (afe->irq > 0) { >> + afe->trig = devm_iio_trigger_alloc(afe->dev, >> + "%s-dev%d", >> + indio_dev->name, >> + indio_dev->id); >> + if (!afe->trig) { >> + dev_err(afe->dev, "Unable to allocate IIO trigger\n"); >> + return -ENOMEM; >> + } >> + >> + iio_trigger_set_drvdata(afe->trig, indio_dev); >> + >> + afe->trig->ops = &afe4403_trigger_ops; >> + afe->trig->dev.parent = afe->dev; >> + >> + ret = iio_trigger_register(afe->trig); >> + if (ret) { >> + dev_err(afe->dev, "Unable to register IIO trigger\n"); >> + return ret; >> + } >> + >> + ret = devm_request_threaded_irq(afe->dev, afe->irq, >> + iio_trigger_generic_data_rdy_poll, >> + NULL, IRQF_ONESHOT, >> + AFE4403_DRIVER_NAME, >> + afe->trig); >> + if (ret) { >> + dev_err(afe->dev, "Unable to request IRQ\n"); >> + return ret; >> + } >> + } >> + >> + ret = iio_triggered_buffer_setup(indio_dev, &iio_pollfunc_store_time, >> + afe4403_trigger_handler, NULL); >> + if (ret) { >> + dev_err(afe->dev, "Unable to setup buffer\n"); >> + return ret; >> + } >> + >> + ret = iio_device_register(indio_dev); >> + if (ret) { >> + dev_err(afe->dev, "Unable to register IIO device\n"); >> + return ret; >> + } >> + >> + return 0; >> +} >> + >> +static int afe4403_remove(struct spi_device *spi) >> +{ >> + struct iio_dev *indio_dev = spi_get_drvdata(spi); >> + struct afe4403_data *afe = iio_priv(indio_dev); >> + int ret; >> + >> + iio_device_unregister(indio_dev); >> + >> + iio_triggered_buffer_cleanup(indio_dev); >> + >> + if (afe->irq > 0) >> + iio_trigger_unregister(afe->trig); >> + >> + ret = regulator_disable(afe->regulator); >> + if (ret) { >> + dev_err(afe->dev, "Unable to disable regulator\n"); >> + return ret; >> + } >> + >> + return 0; >> +} >> + >> +static const struct spi_device_id afe4403_ids[] = { >> + { "afe4403", 0 }, >> + { /* sentinel */ } >> +}; >> +MODULE_DEVICE_TABLE(spi, afe4403_ids); >> + >> +static struct spi_driver afe4403_spi_driver = { >> + .driver = { >> + .name = AFE4403_DRIVER_NAME, >> + .of_match_table = of_match_ptr(afe4403_of_match), >> + .pm = &afe4403_pm_ops, >> + }, >> + .probe = afe4403_probe, >> + .remove = afe4403_remove, >> + .id_table = afe4403_ids, >> +}; >> +module_spi_driver(afe4403_spi_driver); >> + >> +MODULE_AUTHOR("Andrew F. Davis "); >> +MODULE_DESCRIPTION("TI AFE4403 Heart Rate and Pulse Oximeter"); >> +MODULE_LICENSE("GPL v2"); >>