| Message ID | 1474151235-16558-1-git-send-email-f.fainelli@gmail.com |
|---|---|
| State | Changes Requested, archived |
| Delegated to: | David Miller |
| Headers | show |
On Sat, 2016-09-17 at 15:27 -0700, Florian Fainelli wrote: > Gary has not been with Broadcom for some time now, replace his address > with the internal mailing-list used for other entries. > > > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> > --- > Michael, > > Since this is an old driver, not sure who could step up as a maintainer > for b44? [] > diff --git a/MAINTAINERS b/MAINTAINERS [] > @@ -2500,8 +2500,8 @@ S: Supported > F: kernel/bpf/ > > BROADCOM B44 10/100 ETHERNET DRIVER > -M: Gary Zambrano <zambrano@broadcom.com> > L: netdev@vger.kernel.org > +M: bcm-kernel-feedback-list@broadcom.com > S: Supported > F: drivers/net/ethernet/broadcom/b44.* Without an actual maintainer, this should really be orphan and not supported. And the M: bcm-kernel-feedback-list@ should be L: BCM4401 NICs are essentially from 2002. Does anyone really use these any longer with a current distribution or kernel version?
2016-09-17 15:51 GMT-07:00 Joe Perches <joe@perches.com>: > On Sat, 2016-09-17 at 15:27 -0700, Florian Fainelli wrote: >> Gary has not been with Broadcom for some time now, replace his address >> with the internal mailing-list used for other entries. >> >> > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> >> --- >> Michael, >> >> Since this is an old driver, not sure who could step up as a maintainer >> for b44? > [] >> diff --git a/MAINTAINERS b/MAINTAINERS > [] >> @@ -2500,8 +2500,8 @@ S: Supported > >> F: kernel/bpf/ >> >> BROADCOM B44 10/100 ETHERNET DRIVER >> -M: Gary Zambrano <zambrano@broadcom.com> >> L: netdev@vger.kernel.org >> +M: bcm-kernel-feedback-list@broadcom.com >> S: Supported >> F: drivers/net/ethernet/broadcom/b44.* > > Without an actual maintainer, this should really be > orphan and not supported. I would like to hear from Michael before concluding that > > And the M: bcm-kernel-feedback-list@ should be L: The list does not accept public subscribers, so this is the correct entry to use. > > BCM4401 NICs are essentially from 2002. > > Does anyone really use these any longer with a > current distribution or kernel version? This NIC is also embedded inside BCM47xx/BCM53xx which is still getting active support from Rafal and Hauke.
On Sat, 2016-09-17 at 16:17 -0700, Florian Fainelli wrote: > 2016-09-17 15:51 GMT-07:00 Joe Perches <joe@perches.com>: [] > > Without an actual maintainer, this should really be > > orphan and not supported. > I would like to hear from Michael before concluding that No worries. > > And the M: bcm-kernel-feedback-list@ should be L: > The list does not accept public subscribers, so this is the correct > entry to use. Then M: is definitely _not_ the correct entry for this and it should be: L: bcm-kernel-feedback-list@broadcom.com (subscribers-only)
2016-09-17 16:23 GMT-07:00 Joe Perches <joe@perches.com>: > On Sat, 2016-09-17 at 16:17 -0700, Florian Fainelli wrote: >> 2016-09-17 15:51 GMT-07:00 Joe Perches <joe@perches.com>: > [] >> > Without an actual maintainer, this should really be >> > orphan and not supported. >> I would like to hear from Michael before concluding that > > No worries. > >> > And the M: bcm-kernel-feedback-list@ should be L: >> The list does not accept public subscribers, so this is the correct >> entry to use. > > Then M: is definitely _not_ the correct entry for this > and it should be: > > L: bcm-kernel-feedback-list@broadcom.com (subscribers-only) Olof indicated otherwise, so who is right here? https://www.spinics.net/lists/arm-kernel/msg512572.html prompting to this patch series: http://linux-arm-kernel.infradead.narkive.com/CRyvGOKd/patch-maintainers-change-l-to-m-for-broadcom-arm-soc-entries
On Sat, Sep 17, 2016 at 4:17 PM, Florian Fainelli <f.fainelli@gmail.com> wrote: > 2016-09-17 15:51 GMT-07:00 Joe Perches <joe@perches.com>: >> On Sat, 2016-09-17 at 15:27 -0700, Florian Fainelli wrote: >>> Gary has not been with Broadcom for some time now, replace his address >>> with the internal mailing-list used for other entries. >>> >>> > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> >>> --- >>> Michael, >>> >>> Since this is an old driver, not sure who could step up as a maintainer >>> for b44? >> [] >>> diff --git a/MAINTAINERS b/MAINTAINERS >> [] >>> @@ -2500,8 +2500,8 @@ S: Supported >> >>> F: kernel/bpf/ >>> >>> BROADCOM B44 10/100 ETHERNET DRIVER >>> -M: Gary Zambrano <zambrano@broadcom.com> >>> L: netdev@vger.kernel.org >>> +M: bcm-kernel-feedback-list@broadcom.com >>> S: Supported >>> F: drivers/net/ethernet/broadcom/b44.* >> >> Without an actual maintainer, this should really be >> orphan and not supported. > > I would like to hear from Michael before concluding that > I have worked on this NIC more than 10 years ago. Last time I checked, I don't have this NIC anymore after moving offices several times. I don't mind being the maintainer, if no one else more suitable and have access to hardware wants to do it.
On Sat, 2016-09-17 at 16:38 -0700, Florian Fainelli wrote: > 2016-09-17 16:23 GMT-07:00 Joe Perches <joe@perches.com>: > > On Sat, 2016-09-17 at 16:17 -0700, Florian Fainelli wrote: > > > The list does not accept public subscribers, so this is the correct > > > entry to use. > > Then M: is definitely _not_ the correct entry for this > > and it should be: > > L: bcm-kernel-feedback-list@broadcom.com (subscribers-only) > Olof indicated otherwise, so who is right here?https://www.spinics.net/lists/arm-kernel/msg512572.html > prompting to this patch series: > http://linux-arm-kernel.infradead.narkive.com/CRyvGOKd/patch-maintainers-change-l-to- This hasn't been applied to -next, and I looked at the existing L: entries. No worries here either if it's an exploder and not a mailing list. Pity it's not called something like bcm-linux-drivers@broadcom.com if it's really an exploder.
On 09/17/2016 04:39 PM, Michael Chan wrote: > On Sat, Sep 17, 2016 at 4:17 PM, Florian Fainelli <f.fainelli@gmail.com> wrote: >> 2016-09-17 15:51 GMT-07:00 Joe Perches <joe@perches.com>: >>> On Sat, 2016-09-17 at 15:27 -0700, Florian Fainelli wrote: >>>> Gary has not been with Broadcom for some time now, replace his address >>>> with the internal mailing-list used for other entries. >>>> >>>>> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> >>>> --- >>>> Michael, >>>> >>>> Since this is an old driver, not sure who could step up as a maintainer >>>> for b44? >>> [] >>>> diff --git a/MAINTAINERS b/MAINTAINERS >>> [] >>>> @@ -2500,8 +2500,8 @@ S: Supported >>> >>>> F: kernel/bpf/ >>>> >>>> BROADCOM B44 10/100 ETHERNET DRIVER >>>> -M: Gary Zambrano <zambrano@broadcom.com> >>>> L: netdev@vger.kernel.org >>>> +M: bcm-kernel-feedback-list@broadcom.com >>>> S: Supported >>>> F: drivers/net/ethernet/broadcom/b44.* >>> >>> Without an actual maintainer, this should really be >>> orphan and not supported. >> >> I would like to hear from Michael before concluding that >> > > I have worked on this NIC more than 10 years ago. Last time I > checked, I don't have this NIC anymore after moving offices several > times. > > I don't mind being the maintainer, if no one else more suitable and > have access to hardware wants to do it. Should I resubmit with your name as a maintainer or do you want to do it?
On Mon, Sep 19, 2016 at 12:54 PM, Florian Fainelli <f.fainelli@gmail.com> wrote: > On 09/17/2016 04:39 PM, Michael Chan wrote: >> On Sat, Sep 17, 2016 at 4:17 PM, Florian Fainelli <f.fainelli@gmail.com> wrote: >>> 2016-09-17 15:51 GMT-07:00 Joe Perches <joe@perches.com>: >>>> On Sat, 2016-09-17 at 15:27 -0700, Florian Fainelli wrote: >>>>> Gary has not been with Broadcom for some time now, replace his address >>>>> with the internal mailing-list used for other entries. >>>>> >>>>>> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> >>>>> --- >>>>> Michael, >>>>> >>>>> Since this is an old driver, not sure who could step up as a maintainer >>>>> for b44? >>>> [] >>>>> diff --git a/MAINTAINERS b/MAINTAINERS >>>> [] >>>>> @@ -2500,8 +2500,8 @@ S: Supported >>>> >>>>> F: kernel/bpf/ >>>>> >>>>> BROADCOM B44 10/100 ETHERNET DRIVER >>>>> -M: Gary Zambrano <zambrano@broadcom.com> >>>>> L: netdev@vger.kernel.org >>>>> +M: bcm-kernel-feedback-list@broadcom.com >>>>> S: Supported >>>>> F: drivers/net/ethernet/broadcom/b44.* >>>> >>>> Without an actual maintainer, this should really be >>>> orphan and not supported. >>> >>> I would like to hear from Michael before concluding that >>> >> >> I have worked on this NIC more than 10 years ago. Last time I >> checked, I don't have this NIC anymore after moving offices several >> times. >> >> I don't mind being the maintainer, if no one else more suitable and >> have access to hardware wants to do it. > > Should I resubmit with your name as a maintainer or do you want to do it? I will submit the patch in a day or 2. Unless of course someone wants to be the maintainer instead.
diff --git a/MAINTAINERS b/MAINTAINERS index a5e1270dfbf1..dffc3bca17ee 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2500,8 +2500,8 @@ S: Supported F: kernel/bpf/ BROADCOM B44 10/100 ETHERNET DRIVER -M: Gary Zambrano <zambrano@broadcom.com> L: netdev@vger.kernel.org +M: bcm-kernel-feedback-list@broadcom.com S: Supported F: drivers/net/ethernet/broadcom/b44.*
Gary has not been with Broadcom for some time now, replace his address with the internal mailing-list used for other entries. Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> --- Michael, Since this is an old driver, not sure who could step up as a maintainer for b44? MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)