Browse Source

greybus: operation: clean up early connection-state check

Now that the legacy invalid state is gone, clean up the early
connection-state check in the receive path and explicitly drop incoming
messages for disabled connection.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Johan Hovold 9 years ago
parent
commit
8890f95710
1 changed files with 1 additions and 2 deletions
  1. 1 2
      drivers/staging/greybus/operation.c

+ 1 - 2
drivers/staging/greybus/operation.c

@@ -968,8 +968,7 @@ void gb_connection_recv(struct gb_connection *connection,
 	size_t msg_size;
 	u16 operation_id;
 
-	if ((connection->state != GB_CONNECTION_STATE_ENABLED &&
-			connection->state != GB_CONNECTION_STATE_ENABLED_TX) ||
+	if (connection->state == GB_CONNECTION_STATE_DISABLED ||
 			gb_connection_is_offloaded(connection)) {
 		dev_warn_ratelimited(dev, "%s: dropping %zu received bytes\n",
 				connection->name, size);