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@xxxxxxx> > > > > R: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> > > > > L: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated for non-subscribers) > > > > +L: linux-actions@xxxxxxxxxxxxxxxxxxx (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 >
