diff options
author | Guenter Roeck <linux@roeck-us.net> | 2013-01-09 11:09:34 -0500 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2013-01-26 00:03:54 -0500 |
commit | 2a844c148e1f714ebf42cb96e1b172ce394c36c9 (patch) | |
tree | eb68eb8438f0470e7a81b022199abe5f6d866879 /drivers/hwmon/vt1211.c | |
parent | 142c090184ac7f9763c5d22509405da3486f9801 (diff) |
hwmon: Replace SENSORS_LIMIT with clamp_val
SENSORS_LIMIT and the generic clamp_val have the same functionality,
and clamp_val is more efficient.
This patch reduces text size by 9052 bytes and bss size by 11624 bytes
for x86_64 builds.
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: George Joseph <george.joseph@fairview5.com>
Acked-by: Jean Delvare <khali@linux-fr.org>
Diffstat (limited to 'drivers/hwmon/vt1211.c')
-rw-r--r-- | drivers/hwmon/vt1211.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/hwmon/vt1211.c b/drivers/hwmon/vt1211.c index 751703059fae..dcc62f80f67b 100644 --- a/drivers/hwmon/vt1211.c +++ b/drivers/hwmon/vt1211.c | |||
@@ -158,7 +158,7 @@ struct vt1211_data { | |||
158 | #define IN_FROM_REG(ix, reg) ((reg) < 3 ? 0 : (ix) == 5 ? \ | 158 | #define IN_FROM_REG(ix, reg) ((reg) < 3 ? 0 : (ix) == 5 ? \ |
159 | (((reg) - 3) * 15882 + 479) / 958 : \ | 159 | (((reg) - 3) * 15882 + 479) / 958 : \ |
160 | (((reg) - 3) * 10000 + 479) / 958) | 160 | (((reg) - 3) * 10000 + 479) / 958) |
161 | #define IN_TO_REG(ix, val) (SENSORS_LIMIT((ix) == 5 ? \ | 161 | #define IN_TO_REG(ix, val) (clamp_val((ix) == 5 ? \ |
162 | ((val) * 958 + 7941) / 15882 + 3 : \ | 162 | ((val) * 958 + 7941) / 15882 + 3 : \ |
163 | ((val) * 958 + 5000) / 10000 + 3, 0, 255)) | 163 | ((val) * 958 + 5000) / 10000 + 3, 0, 255)) |
164 | 164 | ||
@@ -173,7 +173,7 @@ struct vt1211_data { | |||
173 | (ix) == 1 ? (reg) < 51 ? 0 : \ | 173 | (ix) == 1 ? (reg) < 51 ? 0 : \ |
174 | ((reg) - 51) * 1000 : \ | 174 | ((reg) - 51) * 1000 : \ |
175 | ((253 - (reg)) * 2200 + 105) / 210) | 175 | ((253 - (reg)) * 2200 + 105) / 210) |
176 | #define TEMP_TO_REG(ix, val) SENSORS_LIMIT( \ | 176 | #define TEMP_TO_REG(ix, val) clamp_val( \ |
177 | ((ix) == 0 ? ((val) + 500) / 1000 : \ | 177 | ((ix) == 0 ? ((val) + 500) / 1000 : \ |
178 | (ix) == 1 ? ((val) + 500) / 1000 + 51 : \ | 178 | (ix) == 1 ? ((val) + 500) / 1000 + 51 : \ |
179 | 253 - ((val) * 210 + 1100) / 2200), 0, 255) | 179 | 253 - ((val) * 210 + 1100) / 2200), 0, 255) |
@@ -183,7 +183,7 @@ struct vt1211_data { | |||
183 | #define RPM_FROM_REG(reg, div) (((reg) == 0) || ((reg) == 255) ? 0 : \ | 183 | #define RPM_FROM_REG(reg, div) (((reg) == 0) || ((reg) == 255) ? 0 : \ |
184 | 1310720 / (reg) / DIV_FROM_REG(div)) | 184 | 1310720 / (reg) / DIV_FROM_REG(div)) |
185 | #define RPM_TO_REG(val, div) ((val) == 0 ? 255 : \ | 185 | #define RPM_TO_REG(val, div) ((val) == 0 ? 255 : \ |
186 | SENSORS_LIMIT((1310720 / (val) / \ | 186 | clamp_val((1310720 / (val) / \ |
187 | DIV_FROM_REG(div)), 1, 254)) | 187 | DIV_FROM_REG(div)), 1, 254)) |
188 | 188 | ||
189 | /* --------------------------------------------------------------------- | 189 | /* --------------------------------------------------------------------- |
@@ -687,7 +687,7 @@ static ssize_t set_pwm(struct device *dev, struct device_attribute *attr, | |||
687 | data->fan_ctl)); | 687 | data->fan_ctl)); |
688 | break; | 688 | break; |
689 | case SHOW_SET_PWM_FREQ: | 689 | case SHOW_SET_PWM_FREQ: |
690 | val = 135000 / SENSORS_LIMIT(val, 135000 >> 7, 135000); | 690 | val = 135000 / clamp_val(val, 135000 >> 7, 135000); |
691 | /* calculate tmp = log2(val) */ | 691 | /* calculate tmp = log2(val) */ |
692 | tmp = 0; | 692 | tmp = 0; |
693 | for (val >>= 1; val > 0; val >>= 1) | 693 | for (val >>= 1; val > 0; val >>= 1) |
@@ -845,7 +845,7 @@ static ssize_t set_pwm_auto_point_pwm(struct device *dev, | |||
845 | return err; | 845 | return err; |
846 | 846 | ||
847 | mutex_lock(&data->update_lock); | 847 | mutex_lock(&data->update_lock); |
848 | data->pwm_auto_pwm[ix][ap] = SENSORS_LIMIT(val, 0, 255); | 848 | data->pwm_auto_pwm[ix][ap] = clamp_val(val, 0, 255); |
849 | vt1211_write8(data, VT1211_REG_PWM_AUTO_PWM(ix, ap), | 849 | vt1211_write8(data, VT1211_REG_PWM_AUTO_PWM(ix, ap), |
850 | data->pwm_auto_pwm[ix][ap]); | 850 | data->pwm_auto_pwm[ix][ap]); |
851 | mutex_unlock(&data->update_lock); | 851 | mutex_unlock(&data->update_lock); |