Development discussion of WireGuard
 help / color / mirror / Atom feed
* [PATCH net-next] net: Remove unused including <linux/version.h>
@ 2019-12-12  9:15 YueHaibing
  2019-12-12  9:52 ` Jason A. Donenfeld
  2019-12-12 18:52 ` David Miller
  0 siblings, 2 replies; 4+ messages in thread
From: YueHaibing @ 2019-12-12  9:15 UTC (permalink / raw)
  To: Jason A . Donenfeld, David S . Miller
  Cc: netdev, kernel-janitors, YueHaibing, linux-kernel, wireguard

Remove including <linux/version.h> that don't need it.

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 drivers/net/wireguard/main.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/wireguard/main.c b/drivers/net/wireguard/main.c
index 10c0a40f6a9e..7a7d5f1a80fc 100644
--- a/drivers/net/wireguard/main.c
+++ b/drivers/net/wireguard/main.c
@@ -12,7 +12,6 @@
 
 #include <uapi/linux/wireguard.h>
 
-#include <linux/version.h>
 #include <linux/init.h>
 #include <linux/module.h>
 #include <linux/genetlink.h>



_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next] net: Remove unused including <linux/version.h>
  2019-12-12  9:15 [PATCH net-next] net: Remove unused including <linux/version.h> YueHaibing
@ 2019-12-12  9:52 ` Jason A. Donenfeld
  2019-12-12 18:52 ` David Miller
  1 sibling, 0 replies; 4+ messages in thread
From: Jason A. Donenfeld @ 2019-12-12  9:52 UTC (permalink / raw)
  To: YueHaibing
  Cc: Netdev, kernel-janitors, David S . Miller, WireGuard mailing list, LKML

On Thu, Dec 12, 2019 at 10:18 AM YueHaibing <yuehaibing@huawei.com> wrote:
>
> Remove including <linux/version.h> that don't need it.
>
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Thanks. Committed with some wording changes to wireguard-linux.git for staging:

https://git.kernel.org/pub/scm/linux/kernel/git/zx2c4/wireguard-linux.git/commit/?id=062fd1993cdbf64a57395320c719e58b9a75b767

I'll re-send this to net-next as part of a cleanup series I'm
preparing in a few days.

Jason
_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next] net: Remove unused including <linux/version.h>
  2019-12-12  9:15 [PATCH net-next] net: Remove unused including <linux/version.h> YueHaibing
  2019-12-12  9:52 ` Jason A. Donenfeld
@ 2019-12-12 18:52 ` David Miller
  2019-12-12 19:06   ` Jason A. Donenfeld
  1 sibling, 1 reply; 4+ messages in thread
From: David Miller @ 2019-12-12 18:52 UTC (permalink / raw)
  To: yuehaibing; +Cc: netdev, kernel-janitors, linux-kernel, wireguard

From: YueHaibing <yuehaibing@huawei.com>
Date: Thu, 12 Dec 2019 09:15:27 +0000

> Remove including <linux/version.h> that don't need it.
> 
> Signed-off-by: YueHaibing <yuehaibing@huawei.com>

Appropriate subject line for this should have been:

	Subject: [PATCH net-next] wireguard: Remove unused include <linux/version.h>

'net' is too broad a subsystem prefix as it basically encompases half of the
entire kernel tree.  When people look at the git shortlog output you need to
be specific enough that people can tell what touches what.
_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next] net: Remove unused including <linux/version.h>
  2019-12-12 18:52 ` David Miller
@ 2019-12-12 19:06   ` Jason A. Donenfeld
  0 siblings, 0 replies; 4+ messages in thread
From: Jason A. Donenfeld @ 2019-12-12 19:06 UTC (permalink / raw)
  To: David Miller
  Cc: Netdev, kernel-janitors, YueHaibing, LKML, WireGuard mailing list

Hi Dave,

On Thu, Dec 12, 2019 at 7:53 PM David Miller <davem@davemloft.net> wrote:
>
> From: YueHaibing <yuehaibing@huawei.com>
> Date: Thu, 12 Dec 2019 09:15:27 +0000
>
> > Remove including <linux/version.h> that don't need it.
> >
> > Signed-off-by: YueHaibing <yuehaibing@huawei.com>
>
> Appropriate subject line for this should have been:
>
>         Subject: [PATCH net-next] wireguard: Remove unused include <linux/version.h>
>
> 'net' is too broad a subsystem prefix as it basically encompases half of the
> entire kernel tree.  When people look at the git shortlog output you need to
> be specific enough that people can tell what touches what.

I have these fixed up how you like in the wireguard-linux.git repo,
and I'll submit these in a series to net-next next week all together.

https://git.kernel.org/pub/scm/linux/kernel/git/zx2c4/wireguard-linux.git/log

Jason
_______________________________________________
WireGuard mailing list
WireGuard@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/wireguard

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-12-12 19:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-12  9:15 [PATCH net-next] net: Remove unused including <linux/version.h> YueHaibing
2019-12-12  9:52 ` Jason A. Donenfeld
2019-12-12 18:52 ` David Miller
2019-12-12 19:06   ` Jason A. Donenfeld

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).