From fd6d6f61d83c8217a052ba494f5b44abcec7ff43 Mon Sep 17 00:00:00 2001 From: Bryan O'Donoghue Date: Wed, 22 Jun 2016 14:58:44 +0100 Subject: [PATCH] greybus: timesync: Make printout consitent with other greybus messages The current printout on a TimeSync ping doesn't actually contain the word greybus and uses the word ping-time in the print string, something that appears nowhere in our official documentation on this feature. This patch changes the format string to contain 'greybus' and 'frametime' to bring the TimeSync printout more in-line with other greybus kernel log strings. before: timesync ping-time: ap=8632564 1-svc=8630712 greybus1=8633031 1-8.8=8633026 after: greybus frametime: ap=8632564 1-svc=8630712 greybus1=8633031 1-8.8=8633026 Signed-off-by: Bryan O'Donoghue Signed-off-by: Greg Kroah-Hartman --- drivers/staging/greybus/timesync.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/greybus/timesync.c b/drivers/staging/greybus/timesync.c index 3391febedc63..14531edb824c 100644 --- a/drivers/staging/greybus/timesync.c +++ b/drivers/staging/greybus/timesync.c @@ -581,7 +581,8 @@ static size_t gb_timesync_log_frame_time(struct gb_timesync_svc *timesync_svc, size_t off; /* AP/SVC */ - off = snprintf(buf, buflen, "timesync: ping-time ap=%llu %s=%llu ", + off = snprintf(buf, buflen, "%s frametime: ap=%llu %s=%llu ", + greybus_bus_type.name, timesync_svc->ap_ping_frame_time, dev_name(&svc->dev), timesync_svc->svc_ping_frame_time); len = buflen - off; @@ -623,8 +624,8 @@ static size_t gb_timesync_log_frame_ktime(struct gb_timesync_svc *timesync_svc, /* AP */ gb_timesync_to_timespec(timesync_svc, timesync_svc->ap_ping_frame_time, &ts); - off = snprintf(buf, buflen, "timesync: ping-time ap=%lu.%lu ", - ts.tv_sec, ts.tv_nsec); + off = snprintf(buf, buflen, "%s frametime: ap=%lu.%lu ", + greybus_bus_type.name, ts.tv_sec, ts.tv_nsec); len = buflen - off; if (len >= buflen) goto done; -- 2.39.5