greybus: Fixup __u64, __u32 to __le64, __le32 in timesync declarations
authorBryan O'Donoghue <bryan.odonoghue@linaro.org>
Tue, 12 Apr 2016 16:15:11 +0000 (17:15 +0100)
committerGreg Kroah-Hartman <gregkh@google.com>
Wed, 13 Apr 2016 05:33:39 +0000 (22:33 -0700)
A number of data in TimeSync command structures are declared __u64/__u32
instead of __le64/__le32, I forgot to put this through an x86_64 compile
before presentation for merge and as a result didn't catch this error. This
patch fixes.

Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
drivers/staging/greybus/control.c
drivers/staging/greybus/greybus_protocols.h

index 8475f15773253265acfde8b4837209cb39af62b1..91a7fcac4988bba3b5d767317a890adc43737f2e 100644 (file)
@@ -269,7 +269,7 @@ int gb_control_timesync_authoritative(struct gb_control *control,
        int i;
 
        for (i = 0; i < GB_TIMESYNC_MAX_STROBES; i++)
-               request.frame_time[i] = frame_time[i];
+               request.frame_time[i] = cpu_to_le64(frame_time[i]);
 
        return gb_operation_sync(control->connection,
                                 GB_CONTROL_TYPE_TIMESYNC_AUTHORITATIVE,
index e3ef3c96456de4c3ef67ac6fb85dfb6c4a63b500..e0c0493605ca11fcd7fd42f8a4c8a152ddba9113 100644 (file)
@@ -180,7 +180,7 @@ struct gb_control_timesync_enable_request {
 /* timesync enable response has no payload */
 
 struct gb_control_timesync_authoritative_request {
-       __u64   frame_time[GB_TIMESYNC_MAX_STROBES];
+       __le64  frame_time[GB_TIMESYNC_MAX_STROBES];
 } __packed;
 /* timesync authoritative response has no payload */
 
@@ -911,16 +911,16 @@ struct gb_svc_route_destroy_request {
 
 struct gb_svc_timesync_enable_request {
        __u8    count;
-       __u64   frame_time;
-       __u32   strobe_delay;
-       __u32   strobe_mask;
-       __u32   refclk;
+       __le64  frame_time;
+       __le32  strobe_delay;
+       __le32  strobe_mask;
+       __le32  refclk;
 } __packed;
 /* timesync enable response has no payload */
 
 /* timesync authoritative request has no payload */
 struct gb_svc_timesync_authoritative_response {
-       __u64   frame_time[GB_TIMESYNC_MAX_STROBES];
+       __le64  frame_time[GB_TIMESYNC_MAX_STROBES];
 };
 
 #define GB_SVC_UNIPRO_FAST_MODE                        0x01