diff mbox series

[6/6] MAINTAINERS: Add linux-actions mailing list for Actions Semi

Message ID 20181231185517.18517-7-manivannan.sadhasivam@linaro.org (mailing list archive)
State New, archived
Headers show
Series Add clock support for Actions Semi S500 SoC | expand

Commit Message

Manivannan Sadhasivam Dec. 31, 2018, 6:55 p.m. UTC
Add the linux-actions mailing list for Actions Semi SoC architecture.

Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

Joe Perches Dec. 31, 2018, 7:12 p.m. UTC | #1
On Tue, 2019-01-01 at 00:25 +0530, Manivannan Sadhasivam wrote:
> Add the linux-actions mailing list for Actions Semi SoC architecture.

Unrelated to adding this L: entry

> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -1240,6 +1240,7 @@ ARM/ACTIONS SEMI ARCHITECTURE
>  M:	Andreas Färber <afaerber@suse.de>
>  R:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> +L:	linux-actions@lists.infradead.org (moderated for non-subscribers)
>  S:	Maintained
>  N:	owl

The N: entry matches too many non-owl entries like:

Documentation/parport-lowlevel.txt
drivers/misc/ibmasm/lowlevel.c
drivers/misc/ibmasm/lowlevel.h
fs/udf/lowle
vel.c
net/ipv6/ip6_flowlabel.c
sound/soc/samsung/lowland.c

Maybe there should be two entries

N:	/owl
N:	-owl
Manivannan Sadhasivam Jan. 1, 2019, 3:26 a.m. UTC | #2
Hi Joe,

On Mon, Dec 31, 2018 at 11:12:18AM -0800, Joe Perches wrote:
> On Tue, 2019-01-01 at 00:25 +0530, Manivannan Sadhasivam wrote:
> > Add the linux-actions mailing list for Actions Semi SoC architecture.
> 
> Unrelated to adding this L: entry
> 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> []
> > @@ -1240,6 +1240,7 @@ ARM/ACTIONS SEMI ARCHITECTURE
> >  M:	Andreas Färber <afaerber@suse.de>
> >  R:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> >  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> > +L:	linux-actions@lists.infradead.org (moderated for non-subscribers)
> >  S:	Maintained
> >  N:	owl
> 
> The N: entry matches too many non-owl entries like:
> 
> Documentation/parport-lowlevel.txt
> drivers/misc/ibmasm/lowlevel.c
> drivers/misc/ibmasm/lowlevel.h
> fs/udf/lowle
> vel.c
> net/ipv6/ip6_flowlabel.c
> sound/soc/samsung/lowland.c
> 
> Maybe there should be two entries
> 
> N:	/owl
> N:	-owl
> 

Thanks for letting us know. Yes, we need to change it. How about below?

N: owl-
N: -owl

Will do it as a follow-up patch if agreed!

Regards,
Mani

>
Andreas Färber Jan. 1, 2019, 3:32 a.m. UTC | #3
Am 01.01.19 um 04:26 schrieb Manivannan Sadhasivam:
> On Mon, Dec 31, 2018 at 11:12:18AM -0800, Joe Perches wrote:
>> On Tue, 2019-01-01 at 00:25 +0530, Manivannan Sadhasivam wrote:
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>> []
>>> @@ -1240,6 +1240,7 @@ ARM/ACTIONS SEMI ARCHITECTURE
>>>  M:	Andreas Färber <afaerber@suse.de>
>>>  R:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>>>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>> +L:	linux-actions@lists.infradead.org (moderated for non-subscribers)
>>>  S:	Maintained
>>>  N:	owl
>>
>> The N: entry matches too many non-owl entries like:
>>
>> Documentation/parport-lowlevel.txt
>> drivers/misc/ibmasm/lowlevel.c
>> drivers/misc/ibmasm/lowlevel.h
>> fs/udf/lowle
>> vel.c
>> net/ipv6/ip6_flowlabel.c
>> sound/soc/samsung/lowland.c
>>
>> Maybe there should be two entries
>>
>> N:	/owl
>> N:	-owl
>>
> 
> Thanks for letting us know. Yes, we need to change it. How about below?
> 
> N: owl-
> N: -owl

No underscores anywhere?

Cheers,
Andreas

> 
> Will do it as a follow-up patch if agreed!
> 
> Regards,
> Mani
> 
>>
Manivannan Sadhasivam Jan. 1, 2019, 7:37 a.m. UTC | #4
On Tue, Jan 01, 2019 at 04:32:40AM +0100, Andreas Färber wrote:
> Am 01.01.19 um 04:26 schrieb Manivannan Sadhasivam:
> > On Mon, Dec 31, 2018 at 11:12:18AM -0800, Joe Perches wrote:
> >> On Tue, 2019-01-01 at 00:25 +0530, Manivannan Sadhasivam wrote:
> >>> diff --git a/MAINTAINERS b/MAINTAINERS
> >> []
> >>> @@ -1240,6 +1240,7 @@ ARM/ACTIONS SEMI ARCHITECTURE
> >>>  M:	Andreas Färber <afaerber@suse.de>
> >>>  R:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> >>>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> >>> +L:	linux-actions@lists.infradead.org (moderated for non-subscribers)
> >>>  S:	Maintained
> >>>  N:	owl
> >>
> >> The N: entry matches too many non-owl entries like:
> >>
> >> Documentation/parport-lowlevel.txt
> >> drivers/misc/ibmasm/lowlevel.c
> >> drivers/misc/ibmasm/lowlevel.h
> >> fs/udf/lowle
> >> vel.c
> >> net/ipv6/ip6_flowlabel.c
> >> sound/soc/samsung/lowland.c
> >>
> >> Maybe there should be two entries
> >>
> >> N:	/owl
> >> N:	-owl
> >>
> > 
> > Thanks for letting us know. Yes, we need to change it. How about below?
> > 
> > N: owl-
> > N: -owl
> 
> No underscores anywhere?
>

There is nothing with underscore yet and I don't think we'll add it in
future.

Thanks,
Mani

> Cheers,
> Andreas
> 
> > 
> > Will do it as a follow-up patch if agreed!
> > 
> > Regards,
> > Mani
> > 
> >>
> 
> 
> -- 
> SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Felix Imendörffer, Jane Smithard, Graham Norton
> HRB 21284 (AG Nürnberg)
Joe Perches Jan. 1, 2019, 4:56 p.m. UTC | #5
On Tue, 2019-01-01 at 08:56 +0530, Manivannan Sadhasivam wrote:
> Hi Joe,

Hi Manivannan.

> On Mon, Dec 31, 2018 at 11:12:18AM -0800, Joe Perches wrote:
> > On Tue, 2019-01-01 at 00:25 +0530, Manivannan Sadhasivam wrote:
> > > Add the linux-actions mailing list for Actions Semi SoC architecture.
> > 
> > Unrelated to adding this L: entry
> > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > []
> > > @@ -1240,6 +1240,7 @@ ARM/ACTIONS SEMI ARCHITECTURE
> > >  M:	Andreas Färber <afaerber@suse.de>
> > >  R:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > >  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> > > +L:	linux-actions@lists.infradead.org (moderated for non-subscribers)
> > >  S:	Maintained
> > >  N:	owl
> > 
> > The N: entry matches too many non-owl entries like:
> > 
> > Documentation/parport-lowlevel.txt
> > drivers/misc/ibmasm/lowlevel.c
> > drivers/misc/ibmasm/lowlevel.h
> > fs/udf/lowlevel.c
> > net/ipv6/ip6_flowlabel.c
> > sound/soc/samsung/lowland.c
> > 
> > Maybe there should be two entries
> > 
> > N:	/owl
> > N:	-owl
> > 
> 
> Thanks for letting us know. Yes, we need to change it. How about below?
> 
> N: owl-
> N: -owl
> 
> Will do it as a follow-up patch if agreed!

from MAINTAINERS:

	   scripts/get_maintainer.pl has different behavior for files that
	   match F: pattern and matches of N: patterns.  By default,
	   get_maintainer will not look at git log history when an F: pattern
	   match occurs.  When an N: match occurs, git log history is used
	   to also notify the people that have git commit signatures.

F: entries are preferred to N: entries because the
get_maintainer.pl script does not assume that the
N: entries are exclusively maintained.  Maybe use a
more comprehensive F: list like:

F:	Documentation/devicetree/bindings/*/*owl-*
F:	arch/arm/boot/dts/owl-*
F:	drivers/*/owl-*
F:	drivers/*/*-owl*
F:	drivers/*/*/owl-*
F:	drivers/*/*/*-owl*
F:	include/dt-bindings/power/owl-*
F:	include/linux/soc/actions/owl-sps.h
Manivannan Sadhasivam Jan. 2, 2019, 2:18 a.m. UTC | #6
Hi Joe,

On Tue, Jan 01, 2019 at 08:56:49AM -0800, Joe Perches wrote:
> On Tue, 2019-01-01 at 08:56 +0530, Manivannan Sadhasivam wrote:
> > Hi Joe,
> 
> Hi Manivannan.
> 
> > On Mon, Dec 31, 2018 at 11:12:18AM -0800, Joe Perches wrote:
> > > On Tue, 2019-01-01 at 00:25 +0530, Manivannan Sadhasivam wrote:
> > > > Add the linux-actions mailing list for Actions Semi SoC architecture.
> > > 
> > > Unrelated to adding this L: entry
> > > 
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > []
> > > > @@ -1240,6 +1240,7 @@ ARM/ACTIONS SEMI ARCHITECTURE
> > > >  M:	Andreas Färber <afaerber@suse.de>
> > > >  R:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > > >  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> > > > +L:	linux-actions@lists.infradead.org (moderated for non-subscribers)
> > > >  S:	Maintained
> > > >  N:	owl
> > > 
> > > The N: entry matches too many non-owl entries like:
> > > 
> > > Documentation/parport-lowlevel.txt
> > > drivers/misc/ibmasm/lowlevel.c
> > > drivers/misc/ibmasm/lowlevel.h
> > > fs/udf/lowlevel.c
> > > net/ipv6/ip6_flowlabel.c
> > > sound/soc/samsung/lowland.c
> > > 
> > > Maybe there should be two entries
> > > 
> > > N:	/owl
> > > N:	-owl
> > > 
> > 
> > Thanks for letting us know. Yes, we need to change it. How about below?
> > 
> > N: owl-
> > N: -owl
> > 
> > Will do it as a follow-up patch if agreed!
> 
> from MAINTAINERS:
> 
> 	   scripts/get_maintainer.pl has different behavior for files that
> 	   match F: pattern and matches of N: patterns.  By default,
> 	   get_maintainer will not look at git log history when an F: pattern
> 	   match occurs.  When an N: match occurs, git log history is used
> 	   to also notify the people that have git commit signatures.
> 
> F: entries are preferred to N: entries because the
> get_maintainer.pl script does not assume that the
> N: entries are exclusively maintained.  Maybe use a
> more comprehensive F: list like:
> 
> F:	Documentation/devicetree/bindings/*/*owl-*
> F:	arch/arm/boot/dts/owl-*
> F:	drivers/*/owl-*
> F:	drivers/*/*-owl*
> F:	drivers/*/*/owl-*
> F:	drivers/*/*/*-owl*
> F:	include/dt-bindings/power/owl-*
> F:	include/linux/soc/actions/owl-sps.h
> 

Makes sense!

Andreas: What is your opinion on above entries?

Thanks,
Mani

>
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 7d37f8a4743c..f30e25c8e4a7 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1240,6 +1240,7 @@  ARM/ACTIONS SEMI ARCHITECTURE
 M:	Andreas Färber <afaerber@suse.de>
 R:	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
 L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
+L:	linux-actions@lists.infradead.org (moderated for non-subscribers)
 S:	Maintained
 N:	owl
 F:	arch/arm/mach-actions/