[PATCH net 0/4] MAINTAINERS: Networking updates

[PATCH net 0/4] MAINTAINERS: Networking updates

[Simon Horman]

Hi,

This series includes Networking-related updates to MAINTAINERS.

* Patches 1 -3 aim to assign header files with "*net*' and '*skbuff*'
  in their name to Networking-related sections within Maintainers.

  There are a few such files left over after this patches.
  I plan to send separate patches to add them to SCSI SUBSYSTEM
  and NETWORKING DRIVERS (WIRELESS) sections.

* Patch 4 updates the status of the JME driver to 'Odd Fixes'

---
Simon Horman (4):
      MAINTAINERS: Add sonet.h to ATM section of MAINTAINERS
      MAINTAINERS: Add net_tstamp.h to PTP HARDWARE CLOCK SUPPORT section
      MAINTAINERS: Add header files to NETWORKING sections
      MAINTAINERS: Mark JME Network Driver as Odd Fixes

 MAINTAINERS | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

base-commit: 3d93a1448ed017f3536319a2445568b5f3969547

[PATCH net 1/4] MAINTAINERS: Add sonet.h to ATM section of MAINTAINERS

[Simon Horman]

This is part of an effort to assign a section in MAINTAINERS to header
files that relate to Networking. In this case the files with "net" in
their name.

It seems that sonet.h is included in ATM related source files,
and thus that ATM is the most relevant section for these files.

Cc: Chas Williams <3chas3@gmail.com>
Signed-off-by: Simon Horman <horms@kernel.org>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index a964a34651f5..c682203915a2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3504,7 +3504,9 @@ S:	Maintained
 W:	http://linux-atm.sourceforge.net
 F:	drivers/atm/
 F:	include/linux/atm*
+F:	include/linux/sonet.h
 F:	include/uapi/linux/atm*
+F:	include/uapi/linux/sonet.h
 
 ATMEL MACB ETHERNET DRIVER
 M:	Nicolas Ferre <nicolas.ferre@microchip.com>

-- 
2.43.0

[PATCH net 2/4] MAINTAINERS: Add net_tstamp.h to PTP HARDWARE CLOCK SUPPORT section

[Simon Horman]

This is part of an effort to assign a section in MAINTAINERS to header
files that relate to Networking. In this case the files with "net" in
their name.

As it relates to hardware timestamping, it seems that PTP HARDWARE CLOCK
SUPPORT is the most suitable section for net_tstamp.h.

Cc: Richard Cochran <richardcochran@gmail.com>
Signed-off-by: Simon Horman <horms@kernel.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index c682203915a2..28a67b93cef1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -18403,6 +18403,7 @@ F:	Documentation/ABI/testing/sysfs-ptp
 F:	Documentation/driver-api/ptp.rst
 F:	drivers/net/phy/dp83640*
 F:	drivers/ptp/*
+F:	include/linux/net_tstamp.h
 F:	include/linux/ptp_cl*
 K:	(?:\b|_)ptp(?:\b|_)
 

-- 
2.43.0

[PATCH net 3/4] MAINTAINERS: Add header files to NETWORKING sections

[Simon Horman]

This is part of an effort to assign a section in MAINTAINERS to header
files that relate to Networking. In this case the files with "net" or
"skbuff" in their name.

This patch adds a number of such files to the NETWORKING DRIVERS
and NETWORKING [GENERAL] sections.

Signed-off-by: Simon Horman <horms@kernel.org>
---
 MAINTAINERS | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 28a67b93cef1..38f3731d0e3b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15879,14 +15879,19 @@ F:	drivers/net/
 F:	include/dt-bindings/net/
 F:	include/linux/cn_proc.h
 F:	include/linux/etherdevice.h
+F:	include/linux/ethtool_netlink.h
 F:	include/linux/fcdevice.h
 F:	include/linux/fddidevice.h
 F:	include/linux/hippidevice.h
 F:	include/linux/if_*
 F:	include/linux/inetdevice.h
 F:	include/linux/netdevice.h
+F:	include/linux/netdevice_xmit.h
+F:	include/linux/platform_data/wiznet.h
 F:	include/uapi/linux/cn_proc.h
+F:	include/uapi/linux/ethtool_netlink.h
 F:	include/uapi/linux/if_*
+F:	include/uapi/linux/netdev.h
 F:	include/uapi/linux/netdevice.h
 F:	tools/testing/selftests/drivers/net/
 X:	drivers/net/wireless/
@@ -15939,14 +15944,31 @@ F:	include/linux/framer/framer-provider.h
 F:	include/linux/framer/framer.h
 F:	include/linux/in.h
 F:	include/linux/indirect_call_wrapper.h
+F:	include/linux/inet.h
+F:	include/linux/inet_diag.h
 F:	include/linux/net.h
+F:	include/linux/netdev_features.h
 F:	include/linux/netdevice.h
+F:	include/linux/netlink.h
+F:	include/linux/netpoll.h
+F:	include/linux/rtnetlink.h
+F:	include/linux/seq_file_net.h
 F:	include/linux/skbuff.h
+F:	include/linux/skbuff_ref.h
 F:	include/net/
+F:	include/uapi/linux/genetlink.h
+F:	include/uapi/linux/hsr_netlink.h
 F:	include/uapi/linux/in.h
+F:	include/uapi/linux/inet_diag.h
+F:	include/uapi/linux/nbd-netlink.h
 F:	include/uapi/linux/net.h
 F:	include/uapi/linux/net_namespace.h
+F:	include/uapi/linux/net_shaper.h
+F:	include/uapi/linux/netconf.h
 F:	include/uapi/linux/netdevice.h
+F:	include/uapi/linux/netlink.h
+F:	include/uapi/linux/netlink_diag.h
+F:	include/uapi/linux/rtnetlink.h
 F:	lib/net_utils.c
 F:	lib/random32.c
 F:	net/

-- 
2.43.0

[PATCH net 4/4] MAINTAINERS: Mark JME Network Driver as Odd Fixes

[Simon Horman]

This driver only appears to have received sporadic clean-ups, typically
part of some tree-wide activity, and fixes for quite some time.  And
according to the maintainer, Guo-Fu Tseng, the device has been EOLed for
a long time [1].

[1] https://lore.kernel.org/netdev/20240805003139.M94125@cooldavid.org/

Accordingly, it seems appropriate to mark this driver as odd fixes.

Cc: Moon Yeounsu <yyyynoom@gmail.com>
Cc: Guo-Fu Tseng <cooldavid@cooldavid.org>
Signed-off-by: Simon Horman <horms@kernel.org>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 38f3731d0e3b..6ed906576942 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11995,7 +11995,7 @@ F:	fs/jfs/
 JME NETWORK DRIVER
 M:	Guo-Fu Tseng <cooldavid@cooldavid.org>
 L:	netdev@vger.kernel.org
-S:	Maintained
+S:	Odd Fixes
 F:	drivers/net/ethernet/jme.*
 
 JOURNALLING FLASH FILE SYSTEM V2 (JFFS2)

-- 
2.43.0

Re: [PATCH net 3/4] MAINTAINERS: Add header files to NETWORKING sections

[Simon Horman]

On Fri, Aug 16, 2024 at 01:38:02PM +0100, Simon Horman wrote:
> This is part of an effort to assign a section in MAINTAINERS to header
> files that relate to Networking. In this case the files with "net" or
> "skbuff" in their name.
> 
> This patch adds a number of such files to the NETWORKING DRIVERS
> and NETWORKING [GENERAL] sections.
> 
> Signed-off-by: Simon Horman <horms@kernel.org>

...

> @@ -15939,14 +15944,31 @@ F:	include/linux/framer/framer-provider.h
>  F:	include/linux/framer/framer.h
>  F:	include/linux/in.h
>  F:	include/linux/indirect_call_wrapper.h
> +F:	include/linux/inet.h
> +F:	include/linux/inet_diag.h
>  F:	include/linux/net.h
> +F:	include/linux/netdev_features.h
>  F:	include/linux/netdevice.h
> +F:	include/linux/netlink.h
> +F:	include/linux/netpoll.h
> +F:	include/linux/rtnetlink.h
> +F:	include/linux/seq_file_net.h
>  F:	include/linux/skbuff.h
> +F:	include/linux/skbuff_ref.h
>  F:	include/net/
> +F:	include/uapi/linux/genetlink.h
> +F:	include/uapi/linux/hsr_netlink.h
>  F:	include/uapi/linux/in.h
> +F:	include/uapi/linux/inet_diag.h
> +F:	include/uapi/linux/nbd-netlink.h
>  F:	include/uapi/linux/net.h
>  F:	include/uapi/linux/net_namespace.h
> +F:	include/uapi/linux/net_shaper.h

Sorry, net_shaper.h doesn't exist upstream, but must have in my local
tree when I prepared this. I'll send a v2 in due course.

...

-- 
pw-bot: cr

Re: [PATCH net 3/4] MAINTAINERS: Add header files to NETWORKING sections

[Jakub Kicinski]

On Fri, 16 Aug 2024 14:25:35 +0100 Simon Horman wrote:
> Sorry, net_shaper.h doesn't exist upstream, but must have in my local
> tree when I prepared this. I'll send a v2 in due course.

Any thoughts on using regexps more?

Like for instance:

include/uapi/linux/net_*

should cover existing missing cases, and net_shaper when it comes.

include/linux/netdev*

could be anther from the context of the quote.

Re: [PATCH net 3/4] MAINTAINERS: Add header files to NETWORKING sections

[Simon Horman]

On Fri, Aug 16, 2024 at 08:38:58AM -0700, Jakub Kicinski wrote:
> On Fri, 16 Aug 2024 14:25:35 +0100 Simon Horman wrote:
> > Sorry, net_shaper.h doesn't exist upstream, but must have in my local
> > tree when I prepared this. I'll send a v2 in due course.
> 
> Any thoughts on using regexps more?
> 
> Like for instance:
> 
> include/uapi/linux/net_*
> 
> should cover existing missing cases, and net_shaper when it comes.
> 
> include/linux/netdev*
> 
> could be anther from the context of the quote.

Sure, let me try and verify that we don't get any unintended consequences,
e.g. via false positives. If not I'll update the patch accordingly.

I agree that it would be good to catch things that don't exist yet.