summaryrefslogtreecommitdiffstats
path: root/drivers/hwmon/vt1211.c
diff options
context:
space:
mode:
authorGuenter Roeck <linux@roeck-us.net>2013-01-09 11:09:34 -0500
committerGuenter Roeck <linux@roeck-us.net>2013-01-26 00:03:54 -0500
commit2a844c148e1f714ebf42cb96e1b172ce394c36c9 (patch)
treeeb68eb8438f0470e7a81b022199abe5f6d866879 /drivers/hwmon/vt1211.c
parent142c090184ac7f9763c5d22509405da3486f9801 (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.c10
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);