greybus: i2c: fix name conflict between function and struct: gb_i2c_transfer_response
authorViresh Kumar <viresh.kumar@linaro.org>
Thu, 22 Jan 2015 07:12:39 +0000 (12:42 +0530)
committerGreg Kroah-Hartman <greg@kroah.com>
Thu, 22 Jan 2015 13:15:39 +0000 (21:15 +0800)
'gb_i2c_transfer_response' is the name given to a function and a struct. Though
we don't get any compilation errors/warnings about it, but the names should be
unique.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
drivers/staging/greybus/i2c.c

index ca88914..31528af 100644 (file)
@@ -230,7 +230,7 @@ gb_i2c_transfer_request(struct gb_connection *connection,
        return operation;
 }
 
-static void gb_i2c_transfer_response(struct i2c_msg *msgs, u32 msg_count,
+static void gb_i2c_decode_response(struct i2c_msg *msgs, u32 msg_count,
                                struct gb_i2c_transfer_response *response)
 {
        struct i2c_msg *msg = msgs;
@@ -273,7 +273,7 @@ static int gb_i2c_transfer_operation(struct gb_i2c_device *gb_i2c_dev,
                struct gb_i2c_transfer_response *response;
 
                response = operation->response->payload;
-               gb_i2c_transfer_response(msgs, msg_count, response);
+               gb_i2c_decode_response(msgs, msg_count, response);
                ret = msg_count;
        } else if (!gb_i2c_expected_transfer_error(ret)) {
                pr_err("transfer operation failed (%d)\n", ret);