[PATCH] br2684: Remove unnecessary formatting macros b1 and bs

Subsystems: atm, networking [general], the rest

STALE4104d

2 messages, 2 authors, 2015-07-31 · open the first message on its own page

[PATCH] br2684: Remove unnecessary formatting macros b1 and bs

From: Joe Perches <joe@perches.com>
Date: 2015-07-31 06:54:31

Use vsprintf extension %pI4 instead.

Signed-off-by: Joe Perches <joe@perches.com>
---
 net/atm/br2684.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/net/atm/br2684.c b/net/atm/br2684.c
index cc78538..aa0047c 100644
--- a/net/atm/br2684.c
+++ b/net/atm/br2684.c
@@ -802,13 +802,10 @@ static int br2684_seq_show(struct seq_file *seq, void *v)
 			   (brdev->payload == p_bridged) ? "bridged" : "routed",
 			   brvcc->copies_failed, brvcc->copies_needed);
 #ifdef CONFIG_ATM_BR2684_IPFILTER
-#define b1(var, byte)	((u8 *) &brvcc->filter.var)[byte]
-#define bs(var)		b1(var, 0), b1(var, 1), b1(var, 2), b1(var, 3)
 		if (brvcc->filter.netmask != 0)
-			seq_printf(seq, "    filter=%d.%d.%d.%d/"
-				   "%d.%d.%d.%d\n", bs(prefix), bs(netmask));
-#undef bs
-#undef b1
+			seq_printf(seq, "    filter=%pI4/%pI4\n",
+				   &brvcc->filter.prefix,
+				   &brvcc->filter.netmask);
 #endif /* CONFIG_ATM_BR2684_IPFILTER */
 	}
 	return 0;

Re: [PATCH] br2684: Remove unnecessary formatting macros b1 and bs

From: David Miller <davem@davemloft.net>
Date: 2015-07-31 22:26:07

From: Joe Perches <joe@perches.com>
Date: Thu, 30 Jul 2015 23:54:28 -0700
Use vsprintf extension %pI4 instead.

Signed-off-by: Joe Perches <joe@perches.com>
Applied.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help