batman-adv: Remove unnecessary length qualifier in %14pM

Message ID c3693cdd478c68f078f105f55972a4406b333e69.1497432565.git.joe@perches.com (mailing list archive)
State Accepted, archived
Commit ea9e7718812648894c2aeca3cd4733d4f29f21e9
Delegated to: Simon Wunderlich
Headers

Commit Message

Joe Perches June 14, 2017, 9:33 a.m. UTC
  It's misleading and unnecessary.

Signed-off-by: Joe Perches <joe@perches.com>
---
 net/batman-adv/distributed-arp-table.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

David Miller June 14, 2017, 7:20 p.m. UTC | #1
From: Joe Perches <joe@perches.com>
Date: Wed, 14 Jun 2017 02:33:52 -0700

> It's misleading and unnecessary.
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Simon will you take this into your batman-adv tree?

Thanks.
  
Sven Eckelmann June 14, 2017, 7:41 p.m. UTC | #2
On Mittwoch, 14. Juni 2017 02:33:52 CEST Joe Perches wrote:
> It's misleading and unnecessary.
> 
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
>  net/batman-adv/distributed-arp-table.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

The patch was applied and is currently queued up as 
454ea1866ef45c6f1b747bd27a8203318be72b5d in batadv/net-next.

Thanks,
	Sven
  

Patch

diff --git a/net/batman-adv/distributed-arp-table.c b/net/batman-adv/distributed-arp-table.c
index 6930d6b50f99..b6cfa78e9381 100644
--- a/net/batman-adv/distributed-arp-table.c
+++ b/net/batman-adv/distributed-arp-table.c
@@ -834,7 +834,7 @@  int batadv_dat_cache_seq_print_text(struct seq_file *seq, void *offset)
 			last_seen_msecs = last_seen_msecs % 60000;
 			last_seen_secs = last_seen_msecs / 1000;
 
-			seq_printf(seq, " * %15pI4 %14pM %4i %6i:%02i\n",
+			seq_printf(seq, " * %15pI4 %pM %4i %6i:%02i\n",
 				   &dat_entry->ip, dat_entry->mac_addr,
 				   batadv_print_vid(dat_entry->vid),
 				   last_seen_mins, last_seen_secs);