Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1258610 > unrolled thread

[PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained

Started byWolfram Sang <wsa@the-dreams.de>
First post2015-10-29 08:30 +0100
Last post2015-10-30 12:40 +0100
Articles 5 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained Wolfram Sang <wsa@the-dreams.de> - 2015-10-29 08:30 +0100
    Re: [PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained Joe Perches <joe@perches.com> - 2015-10-29 15:20 +0100
      Re: [PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained Wolfram Sang <wsa@the-dreams.de> - 2015-10-29 17:00 +0100
        Re: [PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained Joe Perches <joe@perches.com> - 2015-10-29 18:20 +0100
    Re: [PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained Wolfram Sang <wsa@the-dreams.de> - 2015-10-30 12:40 +0100

#1258610 — [PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained

FromWolfram Sang <wsa@the-dreams.de>
Date2015-10-29 08:30 +0100
Subject[PATCH] MAINTAINERS: i2c: mark also subdirectories as maintained
Message-ID<qoPON-5DQ-3@gated-at.bofh.it>
Otherwise get_maintainer.pl will fall back to git history and CC
more people than needed.

Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Acked-by: Lee Jones <lee.jones@linaro.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 2f23aab10398bb..c3f01dc36b26b4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5090,6 +5090,7 @@ S:	Maintained
 F:	Documentation/devicetree/bindings/i2c/
 F:	Documentation/i2c/
 F:	drivers/i2c/
+F:	drivers/i2c/*/
 F:	include/linux/i2c.h
 F:	include/linux/i2c-*.h
 F:	include/uapi/linux/i2c.h
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1258816

FromJoe Perches <joe@perches.com>
Date2015-10-29 15:20 +0100
Message-ID<qoWdz-1dD-11@gated-at.bofh.it>
In reply to#1258610
On Thu, 2015-10-29 at 08:25 +0100, Wolfram Sang wrote:
> Otherwise get_maintainer.pl will fall back to git history and CC
> more people than needed.
> 
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> Acked-by: Lee Jones <lee.jones@linaro.org>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2f23aab10398bb..c3f01dc36b26b4 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5090,6 +5090,7 @@ S:	Maintained
>  F:	Documentation/devicetree/bindings/i2c/
>  F:	Documentation/i2c/
>  F:	drivers/i2c/
> +F:	drivers/i2c/*/
>  F:	include/linux/i2c.h
>  F:	include/linux/i2c-*.h
>  F:	include/uapi/linux/i2c.h

Maybe something like this too:

diff --git a/MAINTAINERS b/MAINTAINERS
index a2d50fe..ff0ca88 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -93,6 +93,14 @@ Descriptions of section entries:
 	   F:	drivers/net/*	all files in drivers/net, but not below
 	   F:	*/net/*		all files in "any top level directory"/net
 	   One pattern per line.  Multiple F: lines acceptable.
+	   The scripts/get_maintainer.pl tool has a quirk about subdirectories.
+	   If the file pattern is for a directory, then any subdirectories of
+	   the parent directory should also be specified.  For instance:
+		F:	drivers/i2c/
+		F:	drivers/i2c/*/
+	   Otherwise, the scripts/get_maintainer.pl tool by default will use
+	   git history to add additional people that have "Signed-off-by:"
+	   style signatures on commits to files in those subdirectories.
 	N: Files and directories with regex patterns.
 	   N:	[^a-z]tegra	all files whose path contains the word tegra
 	   One pattern per line.  Multiple N: lines acceptable.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1258881

FromWolfram Sang <wsa@the-dreams.de>
Date2015-10-29 17:00 +0100
Message-ID<qoXMn-26W-27@gated-at.bofh.it>
In reply to#1258816

[Multipart message — attachments visible in raw view] — view raw

> +	   The scripts/get_maintainer.pl tool has a quirk about subdirectories.
> +	   If the file pattern is for a directory, then any subdirectories of
> +	   the parent directory should also be specified.  For instance:
> +		F:	drivers/i2c/
> +		F:	drivers/i2c/*/
> +	   Otherwise, the scripts/get_maintainer.pl tool by default will use
> +	   git history to add additional people that have "Signed-off-by:"
> +	   style signatures on commits to files in those subdirectories.

I'd be fine with the change. The patch description should probably state
why fixing the tool is not worth it.

[toc] | [prev] | [next] | [standalone]


#1258924

FromJoe Perches <joe@perches.com>
Date2015-10-29 18:20 +0100
Message-ID<qoZ1M-34X-7@gated-at.bofh.it>
In reply to#1258881
On Thu, 2015-10-29 at 16:52 +0100, Wolfram Sang wrote:
> > +	   The scripts/get_maintainer.pl tool has a quirk about subdirectories.
> > +	   If the file pattern is for a directory, then any subdirectories of
> > +	   the parent directory should also be specified.  For instance:
> > +		F:	drivers/i2c/
> > +		F:	drivers/i2c/*/
> > +	   Otherwise, the scripts/get_maintainer.pl tool by default will use
> > +	   git history to add additional people that have "Signed-off-by:"
> > +	   style signatures on commits to files in those subdirectories.
> 
> I'd be fine with the change. The patch description should probably state
> why fixing the tool is not worth it.

Not worth it vs inappropriate (arch/arm, etc...)

Maybe the wording around "quirk" could be improved or expanded.

Any suggestions?

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1259419

FromWolfram Sang <wsa@the-dreams.de>
Date2015-10-30 12:40 +0100
Message-ID<qpgci-5f8-23@gated-at.bofh.it>
In reply to#1258610

[Multipart message — attachments visible in raw view] — view raw

On Thu, Oct 29, 2015 at 08:25:48AM +0100, Wolfram Sang wrote:
> Otherwise get_maintainer.pl will fall back to git history and CC
> more people than needed.
> 
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> Acked-by: Lee Jones <lee.jones@linaro.org>

Applied to for-next, thanks!

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web