greybus: timesync: reduce initial startup time
authorBryan O'Donoghue <bryan.odonoghue@linaro.org>
Tue, 19 Jul 2016 14:04:37 +0000 (15:04 +0100)
committerGreg Kroah-Hartman <gregkh@google.com>
Wed, 20 Jul 2016 18:16:25 +0000 (11:16 -0700)
Currently we have a long initial delay when initiating the first operation
to synchronize the FrameTime. That made sense during development of the
feature since we did synchronization asynchronously and wanted to give some
grace time for new Interfaces to be added before initiating a new
synchronization. On the PM runtime resume path though we do a synchronous
restoration of FrameTime and in this case waiting for 1 second before
initiating the first operation is clearly wrong.

This patch reduces the initial operation time bringing the synchronous call
time down by 990 milliseconds.

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 ddbdf2b..23b2091 100644 (file)
@@ -28,7 +28,7 @@
 #define GB_TIMESYNC_DEFAULT_OFFSET_US          1000
 
 /* Work queue timers long, short and SVC strobe timeout */
-#define GB_TIMESYNC_DELAYED_WORK_LONG          msecs_to_jiffies(1000)
+#define GB_TIMESYNC_DELAYED_WORK_LONG          msecs_to_jiffies(10)
 #define GB_TIMESYNC_DELAYED_WORK_SHORT         msecs_to_jiffies(1)
 #define GB_TIMESYNC_MAX_WAIT_SVC               msecs_to_jiffies(5000)
 #define GB_TIMESYNC_KTIME_UPDATE               msecs_to_jiffies(1000)