[PATCH] vfree and kfree cleanup in drivers/

This patch does a full cleanup of 'NULL checks before vfree', and a partial
cleanup of calls to kfree for all of drivers/ - the kfree bit is partial in
that I only did the files that also had vfree calls in them.  The patch
also gets rid of some redundant (void *) casts of pointers being passed to
[vk]free, and a some tiny whitespace corrections also crept in.

Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

1 files changed, 3 insertions, 4 deletions

diff --git a/drivers/char/agp/backend.c b/drivers/char/agp/backend.c
index 4d4e602fdc7e..82b43c541c8d 100644
--- a/drivers/char/agp/backend.c
+++ b/drivers/char/agp/backend.c
@@ -206,10 +206,9 @@ static void agp_backend_cleanup(struct agp_bridge_data *bridge)
                 bridge->driver->cleanup();
         if (bridge->driver->free_gatt_table)
                 bridge->driver->free_gatt_table(bridge);
-        if (bridge->key_list) {
-                vfree(bridge->key_list);
-                bridge->key_list = NULL;
-        }
+
+        vfree(bridge->key_list);
+        bridge->key_list = NULL;
 
         if (bridge->driver->agp_destroy_page &&
             bridge->driver->needs_scratch_page)