Re: [PATCH wireless v2] wifi: rework MAINTAINERS entries a bit

[Jeff Johnson <jeff.johnson@oss.qualcomm.com>]

On 3/26/2025 9:46 AM, Johannes Berg wrote:
> On Wed, 2025-03-26 at 09:45 -0700, Jeff Johnson wrote:
>> On 3/26/2025 8:55 AM, Johannes Berg wrote:
>>> On Wed, 2025-03-26 at 08:49 -0700, Jeff Johnson wrote:
>>>>
>>>> This patch has the unfortunate side-effect that get_maintainer.pl (and b4) no
>>>> longer add linux-wireless to ath* patches, and hence they no longer show up in
>>>> https://patchwork.kernel.org/project/linux-wireless/list/.
>>>>
>>>
>>> Oops. I _wanted_ that entry to no longer match so I don't get CC'ed on
>>> everything ... I guess we need to add L: entries to all the things I
>>> X:'ed. I sort of thought that might be there anyway, but I see that's
>>> not the case for everything ... below change perhaps?
>>>
>>> johannes
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index d19627afa652..09bd9827f318 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -3655,11 +3655,11 @@ T:	git git://github.com/AlbanBedel/linux
>>>  F:	Documentation/devicetree/bindings/phy/phy-ath79-usb.txt
>>>  F:	drivers/phy/qualcomm/phy-ath79-usb.c
>>>  
>>> -ATHEROS ATH GENERIC UTILITIES
>>> +ATHEROS ATH WIRELESS UTILITIES AND DRIVERS
>>>  M:	Jeff Johnson <jjohnson@kernel.org>
>>>  L:	linux-wireless@vger.kernel.org
>>>  S:	Supported
>>> -F:	drivers/net/wireless/ath/*
>>> +F:	drivers/net/wireless/ath/
>>
>> unfortunately the ath* specific entries will override so I think we need:
> 
> Hmm, really? Seems that if that were the case I wouldn't have been CC'ed
> on every ath*k patch?

ok, that was operator error. I thought I was applying your fixup but I was
actually applying the original (merged) patch.

but this now indicates I maintain all of the drivers under ath instead of just
maintaining the utilities that are in the top-level ath folder itself.

so from that perspective I think the ath entry should not be modified and
instead the additions should be made to the ath{10,11,12}k entries

/jeff