From: Bryan O'Donoghue Date: Wed, 22 Jun 2016 13:58:43 +0000 (+0100) Subject: greybus: timesync: Initialize the timesync ping fields to zero X-Git-Tag: v4.14-rc1~2366^2~378^2~21^2~225 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=50687f360958722de8610998a3cc886d9c9ba0b5;p=platform%2Fkernel%2Flinux-rpi.git greybus: timesync: Initialize the timesync ping fields to zero 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 Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/greybus/timesync.c b/drivers/staging/greybus/timesync.c index b9b29f5..3391feb 100644 --- a/drivers/staging/greybus/timesync.c +++ b/drivers/staging/greybus/timesync.c @@ -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, ×ync_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, ×ync_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, ×ync_svc->interface_list, list) { control = timesync_interface->interface->control; + timesync_interface->ping_frame_time = 0; ping_frame_time = ×ync_interface->ping_frame_time; ret = gb_control_timesync_get_last_event(control, ping_frame_time);