diff options
author | Alex Elder <elder@linaro.org> | 2016-05-17 09:13:16 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-05-18 16:38:11 -0700 |
commit | 814ae531d161d789496503969657ea9550e286b5 (patch) | |
tree | 76ff5bc52de022c44403859e1535279ff47eee59 /drivers/staging/greybus/connection.c | |
parent | ac00154a7c6a8f3b2a98c1cf3cef893af2b0abac (diff) | |
download | linux-stable-814ae531d161d789496503969657ea9550e286b5.tar.gz linux-stable-814ae531d161d789496503969657ea9550e286b5.tar.bz2 linux-stable-814ae531d161d789496503969657ea9550e286b5.zip |
greybus: connection: verify disabled when destroyed
A connection must be in DISABLED state before it gets destroyed.
Warn if this is ever not the case (and do the disconnect) before
proceeding with connection destruction.
Signed-off-by: Alex Elder <elder@linaro.org>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/connection.c')
-rw-r--r-- | drivers/staging/greybus/connection.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/staging/greybus/connection.c b/drivers/staging/greybus/connection.c index f803d40413ac..ac3be2fceade 100644 --- a/drivers/staging/greybus/connection.c +++ b/drivers/staging/greybus/connection.c @@ -660,6 +660,9 @@ void gb_connection_destroy(struct gb_connection *connection) if (!connection) return; + if (WARN_ON(connection->state != GB_CONNECTION_STATE_DISABLED)) + gb_connection_disable(connection); + mutex_lock(&gb_connection_mutex); spin_lock_irq(&gb_connections_lock); |