diff options
author | Johan Hovold <johan@hovoldconsulting.com> | 2015-11-03 12:11:26 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2015-11-04 20:20:23 -0800 |
commit | fcfc762f17616e9c720214d85b88ed672e767499 (patch) | |
tree | a51d11ad0c422fcd455bb8f59d96aa53fc1454a9 /drivers/staging/greybus/core.c | |
parent | greybus: connection: fix potential null-deref when binding protocol (diff) | |
download | linux-dev-fcfc762f17616e9c720214d85b88ed672e767499.tar.xz linux-dev-fcfc762f17616e9c720214d85b88ed672e767499.zip |
greybus: connection: kill gb_hd_connections_exit
Connections are destroyed as part of interface tear down. If we fail to
do that properly it's a bug that should be fixed rather than papered
over by a fall-back clean up function.
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>
Diffstat (limited to 'drivers/staging/greybus/core.c')
-rw-r--r-- | drivers/staging/greybus/core.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/staging/greybus/core.c b/drivers/staging/greybus/core.c index be75456a835d..726bf6480af3 100644 --- a/drivers/staging/greybus/core.c +++ b/drivers/staging/greybus/core.c @@ -242,12 +242,6 @@ void greybus_remove_hd(struct greybus_host_device *hd) if (hd->initial_svc_connection) gb_connection_destroy(hd->initial_svc_connection); - /* - * Make sure there are no leftovers that can potentially corrupt sysfs. - */ - if (WARN_ON(!list_empty(&hd->connections))) - gb_hd_connections_exit(hd); - kref_put_mutex(&hd->kref, free_hd, &hd_mutex); } EXPORT_SYMBOL_GPL(greybus_remove_hd); |