| Message ID | 20181231185517.18517-7-manivannan.sadhasivam@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | Add clock support for Actions Semi S500 SoC | expand |
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
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 >
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 > >>
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)
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
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 --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/
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(+)