From: Chi-Hsien Lin <[email protected]>
Add new maintainer and remove old maintainers.
Signed-off-by: Chi-Hsien Lin <[email protected]>
Signed-off-by: Wright Feng <[email protected]>
---
MAINTAINERS | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 5ebfc9de0caf..845058803f95 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3759,9 +3759,9 @@ BROADCOM BRCM80211 IEEE802.11n WIRELESS DRIVER
M: Arend van Spriel <[email protected]>
M: Franky Lin <[email protected]>
M: Hante Meuleman <[email protected]>
-M: Chi-hsien Lin <[email protected]>
M: Wright Feng <[email protected]>
-M: Chung-hsien Hsu <[email protected]>
+M: Double Lo <[email protected]>
+M: Kurt Lee <[email protected]>
L: [email protected]
L: [email protected]
L: [email protected]
--
2.25.0
Wright Feng <[email protected]> writes:
> From: Chi-Hsien Lin <[email protected]>
>
> Add new maintainer and remove old maintainers.
>
> Signed-off-by: Chi-Hsien Lin <[email protected]>
> Signed-off-by: Wright Feng <[email protected]>
> ---
> MAINTAINERS | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5ebfc9de0caf..845058803f95 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3759,9 +3759,9 @@ BROADCOM BRCM80211 IEEE802.11n WIRELESS DRIVER
> M: Arend van Spriel <[email protected]>
> M: Franky Lin <[email protected]>
> M: Hante Meuleman <[email protected]>
> -M: Chi-hsien Lin <[email protected]>
> M: Wright Feng <[email protected]>
> -M: Chung-hsien Hsu <[email protected]>
> +M: Double Lo <[email protected]>
> +M: Kurt Lee <[email protected]>
I'm not really a fan of having too many maintainers, I would say max is
3 and even that needs good justification. And most importantly, is
anyone else than Arend even doing any maintainer work for brcmfmac and
brcmsmac?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
On January 31, 2022 4:08:29 PM Kalle Valo <[email protected]> wrote:
> Wright Feng <[email protected]> writes:
>
>> From: Chi-Hsien Lin <[email protected]>
>>
>> Add new maintainer and remove old maintainers.
>>
>> Signed-off-by: Chi-Hsien Lin <[email protected]>
>> Signed-off-by: Wright Feng <[email protected]>
>> ---
>> MAINTAINERS | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 5ebfc9de0caf..845058803f95 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -3759,9 +3759,9 @@ BROADCOM BRCM80211 IEEE802.11n WIRELESS DRIVER
>> M: Arend van Spriel <[email protected]>
>> M: Franky Lin <[email protected]>
>> M: Hante Meuleman <[email protected]>
>> -M: Chi-hsien Lin <[email protected]>
>> M: Wright Feng <[email protected]>
>> -M: Chung-hsien Hsu <[email protected]>
>> +M: Double Lo <[email protected]>
>> +M: Kurt Lee <[email protected]>
>
> I'm not really a fan of having too many maintainers, I would say max is
> 3 and even that needs good justification. And most importantly, is
> anyone else than Arend even doing any maintainer work for brcmfmac and
> brcmsmac?
Guess it should be clear what falls under the term "maintainer work".
Infineon does a better job these days when it comes to contributions adding
features and new chip support, but that doesn't fall under the category
although maintainers do submit their own patches. It's good to have a go-to
guy at Infineon as they independently work on firmware and it's APIs. On
our side I can probably remove Hante from the list.
Regards,
Arend
Arend Van Spriel <[email protected]> writes:
> On January 31, 2022 4:08:29 PM Kalle Valo <[email protected]> wrote:
>
>> Wright Feng <[email protected]> writes:
>>
>>> From: Chi-Hsien Lin <[email protected]>
>>>
>>> Add new maintainer and remove old maintainers.
>>>
>>> Signed-off-by: Chi-Hsien Lin <[email protected]>
>>> Signed-off-by: Wright Feng <[email protected]>
>>> ---
>>> MAINTAINERS | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 5ebfc9de0caf..845058803f95 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -3759,9 +3759,9 @@ BROADCOM BRCM80211 IEEE802.11n WIRELESS DRIVER
>>> M: Arend van Spriel <[email protected]>
>>> M: Franky Lin <[email protected]>
>>> M: Hante Meuleman <[email protected]>
>>> -M: Chi-hsien Lin <[email protected]>
>>> M: Wright Feng <[email protected]>
>>> -M: Chung-hsien Hsu <[email protected]>
>>> +M: Double Lo <[email protected]>
>>> +M: Kurt Lee <[email protected]>
>>
>> I'm not really a fan of having too many maintainers, I would say max is
>> 3 and even that needs good justification. And most importantly, is
>> anyone else than Arend even doing any maintainer work for brcmfmac and
>> brcmsmac?
>
> Guess it should be clear what falls under the term "maintainer work".
From my point of view most important tasks of a maintainer are reviewing
patches and addressessing regressions (both runtime and build issues).
Of course there are others tasks as well, but these I consider crucial.
> Infineon does a better job these days when it comes to contributions
> adding features and new chip support, but that doesn't fall under the
> category although maintainers do submit their own patches. It's good
> to have a go-to guy at Infineon as they independently work on firmware
> and it's APIs. On our side I can probably remove Hante from the list.
Anyone can submit patches, and there can be a contact person from a
company even if it's not mentioned in MAINTAINERS.
I want to see that a maintainer really cares about upstream and works
with the community, like you Arend do. I do not want to add persons to
MAINTAINERS file and never hear from them again, that's just wasting
everyone's time.
To give a positive example, Ping does a great job maintaining Realtek's
rtw88 and rtw89 drivers. So if a company wants to learn how to become a
good maintainer, follow what Ping does :)
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches