diff options
| author | Russ Dill <russ.dill@gmail.com> | 2008-09-02 17:35:40 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-09-02 22:21:37 -0400 |
| commit | 39dbbb4523754df4a822c69191a848a03e556dc7 (patch) | |
| tree | ef0e116623d66534fcd694b0fc6c87c1463dd395 /drivers/misc/acer-wmi.c | |
| parent | 6ccfa806a9cfbbf1cd43d5b6aa47ef2c0eb518fd (diff) | |
acer-wmi: remove debugfs entries upon unloading
The exit function neglects to remove debugfs entries, leading to a BUG
on reload.
[akpm@linux-foundation.org: cleanups]
Signed-off-by: Russ Dill <Russ.Dill@gmail.com>
Acked-by: Carlos Corbacho <carlos@strangeworlds.co.uk>
Cc: Andi Kleen <ak@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/misc/acer-wmi.c')
| -rw-r--r-- | drivers/misc/acer-wmi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/misc/acer-wmi.c b/drivers/misc/acer-wmi.c index c6c77a505ec1..d8b0d326e452 100644 --- a/drivers/misc/acer-wmi.c +++ b/drivers/misc/acer-wmi.c | |||
| @@ -1189,7 +1189,7 @@ static int create_debugfs(void) | |||
| 1189 | return 0; | 1189 | return 0; |
| 1190 | 1190 | ||
| 1191 | error_debugfs: | 1191 | error_debugfs: |
| 1192 | remove_debugfs(); | 1192 | remove_debugfs(); |
| 1193 | return -ENOMEM; | 1193 | return -ENOMEM; |
| 1194 | } | 1194 | } |
| 1195 | 1195 | ||
| @@ -1272,6 +1272,7 @@ error_platform_register: | |||
| 1272 | static void __exit acer_wmi_exit(void) | 1272 | static void __exit acer_wmi_exit(void) |
| 1273 | { | 1273 | { |
| 1274 | remove_sysfs(acer_platform_device); | 1274 | remove_sysfs(acer_platform_device); |
| 1275 | remove_debugfs(); | ||
| 1275 | platform_device_del(acer_platform_device); | 1276 | platform_device_del(acer_platform_device); |
| 1276 | platform_driver_unregister(&acer_platform_driver); | 1277 | platform_driver_unregister(&acer_platform_driver); |
| 1277 | 1278 | ||
