Skip to contentSkip to conversation

timestamping: fix example build

PATCH Applied Randy Dunlap 2 participants 2 messages Mar 13, 2010 – Mar 15, 2010

Subsystems (3)
Networking Network drivers Documentation
lore
Share & tools
lore

Randy Dunlap

author
#0 lore raw

PATCH 1/1 timestamping: fix example build

From: Randy Dunlap <[email protected]>

Fix Makefiles so that Documentation/networking/timestamping/timestamping.c will build when using the CONFIG_BUILD_DOCSRC kconfig option. (timestamping.c does not build currently with its simple Makefile.)

Also fix printf format warnings.

Signed-off-by Randy Dunlap <[email protected]>
Cc Patrick Ohly <[email protected]>
---
 Documentation/networking/Makefile                    |    2 +
 Documentation/networking/timestamping/Makefile       |   11 ++++++++--
 Documentation/networking/timestamping/timestamping.c |   10 ++++-----
 3 files changed, 16 insertions(+), 7 deletions(-)
Documentation/networking/Makefile +2 -0
@@ -6,3 +6,5 @@ hostprogs-y := ifenslave
 
 # Tell kbuild to always build the programs
 always := $(hostprogs-y)
+
+obj-m := timestamping/
Documentation/networking/timestamping/Makefile +9 -2
@@ -1,6 +1,13 @@
-CPPFLAGS = -I../../../include
+# kbuild trick to avoid linker error. Can be omitted if a module is built.
+obj- := dummy.o
 
-timestamping: timestamping.c
+# List of programs to build
+hostprogs-y := timestamping
+
+# Tell kbuild to always build the programs
+always := $(hostprogs-y)
+
+HOSTCFLAGS_timestamping.o += -I$(objtree)/usr/include
 
 clean:
 	rm -f timestamping
Documentation/networking/timestamping/timestamping.c +5 -5
@@ -41,9 +41,9 @@
 #include <arpa/inet.h>
 #include <net/if.h>
 
-#include "asm/types.h"
-#include "linux/net_tstamp.h"
-#include "linux/errqueue.h"
+#include <asm/types.h>
+#include <linux/net_tstamp.h>
+#include <linux/errqueue.h>
 
 #ifndef SO_TIMESTAMPING
 # define SO_TIMESTAMPING         37

@@ -164,7 +164,7 @@ static void printpacket(struct msghdr *m
 
 	gettimeofday(&now, 0);
 
-	printf("%ld.%06ld: received %s data, %d bytes from %s, %d bytes control messages\n",
+	printf("%ld.%06ld: received %s data, %d bytes from %s, %zu bytes control messages\n",
 	       (long)now.tv_sec, (long)now.tv_usec,
 	       (recvmsg_flags & MSG_ERRQUEUE) ? "error" : "regular",
 	       res,

@@ -173,7 +173,7 @@ static void printpacket(struct msghdr *m
 	for (cmsg = CMSG_FIRSTHDR(msg);
 	     cmsg;
 	     cmsg = CMSG_NXTHDR(msg, cmsg)) {
-		printf("   cmsg len %d: ", cmsg->cmsg_len);
+		printf("   cmsg len %zu: ", cmsg->cmsg_len);
 		switch (cmsg->cmsg_level) {
 		case SOL_SOCKET:
 			printf("SOL_SOCKET ");
maintainer Applied
#1 lore raw

From: Randy Dunlap <[email protected]>
Date: Fri, 12 Mar 2010 21:23:27 -0800

Quoted 10 lines

From: Randy Dunlap <[email protected]>

Fix Makefiles so that Documentation/networking/timestamping/timestamping.c will build when using the CONFIG_BUILD_DOCSRC kconfig option. (timestamping.c does not build currently with its simple Makefile.)

Also fix printf format warnings.

Signed-off-by Randy Dunlap <[email protected]>
Cc Patrick Ohly <[email protected]>

Applied, thanks Randy.