greybus: operation: clean up greybus_message_sent

Add connection variable to greybus_message_sent.

This will be put to more use by a follow-up up patch.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Tested-by: Rui Miguel Silva <rui.silva@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
This commit is contained in:
Johan Hovold 2015-07-22 17:49:18 +02:00 committed by Greg Kroah-Hartman
parent b4bd734ea6
commit a4e08469eb

View file

@ -778,7 +778,8 @@ static int gb_operation_response_send(struct gb_operation *operation,
void greybus_message_sent(struct greybus_host_device *hd,
struct gb_message *message, int status)
{
struct gb_operation *operation;
struct gb_operation *operation = message->operation;
struct gb_connection *connection = operation->connection;
/*
* If the message was a response, we just need to drop our
@ -790,10 +791,9 @@ void greybus_message_sent(struct greybus_host_device *hd,
* attempting to send it, record that as the result of
* the operation and schedule its completion.
*/
operation = message->operation;
if (message == operation->response) {
if (status) {
dev_err(&operation->connection->dev,
dev_err(&connection->dev,
"error sending response: %d\n", status);
}
gb_operation_put_active(operation);