Commit 742647e9 authored by Jean Delvare's avatar Jean Delvare Committed by Patrick Mochel

[PATCH] I2C: Setting w83627hf fan_div preserves fan_min

Here is a patch that updates the w83627hf driver in the exact same way I
did recently for the w83781d driver. There were two problems:
1* Fan divisor storing code was ugly, badly ripped from the 2.4 w83627hf
   driver and/or the 2.6 w83781d driver.
2* Setting fan divisors wouldn't preserve fan mins.

Exactly the same as w83781d:
http://archives.andrew.net.au/lm-sensors/msg06952.html
http://archives.andrew.net.au/lm-sensors/msg07008.html

No surprise since the w83627hf driver is a fork of the w83781d driver.

Since the two drivers are strongly similar, I took the code directly
from the updated w83781d driver. I cannot test the w83627hf driver
(testers welcome BTW) but this makes me feel confident that the code is
correct.

To make it clear, this single patch is the w83627f equivalent of the
three patches I submitted for the w83781d:
* Cleanup
* Refactoring
* Setting fan_div preserves fan_min
All in one (much better looking BTW).
parent 4b65ed3a
...@@ -659,34 +659,37 @@ show_fan_div_reg(struct device *dev, char *buf, int nr) ...@@ -659,34 +659,37 @@ show_fan_div_reg(struct device *dev, char *buf, int nr)
(long) DIV_FROM_REG(data->fan_div[nr - 1])); (long) DIV_FROM_REG(data->fan_div[nr - 1]));
} }
/* Note: we save and restore the fan minimum here, because its value is
determined in part by the fan divisor. This follows the principle of
least suprise; the user doesn't expect the fan minimum to change just
because the divisor changed. */
static ssize_t static ssize_t
store_fan_div_reg(struct device *dev, const char *buf, size_t count, int nr) store_fan_div_reg(struct device *dev, const char *buf, size_t count, int nr)
{ {
struct i2c_client *client = to_i2c_client(dev); struct i2c_client *client = to_i2c_client(dev);
struct w83627hf_data *data = i2c_get_clientdata(client); struct w83627hf_data *data = i2c_get_clientdata(client);
u32 val, old, old2, old3 = 0; unsigned long min;
u8 reg;
val = simple_strtoul(buf, NULL, 10); /* Save fan_min */
old = w83627hf_read_value(client, W83781D_REG_VID_FANDIV); min = FAN_FROM_REG(data->fan_min[nr],
old3 = w83627hf_read_value(client, W83781D_REG_VBAT); DIV_FROM_REG(data->fan_div[nr]));
data->fan_div[nr - 1] = DIV_TO_REG(val);
data->fan_div[nr] = DIV_TO_REG(simple_strtoul(buf, NULL, 10));
if (nr >= 3 && data->type != w83697hf) {
old2 = w83627hf_read_value(client, W83781D_REG_PIN); reg = (w83627hf_read_value(client, nr==2 ? W83781D_REG_PIN : W83781D_REG_VID_FANDIV)
old2 = (old2 & 0x3f) | ((data->fan_div[2] & 0x03) << 6); & (nr==0 ? 0xcf : 0x3f))
w83627hf_write_value(client, W83781D_REG_PIN, old2); | ((data->fan_div[nr] & 0x03) << (nr==0 ? 4 : 6));
old3 = (old3 & 0x7f) | ((data->fan_div[2] & 0x04) << 5); w83627hf_write_value(client, nr==2 ? W83781D_REG_PIN : W83781D_REG_VID_FANDIV, reg);
}
if (nr >= 2) { reg = (w83627hf_read_value(client, W83781D_REG_VBAT)
old = (old & 0x3f) | ((data->fan_div[1] & 0x03) << 6); & ~(1 << (5 + nr)))
old3 = (old3 & 0xbf) | ((data->fan_div[1] & 0x04) << 4); | ((data->fan_div[nr] & 0x04) << (3 + nr));
} w83627hf_write_value(client, W83781D_REG_VBAT, reg);
if (nr >= 1) {
old = (old & 0xcf) | ((data->fan_div[0] & 0x03) << 4); /* Restore fan_min */
w83627hf_write_value(client, W83781D_REG_VID_FANDIV, old); data->fan_min[nr] = FAN_TO_REG(min, DIV_FROM_REG(data->fan_div[nr]));
old3 = (old3 & 0xdf) | ((data->fan_div[0] & 0x04) << 3); w83627hf_write_value(client, W83781D_REG_FAN_MIN(nr+1), data->fan_min[nr]);
w83627hf_write_value(client, W83781D_REG_VBAT, old3);
}
return count; return count;
} }
...@@ -700,7 +703,7 @@ static ssize_t \ ...@@ -700,7 +703,7 @@ static ssize_t \
store_regs_fan_div_##offset (struct device *dev, \ store_regs_fan_div_##offset (struct device *dev, \
const char *buf, size_t count) \ const char *buf, size_t count) \
{ \ { \
return store_fan_div_reg(dev, buf, count, offset); \ return store_fan_div_reg(dev, buf, count, offset - 1); \
} \ } \
static DEVICE_ATTR(fan##offset##_div, S_IRUGO | S_IWUSR, \ static DEVICE_ATTR(fan##offset##_div, S_IRUGO | S_IWUSR, \
show_regs_fan_div_##offset, store_regs_fan_div_##offset) show_regs_fan_div_##offset, store_regs_fan_div_##offset)
...@@ -982,6 +985,11 @@ int w83627hf_detect(struct i2c_adapter *adapter, int address, ...@@ -982,6 +985,11 @@ int w83627hf_detect(struct i2c_adapter *adapter, int address,
/* Initialize the chip */ /* Initialize the chip */
w83627hf_init_client(new_client); w83627hf_init_client(new_client);
/* A few vars need to be filled upon startup */
data->fan_min[0] = w83627hf_read_value(new_client, W83781D_REG_FAN_MIN(1));
data->fan_min[1] = w83627hf_read_value(new_client, W83781D_REG_FAN_MIN(2));
data->fan_min[2] = w83627hf_read_value(new_client, W83781D_REG_FAN_MIN(3));
/* Register sysfs hooks */ /* Register sysfs hooks */
device_create_file_in(new_client, 0); device_create_file_in(new_client, 0);
if (kind != w83697hf) if (kind != w83697hf)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment