Skip to content

Commit

Permalink
hwmon: (adt7470) Add write support to alarm_mask
Browse files Browse the repository at this point in the history
Add write support for the alarm_mask. A base of 0 is provided so that
either hex or decimal can be used. The hex format when reading alarm_mask
is unchanged.

Signed-off-by: Joshua Scott <joshua.scott@alliedtelesis.co.nz>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
  • Loading branch information
Joshua Scott authored and Guenter Roeck committed Sep 10, 2016
1 parent c7cefce commit feca313
Showing 1 changed file with 24 additions and 1 deletion.
25 changes: 24 additions & 1 deletion drivers/hwmon/adt7470.c
Original file line number Diff line number Diff line change
Expand Up @@ -545,6 +545,28 @@ static ssize_t show_alarm_mask(struct device *dev,
return sprintf(buf, "%x\n", data->alarms_mask);
}

static ssize_t set_alarm_mask(struct device *dev,
struct device_attribute *devattr,
const char *buf,
size_t count)
{
struct adt7470_data *data = dev_get_drvdata(dev);
long mask;

if (kstrtoul(buf, 0, &mask))
return -EINVAL;

if (mask & ~0xffff)
return -EINVAL;

mutex_lock(&data->lock);
data->alarms_mask = mask;
adt7470_write_word_data(data->client, ADT7470_REG_ALARM1_MASK, mask);
mutex_unlock(&data->lock);

return count;
}

static ssize_t show_fan_max(struct device *dev,
struct device_attribute *devattr,
char *buf)
Expand Down Expand Up @@ -989,7 +1011,8 @@ static ssize_t show_alarm(struct device *dev,
return sprintf(buf, "0\n");
}

static DEVICE_ATTR(alarm_mask, S_IRUGO, show_alarm_mask, NULL);
static DEVICE_ATTR(alarm_mask, S_IWUSR | S_IRUGO, show_alarm_mask,
set_alarm_mask);
static DEVICE_ATTR(num_temp_sensors, S_IWUSR | S_IRUGO, show_num_temp_sensors,
set_num_temp_sensors);
static DEVICE_ATTR(auto_update_interval, S_IWUSR | S_IRUGO,
Expand Down

0 comments on commit feca313

Please sign in to comment.