diff options
author | Anton Vorontsov <anton.vorontsov@linaro.org> | 2012-03-13 20:27:20 -0400 |
---|---|---|
committer | Anton Vorontsov <anton.vorontsov@linaro.org> | 2012-03-26 12:41:09 -0400 |
commit | d329129e9e10e3089550fd9bd692f67687503136 (patch) | |
tree | 5bb28bac5bdaa5b42a518ed6a246697cf5a983c6 /include | |
parent | efd71c89a411d72f1e20d91e34f0779e0e0019b4 (diff) |
ab8500_btemp: Get rid of 'enum adc_therm'
This is the same as abx500_adc_therm, but when the former is used,
the following warning flood pops up:
drivers/power/ab8500_btemp.c: In function 'ab8500_btemp_batctrl_volt_to_res':
ab8500_btemp.c:150:25: warning: comparison between 'enum abx500_adc_therm' and 'enum adc_therm' [-Wenum-compare]
ab8500_btemp.c: In function 'ab8500_btemp_curr_source_enable':
ab8500_btemp.c:212:25: warning: comparison between 'enum abx500_adc_therm' and 'enum adc_therm' [-Wenum-compare]
ab8500_btemp.c:244:32: warning: comparison between 'enum abx500_adc_therm' and 'enum adc_therm' [-Wenum-compare]
ab8500_btemp.c: In function 'ab8500_btemp_measure_temp':
ab8500_btemp.c:462:25: warning: comparison between 'enum abx500_adc_therm' and 'enum adc_therm' [-Wenum-compare]
ab8500_btemp.c: In function 'ab8500_btemp_id':
ab8500_btemp.c:528:121: warning: comparison between 'enum abx500_adc_therm' and 'enum adc_therm' [-Wenum-compare]
ab8500_btemp.c:551:25: warning: comparison between 'enum abx500_adc_therm' and 'enum adc_therm' [-Wenum-compare]
This patch fixes the issue by switching the driver to use more
namespace-friendly enum.
Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/mfd/abx500/ab8500-bm.h | 15 |
1 files changed, 2 insertions, 13 deletions
diff --git a/include/linux/mfd/abx500/ab8500-bm.h b/include/linux/mfd/abx500/ab8500-bm.h index 4b7342c54b41..f61b7b981ce2 100644 --- a/include/linux/mfd/abx500/ab8500-bm.h +++ b/include/linux/mfd/abx500/ab8500-bm.h | |||
@@ -9,6 +9,7 @@ | |||
9 | #define _AB8500_BM_H | 9 | #define _AB8500_BM_H |
10 | 10 | ||
11 | #include <linux/kernel.h> | 11 | #include <linux/kernel.h> |
12 | #include <linux/mfd/abx500.h> | ||
12 | 13 | ||
13 | /* | 14 | /* |
14 | * System control 2 register offsets. | 15 | * System control 2 register offsets. |
@@ -231,18 +232,6 @@ | |||
231 | /* Battery type */ | 232 | /* Battery type */ |
232 | #define BATTERY_UNKNOWN 00 | 233 | #define BATTERY_UNKNOWN 00 |
233 | 234 | ||
234 | /* | ||
235 | * ADC for the battery thermistor. | ||
236 | * When using the ADC_THERM_BATCTRL the battery ID resistor is combined with | ||
237 | * a NTC resistor to both identify the battery and to measure its temperature. | ||
238 | * Different phone manufactures uses different techniques to both identify the | ||
239 | * battery and to read its temperature. | ||
240 | */ | ||
241 | enum adc_therm { | ||
242 | ADC_THERM_BATCTRL, | ||
243 | ADC_THERM_BATTEMP, | ||
244 | }; | ||
245 | |||
246 | /** | 235 | /** |
247 | * struct res_to_temp - defines one point in a temp to res curve. To | 236 | * struct res_to_temp - defines one point in a temp to res curve. To |
248 | * be used in battery packs that combines the identification resistor with a | 237 | * be used in battery packs that combines the identification resistor with a |
@@ -464,7 +453,7 @@ struct ab8500_bm_data { | |||
464 | bool no_maintenance; | 453 | bool no_maintenance; |
465 | bool chg_unknown_bat; | 454 | bool chg_unknown_bat; |
466 | bool enable_overshoot; | 455 | bool enable_overshoot; |
467 | enum adc_therm adc_therm; | 456 | enum abx500_adc_therm adc_therm; |
468 | int fg_res; | 457 | int fg_res; |
469 | int n_btypes; | 458 | int n_btypes; |
470 | int batt_id; | 459 | int batt_id; |