diff options
author | Andrey Borzenkov <arvidjaar@mail.ru> | 2007-11-14 20:00:37 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-11-14 21:45:44 -0500 |
commit | 8a246ee43f4b1df3fa5cbf9c4a3d3dcad0b1e08c (patch) | |
tree | ad9027fb48466d7faa6db2849d4538216aa4859a /drivers | |
parent | f96159840bc5f605aca5113ab2d24308d3dc2eff (diff) |
make /proc/acpi/ac_adapter dependent on ACPI_PROCFS
Do not provide /proc/acpi/ac_adapter if ACPI_PROCFS is not defined. This
eliminates duplicated power adapters in HAL and makes it consistent with
battery module
Signed-off-by: Andrey Borzenkov <arvidjaar@mail.ru>
Acked-by: Alexey Starikovskiy <astarikovskiy@suse.de>
Cc: Len Brown <lenb@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/acpi/ac.c | 23 |
1 files changed, 22 insertions, 1 deletions
diff --git a/drivers/acpi/ac.c b/drivers/acpi/ac.c index e03de37a750d..1461dc9944a3 100644 --- a/drivers/acpi/ac.c +++ b/drivers/acpi/ac.c | |||
@@ -27,8 +27,10 @@ | |||
27 | #include <linux/module.h> | 27 | #include <linux/module.h> |
28 | #include <linux/init.h> | 28 | #include <linux/init.h> |
29 | #include <linux/types.h> | 29 | #include <linux/types.h> |
30 | #ifdef CONFIG_ACPI_PROCFS | ||
30 | #include <linux/proc_fs.h> | 31 | #include <linux/proc_fs.h> |
31 | #include <linux/seq_file.h> | 32 | #include <linux/seq_file.h> |
33 | #endif | ||
32 | #include <linux/power_supply.h> | 34 | #include <linux/power_supply.h> |
33 | #include <acpi/acpi_bus.h> | 35 | #include <acpi/acpi_bus.h> |
34 | #include <acpi/acpi_drivers.h> | 36 | #include <acpi/acpi_drivers.h> |
@@ -49,12 +51,14 @@ MODULE_AUTHOR("Paul Diefenbaugh"); | |||
49 | MODULE_DESCRIPTION("ACPI AC Adapter Driver"); | 51 | MODULE_DESCRIPTION("ACPI AC Adapter Driver"); |
50 | MODULE_LICENSE("GPL"); | 52 | MODULE_LICENSE("GPL"); |
51 | 53 | ||
54 | #ifdef CONFIG_ACPI_PROCFS | ||
52 | extern struct proc_dir_entry *acpi_lock_ac_dir(void); | 55 | extern struct proc_dir_entry *acpi_lock_ac_dir(void); |
53 | extern void *acpi_unlock_ac_dir(struct proc_dir_entry *acpi_ac_dir); | 56 | extern void *acpi_unlock_ac_dir(struct proc_dir_entry *acpi_ac_dir); |
57 | static int acpi_ac_open_fs(struct inode *inode, struct file *file); | ||
58 | #endif | ||
54 | 59 | ||
55 | static int acpi_ac_add(struct acpi_device *device); | 60 | static int acpi_ac_add(struct acpi_device *device); |
56 | static int acpi_ac_remove(struct acpi_device *device, int type); | 61 | static int acpi_ac_remove(struct acpi_device *device, int type); |
57 | static int acpi_ac_open_fs(struct inode *inode, struct file *file); | ||
58 | 62 | ||
59 | const static struct acpi_device_id ac_device_ids[] = { | 63 | const static struct acpi_device_id ac_device_ids[] = { |
60 | {"ACPI0003", 0}, | 64 | {"ACPI0003", 0}, |
@@ -80,12 +84,15 @@ struct acpi_ac { | |||
80 | 84 | ||
81 | #define to_acpi_ac(x) container_of(x, struct acpi_ac, charger); | 85 | #define to_acpi_ac(x) container_of(x, struct acpi_ac, charger); |
82 | 86 | ||
87 | #ifdef CONFIG_ACPI_PROCFS | ||
83 | static const struct file_operations acpi_ac_fops = { | 88 | static const struct file_operations acpi_ac_fops = { |
84 | .open = acpi_ac_open_fs, | 89 | .open = acpi_ac_open_fs, |
85 | .read = seq_read, | 90 | .read = seq_read, |
86 | .llseek = seq_lseek, | 91 | .llseek = seq_lseek, |
87 | .release = single_release, | 92 | .release = single_release, |
88 | }; | 93 | }; |
94 | #endif | ||
95 | |||
89 | static int get_ac_property(struct power_supply *psy, | 96 | static int get_ac_property(struct power_supply *psy, |
90 | enum power_supply_property psp, | 97 | enum power_supply_property psp, |
91 | union power_supply_propval *val) | 98 | union power_supply_propval *val) |
@@ -127,6 +134,7 @@ static int acpi_ac_get_state(struct acpi_ac *ac) | |||
127 | return 0; | 134 | return 0; |
128 | } | 135 | } |
129 | 136 | ||
137 | #ifdef CONFIG_ACPI_PROCFS | ||
130 | /* -------------------------------------------------------------------------- | 138 | /* -------------------------------------------------------------------------- |
131 | FS Interface (/proc) | 139 | FS Interface (/proc) |
132 | -------------------------------------------------------------------------- */ | 140 | -------------------------------------------------------------------------- */ |
@@ -206,6 +214,7 @@ static int acpi_ac_remove_fs(struct acpi_device *device) | |||
206 | 214 | ||
207 | return 0; | 215 | return 0; |
208 | } | 216 | } |
217 | #endif | ||
209 | 218 | ||
210 | /* -------------------------------------------------------------------------- | 219 | /* -------------------------------------------------------------------------- |
211 | Driver Model | 220 | Driver Model |
@@ -264,7 +273,9 @@ static int acpi_ac_add(struct acpi_device *device) | |||
264 | if (result) | 273 | if (result) |
265 | goto end; | 274 | goto end; |
266 | 275 | ||
276 | #ifdef CONFIG_ACPI_PROCFS | ||
267 | result = acpi_ac_add_fs(device); | 277 | result = acpi_ac_add_fs(device); |
278 | #endif | ||
268 | if (result) | 279 | if (result) |
269 | goto end; | 280 | goto end; |
270 | ac->charger.name = acpi_device_bid(device); | 281 | ac->charger.name = acpi_device_bid(device); |
@@ -287,7 +298,9 @@ static int acpi_ac_add(struct acpi_device *device) | |||
287 | 298 | ||
288 | end: | 299 | end: |
289 | if (result) { | 300 | if (result) { |
301 | #ifdef CONFIG_ACPI_PROCFS | ||
290 | acpi_ac_remove_fs(device); | 302 | acpi_ac_remove_fs(device); |
303 | #endif | ||
291 | kfree(ac); | 304 | kfree(ac); |
292 | } | 305 | } |
293 | 306 | ||
@@ -309,7 +322,9 @@ static int acpi_ac_remove(struct acpi_device *device, int type) | |||
309 | ACPI_ALL_NOTIFY, acpi_ac_notify); | 322 | ACPI_ALL_NOTIFY, acpi_ac_notify); |
310 | if (ac->charger.dev) | 323 | if (ac->charger.dev) |
311 | power_supply_unregister(&ac->charger); | 324 | power_supply_unregister(&ac->charger); |
325 | #ifdef CONFIG_ACPI_PROCFS | ||
312 | acpi_ac_remove_fs(device); | 326 | acpi_ac_remove_fs(device); |
327 | #endif | ||
313 | 328 | ||
314 | kfree(ac); | 329 | kfree(ac); |
315 | 330 | ||
@@ -323,13 +338,17 @@ static int __init acpi_ac_init(void) | |||
323 | if (acpi_disabled) | 338 | if (acpi_disabled) |
324 | return -ENODEV; | 339 | return -ENODEV; |
325 | 340 | ||
341 | #ifdef CONFIG_ACPI_PROCFS | ||
326 | acpi_ac_dir = acpi_lock_ac_dir(); | 342 | acpi_ac_dir = acpi_lock_ac_dir(); |
327 | if (!acpi_ac_dir) | 343 | if (!acpi_ac_dir) |
328 | return -ENODEV; | 344 | return -ENODEV; |
345 | #endif | ||
329 | 346 | ||
330 | result = acpi_bus_register_driver(&acpi_ac_driver); | 347 | result = acpi_bus_register_driver(&acpi_ac_driver); |
331 | if (result < 0) { | 348 | if (result < 0) { |
349 | #ifdef CONFIG_ACPI_PROCFS | ||
332 | acpi_unlock_ac_dir(acpi_ac_dir); | 350 | acpi_unlock_ac_dir(acpi_ac_dir); |
351 | #endif | ||
333 | return -ENODEV; | 352 | return -ENODEV; |
334 | } | 353 | } |
335 | 354 | ||
@@ -341,7 +360,9 @@ static void __exit acpi_ac_exit(void) | |||
341 | 360 | ||
342 | acpi_bus_unregister_driver(&acpi_ac_driver); | 361 | acpi_bus_unregister_driver(&acpi_ac_driver); |
343 | 362 | ||
363 | #ifdef CONFIG_ACPI_PROCFS | ||
344 | acpi_unlock_ac_dir(acpi_ac_dir); | 364 | acpi_unlock_ac_dir(acpi_ac_dir); |
365 | #endif | ||
345 | 366 | ||
346 | return; | 367 | return; |
347 | } | 368 | } |