greybus: connection: Exit connection before destroying it

gb_connection_create() can initialize a connection if bundle->device id
is valid. And so the connection must be destroyed by calling
gb_connection_exit() before gb_connection_destroy(). This wasn't done in
the code that is parsing the manifest.

Fix it.

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

diff --git a/drivers/staging/greybus/manifest.c b/drivers/staging/greybus/manifest.c
index ce4e89c6d13ede232f6e1f7517b033eea4474738..bea565f1c1d10270e839b1df4069c4d1aab8d6b1 100644 (file)
--- a/drivers/staging/greybus/manifest.c
+++ b/drivers/staging/greybus/manifest.c
@@ -255,6 +255,7 @@ cleanup:
        /* An error occurred; undo any changes we've made */
        list_for_each_entry_safe(connection, connection_next,
                        &bundle->connections, bundle_links) {
+               gb_connection_exit(connection);
                gb_connection_destroy(connection);
                count--;
        }