diff options
author | Bryan O'Donoghue <bryan.odonoghue@linaro.org> | 2016-07-18 14:59:36 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-07-18 09:22:45 -0700 |
commit | 6a1d29595ed7af5aebbcdb8f4cc045ae7893dc6c (patch) | |
tree | 0ed8b215bcd46cb41349a155133d120adf81f3c2 /drivers/staging/greybus/greybus_protocols.h | |
parent | 433aa123f51b85b5ad8fb2f088a6365d54e2763b (diff) | |
download | op-kernel-dev-6a1d29595ed7af5aebbcdb8f4cc045ae7893dc6c.zip op-kernel-dev-6a1d29595ed7af5aebbcdb8f4cc045ae7893dc6c.tar.gz |
greybus: greybus_protocols.h/es2: Ensure __le32 is used not u32
There is a dangling u32 in es2/greybus_protocols.h that is only obvious
when you try to compile up gbsim. We need to do a cpu_to_le32 and declare
the 32-bit int in the header as __le32. This patch does that and splits out
the assignment of the req->flags parameter to a temporary variable to allow
for later printing.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Suggested-by: Johan Hovold <johan@hovoldconsulting.com>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/greybus_protocols.h')
-rw-r--r-- | drivers/staging/greybus/greybus_protocols.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/greybus/greybus_protocols.h b/drivers/staging/greybus/greybus_protocols.h index 178ec89..d39a580 100644 --- a/drivers/staging/greybus/greybus_protocols.h +++ b/drivers/staging/greybus/greybus_protocols.h @@ -272,7 +272,7 @@ struct gb_control_intf_pm_response { #define APBA_REQUEST_ARPC_RUN 0x12 struct gb_apb_request_cport_flags { - u32 flags; + __le32 flags; #define GB_APB_CPORT_FLAG_CONTROL 0x01 #define GB_APB_CPORT_FLAG_HIGH_PRIO 0x02 } __packed; |