Commit f9624ded authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

greybus: operation: fix some sparse warnings

One of which was "real".
Signed-off-by: default avatarGreg Kroah-Hartman <greg@kroah.com>
parent 066799c1
......@@ -274,8 +274,14 @@ static void gb_operation_gbuf_complete(struct gbuf *gbuf)
header = operation->response_payload;
else
header = NULL;
id = header ? (int)header->id : -1;
type = header ? (int)header->type : -1;
if (header) {
id = le16_to_cpu(header->id);
type = header->type;
} else {
id = -1;
type = -1;
}
gb_connection_err(operation->connection,
"operation %d type %d gbuf error %d",
......@@ -292,8 +298,9 @@ static void gb_operation_gbuf_complete(struct gbuf *gbuf)
* initialize it here (it'll be overwritten by the incoming
* message).
*/
struct gbuf *gb_operation_gbuf_create(struct gb_operation *operation,
u8 type, size_t size, bool data_out)
static struct gbuf *gb_operation_gbuf_create(struct gb_operation *operation,
u8 type, size_t size,
bool data_out)
{
struct gb_connection *connection = operation->connection;
struct gb_operation_msg_hdr *header;
......
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