From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com ([134.134.136.65]:28331 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753634AbcCINap (ORCPT ); Wed, 9 Mar 2016 08:30:45 -0500 From: Tiberiu Breana To: linux-iio@vger.kernel.org Cc: Jonathan Cameron Subject: [PATCH 1/2] iio: temperature: Add support for MAX31722/MAX31723 temperature sensors Date: Wed, 9 Mar 2016 15:30:51 +0200 Message-Id: <1457530252-4984-2-git-send-email-tiberiu.a.breana@intel.com> In-Reply-To: <1457530252-4984-1-git-send-email-tiberiu.a.breana@intel.com> References: <1457530252-4984-1-git-send-email-tiberiu.a.breana@intel.com> Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org Add basic support for the Maxim Integrated MAX31722/MAX31723 SPI temperature sensors / thermostats. Includes: - ACPI support; - raw temperature readings; - ability to set measurement resolution; - power management Datasheet: https://datasheets.maximintegrated.com/en/ds/MAX31722-MAX31723.pdf Signed-off-by: Tiberiu Breana --- drivers/iio/temperature/Kconfig | 12 ++ drivers/iio/temperature/Makefile | 1 + drivers/iio/temperature/max31722.c | 374 +++++++++++++++++++++++++++++++++++++ 3 files changed, 387 insertions(+) create mode 100644 drivers/iio/temperature/max31722.c diff --git a/drivers/iio/temperature/Kconfig b/drivers/iio/temperature/Kconfig index c4664e5..7587887 100644 --- a/drivers/iio/temperature/Kconfig +++ b/drivers/iio/temperature/Kconfig @@ -3,6 +3,18 @@ # menu "Temperature sensors" +config MAX31722 + tristate "MAX31722 temperature sensor" + depends on SPI + select REGMAP_SPI + help + If you say yes here you get support for the Maxim Integrated + MAX31722/MAX31723 digital thermometers / thermostats operating + over an SPI interface. + + This driver can also be built as a module. If so, the module will + be called max31722. + config MLX90614 tristate "MLX90614 contact-less infrared sensor" depends on I2C diff --git a/drivers/iio/temperature/Makefile b/drivers/iio/temperature/Makefile index 02bc79d..a9484d6 100644 --- a/drivers/iio/temperature/Makefile +++ b/drivers/iio/temperature/Makefile @@ -2,6 +2,7 @@ # Makefile for industrial I/O temperature drivers # +obj-$(CONFIG_MAX31722) += max31722.o obj-$(CONFIG_MLX90614) += mlx90614.o obj-$(CONFIG_TMP006) += tmp006.o obj-$(CONFIG_TSYS01) += tsys01.o diff --git a/drivers/iio/temperature/max31722.c b/drivers/iio/temperature/max31722.c new file mode 100644 index 0000000..fa833b6 --- /dev/null +++ b/drivers/iio/temperature/max31722.c @@ -0,0 +1,374 @@ +/** + * MAX31722/MAX31723 digital thermometer and thermostat SPI driver + * + * Copyright (c) 2016, Intel Corporation. + * + * This file is subject to the terms and conditions of version 2 of + * the GNU General Public License. See the file COPYING in the main + * directory of this archive for more details. + */ + +#include +#include +#include +#include +#include +#include +#include + +#define MAX31722_REG_CFG 0x00 +#define MAX31722_REG_TEMP_LSB 0x01 +#define MAX31722_REG_TEMP_MSB 0x02 +#define MAX31722_MAX_REG 0x86 + +#define MAX31722_MODE_CONTINUOUS 0x00 +#define MAX31722_MODE_STANDBY 0x01 + +#define MAX31722_REGMAP_NAME "max31722_regmap" + +#define MAX31722_SCALE_AVAILABLE "0.5 0.25 0.125 0.0625" + +#define MAX31722_REGFIELD(name) \ + do { \ + data->reg_##name = \ + devm_regmap_field_alloc(&spi->dev, regmap, \ + max31722_reg_field_##name); \ + if (IS_ERR(data->reg_##name)) { \ + dev_err(&spi->dev, "reg field alloc failed.\n"); \ + return PTR_ERR(data->reg_##name); \ + } \ + } while (0) + +static const struct reg_field max31722_reg_field_state = + REG_FIELD(MAX31722_REG_CFG, 0, 0); +static const struct reg_field max31722_reg_field_resolution = + REG_FIELD(MAX31722_REG_CFG, 1, 2); + +struct max31722_data { + struct spi_device *spi_device; + struct mutex lock; + struct regmap *regmap; + struct regmap_field *reg_state; + struct regmap_field *reg_resolution; +}; + +/* + * This table can double as a negative exponent value lookup table + * (as micro units) for calculating LSB temperature values. + */ +static const int max31722_scale_table[4] = { + 500000, 250000, 125000, 62500 +}; + +static IIO_CONST_ATTR(in_temp_scale_available, MAX31722_SCALE_AVAILABLE); + +static struct attribute *max31722_attributes[] = { + &iio_const_attr_in_temp_scale_available.dev_attr.attr, + NULL +}; + +static const struct attribute_group max31722_attribute_group = { + .attrs = max31722_attributes +}; + +static const struct iio_chan_spec max31722_channels[] = { + { + .type = IIO_TEMP, + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) | + BIT(IIO_CHAN_INFO_SCALE), + }, +}; + +static bool max31722_is_volatile_reg(struct device *dev, unsigned int reg) +{ + switch (reg) { + case MAX31722_REG_TEMP_LSB: + case MAX31722_REG_TEMP_MSB: + return true; + default: + return false; + } +} + +static bool max31722_is_writeable_reg(struct device *dev, unsigned int reg) +{ + switch (reg) { + case MAX31722_REG_TEMP_LSB: + case MAX31722_REG_TEMP_MSB: + return false; + default: + return true; + } +} + +static struct regmap_config max31722_regmap_config = { + .name = MAX31722_REGMAP_NAME, + .reg_bits = 8, + .val_bits = 8, + + .max_register = MAX31722_MAX_REG, + .cache_type = REGCACHE_RBTREE, + + .volatile_reg = max31722_is_volatile_reg, + .writeable_reg = max31722_is_writeable_reg, + + .read_flag_mask = 0x00, + .write_flag_mask = 0x80, +}; + +/* + * Convert a temperature register value to a floating point value. + * Data is represented in 9-12 bits, two's complement. + * The MSB contains the integer part of the temperature value + * (signed), while the LSB contains the fractional part as + * negative powers of 2, e.g.: bit 0 is 2^-1, bit 1 is 2^-2, etc. + */ +static void max31722_reg_to_float(u16 reg_val, int *val, int *val2) +{ + int i; + u8 msb; + u8 lsb; + + msb = reg_val >> 8; + lsb = (reg_val << 8) >> 8; + + /* Calculate the integer part. */ + if (msb & 0x80) { + /* Negative value. Reverse bits and add 1. */ + u16 orig = reg_val; + u16 rev = 0; + int num_bits = sizeof(rev) * 8; + + for (i = 0 ; i < num_bits ; i++) { + rev |= !(orig & 0x01) << i; + orig >>= 1; + } + rev += 1; + msb = rev >> 8; + lsb = (rev << 8) >> 8; + *val = (-1) * msb; + } else { /* Positive value. */ + *val = msb; + } + + /* + * Calculate the fractional part. + * Only the first four LSB bits contain data. + */ + lsb >>= 4; + i = 0; + *val2 = 0; + while (i < 4) { + if ((lsb >> (4 - i - 1)) & 0x01) + *val2 += max31722_scale_table[i]; + lsb <<= 1; + lsb >>= 1; + i++; + } +} + +static int max31722_read_raw(struct iio_dev *indio_dev, + struct iio_chan_spec const *chan, + int *val, int *val2, long mask) +{ + int ret; + int index; + u16 buf; + struct max31722_data *data = iio_priv(indio_dev); + struct spi_device *spi = data->spi_device; + + switch (mask) { + case IIO_CHAN_INFO_RAW: + mutex_lock(&data->lock); + ret = regmap_bulk_read(data->regmap, + MAX31722_REG_TEMP_LSB, &buf, 2); + if (ret < 0) { + dev_err(&spi->dev, "failed to read temperature register\n"); + mutex_unlock(&data->lock); + return ret; + } + max31722_reg_to_float(buf, val, val2); + mutex_unlock(&data->lock); + return IIO_VAL_INT_PLUS_MICRO; + case IIO_CHAN_INFO_SCALE: + mutex_lock(&data->lock); + ret = regmap_field_read(data->reg_resolution, &index); + if (ret < 0) { + dev_err(&spi->dev, "failed to read configuration register\n"); + mutex_unlock(&data->lock); + return ret; + } + mutex_unlock(&data->lock); + *val = 0; + *val2 = max31722_scale_table[index]; + return IIO_VAL_INT_PLUS_MICRO; + } + + return -EINVAL; +} + +static int max31722_write_raw(struct iio_dev *indio_dev, + struct iio_chan_spec const *chan, + int val, int val2, long mask) +{ + int i; + int ret; + int index = -1; + struct max31722_data *data = iio_priv(indio_dev); + + switch (mask) { + case IIO_CHAN_INFO_SCALE: + for (i = 0; i < ARRAY_SIZE(max31722_scale_table); i++) { + if (val == 0 && val2 == max31722_scale_table[i]) { + index = i; + break; + } + } + if (index < 0) + return -EINVAL; + + mutex_lock(&data->lock); + ret = regmap_field_write(data->reg_resolution, index); + mutex_unlock(&data->lock); + + return ret; + } + return -EINVAL; +} + +static const struct iio_info max31722_info = { + .driver_module = THIS_MODULE, + .read_raw = max31722_read_raw, + .write_raw = max31722_write_raw, + .attrs = &max31722_attribute_group, +}; + +static int max31722_init(struct max31722_data *data) +{ + int ret = 0; + struct spi_device *spi = data->spi_device; + struct regmap *regmap; + + /* Initialize the regmap */ + regmap = devm_regmap_init_spi(spi, &max31722_regmap_config); + if (IS_ERR(regmap)) { + dev_err(&spi->dev, "regmap initialization failed.\n"); + return PTR_ERR(regmap); + } + data->regmap = regmap; + + MAX31722_REGFIELD(state); + MAX31722_REGFIELD(resolution); + + /* Set SD bit to 0 so we can have continuous measurements. */ + ret = regmap_field_write(data->reg_state, MAX31722_MODE_CONTINUOUS); + if (ret < 0) + dev_err(&spi->dev, "failed to configure sensor.\n"); + + return ret; +} + +static int max31722_probe(struct spi_device *spi) +{ + int ret; + struct iio_dev *indio_dev; + struct max31722_data *data; + + indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*data)); + if (!indio_dev) { + dev_err(&spi->dev, "iio allocation failed!\n"); + return -ENOMEM; + } + + data = iio_priv(indio_dev); + data->spi_device = spi; + spi_set_drvdata(spi, indio_dev); + mutex_init(&data->lock); + + indio_dev->dev.parent = &spi->dev; + indio_dev->info = &max31722_info; + indio_dev->name = spi_get_device_id(spi)->name; + indio_dev->modes = INDIO_DIRECT_MODE; + indio_dev->channels = max31722_channels; + indio_dev->num_channels = ARRAY_SIZE(max31722_channels); + + ret = max31722_init(data); + if (ret < 0) + return ret; + + ret = iio_device_register(indio_dev); + if (ret < 0) { + dev_err(&spi->dev, "iio_device_register failed\n"); + regmap_field_write(data->reg_state, MAX31722_MODE_STANDBY); + } + + return ret; +} + +static int max31722_remove(struct spi_device *spi) +{ + struct max31722_data *data; + struct iio_dev *indio_dev = spi_get_drvdata(spi); + + iio_device_unregister(indio_dev); + data = iio_priv(indio_dev); + + return regmap_field_write(data->reg_state, MAX31722_MODE_STANDBY); +} + +#ifdef CONFIG_PM_SLEEP +static int max31722_suspend(struct device *dev) +{ + struct max31722_data *data; + + data = iio_priv(spi_get_drvdata(to_spi_device(dev))); + + return regmap_field_write(data->reg_state, MAX31722_MODE_STANDBY); +} + +static int max31722_resume(struct device *dev) +{ + struct max31722_data *data; + + data = iio_priv(spi_get_drvdata(to_spi_device(dev))); + + return regmap_field_write(data->reg_state, MAX31722_MODE_CONTINUOUS); +} + +static SIMPLE_DEV_PM_OPS(max31722_pm_ops, max31722_suspend, max31722_resume); + +#define MAX31722_PM_OPS (&max31722_pm_ops) +#else +#define MAX31722_PM_OPS NULL +#endif + +static const struct spi_device_id max31722_spi_id[] = { + {"max31722", 0}, + {"max31723", 0}, + {} +}; + +static const struct acpi_device_id max31722_acpi_id[] = { + {"MAX31722", 0}, + {"MAX31723", 0}, + {} +}; + +MODULE_DEVICE_TABLE(spi, max31722_spi_id); + +static struct spi_driver max31722_driver = { + .driver = { + .name = "max31722", + .pm = MAX31722_PM_OPS, + .acpi_match_table = ACPI_PTR(max31722_acpi_id), + }, + .probe = max31722_probe, + .remove = max31722_remove, + .id_table = max31722_spi_id, +}; + +module_spi_driver(max31722_driver); + +MODULE_AUTHOR("Tiberiu Breana "); +MODULE_DESCRIPTION("MAX31722 digital thermometer and thermostat SPI driver"); +MODULE_LICENSE("GPL v2"); -- 1.9.1