greybus: timesync: Initialize the timesync ping fields to zero
authorBryan O'Donoghue <bryan.odonoghue@linaro.org>
Wed, 22 Jun 2016 13:58:43 +0000 (14:58 +0100)
committerGreg Kroah-Hartman <gregkh@google.com>
Fri, 24 Jun 2016 22:49:58 +0000 (15:49 -0700)
Remember to initialize the TimeSync ping fields to zero so that if a
timesync_get_last_event() returns an error - we display a FrameTime that is
obviously incorrect.

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

index b9b29f5..3391feb 100644 (file)
@@ -698,6 +698,7 @@ static void gb_timesync_ping(struct gb_timesync_svc *timesync_svc)
 
        /* Have SVC generate a timesync ping */
        timesync_svc->capture_ping = true;
+       timesync_svc->svc_ping_frame_time = 0;
        ret = gb_svc_timesync_ping(svc, &timesync_svc->svc_ping_frame_time);
        timesync_svc->capture_ping = false;
        if (ret) {
@@ -709,6 +710,7 @@ static void gb_timesync_ping(struct gb_timesync_svc *timesync_svc)
 
        /* Get the ping FrameTime from each APB/GPB */
        hd = timesync_svc->timesync_hd->hd;
+       timesync_svc->timesync_hd->ping_frame_time = 0;
        ret = hd->driver->timesync_get_last_event(hd,
                &timesync_svc->timesync_hd->ping_frame_time);
        if (ret)
@@ -717,6 +719,7 @@ static void gb_timesync_ping(struct gb_timesync_svc *timesync_svc)
        list_for_each_entry(timesync_interface,
                            &timesync_svc->interface_list, list) {
                control = timesync_interface->interface->control;
+               timesync_interface->ping_frame_time = 0;
                ping_frame_time = &timesync_interface->ping_frame_time;
                ret = gb_control_timesync_get_last_event(control,
                                                         ping_frame_time);