diff options
author | Russ Dill <russ.dill@gmail.com> | 2008-09-02 17:35:40 -0400 |
---|---|---|
committer | Andi Kleen <ak@linux.intel.com> | 2008-09-04 08:40:48 -0400 |
commit | 7d964c352b06aabb895e39d3b479e105bd9d1ca0 (patch) | |
tree | 0b6684844032b6aa7df2585b44d65bf74b344e12 /drivers | |
parent | b635acec48bcaa9183fcbf4e3955616b0d4119b5 (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: Andi Kleen <ak@linux.intel.com>
Diffstat (limited to 'drivers')
-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 e7a3fe508dff..9c883f8b3896 100644 --- a/drivers/misc/acer-wmi.c +++ b/drivers/misc/acer-wmi.c | |||
@@ -1167,7 +1167,7 @@ static int create_debugfs(void) | |||
1167 | return 0; | 1167 | return 0; |
1168 | 1168 | ||
1169 | error_debugfs: | 1169 | error_debugfs: |
1170 | remove_debugfs(); | 1170 | remove_debugfs(); |
1171 | return -ENOMEM; | 1171 | return -ENOMEM; |
1172 | } | 1172 | } |
1173 | 1173 | ||
@@ -1248,6 +1248,7 @@ error_platform_register: | |||
1248 | static void __exit acer_wmi_exit(void) | 1248 | static void __exit acer_wmi_exit(void) |
1249 | { | 1249 | { |
1250 | remove_sysfs(acer_platform_device); | 1250 | remove_sysfs(acer_platform_device); |
1251 | remove_debugfs(); | ||
1251 | platform_device_del(acer_platform_device); | 1252 | platform_device_del(acer_platform_device); |
1252 | platform_driver_unregister(&acer_platform_driver); | 1253 | platform_driver_unregister(&acer_platform_driver); |
1253 | 1254 | ||