diff options
author | Jesper Juhl <jesper.juhl@gmail.com> | 2005-11-07 04:01:32 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-11-07 10:54:05 -0500 |
commit | 6044ec8882c726e325017bd948aa0cd94ad33abc (patch) | |
tree | 3bfc5dc93434e8ad556540f6689abcd2699aa45d /drivers/w1 | |
parent | 2ea7533060e361810c21b2f5ee02151c4dfb85d8 (diff) |
[PATCH] kfree cleanup: misc remaining drivers
This is the remaining misc drivers/ part of the big kfree cleanup patch.
Remove pointless checks for NULL prior to calling kfree() in misc files in
drivers/.
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Acked-by: Aristeu Sergio Rozanski Filho <aris@cathedrallabs.org>
Acked-by: Roland Dreier <rolandd@cisco.com>
Acked-by: Pierre Ossman <drzeus@drzeus.cx>
Acked-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Greg Kroah-Hartman <gregkh@suse.de>
Acked-by: Len Brown <len.brown@intel.com>
Acked-by: "Antonino A. Daplas" <adaplas@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/w1')
-rw-r--r-- | drivers/w1/w1_ds2433.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/w1/w1_ds2433.c b/drivers/w1/w1_ds2433.c index 279e0e0363d6..1e3d98aac12d 100644 --- a/drivers/w1/w1_ds2433.c +++ b/drivers/w1/w1_ds2433.c | |||
@@ -299,10 +299,8 @@ static int w1_f23_add_slave(struct w1_slave *sl) | |||
299 | static void w1_f23_remove_slave(struct w1_slave *sl) | 299 | static void w1_f23_remove_slave(struct w1_slave *sl) |
300 | { | 300 | { |
301 | #ifdef CONFIG_W1_F23_CRC | 301 | #ifdef CONFIG_W1_F23_CRC |
302 | if (sl->family_data) { | 302 | kfree(sl->family_data); |
303 | kfree(sl->family_data); | 303 | sl->family_data = NULL; |
304 | sl->family_data = NULL; | ||
305 | } | ||
306 | #endif /* CONFIG_W1_F23_CRC */ | 304 | #endif /* CONFIG_W1_F23_CRC */ |
307 | sysfs_remove_bin_file(&sl->dev.kobj, &w1_f23_bin_attr); | 305 | sysfs_remove_bin_file(&sl->dev.kobj, &w1_f23_bin_attr); |
308 | } | 306 | } |