[PATCH] ptp.c: print field names and use macros form print_fields.h

[Harsha Sharma]

* ptp.c: print field names with field values for PTP_PEROUT_REQUEST
and use macros from print_fields.h
---
 ptp.c | 27 ++++++++++++++++-----------
 1 file changed, 16 insertions(+), 11 deletions(-)

diff --git a/ptp.c b/ptp.c
index 3cb3ae87..476d70c7 100644
--- a/ptp.c
+++ b/ptp.c
@@ -34,6 +34,7 @@
 # include <linux/ioctl.h>
 # include <linux/ptp_clock.h>
 
+#include "print_fields.h"
 # include "xlat/ptp_flags_options.h"
 
 int
@@ -51,7 +52,8 @@ ptp_ioctl(struct tcb *const tcp, const unsigned int code,
 		if (umove_or_printaddr(tcp, arg, &extts))
 			break;
 
-		tprintf("{index=%d, flags=", extts.index);
+		PRINT_FIELD_D("{", extts, index);
+		tprints(", flags=");
 		printflags(ptp_flags_options, extts.flags, "PTP_???");
 		tprints("}");
 		break;
@@ -64,12 +66,12 @@ ptp_ioctl(struct tcb *const tcp, const unsigned int code,
 		if (umove_or_printaddr(tcp, arg, &perout))
 			break;
 
-		tprintf("{start={%" PRId64 ", %" PRIu32 "}"
-			   ", period={%" PRId64 ", %" PRIu32 "}"
-			   ", index=%d, flags=",
+		tprintf("{start={sec=%" PRId64 ", nsec=%" PRIu32 "}"
+			   ", period={sec=%" PRId64 ", nsec=%" PRIu32 "}",
 			(int64_t)perout.start.sec, perout.start.nsec,
-			(int64_t)perout.period.sec, perout.period.nsec,
-			perout.index);
+			(int64_t)perout.period.sec, perout.period.nsec);
+		PRINT_FIELD_D(", ", perout, index);
+		tprints(", flags=");
 		printflags(ptp_flags_options, perout.flags, "PTP_???");
 		tprints("}");
 		break;
@@ -87,7 +89,7 @@ ptp_ioctl(struct tcb *const tcp, const unsigned int code,
 			if (umove_or_printaddr(tcp, arg, &sysoff))
 				break;
 
-			tprintf("{n_samples=%u", sysoff.n_samples);
+			PRINT_FIELD_U("{", sysoff, n_samples);
 			return 0;
 		} else {
 			unsigned int n_samples, i;
@@ -109,7 +111,7 @@ ptp_ioctl(struct tcb *const tcp, const unsigned int code,
 			for (i = 0; i < 2 * n_samples + 1; ++i) {
 				if (i > 0)
 					tprints(", ");
-				tprintf("{%" PRId64 ", %" PRIu32 "}",
+				tprintf("{sec=%" PRId64 ", nsec=%" PRIu32 "}",
 					(int64_t)sysoff.ts[i].sec,
 					sysoff.ts[i].nsec);
 			}
@@ -129,9 +131,12 @@ ptp_ioctl(struct tcb *const tcp, const unsigned int code,
 		if (umove_or_printaddr(tcp, arg, &caps))
 			break;
 
-		tprintf("{max_adj=%d, n_alarm=%d, n_ext_ts=%d, n_per_out=%d, pps=%d}",
-			caps.max_adj, caps.n_alarm, caps.n_ext_ts,
-			caps.n_per_out, caps.pps);
+		PRINT_FIELD_D("{", caps, max_adj);
+		PRINT_FIELD_D(", ", caps, n_alarm);
+		PRINT_FIELD_D(", ", caps, n_ext_ts);
+		PRINT_FIELD_D(", ", caps, n_per_out);
+		PRINT_FIELD_D(", ", caps, pps);
+		tprints("}");
 		break;
 	}
 
-- 
2.14.1