[PATCH] MAINTAINERS: add keyword "tegra" to Tegra section

[PATCH] MAINTAINERS: add keyword "tegra" to Tegra section

[Stephen Warren]

From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

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. Use a negative look-
behind assertion to make sure the keyword doesn't match those.

Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
 MAINTAINERS |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 972fd51..ebd3c64 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7850,9 +7850,7 @@ L:	linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
 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:	(?<!in)tegra
 
 TEHUTI ETHERNET DRIVER
 M:	Andy Gospodarek <andy-QlMahl40kYEqcZcGjlUOXw@public.gmane.org>
-- 
1.7.10.4

Re: [PATCH] MAINTAINERS: add keyword "tegra" to Tegra section

[Joe Perches]

On Fri, 2013-02-08 at 13:04 -0700, Stephen Warren wrote:
> (?<!in)tegra

This doesn't work well.   It matches too many
words like integral.

I suggest

K:	(?:\btegra\b|\bTEGRA_)

Re: [PATCH] MAINTAINERS: add keyword "tegra" to Tegra section

[Stephen Warren]

On 02/08/2013 05:40 PM, Joe Perches wrote:
> On Fri, 2013-02-08 at 13:04 -0700, Stephen Warren wrote:
>> (?<!in)tegra
> 
> This doesn't work well.   It matches too many
> words like integral.

Hmmm. That's exactly what the (?<!in) negative look-behind assertion is
attempting to avoid. It works fine here. Is there some dependency on
Perl version or something? My Perl version is v5.14.2.

> I suggest
> 
> K:	(?:\btegra\b|\bTEGRA_)
> 

Re: [PATCH] MAINTAINERS: add keyword "tegra" to Tegra section

[Joe Perches]

On Fri, 2013-02-08 at 17:47 -0700, Stephen Warren wrote:
> On 02/08/2013 05:40 PM, Joe Perches wrote:
> > On Fri, 2013-02-08 at 13:04 -0700, Stephen Warren wrote:
> >> (?<!in)tegra
> > 
> > This doesn't work well.   It matches too many
> > words like integral.
> 
> Hmmm. That's exactly what the (?<!in) negative look-behind assertion is
> attempting to avoid. It works fine here. Is there some dependency on
> Perl version or something? My Perl version is v5.14.2.

Integral matches

I don't see any issue with using a leading \b

Re: [PATCH] MAINTAINERS: add keyword "tegra" to Tegra section

[Stephen Warren]

On 02/08/2013 06:05 PM, Joe Perches wrote:
> On Fri, 2013-02-08 at 17:47 -0700, Stephen Warren wrote:
>> On 02/08/2013 05:40 PM, Joe Perches wrote:
>>> On Fri, 2013-02-08 at 13:04 -0700, Stephen Warren wrote:
>>>> (?<!in)tegra
>>>
>>> This doesn't work well.   It matches too many
>>> words like integral.
>>
>> Hmmm. That's exactly what the (?<!in) negative look-behind assertion is
>> attempting to avoid. It works fine here. Is there some dependency on
>> Perl version or something? My Perl version is v5.14.2.
> 
> Integral matches
> 
> I don't see any issue with using a leading \b

Oh right, it's a capitalization issue; I only tested with "integral" not
"Integral".

The problem with \btegra is that it wouldn't match _tegra, which would
be useful. Perhaps a custom almost-equivalent (?i)[^a-z]tegra would work?

Re: [PATCH] MAINTAINERS: add keyword "tegra" to Tegra section

[Joe Perches]

On Fri, 2013-02-08 at 20:15 -0700, Stephen Warren wrote:
> Perhaps a custom almost-equivalent (?i)[^a-z]tegra would work?

I didn't notice false positives with that one.