diff options
author | Jean Delvare <khali@linux-fr.org> | 2005-10-07 18:12:01 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-10-28 17:02:10 -0400 |
commit | 862bc36f682b25dd97833f702b4f8c97661b8811 (patch) | |
tree | bdafe5326e72ec7cc9a677746ca7332ae73e7bd3 /drivers/hwmon/w83627hf.c | |
parent | 91749996287c230671788014f422f72e594e7b86 (diff) |
[PATCH] hwmon: Drop useless w83627hf initialization step
Drop a useless initialization step in the w83627hf driver. The comment
says that the W83627HF PWM2 can be disabled, but it can't. I suppose
this is a leftover from the w83781d driver (from which the w83627hf
driver is derived), as for example the W83782D had the ability to
disable PWM2.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/hwmon/w83627hf.c')
-rw-r--r-- | drivers/hwmon/w83627hf.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/hwmon/w83627hf.c b/drivers/hwmon/w83627hf.c index 494274d27f01..48270819ff99 100644 --- a/drivers/hwmon/w83627hf.c +++ b/drivers/hwmon/w83627hf.c | |||
@@ -201,7 +201,6 @@ superio_exit(void) | |||
201 | 201 | ||
202 | #define W83627HF_REG_PWM1 0x5A | 202 | #define W83627HF_REG_PWM1 0x5A |
203 | #define W83627HF_REG_PWM2 0x5B | 203 | #define W83627HF_REG_PWM2 0x5B |
204 | #define W83627HF_REG_PWMCLK12 0x5C | ||
205 | 204 | ||
206 | #define W83627THF_REG_PWM1 0x01 /* 697HF and 637HF too */ | 205 | #define W83627THF_REG_PWM1 0x01 /* 697HF and 637HF too */ |
207 | #define W83627THF_REG_PWM2 0x03 /* 697HF and 637HF too */ | 206 | #define W83627THF_REG_PWM2 0x03 /* 697HF and 637HF too */ |
@@ -1373,12 +1372,6 @@ static void w83627hf_init_client(struct i2c_client *client) | |||
1373 | } | 1372 | } |
1374 | } | 1373 | } |
1375 | 1374 | ||
1376 | if (type == w83627hf) { | ||
1377 | /* enable PWM2 control (can't hurt since PWM reg | ||
1378 | should have been reset to 0xff) */ | ||
1379 | w83627hf_write_value(client, W83627HF_REG_PWMCLK12, | ||
1380 | 0x19); | ||
1381 | } | ||
1382 | /* enable comparator mode for temp2 and temp3 so | 1375 | /* enable comparator mode for temp2 and temp3 so |
1383 | alarm indication will work correctly */ | 1376 | alarm indication will work correctly */ |
1384 | i = w83627hf_read_value(client, W83781D_REG_IRQ); | 1377 | i = w83627hf_read_value(client, W83781D_REG_IRQ); |