On 02/11/2013 11:04 AM, Joe Perches wrote: > On Mon, 2013-02-11 at 11:01 -0700, Stephen Warren wrote: >> On 02/11/2013 10:56 AM, Joe Perches wrote: >>> On Mon, 2013-02-11 at 10:51 -0700, Stephen Warren wrote: >>>> From: Stephen Warren <swarren@xxxxxxxxxx> >>>> >>>> The intent is to ensure that all Tegra-related patches are sent to the >>>> linux-tegra@ mailing list, so people can keep up-to-date on all misc >>>> driver changes. >>>> >>>> Doing this with a keyword is far simpler and more compact than listing >>>> all Tegra-related drivers, even if wildcards were used. >>>> >>>> Words such as integrate or integrator are common. Ensure the character >>>> right before "tegra" isn't a-z (case-insensitive), to make sure the >>>> keyword doesn't match those. >>>> >>>> Cc: Joe Perches <joe@xxxxxxxxxxx> >>>> Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx> >>>> --- >>>> MAINTAINERS | 4 +--- >>>> 1 file changed, 1 insertion(+), 3 deletions(-) >>>> >>>> diff --git a/MAINTAINERS b/MAINTAINERS >>>> index b98c89b..10e9b31 100644 >>>> --- a/MAINTAINERS >>>> +++ b/MAINTAINERS >>>> @@ -7829,9 +7829,7 @@ L: linux-tegra@xxxxxxxxxxxxxxx >>>> Q: http://patchwork.ozlabs.org/project/linux-tegra/list/ >>>> T: git git://git.kernel.org/pub/scm/linux/kernel/git/swarren/linux-tegra.git >>>> S: Supported >>>> -F: arch/arm/mach-tegra >>>> -F: arch/arm/boot/dts/tegra* >>>> -F: arch/arm/configs/tegra_defconfig >>>> +K: (?i)[^a-z]tegra >>>> >>>> TEHUTI ETHERNET DRIVER >>>> M: Andy Gospodarek <andy@xxxxxxxxxxxxx> >>> >>> It's probably not a good idea to remove >>> any of the F: lines as there are files >>> that could get modified in those paths >>> without any tegra word in the patch. >> >> At least for those F: lines that exist already and are removed above, >> the keyword will match their filename, so there's no issue here. >> > > Not every use of get_maintainers is by patch. > > ./scripts/get_maintainer.pl -f arch/arm/mach-tegra > > wouldn't match anything. That's annoying; it kinda defeats the whole purpose of this patch then. The way this patch works out with a patch is because the file-names are matched against the keywords since they appear in the patch. A solution to the issue you raise would be to do the same in -f mode, by adding an explicit pass to check the filenames that are found against MAINTAINERS too. Do you have any objection to that? -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
