diff mbox series

MAINTAINERS: Add net-cw1200.h to NETWORKING DRIVERS (WIRELESS) section

Message ID 20240816-wifi-mnt-v1-1-3fb3bf5d44aa@kernel.org
State New
Headers show
Series MAINTAINERS: Add net-cw1200.h to NETWORKING DRIVERS (WIRELESS) section | expand

Commit Message

Simon Horman Aug. 16, 2024, 12:50 p.m. UTC
This is part of an effort [1] to assign a section in MAINTAINERS to header
files that relate to Networking. In this case the files with "net" in
their name.

[1] https://lore.kernel.org/netdev/20240816-net-mnt-v1-0-ef946b47ced4@kernel.org/

It seems that net-cw1200.h is included by CW1200 Wireless drivers, and
thus that NETWORKING DRIVERS (WIRELESS) is the most relevant section for
this file.

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

Comments

Kalle Valo Aug. 22, 2024, 9:08 a.m. UTC | #1
Simon Horman <horms@kernel.org> wrote:

> This is part of an effort [1] to assign a section in MAINTAINERS to header
> files that relate to Networking. In this case the files with "net" in
> their name.
> 
> [1] https://lore.kernel.org/netdev/20240816-net-mnt-v1-0-ef946b47ced4@kernel.org/
> 
> It seems that net-cw1200.h is included by CW1200 Wireless drivers, and
> thus that NETWORKING DRIVERS (WIRELESS) is the most relevant section for
> this file.
> 
> Signed-off-by: Simon Horman <horms@kernel.org>

Why add it to the general wireless drivers entry? To me the cw1200 entry would be more logical:

CW1200 WLAN driver
S:	Orphan
F:	drivers/net/wireless/st/cw1200/

Please change the title to:

MAINTAINERS: wifi: cw1200: add net-cw1200.h
Simon Horman Sept. 2, 2024, 9:09 a.m. UTC | #2
On Thu, Aug 22, 2024 at 09:08:44AM +0000, Kalle Valo wrote:
> Simon Horman <horms@kernel.org> wrote:
> 
> > This is part of an effort [1] to assign a section in MAINTAINERS to header
> > files that relate to Networking. In this case the files with "net" in
> > their name.
> > 
> > [1] https://lore.kernel.org/netdev/20240816-net-mnt-v1-0-ef946b47ced4@kernel.org/
> > 
> > It seems that net-cw1200.h is included by CW1200 Wireless drivers, and
> > thus that NETWORKING DRIVERS (WIRELESS) is the most relevant section for
> > this file.
> > 
> > Signed-off-by: Simon Horman <horms@kernel.org>
> 
> Why add it to the general wireless drivers entry? To me the cw1200 entry would be more logical:
> 
> CW1200 WLAN driver
> S:	Orphan
> F:	drivers/net/wireless/st/cw1200/
> 
> Please change the title to:
> 
> MAINTAINERS: wifi: cw1200: add net-cw1200.h

Thanks, will do.
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 8766f3e5e87e..028884a31640 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15896,6 +15896,7 @@  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/wireless/wireless.git
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/wireless/wireless-next.git
 F:	Documentation/devicetree/bindings/net/wireless/
 F:	drivers/net/wireless/
+F:	include/linux/platform_data/net-cw1200.h
 
 NETWORKING [DSA]
 M:	Andrew Lunn <andrew@lunn.ch>