greybus: legacy: use protocol pointer to determine state

Use the protocol pointer to determine the legacy connection state.

This is needed to allow core disable connections when an interface has
been hot-unplugged without the legacy protocols leaking its resources.

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

diff --git a/drivers/staging/greybus/legacy.c b/drivers/staging/greybus/legacy.c
index fd847f42376fe9f13f6c98e85ff6f27a56c1f847..a2c0b9bed9a9c8af3af99031cdb78556b5b3a98b 100644 (file)
--- a/drivers/staging/greybus/legacy.c
+++ b/drivers/staging/greybus/legacy.c
@@ -100,7 +100,7 @@ err_unbind_protocol:
 
 static void legacy_connection_exit(struct gb_connection *connection)
 {
-       if (connection->state == GB_CONNECTION_STATE_DISABLED)
+       if (!connection->protocol)
                return;
 
        gb_connection_disable(connection);