diff options
author | Anton Vorontsov <cbou@mail.ru> | 2007-07-14 21:18:25 -0400 |
---|---|---|
committer | Anton Vorontsov <cbou@mail.ru> | 2007-07-15 14:32:38 -0400 |
commit | 7b3d54a8c30d2c524889a05d0c1334813d516b93 (patch) | |
tree | 7bcbfadc8604f04bea356a8fc61f8069daf24b85 /drivers/power | |
parent | 5ebf6e6a96e41220edec23a90e4140985d1a5732 (diff) |
Power supply class and drivers: remove non obligatory return statements
Per Jeff Garzik request.
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Signed-off-by: Anton Vorontsov <cbou@mail.ru>
Diffstat (limited to 'drivers/power')
-rw-r--r-- | drivers/power/apm_power.c | 4 | ||||
-rw-r--r-- | drivers/power/ds2760_battery.c | 7 | ||||
-rw-r--r-- | drivers/power/olpc_battery.c | 1 | ||||
-rw-r--r-- | drivers/power/pda_power.c | 5 | ||||
-rw-r--r-- | drivers/power/pmu_battery.c | 2 | ||||
-rw-r--r-- | drivers/power/power_supply_core.c | 6 | ||||
-rw-r--r-- | drivers/power/power_supply_leds.c | 8 | ||||
-rw-r--r-- | drivers/power/power_supply_sysfs.c | 2 |
8 files changed, 0 insertions, 35 deletions
diff --git a/drivers/power/apm_power.c b/drivers/power/apm_power.c index 042bd950d036..39a90a6f0f80 100644 --- a/drivers/power/apm_power.c +++ b/drivers/power/apm_power.c | |||
@@ -48,8 +48,6 @@ static void find_main_battery(void) | |||
48 | } | 48 | } |
49 | if (!main_battery) | 49 | if (!main_battery) |
50 | main_battery = batm; | 50 | main_battery = batm; |
51 | |||
52 | return; | ||
53 | } | 51 | } |
54 | 52 | ||
55 | static int calculate_time(int status) | 53 | static int calculate_time(int status) |
@@ -218,7 +216,6 @@ static void apm_battery_apm_get_power_status(struct apm_power_info *info) | |||
218 | } | 216 | } |
219 | 217 | ||
220 | up(&power_supply_class->sem); | 218 | up(&power_supply_class->sem); |
221 | return; | ||
222 | } | 219 | } |
223 | 220 | ||
224 | static int __init apm_battery_init(void) | 221 | static int __init apm_battery_init(void) |
@@ -232,7 +229,6 @@ static int __init apm_battery_init(void) | |||
232 | static void __exit apm_battery_exit(void) | 229 | static void __exit apm_battery_exit(void) |
233 | { | 230 | { |
234 | apm_get_power_status = NULL; | 231 | apm_get_power_status = NULL; |
235 | return; | ||
236 | } | 232 | } |
237 | 233 | ||
238 | module_init(apm_battery_init); | 234 | module_init(apm_battery_init); |
diff --git a/drivers/power/ds2760_battery.c b/drivers/power/ds2760_battery.c index 00e1ea6f1de2..be7021ee3611 100644 --- a/drivers/power/ds2760_battery.c +++ b/drivers/power/ds2760_battery.c | |||
@@ -254,8 +254,6 @@ static void ds2760_battery_update_status(struct ds2760_device_info *di) | |||
254 | 254 | ||
255 | if (di->charge_status != old_charge_status) | 255 | if (di->charge_status != old_charge_status) |
256 | power_supply_changed(&di->bat); | 256 | power_supply_changed(&di->bat); |
257 | |||
258 | return; | ||
259 | } | 257 | } |
260 | 258 | ||
261 | static void ds2760_battery_work(struct work_struct *work) | 259 | static void ds2760_battery_work(struct work_struct *work) |
@@ -268,8 +266,6 @@ static void ds2760_battery_work(struct work_struct *work) | |||
268 | 266 | ||
269 | ds2760_battery_update_status(di); | 267 | ds2760_battery_update_status(di); |
270 | queue_delayed_work(di->monitor_wqueue, &di->monitor_work, interval); | 268 | queue_delayed_work(di->monitor_wqueue, &di->monitor_work, interval); |
271 | |||
272 | return; | ||
273 | } | 269 | } |
274 | 270 | ||
275 | #define to_ds2760_device_info(x) container_of((x), struct ds2760_device_info, \ | 271 | #define to_ds2760_device_info(x) container_of((x), struct ds2760_device_info, \ |
@@ -283,8 +279,6 @@ static void ds2760_battery_external_power_changed(struct power_supply *psy) | |||
283 | 279 | ||
284 | cancel_delayed_work(&di->monitor_work); | 280 | cancel_delayed_work(&di->monitor_work); |
285 | queue_delayed_work(di->monitor_wqueue, &di->monitor_work, HZ/10); | 281 | queue_delayed_work(di->monitor_wqueue, &di->monitor_work, HZ/10); |
286 | |||
287 | return; | ||
288 | } | 282 | } |
289 | 283 | ||
290 | static int ds2760_battery_get_property(struct power_supply *psy, | 284 | static int ds2760_battery_get_property(struct power_supply *psy, |
@@ -457,7 +451,6 @@ static int __init ds2760_battery_init(void) | |||
457 | static void __exit ds2760_battery_exit(void) | 451 | static void __exit ds2760_battery_exit(void) |
458 | { | 452 | { |
459 | platform_driver_unregister(&ds2760_battery_driver); | 453 | platform_driver_unregister(&ds2760_battery_driver); |
460 | return; | ||
461 | } | 454 | } |
462 | 455 | ||
463 | module_init(ds2760_battery_init); | 456 | module_init(ds2760_battery_init); |
diff --git a/drivers/power/olpc_battery.c b/drivers/power/olpc_battery.c index 878684df7667..c998e68d060f 100644 --- a/drivers/power/olpc_battery.c +++ b/drivers/power/olpc_battery.c | |||
@@ -341,7 +341,6 @@ static void __exit olpc_bat_exit(void) | |||
341 | power_supply_unregister(&olpc_bat); | 341 | power_supply_unregister(&olpc_bat); |
342 | power_supply_unregister(&olpc_ac); | 342 | power_supply_unregister(&olpc_ac); |
343 | platform_device_unregister(bat_pdev); | 343 | platform_device_unregister(bat_pdev); |
344 | return; | ||
345 | } | 344 | } |
346 | 345 | ||
347 | module_init(olpc_bat_init); | 346 | module_init(olpc_bat_init); |
diff --git a/drivers/power/pda_power.c b/drivers/power/pda_power.c index cdc28923e0ce..c058f285be1a 100644 --- a/drivers/power/pda_power.c +++ b/drivers/power/pda_power.c | |||
@@ -97,8 +97,6 @@ static void update_charger(void) | |||
97 | dev_dbg(dev, "charger off\n"); | 97 | dev_dbg(dev, "charger off\n"); |
98 | pdata->set_charge(0); | 98 | pdata->set_charge(0); |
99 | } | 99 | } |
100 | |||
101 | return; | ||
102 | } | 100 | } |
103 | 101 | ||
104 | static void supply_timer_func(unsigned long power_supply_ptr) | 102 | static void supply_timer_func(unsigned long power_supply_ptr) |
@@ -106,7 +104,6 @@ static void supply_timer_func(unsigned long power_supply_ptr) | |||
106 | void *power_supply = (void *)power_supply_ptr; | 104 | void *power_supply = (void *)power_supply_ptr; |
107 | 105 | ||
108 | power_supply_changed(power_supply); | 106 | power_supply_changed(power_supply); |
109 | return; | ||
110 | } | 107 | } |
111 | 108 | ||
112 | static void charger_timer_func(unsigned long power_supply_ptr) | 109 | static void charger_timer_func(unsigned long power_supply_ptr) |
@@ -118,7 +115,6 @@ static void charger_timer_func(unsigned long power_supply_ptr) | |||
118 | supply_timer.data = power_supply_ptr; | 115 | supply_timer.data = power_supply_ptr; |
119 | mod_timer(&supply_timer, | 116 | mod_timer(&supply_timer, |
120 | jiffies + msecs_to_jiffies(pdata->wait_for_charger)); | 117 | jiffies + msecs_to_jiffies(pdata->wait_for_charger)); |
121 | return; | ||
122 | } | 118 | } |
123 | 119 | ||
124 | static irqreturn_t power_changed_isr(int irq, void *power_supply) | 120 | static irqreturn_t power_changed_isr(int irq, void *power_supply) |
@@ -251,7 +247,6 @@ static int __init pda_power_init(void) | |||
251 | static void __exit pda_power_exit(void) | 247 | static void __exit pda_power_exit(void) |
252 | { | 248 | { |
253 | platform_driver_unregister(&pda_power_pdrv); | 249 | platform_driver_unregister(&pda_power_pdrv); |
254 | return; | ||
255 | } | 250 | } |
256 | 251 | ||
257 | module_init(pda_power_init); | 252 | module_init(pda_power_init); |
diff --git a/drivers/power/pmu_battery.c b/drivers/power/pmu_battery.c index 2fea4af0e40a..60a8cf3a0431 100644 --- a/drivers/power/pmu_battery.c +++ b/drivers/power/pmu_battery.c | |||
@@ -203,8 +203,6 @@ static void __exit pmu_bat_exit(void) | |||
203 | } | 203 | } |
204 | power_supply_unregister(&pmu_ac); | 204 | power_supply_unregister(&pmu_ac); |
205 | platform_device_unregister(bat_pdev); | 205 | platform_device_unregister(bat_pdev); |
206 | |||
207 | return; | ||
208 | } | 206 | } |
209 | 207 | ||
210 | module_init(pmu_bat_init); | 208 | module_init(pmu_bat_init); |
diff --git a/drivers/power/power_supply_core.c b/drivers/power/power_supply_core.c index e87ea5156755..a63b75cf75e2 100644 --- a/drivers/power/power_supply_core.c +++ b/drivers/power/power_supply_core.c | |||
@@ -46,8 +46,6 @@ static void power_supply_changed_work(struct work_struct *work) | |||
46 | power_supply_update_leds(psy); | 46 | power_supply_update_leds(psy); |
47 | 47 | ||
48 | kobject_uevent(&psy->dev->kobj, KOBJ_CHANGE); | 48 | kobject_uevent(&psy->dev->kobj, KOBJ_CHANGE); |
49 | |||
50 | return; | ||
51 | } | 49 | } |
52 | 50 | ||
53 | void power_supply_changed(struct power_supply *psy) | 51 | void power_supply_changed(struct power_supply *psy) |
@@ -55,8 +53,6 @@ void power_supply_changed(struct power_supply *psy) | |||
55 | dev_dbg(psy->dev, "%s\n", __FUNCTION__); | 53 | dev_dbg(psy->dev, "%s\n", __FUNCTION__); |
56 | 54 | ||
57 | schedule_work(&psy->changed_work); | 55 | schedule_work(&psy->changed_work); |
58 | |||
59 | return; | ||
60 | } | 56 | } |
61 | 57 | ||
62 | int power_supply_am_i_supplied(struct power_supply *psy) | 58 | int power_supply_am_i_supplied(struct power_supply *psy) |
@@ -129,7 +125,6 @@ void power_supply_unregister(struct power_supply *psy) | |||
129 | power_supply_remove_triggers(psy); | 125 | power_supply_remove_triggers(psy); |
130 | power_supply_remove_attrs(psy); | 126 | power_supply_remove_attrs(psy); |
131 | device_unregister(psy->dev); | 127 | device_unregister(psy->dev); |
132 | return; | ||
133 | } | 128 | } |
134 | 129 | ||
135 | static int __init power_supply_class_init(void) | 130 | static int __init power_supply_class_init(void) |
@@ -147,7 +142,6 @@ static int __init power_supply_class_init(void) | |||
147 | static void __exit power_supply_class_exit(void) | 142 | static void __exit power_supply_class_exit(void) |
148 | { | 143 | { |
149 | class_destroy(power_supply_class); | 144 | class_destroy(power_supply_class); |
150 | return; | ||
151 | } | 145 | } |
152 | 146 | ||
153 | EXPORT_SYMBOL_GPL(power_supply_changed); | 147 | EXPORT_SYMBOL_GPL(power_supply_changed); |
diff --git a/drivers/power/power_supply_leds.c b/drivers/power/power_supply_leds.c index 7232490bb595..7f8f3590b02e 100644 --- a/drivers/power/power_supply_leds.c +++ b/drivers/power/power_supply_leds.c | |||
@@ -40,8 +40,6 @@ static void power_supply_update_bat_leds(struct power_supply *psy) | |||
40 | led_trigger_event(psy->full_trig, LED_OFF); | 40 | led_trigger_event(psy->full_trig, LED_OFF); |
41 | break; | 41 | break; |
42 | } | 42 | } |
43 | |||
44 | return; | ||
45 | } | 43 | } |
46 | 44 | ||
47 | static int power_supply_create_bat_triggers(struct power_supply *psy) | 45 | static int power_supply_create_bat_triggers(struct power_supply *psy) |
@@ -97,7 +95,6 @@ static void power_supply_remove_bat_triggers(struct power_supply *psy) | |||
97 | kfree(psy->full_trig_name); | 95 | kfree(psy->full_trig_name); |
98 | kfree(psy->charging_trig_name); | 96 | kfree(psy->charging_trig_name); |
99 | kfree(psy->charging_full_trig_name); | 97 | kfree(psy->charging_full_trig_name); |
100 | return; | ||
101 | } | 98 | } |
102 | 99 | ||
103 | /* Generated power specific LEDs triggers. */ | 100 | /* Generated power specific LEDs triggers. */ |
@@ -115,8 +112,6 @@ static void power_supply_update_gen_leds(struct power_supply *psy) | |||
115 | led_trigger_event(psy->online_trig, LED_FULL); | 112 | led_trigger_event(psy->online_trig, LED_FULL); |
116 | else | 113 | else |
117 | led_trigger_event(psy->online_trig, LED_OFF); | 114 | led_trigger_event(psy->online_trig, LED_OFF); |
118 | |||
119 | return; | ||
120 | } | 115 | } |
121 | 116 | ||
122 | static int power_supply_create_gen_triggers(struct power_supply *psy) | 117 | static int power_supply_create_gen_triggers(struct power_supply *psy) |
@@ -145,7 +140,6 @@ static void power_supply_remove_gen_triggers(struct power_supply *psy) | |||
145 | { | 140 | { |
146 | led_trigger_unregister_simple(psy->online_trig); | 141 | led_trigger_unregister_simple(psy->online_trig); |
147 | kfree(psy->online_trig_name); | 142 | kfree(psy->online_trig_name); |
148 | return; | ||
149 | } | 143 | } |
150 | 144 | ||
151 | /* Choice what triggers to create&update. */ | 145 | /* Choice what triggers to create&update. */ |
@@ -156,7 +150,6 @@ void power_supply_update_leds(struct power_supply *psy) | |||
156 | power_supply_update_bat_leds(psy); | 150 | power_supply_update_bat_leds(psy); |
157 | else | 151 | else |
158 | power_supply_update_gen_leds(psy); | 152 | power_supply_update_gen_leds(psy); |
159 | return; | ||
160 | } | 153 | } |
161 | 154 | ||
162 | int power_supply_create_triggers(struct power_supply *psy) | 155 | int power_supply_create_triggers(struct power_supply *psy) |
@@ -172,5 +165,4 @@ void power_supply_remove_triggers(struct power_supply *psy) | |||
172 | power_supply_remove_bat_triggers(psy); | 165 | power_supply_remove_bat_triggers(psy); |
173 | else | 166 | else |
174 | power_supply_remove_gen_triggers(psy); | 167 | power_supply_remove_gen_triggers(psy); |
175 | return; | ||
176 | } | 168 | } |
diff --git a/drivers/power/power_supply_sysfs.c b/drivers/power/power_supply_sysfs.c index c07d4258d347..c7c4574729b1 100644 --- a/drivers/power/power_supply_sysfs.c +++ b/drivers/power/power_supply_sysfs.c | |||
@@ -176,8 +176,6 @@ void power_supply_remove_attrs(struct power_supply *psy) | |||
176 | for (i = 0; i < psy->num_properties; i++) | 176 | for (i = 0; i < psy->num_properties; i++) |
177 | device_remove_file(psy->dev, | 177 | device_remove_file(psy->dev, |
178 | &power_supply_attrs[psy->properties[i]]); | 178 | &power_supply_attrs[psy->properties[i]]); |
179 | |||
180 | return; | ||
181 | } | 179 | } |
182 | 180 | ||
183 | static char *kstruprdup(const char *str, gfp_t gfp) | 181 | static char *kstruprdup(const char *str, gfp_t gfp) |