greybus: connection: try cancelling operations only if list isn't empty

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

drivers/staging/greybus/connection.c

@@ -226,11 +226,10 @@ void gb_connection_destroy(struct gb_connection *connection)
 		return;
 
 	/* XXX Need to wait for any outstanding requests to complete */
-	WARN_ON(!list_empty(&connection->operations));
-
-	list_for_each_entry_safe(operation, next, &connection->operations,
-					links) {
-		gb_operation_cancel(operation);
+	if (WARN_ON(!list_empty(&connection->operations))) {
+		list_for_each_entry_safe(operation, next,
+					 &connection->operations, links)
+			gb_operation_cancel(operation);
 	}
 	spin_lock_irq(&gb_connections_lock);
 	list_del(&connection->interface_links);