Commit 8abf4148 authored by Viresh Kumar's avatar Viresh Kumar Committed by Greg Kroah-Hartman

greybus: operation: don't complete operation twice

Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: default avatarAlex Elder <elder@linaro.org>
Signed-off-by: default avatarGreg Kroah-Hartman <greg@kroah.com>
parent d37b1db1
...@@ -176,14 +176,12 @@ static void gb_operation_request_handle(struct gb_operation *operation) ...@@ -176,14 +176,12 @@ static void gb_operation_request_handle(struct gb_operation *operation)
*/ */
if (protocol->request_recv) { if (protocol->request_recv) {
protocol->request_recv(header->type, operation); protocol->request_recv(header->type, operation);
goto out; return;
} }
gb_connection_err(operation->connection, gb_connection_err(operation->connection,
"unexpected incoming request type 0x%02hhx\n", header->type); "unexpected incoming request type 0x%02hhx\n", header->type);
operation->result = GB_OP_PROTOCOL_BAD; operation->result = GB_OP_PROTOCOL_BAD;
out:
gb_operation_complete(operation);
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment