Re: [PATCH] [464/2many] MAINTAINERS - SUPERH (sh)

[Paul M. <le...@li...>]

On Sun, Aug 12, 2007 at 11:36:51PM -0700, jo...@pe... wrote:
> Add file pattern to MAINTAINER entry
> 
> Signed-off-by: Joe Perches <jo...@pe...>
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3e51fc6..f42712c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4414,6 +4414,8 @@ M:	le...@li...
>  L:	lin...@li... (subscribers-only)
>  W:	http://www.linux-sh.org
>  S:	Maintained
> +F:	arch/sh/
> +F:	arch/asm-sh/
>  
I suppose you meant include/ here?

On Sun, Aug 12, 2007 at 11:36:53PM -0700, jo...@pe... wrote:
> Add file pattern to MAINTAINER entry
> 
> Signed-off-by: Joe Perches <jo...@pe...>
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f42712c..df097fb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4423,6 +4423,8 @@ M:	le...@li...
>  L:	lin...@li...
>  W:	http://www.linux-sh.org
>  S:	Maintained
> +F:	arch/sh64/
> +F:	arch/asm-sh64/
>  

Likewise?

The more pressing question that comes to mind is what the point of this
is? Architecture related drivers and things of that nature fall under
this also, as most folks don't create a separate entry for every trivial
driver. Are you intending to have a long list of individual drivers here
that your script can parse?

Is there actually a problem with people just opening up MAINTAINERS in a
text file? Surely this is not such a non-intuitive thing, there are
already keywords there you can grep for.