greybus: raw: fix memory leak on disconnect

Make sure the class device is freed as well as deregistered on
disconnect.

Also deregister the class device before character device as the former
depends on the latter.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>

diff --git a/drivers/staging/greybus/raw.c b/drivers/staging/greybus/raw.c
index fa5025d..a6e7959 100644 (file)
--- a/drivers/staging/greybus/raw.c
+++ b/drivers/staging/greybus/raw.c
@@ -203,9 +203,9 @@ static void gb_raw_connection_exit(struct gb_connection *connection)
        struct raw_data *temp;
 
        // FIXME - handle removing a connection when the char device node is open.
+       device_destroy(raw_class, raw->dev);
        cdev_del(&raw->cdev);
        ida_simple_remove(&minors, MINOR(raw->dev));
-       device_del(raw->device);
        mutex_lock(&raw->list_lock);
        list_for_each_entry_safe(raw_data, temp, &raw->list, entry) {
                list_del(&raw_data->entry);