On Sun, Dec 16, 2012 at 09:33:09PM -0800, Guenter Roeck wrote: > Add support for MAX6581, MAX6602, MAX6622, MAX6636, MAX6689, MAX6693, > MAX6694, MAX6697, MAX6698, and MAX6699 temperature sensors > > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> I plan to line this driver up for 3.9. Only change from the code below is to replace SENSORS_LIMIT with clamp_val. Any objections, make yourself heard. Thanks, Guenter > --- > v2: > - Add suppport for platform data and devicetree based chip initialization > - Drop S_IRUGOWU macro: s/S_IRUGOWU/S_IRUGO | S_IWUSR/ > > Documentation/devicetree/bindings/i2c/max6697.txt | 45 ++ > Documentation/hwmon/max6697 | 57 ++ > drivers/hwmon/Kconfig | 11 + > drivers/hwmon/Makefile | 1 + > drivers/hwmon/max6697.c | 634 +++++++++++++++++++++ > include/linux/platform_data/max6697.h | 25 + > 6 files changed, 773 insertions(+) > create mode 100644 Documentation/devicetree/bindings/i2c/max6697.txt > create mode 100644 Documentation/hwmon/max6697 > create mode 100644 drivers/hwmon/max6697.c > create mode 100644 include/linux/platform_data/max6697.h > > diff --git a/Documentation/devicetree/bindings/i2c/max6697.txt b/Documentation/devicetree/bindings/i2c/max6697.txt > new file mode 100644 > index 0000000..3e867e2 > --- /dev/null > +++ b/Documentation/devicetree/bindings/i2c/max6697.txt > @@ -0,0 +1,45 @@ > +max6697 properties > + > +Required properties: > +- compatible: > + Should be one of > + maxim,max6581 > + maxim,max6602 > + maxim,max6622 > + maxim,max6636 > + maxim,max6689 > + maxim,max6693 > + maxim,max6694 > + maxim,max6697 > + maxim,max6698 > + maxim,max6699 > +- reg: I2C address > + > +Optional properties: > +- smbus-timeout-disable > + Set to enable SMBus timeout > +- extended-range-enable > + Only valid for MAX6581. Set to enable extended temperature range. > +- alert-mask > + Alert bit mask. Alert disabled for bits set. > +- over-temperature-mask > + Over temperature bit mask. Over temperature reporting disabled for > + bits set. > +- resistance-cancellation > + Boolean for all chips other than MAX6581. Enabled if set. > + For MAX6581, resistance cancellation enabled for all channels if > + specified as boolean, otherwise as per bit mask specified. > +- transistor-ideality > + For MAX6581 only. Two values; first is bit mask, second is ideality > + select value as per MAX6581 data sheet. > + > +Example: > + > +temp-sensor@1a { > + compatible = "maxim,max6697"; > + reg = <0x1a>; > + smbus-timeout-disable; > + resistance-cancellation; > + alert-mask = <0xff>; > + over-temperature-mask = <0xff>; > +}; > diff --git a/Documentation/hwmon/max6697 b/Documentation/hwmon/max6697 > new file mode 100644 > index 0000000..35fc2e9 > --- /dev/null > +++ b/Documentation/hwmon/max6697 > @@ -0,0 +1,57 @@ > +Kernel driver max6697 > +===================== > + > +Supported chips: > + * Maxim MAX6581 > + Prefix: 'max6581' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6581.pdf > + * Maxim MAX6602 > + Prefix: 'max6602' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6602.pdf > + * Maxim MAX6622 > + Prefix: 'max6622' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6622.pdf > + * Maxim MAX6636 > + Prefix: 'max6636' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6636.pdf > + * Maxim MAX6689 > + Prefix: 'max6689' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6689.pdf > + * Maxim MAX6693 > + Prefix: 'max6693' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6693.pdf > + * Maxim MAX6694 > + Prefix: 'max6694' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6694.pdf > + * Maxim MAX6697 > + Prefix: 'max6697' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6697.pdf > + * Maxim MAX6698 > + Prefix: 'max6698' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6698.pdf > + * Maxim MAX6699 > + Prefix: 'max6699' > + Datasheet: http://datasheets.maximintegrated.com/en/ds/MAX6699.pdf > + > +Author: > + Guenter Roeck <linux@xxxxxxxxxxxx> > + > +Description > +----------- > + > +This driver implements support for several MAX6697 compatible temperature sensor > +chips. The chips support one local temperature sensor plus four or six remote > +temperature sensors. Remote temperature sensors are diode-connected thermal > +transitors, except for MAX6698 which supports three diode-connected thermal > +transistors plus three thermistors in addition to the local temperature sensor. > + > +The driver provides the following sysfs attributes. temp1 is the local (chip) > +temperature, temp[2..n] are remote temperatures. The actually supported > +per-channel attributes are chip type and channel dependent. > + > +tempX_input ro remote temperature > +tempX_max rw remote temperature maximum threshold for alarm > +tempX_max_alarm ro remote temperature maximum threshold alarm > +tempX_crit rw remote temperature critical threshold for alarm > +tempX_crit_alarm ro remote temperature critical threshold alarm > +tempX_fault ro remote temperature diode fault > diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig > index c4633de..14f7ac9 100644 > --- a/drivers/hwmon/Kconfig > +++ b/drivers/hwmon/Kconfig > @@ -844,6 +844,17 @@ config SENSORS_MAX6650 > This driver can also be built as a module. If so, the module > will be called max6650. > > +config SENSORS_MAX6697 > + tristate "Maxim MAX6697 and compatibles" > + depends on I2C > + help > + If you say yes here you get support for MAX6581, MAX6602, MAX6622, > + MAX6636, MAX6689, MAX6693, MAX6694, MAX6697, MAX6698, and MAX6699 > + temperature sensor chips. > + > + This driver can also be built as a module. If so, the module > + will be called max6697. > + > config SENSORS_MCP3021 > tristate "Microchip MCP3021 and compatibles" > depends on I2C > diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile > index 8d5fcb5..d23646d 100644 > --- a/drivers/hwmon/Makefile > +++ b/drivers/hwmon/Makefile > @@ -98,6 +98,7 @@ obj-$(CONFIG_SENSORS_MAX197) += max197.o > obj-$(CONFIG_SENSORS_MAX6639) += max6639.o > obj-$(CONFIG_SENSORS_MAX6642) += max6642.o > obj-$(CONFIG_SENSORS_MAX6650) += max6650.o > +obj-$(CONFIG_SENSORS_MAX6697) += max6697.o > obj-$(CONFIG_SENSORS_MC13783_ADC)+= mc13783-adc.o > obj-$(CONFIG_SENSORS_MCP3021) += mcp3021.o > obj-$(CONFIG_SENSORS_NTC_THERMISTOR) += ntc_thermistor.o > diff --git a/drivers/hwmon/max6697.c b/drivers/hwmon/max6697.c > new file mode 100644 > index 0000000..597cb98 > --- /dev/null > +++ b/drivers/hwmon/max6697.c > @@ -0,0 +1,634 @@ > +/* > + * Copyright (c) 2012 Guenter Roeck <linux@xxxxxxxxxxxx> > + * > + * based on max1668.c > + * Copyright (c) 2011 David George <david.george@xxxxxxxxx> > + * > + * 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. > + */ > + > +#include <linux/module.h> > +#include <linux/init.h> > +#include <linux/slab.h> > +#include <linux/jiffies.h> > +#include <linux/i2c.h> > +#include <linux/hwmon.h> > +#include <linux/hwmon-sysfs.h> > +#include <linux/err.h> > +#include <linux/mutex.h> > +#include <linux/of.h> > + > +#include <linux/platform_data/max6697.h> > + > +enum chips { max6581, max6602, max6622, max6636, max6689, max6693, max6694, > + max6697, max6698, max6699 }; > + > +/* Report local sensor as temp1 */ > + > +static const u8 MAX6697_REG_TEMP[] = { 7, 1, 2, 3, 4, 5, 6, 8 }; > +static const u8 MAX6697_REG_TEMP_EXT[] = { > + 0x57, 0x09, 0x52, 0x53, 0x54, 0x55, 0x56, 0 }; > +static const u8 MAX6697_REG_MAX[] = { > + 0x17, 0x11, 0x12, 0x13, 0x14, 0x15, 0x16, 0x18 }; > +static const u8 MAX6697_REG_CRIT[] = { > + 0x20, 0x21, 0x22, 0x23, 0x24, 0x25, 0x26, 0x27 }; > + > +#define MAX6697_REG_STAT(n) (0x44 + (n)) > + > +#define MAX6697_REG_CONFIG 0x41 > +#define MAX6581_CONF_EXTENDED (1 << 1) > +#define MAX6697_CONF_RESISTANCE (1 << 3) > +#define MAX6697_CONF_TIMEOUT (1 << 5) > +#define MAX6697_REG_ALERT_MASK 0x42 > +#define MAX6697_REG_OVERT_MASK 0x43 > + > +#define MAX6581_REG_RESISTANCE 0x4a > +#define MAX6581_REG_IDEALITY 0x4b > +#define MAX6581_REG_IDEALITY_SELECT 0x4c > +#define MAX6581_REG_OFFSET 0x4d > +#define MAX6581_REG_OFFSET_SELECT 0x4e > + > +struct max6697_chip_data { > + int channels; > + u32 have_ext; > + u32 have_crit; > + u32 have_fault; > + const u8 *alarm_map; > +}; > + > +struct max6697_data { > + struct device *hwmon_dev; > + > + enum chips type; > + const struct max6697_chip_data *chip; > + > + struct mutex update_lock; > + unsigned long last_updated; /* In jiffies */ > + bool valid; /* true if following fields are valid */ > + > + /* 1x local and up to 7x remote */ > + u8 temp[8][4]; /* [nr][0]=temp [1]=ext [2]=max [3]=crit */ > + u32 alarms; > +}; > + > +/* Diode fault status bits on MAX6581 are right shifted by one bit */ > +static const u8 max6581_alarm_map[] = { > + 0, 0, 1, 2, 3, 4, 5, 6, 8, 9, 10, 11, 12, 13, 14, 15, > + 16, 17, 18, 19, 20, 21, 22, 23 }; > + > +static const struct max6697_chip_data max6697_chip_data[] = { > + [max6581] = { > + .channels = 8, > + .have_crit = 0xff, > + .have_ext = 0xfe, > + .have_fault = 0xfe, > + .alarm_map = max6581_alarm_map, > + }, > + [max6602] = { > + .channels = 5, > + .have_crit = 0x12, > + .have_ext = 0x02, > + .have_fault = 0x1e, > + }, > + [max6636] = { > + .channels = 7, > + .have_crit = 0x72, > + .have_ext = 0x02, > + .have_fault = 0x7e, > + }, > + [max6622] = { > + .channels = 5, > + .have_crit = 0x12, > + .have_ext = 0x02, > + .have_fault = 0x1e, > + }, > + [max6689] = { > + .channels = 7, > + .have_crit = 0x72, > + .have_ext = 0x02, > + .have_fault = 0x7e, > + }, > + [max6693] = { > + .channels = 7, > + .have_crit = 0x72, > + .have_ext = 0x02, > + .have_fault = 0x7e, > + }, > + [max6694] = { > + .channels = 5, > + .have_crit = 0x12, > + .have_ext = 0x02, > + .have_fault = 0x1e, > + }, > + [max6697] = { > + .channels = 7, > + .have_crit = 0x72, > + .have_ext = 0x02, > + .have_fault = 0x7e, > + }, > + [max6698] = { > + .channels = 7, > + .have_crit = 0x72, > + .have_ext = 0x02, > + .have_fault = 0x0e, > + }, > + [max6699] = { > + .channels = 5, > + .have_crit = 0x12, > + .have_ext = 0x02, > + .have_fault = 0x1e, > + }, > +}; > + > +static struct max6697_data *max6697_update_device(struct device *dev) > +{ > + struct i2c_client *client = to_i2c_client(dev); > + struct max6697_data *data = i2c_get_clientdata(client); > + struct max6697_data *ret = data; > + int val; > + int i; > + > + mutex_lock(&data->update_lock); > + > + if (data->valid && > + !time_after(jiffies, data->last_updated + HZ + HZ / 2)) > + goto abort; > + > + for (i = 0; i < data->chip->channels; i++) { > + if (data->chip->have_ext & (1 << i)) { > + val = i2c_smbus_read_byte_data(client, > + MAX6697_REG_TEMP_EXT[i]); > + if (unlikely(val < 0)) { > + ret = ERR_PTR(val); > + goto abort; > + } > + data->temp[i][1] = val; > + } > + > + val = i2c_smbus_read_byte_data(client, MAX6697_REG_TEMP[i]); > + if (unlikely(val < 0)) { > + ret = ERR_PTR(val); > + goto abort; > + } > + data->temp[i][0] = val; > + > + val = i2c_smbus_read_byte_data(client, MAX6697_REG_MAX[i]); > + if (unlikely(val < 0)) { > + ret = ERR_PTR(val); > + goto abort; > + } > + data->temp[i][2] = val; > + > + if (data->chip->have_crit & (1 << i)) { > + val = i2c_smbus_read_byte_data(client, > + MAX6697_REG_CRIT[i]); > + if (unlikely(val < 0)) { > + ret = ERR_PTR(val); > + goto abort; > + } > + data->temp[i][3] = val; > + } > + } > + > + data->alarms = 0; > + for (i = 0; i < 3; i++) { > + val = i2c_smbus_read_byte_data(client, MAX6697_REG_STAT(i)); > + if (unlikely(val < 0)) { > + ret = ERR_PTR(val); > + goto abort; > + } > + data->alarms = (data->alarms << 8) | val; > + } > + > + data->last_updated = jiffies; > + data->valid = true; > +abort: > + mutex_unlock(&data->update_lock); > + > + return ret; > +} > + > +static ssize_t show_temp11(struct device *dev, > + struct device_attribute *devattr, char *buf) > +{ > + int nr = to_sensor_dev_attr_2(devattr)->nr; > + int index = to_sensor_dev_attr_2(devattr)->index; > + struct max6697_data *data = max6697_update_device(dev); > + > + if (IS_ERR(data)) > + return PTR_ERR(data); > + > + return sprintf(buf, "%d\n", > + ((data->temp[nr][index] << 3) | > + (data->temp[nr][index + 1] >> 5)) * 125); > +} > + > +static ssize_t show_temp(struct device *dev, > + struct device_attribute *devattr, char *buf) > +{ > + int nr = to_sensor_dev_attr_2(devattr)->nr; > + int index = to_sensor_dev_attr_2(devattr)->index; > + struct max6697_data *data = max6697_update_device(dev); > + > + if (IS_ERR(data)) > + return PTR_ERR(data); > + > + return sprintf(buf, "%d\n", data->temp[nr][index] * 1000); > +} > + > +static ssize_t show_alarm(struct device *dev, struct device_attribute *attr, > + char *buf) > +{ > + int index = to_sensor_dev_attr(attr)->index; > + struct max6697_data *data = max6697_update_device(dev); > + > + if (IS_ERR(data)) > + return PTR_ERR(data); > + > + if (data->chip->alarm_map) > + index = data->chip->alarm_map[index]; > + > + return sprintf(buf, "%u\n", (data->alarms >> index) & 0x1); > +} > + > +static ssize_t set_temp(struct device *dev, > + struct device_attribute *devattr, > + const char *buf, size_t count) > +{ > + int nr = to_sensor_dev_attr_2(devattr)->nr; > + int index = to_sensor_dev_attr_2(devattr)->index; > + struct i2c_client *client = to_i2c_client(dev); > + struct max6697_data *data = i2c_get_clientdata(client); > + long temp; > + int ret; > + > + ret = kstrtol(buf, 10, &temp); > + if (ret < 0) > + return ret; > + > + mutex_lock(&data->update_lock); > + data->temp[nr][index] = SENSORS_LIMIT(temp / 1000, 0, 127); > + ret = i2c_smbus_write_byte_data(client, > + index == 2 ? MAX6697_REG_MAX[nr] > + : MAX6697_REG_CRIT[nr], > + data->temp[nr][index]); > + if (ret < 0) > + count = ret; > + mutex_unlock(&data->update_lock); > + > + return count; > +} > + > +static SENSOR_DEVICE_ATTR_2(temp1_input, S_IRUGO, show_temp11, NULL, 0, 0); > +static SENSOR_DEVICE_ATTR_2(temp1_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 0, 2); > +static SENSOR_DEVICE_ATTR_2(temp1_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 0, 3); > + > +static SENSOR_DEVICE_ATTR_2(temp2_input, S_IRUGO, show_temp11, NULL, 1, 0); > +static SENSOR_DEVICE_ATTR_2(temp2_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 1, 2); > +static SENSOR_DEVICE_ATTR_2(temp2_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 1, 3); > + > +static SENSOR_DEVICE_ATTR_2(temp3_input, S_IRUGO, show_temp11, NULL, 2, 0); > +static SENSOR_DEVICE_ATTR_2(temp3_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 2, 2); > +static SENSOR_DEVICE_ATTR_2(temp3_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 2, 3); > + > +static SENSOR_DEVICE_ATTR_2(temp4_input, S_IRUGO, show_temp11, NULL, 3, 0); > +static SENSOR_DEVICE_ATTR_2(temp4_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 3, 2); > +static SENSOR_DEVICE_ATTR_2(temp4_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 3, 3); > + > +static SENSOR_DEVICE_ATTR_2(temp5_input, S_IRUGO, show_temp11, NULL, 4, 0); > +static SENSOR_DEVICE_ATTR_2(temp5_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 4, 2); > +static SENSOR_DEVICE_ATTR_2(temp5_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 4, 3); > + > +static SENSOR_DEVICE_ATTR_2(temp6_input, S_IRUGO, show_temp11, NULL, 5, 0); > +static SENSOR_DEVICE_ATTR_2(temp6_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 5, 2); > +static SENSOR_DEVICE_ATTR_2(temp6_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 5, 3); > + > +static SENSOR_DEVICE_ATTR_2(temp7_input, S_IRUGO, show_temp11, NULL, 6, 0); > +static SENSOR_DEVICE_ATTR_2(temp7_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 6, 2); > +static SENSOR_DEVICE_ATTR_2(temp7_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 6, 3); > + > +static SENSOR_DEVICE_ATTR_2(temp8_input, S_IRUGO, show_temp11, NULL, 7, 0); > +static SENSOR_DEVICE_ATTR_2(temp8_max, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 7, 2); > +static SENSOR_DEVICE_ATTR_2(temp8_crit, S_IRUGO | S_IWUSR, show_temp, set_temp, > + 7, 3); > + > +static SENSOR_DEVICE_ATTR(temp1_max_alarm, S_IRUGO, show_alarm, NULL, 22); > +static SENSOR_DEVICE_ATTR(temp2_max_alarm, S_IRUGO, show_alarm, NULL, 16); > +static SENSOR_DEVICE_ATTR(temp3_max_alarm, S_IRUGO, show_alarm, NULL, 17); > +static SENSOR_DEVICE_ATTR(temp4_max_alarm, S_IRUGO, show_alarm, NULL, 18); > +static SENSOR_DEVICE_ATTR(temp5_max_alarm, S_IRUGO, show_alarm, NULL, 19); > +static SENSOR_DEVICE_ATTR(temp6_max_alarm, S_IRUGO, show_alarm, NULL, 20); > +static SENSOR_DEVICE_ATTR(temp7_max_alarm, S_IRUGO, show_alarm, NULL, 21); > +static SENSOR_DEVICE_ATTR(temp8_max_alarm, S_IRUGO, show_alarm, NULL, 23); > + > +static SENSOR_DEVICE_ATTR(temp1_crit_alarm, S_IRUGO, show_alarm, NULL, 14); > +static SENSOR_DEVICE_ATTR(temp2_crit_alarm, S_IRUGO, show_alarm, NULL, 8); > +static SENSOR_DEVICE_ATTR(temp3_crit_alarm, S_IRUGO, show_alarm, NULL, 9); > +static SENSOR_DEVICE_ATTR(temp4_crit_alarm, S_IRUGO, show_alarm, NULL, 10); > +static SENSOR_DEVICE_ATTR(temp5_crit_alarm, S_IRUGO, show_alarm, NULL, 11); > +static SENSOR_DEVICE_ATTR(temp6_crit_alarm, S_IRUGO, show_alarm, NULL, 12); > +static SENSOR_DEVICE_ATTR(temp7_crit_alarm, S_IRUGO, show_alarm, NULL, 13); > +static SENSOR_DEVICE_ATTR(temp8_crit_alarm, S_IRUGO, show_alarm, NULL, 15); > + > +static SENSOR_DEVICE_ATTR(temp1_fault, S_IRUGO, show_alarm, NULL, 0); > +static SENSOR_DEVICE_ATTR(temp2_fault, S_IRUGO, show_alarm, NULL, 1); > +static SENSOR_DEVICE_ATTR(temp3_fault, S_IRUGO, show_alarm, NULL, 2); > +static SENSOR_DEVICE_ATTR(temp4_fault, S_IRUGO, show_alarm, NULL, 3); > +static SENSOR_DEVICE_ATTR(temp5_fault, S_IRUGO, show_alarm, NULL, 4); > +static SENSOR_DEVICE_ATTR(temp6_fault, S_IRUGO, show_alarm, NULL, 5); > +static SENSOR_DEVICE_ATTR(temp7_fault, S_IRUGO, show_alarm, NULL, 6); > +static SENSOR_DEVICE_ATTR(temp8_fault, S_IRUGO, show_alarm, NULL, 7); > + > +static struct attribute *max6697_attributes[] = { > + &sensor_dev_attr_temp1_input.dev_attr.attr, > + &sensor_dev_attr_temp1_max.dev_attr.attr, > + &sensor_dev_attr_temp1_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp1_crit.dev_attr.attr, > + &sensor_dev_attr_temp1_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp1_fault.dev_attr.attr, > + > + &sensor_dev_attr_temp2_input.dev_attr.attr, > + &sensor_dev_attr_temp2_max.dev_attr.attr, > + &sensor_dev_attr_temp2_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp2_crit.dev_attr.attr, > + &sensor_dev_attr_temp2_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp2_fault.dev_attr.attr, > + > + &sensor_dev_attr_temp3_input.dev_attr.attr, > + &sensor_dev_attr_temp3_max.dev_attr.attr, > + &sensor_dev_attr_temp3_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp3_crit.dev_attr.attr, > + &sensor_dev_attr_temp3_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp3_fault.dev_attr.attr, > + > + &sensor_dev_attr_temp4_input.dev_attr.attr, > + &sensor_dev_attr_temp4_max.dev_attr.attr, > + &sensor_dev_attr_temp4_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp4_crit.dev_attr.attr, > + &sensor_dev_attr_temp4_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp4_fault.dev_attr.attr, > + > + &sensor_dev_attr_temp5_input.dev_attr.attr, > + &sensor_dev_attr_temp5_max.dev_attr.attr, > + &sensor_dev_attr_temp5_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp5_crit.dev_attr.attr, > + &sensor_dev_attr_temp5_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp5_fault.dev_attr.attr, > + > + &sensor_dev_attr_temp6_input.dev_attr.attr, > + &sensor_dev_attr_temp6_max.dev_attr.attr, > + &sensor_dev_attr_temp6_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp6_crit.dev_attr.attr, > + &sensor_dev_attr_temp6_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp6_fault.dev_attr.attr, > + > + &sensor_dev_attr_temp7_input.dev_attr.attr, > + &sensor_dev_attr_temp7_max.dev_attr.attr, > + &sensor_dev_attr_temp7_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp7_crit.dev_attr.attr, > + &sensor_dev_attr_temp7_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp7_fault.dev_attr.attr, > + > + &sensor_dev_attr_temp8_input.dev_attr.attr, > + &sensor_dev_attr_temp8_max.dev_attr.attr, > + &sensor_dev_attr_temp8_max_alarm.dev_attr.attr, > + &sensor_dev_attr_temp8_crit.dev_attr.attr, > + &sensor_dev_attr_temp8_crit_alarm.dev_attr.attr, > + &sensor_dev_attr_temp8_fault.dev_attr.attr, > + > + NULL > +}; > + > +static const struct attribute_group max6697_group = { > + .attrs = max6697_attributes, > +}; > + > +static void max6697_get_config_of(struct device_node *node, > + struct max6697_platform_data *pdata) > +{ > + int len; > + const __be32 *prop; > + > + prop = of_get_property(node, "smbus-timeout-disable", &len); > + if (prop) > + pdata->smbus_timeout_disable = true; > + prop = of_get_property(node, "extended_range_enable", &len); > + if (prop) > + pdata->extended_range_enable = true; > + prop = of_get_property(node, "alert-mask", &len); > + if (prop && len == sizeof(u32)) > + pdata->alert_mask = be32_to_cpu(prop[0]); > + prop = of_get_property(node, "over-temperature-mask", &len); > + if (prop && len == sizeof(u32)) > + pdata->over_temperature_mask = be32_to_cpu(prop[0]); > + prop = of_get_property(node, "resistance-cancellation", &len); > + if (prop) { > + if (len == sizeof(u32)) > + pdata->resistance_cancellation = be32_to_cpu(prop[0]); > + else > + pdata->resistance_cancellation = 0xff; > + } > + prop = of_get_property(node, "transistor-ideality", &len); > + if (prop && len == 2 * sizeof(u32)) { > + pdata->ideality_mask = be32_to_cpu(prop[0]); > + pdata->ideality_value = be32_to_cpu(prop[1]); > + } > +} > + > +static int max6697_init_chip(struct i2c_client *client) > +{ > + struct max6697_data *data = i2c_get_clientdata(client); > + struct max6697_platform_data *pdata = dev_get_platdata(&client->dev); > + struct max6697_platform_data p; > + int ret; > + u8 reg; > + > + /* > + * Don't touch configuration if neither platform data nor of > + * configuration was specified. > + */ > + if (!pdata && !client->dev.of_node) > + return 0; > + > + if (!pdata || client->dev.of_node) { > + memset(&p, 0, sizeof(data)); > + max6697_get_config_of(client->dev.of_node, &p); > + pdata = &p; > + } > + > + reg = 0; > + if (pdata->smbus_timeout_disable) > + reg |= MAX6697_CONF_TIMEOUT; > + if (pdata->extended_range_enable && data->type == max6581) > + reg |= MAX6581_CONF_EXTENDED; > + if (pdata->resistance_cancellation && data->type != max6581) > + reg |= MAX6697_CONF_RESISTANCE; > + > + ret = i2c_smbus_write_byte_data(client, MAX6697_REG_CONFIG, reg); > + if (ret < 0) > + return ret; > + > + ret = i2c_smbus_write_byte_data(client, MAX6697_REG_ALERT_MASK, > + pdata->alert_mask); > + if (ret < 0) > + return ret; > + > + ret = i2c_smbus_write_byte_data(client, MAX6697_REG_OVERT_MASK, > + pdata->over_temperature_mask); > + if (ret < 0) > + return ret; > + > + if (data->type == max6581) { > + ret = i2c_smbus_write_byte_data(client, MAX6581_REG_RESISTANCE, > + pdata->resistance_cancellation); > + if (ret < 0) > + return ret; > + ret = i2c_smbus_write_byte_data(client, MAX6581_REG_IDEALITY, > + pdata->ideality_mask); > + if (ret < 0) > + return ret; > + ret = i2c_smbus_write_byte_data(client, > + MAX6581_REG_IDEALITY_SELECT, > + pdata->ideality_value); > + if (ret < 0) > + return ret; > + } > + return ret; > +} > + > +static int max6697_probe(struct i2c_client *client, > + const struct i2c_device_id *id) > +{ > + struct i2c_adapter *adapter = client->adapter; > + struct device *dev = &client->dev; > + struct max6697_data *data; > + int i, err; > + > + if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA)) > + return -ENODEV; > + > + data = devm_kzalloc(dev, sizeof(struct max6697_data), GFP_KERNEL); > + if (!data) > + return -ENOMEM; > + > + data->type = id->driver_data; > + data->chip = &max6697_chip_data[data->type]; > + > + i2c_set_clientdata(client, data); > + mutex_init(&data->update_lock); > + > + err = max6697_init_chip(client); > + if (err) > + return err; > + > + /* Register sysfs hooks */ > + > + for (i = 0; i < data->chip->channels; i++) { > + err = sysfs_create_file(&dev->kobj, > + max6697_attributes[i * 6]); > + if (err) > + goto error; > + err = sysfs_create_file(&dev->kobj, > + max6697_attributes[i * 6 + 1]); > + if (err) > + goto error; > + err = sysfs_create_file(&dev->kobj, > + max6697_attributes[i * 6 + 2]); > + if (err) > + goto error; > + > + if (data->chip->have_crit & (1 << i)) { > + err = sysfs_create_file(&dev->kobj, > + max6697_attributes[i * 6 + 3]); > + if (err) > + goto error; > + err = sysfs_create_file(&dev->kobj, > + max6697_attributes[i * 6 + 4]); > + if (err) > + goto error; > + } > + if (data->chip->have_fault & (1 << i)) { > + err = sysfs_create_file(&dev->kobj, > + max6697_attributes[i * 6 + 5]); > + if (err) > + goto error; > + } > + } > + > + data->hwmon_dev = hwmon_device_register(dev); > + if (IS_ERR(data->hwmon_dev)) { > + err = PTR_ERR(data->hwmon_dev); > + goto error; > + } > + > + return 0; > + > +error: > + sysfs_remove_group(&dev->kobj, &max6697_group); > + return err; > +} > + > +static int max6697_remove(struct i2c_client *client) > +{ > + struct max6697_data *data = i2c_get_clientdata(client); > + > + hwmon_device_unregister(data->hwmon_dev); > + sysfs_remove_group(&client->dev.kobj, &max6697_group); > + > + return 0; > +} > + > +static const struct i2c_device_id max6697_id[] = { > + { "max6581", max6581 }, > + { "max6602", max6602 }, > + { "max6622", max6622 }, > + { "max6636", max6636 }, > + { "max6689", max6689 }, > + { "max6693", max6693 }, > + { "max6694", max6694 }, > + { "max6697", max6697 }, > + { "max6698", max6698 }, > + { "max6699", max6699 }, > + { } > +}; > +MODULE_DEVICE_TABLE(i2c, max6697_id); > + > +/* This is the driver that will be inserted */ > +static struct i2c_driver max6697_driver = { > + .class = I2C_CLASS_HWMON, > + .driver = { > + .name = "max6697", > + }, > + .probe = max6697_probe, > + .remove = max6697_remove, > + .id_table = max6697_id, > +}; > + > +module_i2c_driver(max6697_driver); > + > +MODULE_AUTHOR("Guenter Roeck <linux@xxxxxxxxxxxx>"); > +MODULE_DESCRIPTION("MAX6697 temperature sensor driver"); > +MODULE_LICENSE("GPL"); > diff --git a/include/linux/platform_data/max6697.h b/include/linux/platform_data/max6697.h > new file mode 100644 > index 0000000..5a21b9b > --- /dev/null > +++ b/include/linux/platform_data/max6697.h > @@ -0,0 +1,25 @@ > +/* > + * max6697.h > + * Copyright (c) 2012 Guenter Roeck <linux@xxxxxxxxxxxx> > + * > + * 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. > + */ > + > +#ifndef MAX6697_H > +#define MAX6697_H > + > +#define MAX6697_MAX_CONFIG_REG 8 > + > +struct max6697_platform_data { > + bool smbus_timeout_disable; > + bool extended_range_enable; > + u8 alert_mask; > + u8 over_temperature_mask; > + u8 resistance_cancellation; > + u8 ideality_mask; > + u8 ideality_value; > +}; > + > +#endif /* MAX6697_H */ > -- > 1.7.9.7 > > _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors