Re: [PATCH 02/43] MAINTAINERS: add git URL for at91

Re: [PATCH 02/43] MAINTAINERS: add git URL for at91

[Alexandre Belloni]

On 18/12/2015 at 15:51:25 +0800, Fengguang Wu wrote :
> CC: Nicolas Ferre <nicolas.ferre@atmel.com>
> CC: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> CC: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

> ---
>  MAINTAINERS |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:26.469016840 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:26.469016840 +0800
> @@ -932,6 +932,7 @@ M:	Alexandre Belloni <alexandre.belloni@
>  M:	Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>  W:	http://www.linux4sam.org
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git
>  S:	Supported
>  F:	arch/arm/mach-at91/
>  F:	include/soc/at91/
> 
> 

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Re: [PATCH 02/43] MAINTAINERS: add git URL for at91

[Nicolas Ferre]

Le 18/12/2015 09:26, Alexandre Belloni a écrit :
> On 18/12/2015 at 15:51:25 +0800, Fengguang Wu wrote :
>> CC: Nicolas Ferre <nicolas.ferre@atmel.com>
>> CC: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>> CC: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
>> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

Fengguang, do you take this patch or should we take care of it by
queueing it through arm-soc?

Thanks, bye.

>> ---
>>  MAINTAINERS |    1 +
>>  1 file changed, 1 insertion(+)
>>
>> --- linux.orig/MAINTAINERS	2015-12-18 15:43:26.469016840 +0800
>> +++ linux/MAINTAINERS	2015-12-18 15:43:26.469016840 +0800
>> @@ -932,6 +932,7 @@ M:	Alexandre Belloni <alexandre.belloni@
>>  M:	Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
>>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>  W:	http://www.linux4sam.org
>> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git
>>  S:	Supported
>>  F:	arch/arm/mach-at91/
>>  F:	include/soc/at91/
>>
>>
> 


-- 
Nicolas Ferre

Re: [PATCH 02/43] MAINTAINERS: add git URL for at91

[Fengguang Wu]

On Fri, Dec 18, 2015 at 10:16:06AM +0100, Nicolas Ferre wrote:
> Le 18/12/2015 09:26, Alexandre Belloni a 閏rit :
> > On 18/12/2015 at 15:51:25 +0800, Fengguang Wu wrote :
> >> CC: Nicolas Ferre <nicolas.ferre@atmel.com>
> >> CC: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> >> CC: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
> >> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> 
> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> 
> Fengguang, do you take this patch or should we take care of it by
> queueing it through arm-soc?

I'll collect your Acked-by tags and whole-sale v2 to Andrew Morton. :)

Thanks!

Fengguang

> >> ---
> >>  MAINTAINERS |    1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> --- linux.orig/MAINTAINERS	2015-12-18 15:43:26.469016840 +0800
> >> +++ linux/MAINTAINERS	2015-12-18 15:43:26.469016840 +0800
> >> @@ -932,6 +932,7 @@ M:	Alexandre Belloni <alexandre.belloni@
> >>  M:	Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
> >>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> >>  W:	http://www.linux4sam.org
> >> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/nferre/linux-at91.git
> >>  S:	Supported
> >>  F:	arch/arm/mach-at91/
> >>  F:	include/soc/at91/
> >>
> >>
> > 
> 
> 
> -- 
> Nicolas Ferre

Re: [PATCH 04/43] MAINTAINERS: add git URL for ARM64

[Catalin Marinas]

On Fri, Dec 18, 2015 at 03:51:27PM +0800, Fengguang Wu wrote:
> CC: Catalin Marinas <catalin.marinas@arm.com>
> CC: Will Deacon <will.deacon@arm.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>  MAINTAINERS |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:26.685016847 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:26.685016847 +0800
> @@ -1766,6 +1766,7 @@ ARM64 PORT (AARCH64 ARCHITECTURE)
>  M:	Catalin Marinas <catalin.marinas@arm.com>
>  M:	Will Deacon <will.deacon@arm.com>
>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
>  S:	Maintained
>  F:	arch/arm64/
>  F:	Documentation/arm64/

Acked-by: Catalin Marinas <catalin.marinas@arm.com>

Re: [PATCH 04/43] MAINTAINERS: add git URL for ARM64

[Will Deacon]

On Fri, Dec 18, 2015 at 08:52:04AM +0000, Catalin Marinas wrote:
> On Fri, Dec 18, 2015 at 03:51:27PM +0800, Fengguang Wu wrote:
> > CC: Catalin Marinas <catalin.marinas@arm.com>
> > CC: Will Deacon <will.deacon@arm.com>
> > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > ---
> >  MAINTAINERS |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > --- linux.orig/MAINTAINERS	2015-12-18 15:43:26.685016847 +0800
> > +++ linux/MAINTAINERS	2015-12-18 15:43:26.685016847 +0800
> > @@ -1766,6 +1766,7 @@ ARM64 PORT (AARCH64 ARCHITECTURE)
> >  M:	Catalin Marinas <catalin.marinas@arm.com>
> >  M:	Will Deacon <will.deacon@arm.com>
> >  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git
> >  S:	Maintained
> >  F:	arch/arm64/
> >  F:	Documentation/arm64/
> 
> Acked-by: Catalin Marinas <catalin.marinas@arm.com>

I'll queue this for 4.5.

Will

Re: [PATCH 24/43] MAINTAINERS: add git URL for metag

[James Hogan]

[-- Attachment #1: Type: text/plain, Size: 803 bytes --]

On Fri, Dec 18, 2015 at 03:51:47PM +0800, Fengguang Wu wrote:
> CC: James Hogan <james.hogan@imgtec.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Acked-by: James Hogan <james.hogan@imgtec.com>

Let me know if you want me to take this through the metag tree.

Cheers
James

> ---
>  MAINTAINERS |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.897016917 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:28.897016917 +0800
> @@ -7032,6 +7032,7 @@ F:	Documentation/hwmon/menf21bmc
>  METAG ARCHITECTURE
>  M:	James Hogan <james.hogan@imgtec.com>
>  L:	linux-metag@vger.kernel.org
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/jhogan/metag.git
>  S:	Odd Fixes
>  F:	arch/metag/
>  F:	Documentation/metag/
> 
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH 24/43] MAINTAINERS: add git URL for metag

[Fengguang Wu]

On Fri, Dec 18, 2015 at 09:37:21AM +0000, James Hogan wrote:
> On Fri, Dec 18, 2015 at 03:51:47PM +0800, Fengguang Wu wrote:
> > CC: James Hogan <james.hogan@imgtec.com>
> > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> 
> Acked-by: James Hogan <james.hogan@imgtec.com>
> 
> Let me know if you want me to take this through the metag tree.

I'll collect your Acked-by and whole-sale v2 to Andrew Morton. :)

Thanks!

Fengguang

> > ---
> >  MAINTAINERS |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.897016917 +0800
> > +++ linux/MAINTAINERS	2015-12-18 15:43:28.897016917 +0800
> > @@ -7032,6 +7032,7 @@ F:	Documentation/hwmon/menf21bmc
> >  METAG ARCHITECTURE
> >  M:	James Hogan <james.hogan@imgtec.com>
> >  L:	linux-metag@vger.kernel.org
> > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/jhogan/metag.git
> >  S:	Odd Fixes
> >  F:	arch/metag/
> >  F:	Documentation/metag/
> > 
> > 

Re: [PATCH 21/43] MAINTAINERS: add git URL for KVM/ARM

[Christoffer Dall]

On Fri, Dec 18, 2015 at 03:51:44PM +0800, Fengguang Wu wrote:
> CC: Christoffer Dall <christoffer.dall@linaro.org>
> CC: Marc Zyngier <marc.zyngier@arm.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Acked-by: Christoffer Dall <christoffer.dall@linaro.org>

Re: [PATCH 21/43] MAINTAINERS: add git URL for KVM/ARM

[Marc Zyngier]

On 18/12/15 09:56, Christoffer Dall wrote:
> On Fri, Dec 18, 2015 at 03:51:44PM +0800, Fengguang Wu wrote:
>> CC: Christoffer Dall <christoffer.dall@linaro.org>
>> CC: Marc Zyngier <marc.zyngier@arm.com>
>> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> 
> Acked-by: Christoffer Dall <christoffer.dall@linaro.org>
> 

Applied.

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...

Re: [PATCH 05/43] MAINTAINERS: add git URL for ath6kl

[Kalle Valo]

Fengguang Wu <fengguang.wu@intel.com> writes:

> CC: Kalle Valo <kvalo@qca.qualcomm.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

To nitpick it's actually update, not add. But anyway:

Acked-by: Kalle Valo <kvalo@qca.qualcomm.com>

-- 
Kalle Valo

Re: [PATCH 29/43] MAINTAINERS: update git URL for ath10k

[Kalle Valo]

Fengguang Wu <fengguang.wu@intel.com> writes:

> CC: Kalle Valo <kvalo@qca.qualcomm.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Acked-by: Kalle Valo <kvalo@qca.qualcomm.com>

-- 
Kalle Valo

Re: [PATCH 20/43] MAINTAINERS: add git URL for KVM/s390

[Cornelia Huck]

On Fri, 18 Dec 2015 15:51:43 +0800
Fengguang Wu <fengguang.wu@intel.com> wrote:

> CC: Christian Borntraeger <borntraeger@de.ibm.com>
> CC: Cornelia Huck <cornelia.huck@de.ibm.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>  MAINTAINERS |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.457016903 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:28.457016903 +0800
> @@ -6117,6 +6117,7 @@ M:	Christian Borntraeger <borntraeger@de
>  M:	Cornelia Huck <cornelia.huck@de.ibm.com>
>  L:	linux-s390@vger.kernel.org
>  W:	http://www.ibm.com/developerworks/linux/linux390/
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git
>  S:	Supported
>  F:	Documentation/s390/kvm.txt
>  F:	arch/s390/include/asm/kvm*
> 
> 

Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com>

I wonder about the context? Your mails don't seem to have made it to
lkml...

Re: [PATCH 20/43] MAINTAINERS: add git URL for KVM/s390

[Fengguang Wu]

On Fri, Dec 18, 2015 at 01:16:36PM +0100, Cornelia Huck wrote:
> On Fri, 18 Dec 2015 15:51:43 +0800
> Fengguang Wu <fengguang.wu@intel.com> wrote:
> 
> > CC: Christian Borntraeger <borntraeger@de.ibm.com>
> > CC: Cornelia Huck <cornelia.huck@de.ibm.com>
> > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > ---
> >  MAINTAINERS |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.457016903 +0800
> > +++ linux/MAINTAINERS	2015-12-18 15:43:28.457016903 +0800
> > @@ -6117,6 +6117,7 @@ M:	Christian Borntraeger <borntraeger@de
> >  M:	Cornelia Huck <cornelia.huck@de.ibm.com>
> >  L:	linux-s390@vger.kernel.org
> >  W:	http://www.ibm.com/developerworks/linux/linux390/
> > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git
> >  S:	Supported
> >  F:	Documentation/s390/kvm.txt
> >  F:	arch/s390/include/asm/kvm*
> > 
> > 
> 
> Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com>
> 
> I wonder about the context? Your mails don't seem to have made it to
> lkml...

Thanks! The background is, the 0day patch test script relies on the
information in MAINTAINERS to decide which tree it can apply the LKML
patches to. So I need to add some missing git URLs to the MAINTAINERS
file.

Thanks,
Fengguang

Re: [PATCH 20/43] MAINTAINERS: add git URL for KVM/s390

[Cornelia Huck]

On Fri, 18 Dec 2015 20:32:15 +0800
Fengguang Wu <fengguang.wu@intel.com> wrote:

> Thanks! The background is, the 0day patch test script relies on the
> information in MAINTAINERS to decide which tree it can apply the LKML
> patches to. So I need to add some missing git URLs to the MAINTAINERS
> file.

Don't you also need the branch the patches should apply against, or am
I misunderstanding what you're trying to do?

Re: [PATCH 20/43] MAINTAINERS: add git URL for KVM/s390

[Fengguang Wu]

On Fri, Dec 18, 2015 at 01:42:09PM +0100, Cornelia Huck wrote:
> On Fri, 18 Dec 2015 20:32:15 +0800
> Fengguang Wu <fengguang.wu@intel.com> wrote:
> 
> > Thanks! The background is, the 0day patch test script relies on the
> > information in MAINTAINERS to decide which tree it can apply the LKML
> > patches to. So I need to add some missing git URLs to the MAINTAINERS
> > file.
> 
> Don't you also need the branch the patches should apply against, or am
> I misunderstanding what you're trying to do?

Yes one option is to append the branch name in MAINTAINERS like this:

+T:     git git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git next

The other option is to get the branch name from linux-next, whose Next/Trees
file contains this line:

kvms390         git     git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git#next

Then the robot knows patches should be applied to the 'next' branch.

So for most trees, there is no need to specify the branch name in the
MAINTAINERS file. Not only it's not necessary, but also helps avoid
double configuration.

Thanks,
Fengguang

Re: [PATCH 13/43] MAINTAINERS: fix git URL for EDAC

[Borislav Petkov]

On Fri, Dec 18, 2015 at 03:51:36PM +0800, Fengguang Wu wrote:
> CC: Doug Thompson <dougthompson@xmission.com>
> CC: Borislav Petkov <bp@alien8.de>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>  MAINTAINERS |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:27.677016878 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:27.673016878 +0800
> @@ -3945,8 +3945,8 @@ M:	Borislav Petkov <bp@alien8.de>
>  M:	Mauro Carvalho Chehab <mchehab@osg.samsung.com>
>  L:	linux-edac@vger.kernel.org
>  W:	bluesmoke.sourceforge.net
> -T:	git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git#for-next
> -T:	git://git.kernel.org/pub/linux/kernel/git/mchehab/linux-edac.git#linux_next
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git for-next
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac.git linux_next

Huh, "git git://"? That's twice and the protocol git:// already says
what kind of a repo this is.

This is redundant.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

Re: [PATCH 13/43] MAINTAINERS: fix git URL for EDAC

[Fengguang Wu]

On Fri, Dec 18, 2015 at 11:49:08AM +0100, Borislav Petkov wrote:
> On Fri, Dec 18, 2015 at 03:51:36PM +0800, Fengguang Wu wrote:
> > CC: Doug Thompson <dougthompson@xmission.com>
> > CC: Borislav Petkov <bp@alien8.de>
> > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > ---
> >  MAINTAINERS |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > --- linux.orig/MAINTAINERS	2015-12-18 15:43:27.677016878 +0800
> > +++ linux/MAINTAINERS	2015-12-18 15:43:27.673016878 +0800
> > @@ -3945,8 +3945,8 @@ M:	Borislav Petkov <bp@alien8.de>
> >  M:	Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> >  L:	linux-edac@vger.kernel.org
> >  W:	bluesmoke.sourceforge.net
> > -T:	git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git#for-next
> > -T:	git://git.kernel.org/pub/linux/kernel/git/mchehab/linux-edac.git#linux_next
> > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git for-next
> > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac.git linux_next
> 
> Huh, "git git://"? That's twice and the protocol git:// already says
> what kind of a repo this is.
> 
> This is redundant.

That's the required format, documented earlier in MAINTAINERS:

        T: SCM tree type and location.
           Type is one of: git, hg, quilt, stgit, topgit

Which is followed all other places, eg.

T:      git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git
T:      git https://github.com/Xilinx/linux-xlnx.git

Thanks,
Fengguang

Re: [PATCH 13/43] MAINTAINERS: fix git URL for EDAC

[Borislav Petkov]

On Fri, Dec 18, 2015 at 06:57:33PM +0800, Fengguang Wu wrote:
> That's the required format, documented earlier in MAINTAINERS:
> 
>         T: SCM tree type and location.
>            Type is one of: git, hg, quilt, stgit, topgit
> 
> Which is followed all other places, eg.
> 
> T:      git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git
> T:      git https://github.com/Xilinx/linux-xlnx.git

I saw that it is, I'm just saying that "git git://" is redundant.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

Re: [PATCH 13/43] MAINTAINERS: fix git URL for EDAC

[Borislav Petkov]

On Fri, Dec 18, 2015 at 06:57:33PM +0800, Fengguang Wu wrote:
> That's the required format, documented earlier in MAINTAINERS:
> 
>         T: SCM tree type and location.
>            Type is one of: git, hg, quilt, stgit, topgit
> 
> Which is followed all other places, eg.
> 
> T:      git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git
> T:      git https://github.com/Xilinx/linux-xlnx.git

Right, and IMO it should be done something like the diff below.

The T: should contain the URL and if it is not clear from the URL
type, the *type* of the repository should be appended and not some
commands like "hg" or "quilt" or whatever. Especially if those commands
don't necessarily tell you what the repo is unless you know what the
executables' names mean.

---
diff --git a/MAINTAINERS b/MAINTAINERS
index 7271b6189ccf..09a9c76ff83e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -75,8 +75,8 @@ Descriptions of section entries:
 	L: Mailing list that is relevant to this area
 	W: Web-page with status/info
 	Q: Patchwork web based patch tracking system site
-	T: SCM tree type and location.
-	   Type is one of: git, hg, quilt, stgit, topgit
+	T: SCM tree type and location. If it is not clear from the SCM URL, the
+	   SCM type is appended in brackets.
 	S: Status, one of the following:
 	   Supported:	Someone is actually paid to look after this.
 	   Maintained:	Someone actually looks after it.
@@ -1739,7 +1739,7 @@ M:	Michal Simek <michal.simek@xilinx.com>
 R:	Sören Brinkmann <soren.brinkmann@xilinx.com>
 L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
 W:	http://wiki.xilinx.com
-T:	git https://github.com/Xilinx/linux-xlnx.git
+T:	https://github.com/Xilinx/linux-xlnx.git (git)
 S:	Supported
 F:	arch/arm/mach-zynq/
 F:	drivers/cpuidle/cpuidle-zynq.c
@@ -5604,7 +5604,7 @@ M:	Gang Wei <gang.wei@intel.com>
 M:	Shane Wang <shane.wang@intel.com>
 L:	tboot-devel@lists.sourceforge.net
 W:	http://tboot.sourceforge.net
-T:	hg http://tboot.hg.sourceforge.net:8000/hgroot/tboot/tboot
+T:	http://tboot.hg.sourceforge.net:8000/hgroot/tboot/tboot (mercurial)
 S:	Supported
 F:	Documentation/intel_txt.txt
 F:	include/linux/tboot.h

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

Re: [PATCH 13/43] MAINTAINERS: fix git URL for EDAC

[Fengguang Wu]

On Fri, Dec 18, 2015 at 12:41:55PM +0100, Borislav Petkov wrote:
> On Fri, Dec 18, 2015 at 06:57:33PM +0800, Fengguang Wu wrote:
> > That's the required format, documented earlier in MAINTAINERS:
> > 
> >         T: SCM tree type and location.
> >            Type is one of: git, hg, quilt, stgit, topgit
> > 
> > Which is followed all other places, eg.
> > 
> > T:      git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git
> > T:      git https://github.com/Xilinx/linux-xlnx.git
> 
> Right, and IMO it should be done something like the diff below.
> 
> The T: should contain the URL and if it is not clear from the URL
> type, the *type* of the repository should be appended and not some
> commands like "hg" or "quilt" or whatever. Especially if those commands
> don't necessarily tell you what the repo is unless you know what the
> executables' names mean.

That's all right for humans, however there are also in-tree and
out-of-tree scripts that parse the MAINTAINERS file and depend on
the current format.

So isn't it better to change the several lines for EDAC than to
convert hundreds of lines to the new format, plus converting the
related scripts?

Thanks,
Fengguang

> ---
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7271b6189ccf..09a9c76ff83e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -75,8 +75,8 @@ Descriptions of section entries:
>  	L: Mailing list that is relevant to this area
>  	W: Web-page with status/info
>  	Q: Patchwork web based patch tracking system site
> -	T: SCM tree type and location.
> -	   Type is one of: git, hg, quilt, stgit, topgit
> +	T: SCM tree type and location. If it is not clear from the SCM URL, the
> +	   SCM type is appended in brackets.
>  	S: Status, one of the following:
>  	   Supported:	Someone is actually paid to look after this.
>  	   Maintained:	Someone actually looks after it.
> @@ -1739,7 +1739,7 @@ M:	Michal Simek <michal.simek@xilinx.com>
>  R:	Sören Brinkmann <soren.brinkmann@xilinx.com>
>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>  W:	http://wiki.xilinx.com
> -T:	git https://github.com/Xilinx/linux-xlnx.git
> +T:	https://github.com/Xilinx/linux-xlnx.git (git)
>  S:	Supported
>  F:	arch/arm/mach-zynq/
>  F:	drivers/cpuidle/cpuidle-zynq.c
> @@ -5604,7 +5604,7 @@ M:	Gang Wei <gang.wei@intel.com>
>  M:	Shane Wang <shane.wang@intel.com>
>  L:	tboot-devel@lists.sourceforge.net
>  W:	http://tboot.sourceforge.net
> -T:	hg http://tboot.hg.sourceforge.net:8000/hgroot/tboot/tboot
> +T:	http://tboot.hg.sourceforge.net:8000/hgroot/tboot/tboot (mercurial)
>  S:	Supported
>  F:	Documentation/intel_txt.txt
>  F:	include/linux/tboot.h
> 
> -- 
> Regards/Gruss,
>     Boris.
> 
> ECO tip #101: Trim your mails when you reply.

Re: [PATCH 13/43] MAINTAINERS: fix git URL for EDAC

[Borislav Petkov]

On Fri, Dec 18, 2015 at 07:55:39PM +0800, Fengguang Wu wrote:
> That's all right for humans, however there are also in-tree and
> out-of-tree scripts that parse the MAINTAINERS file and depend on
> the current format.

Hmm, it doesn't say that in the T: description - I thought it was for
humans only. Besides, scripts can parse stuff in brackets too.

> So isn't it better to change the several lines for EDAC than to
> convert hundreds of lines to the new format, plus converting the
> related scripts?

Not better - easier. I'm pointing out that this scheme is suboptimal.
Someone might read it and decide to fix it, who knows. :)

Thanks.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

Re: [PATCH 13/43] MAINTAINERS: fix git URL for EDAC

[Borislav Petkov]

On Fri, Dec 18, 2015 at 03:51:36PM +0800, Fengguang Wu wrote:
> CC: Doug Thompson <dougthompson@xmission.com>
> CC: Borislav Petkov <bp@alien8.de>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>  MAINTAINERS |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:27.677016878 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:27.673016878 +0800
> @@ -3945,8 +3945,8 @@ M:	Borislav Petkov <bp@alien8.de>
>  M:	Mauro Carvalho Chehab <mchehab@osg.samsung.com>
>  L:	linux-edac@vger.kernel.org
>  W:	bluesmoke.sourceforge.net
> -T:	git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git#for-next
> -T:	git://git.kernel.org/pub/linux/kernel/git/mchehab/linux-edac.git#linux_next
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git for-next
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac.git linux_next
>  S:	Supported
>  F:	Documentation/edac.txt
>  F:	drivers/edac/

Applied, thanks.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

Re: [PATCH 25/43] MAINTAINERS: add git URL for wireless drivers

[Kalle Valo]

Fengguang Wu <fengguang.wu@intel.com> writes:

> Also fixup the tailing /
>
> CC: Kalle Valo <kvalo@codeaurora.org>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Acked-by: Kalle Valo <kvalo@codeaurora.org>

-- 
Kalle Valo

Re: [PATCH 38/43] MAINTAINERS: add git URL for tile

[Chris Metcalf]

On 12/18/2015 02:52 AM, Fengguang Wu wrote:
> CC: Chris Metcalf<cmetcalf@ezchip.com>
> Signed-off-by: Fengguang Wu<fengguang.wu@intel.com>
> ---
>   MAINTAINERS |    1 +
>   1 file changed, 1 insertion(+)

Thanks!  Taken into the tile tree.

-- 
Chris Metcalf, EZChip Semiconductor
http://www.ezchip.com

Re: [PATCH 18/43] MAINTAINERS: add kdbus

[Greg Kroah-Hartman]

On Fri, Dec 18, 2015 at 03:51:41PM +0800, Fengguang Wu wrote:
> CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>  MAINTAINERS |   10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> @@ -6005,6 +6005,16 @@ S:	Maintained
>  F:	Documentation/kbuild/kconfig-language.txt
>  F:	scripts/kconfig/
>  
> +KDBUS
> +M:	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git kdbus
> +S:	Supported
> +F:	ipc/kdbus/
> +F:	Documentation/kdbus/
> +F:	samples/kdbus/
> +F:	tools/testing/selftests/kdbus/
> +F:	include/uapi/linux/kdbus.h

What is this patch for?  I didn't send this or create this entry, why
did you?

confused,

greg k-h

Re: [PATCH 18/43] MAINTAINERS: add kdbus

[Fengguang Wu]

On Fri, Dec 18, 2015 at 07:33:36AM -0800, Greg KH wrote:
> On Fri, Dec 18, 2015 at 03:51:41PM +0800, Fengguang Wu wrote:
> > CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > ---
> >  MAINTAINERS |   10 ++++++++++
> >  1 file changed, 10 insertions(+)
> > 
> > --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> > +++ linux/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> > @@ -6005,6 +6005,16 @@ S:	Maintained
> >  F:	Documentation/kbuild/kconfig-language.txt
> >  F:	scripts/kconfig/
> >  
> > +KDBUS
> > +M:	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git kdbus
> > +S:	Supported
> > +F:	ipc/kdbus/
> > +F:	Documentation/kdbus/
> > +F:	samples/kdbus/
> > +F:	tools/testing/selftests/kdbus/
> > +F:	include/uapi/linux/kdbus.h
> 
> What is this patch for?  I didn't send this or create this entry, why
> did you?

Greg, that is necessary for the 0day email testing feature. After
adding the above information, when someone posted patch modifying the
listed kdbus files, the robot will know to apply the patch to your
char-misc/kdbus branch for testing.

Thanks,
Fengguang

Re: [PATCH 18/43] MAINTAINERS: add kdbus

[Greg Kroah-Hartman]

On Sat, Dec 19, 2015 at 10:10:24AM +0800, Fengguang Wu wrote:
> On Fri, Dec 18, 2015 at 07:33:36AM -0800, Greg KH wrote:
> > On Fri, Dec 18, 2015 at 03:51:41PM +0800, Fengguang Wu wrote:
> > > CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > > ---
> > >  MAINTAINERS |   10 ++++++++++
> > >  1 file changed, 10 insertions(+)
> > > 
> > > --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> > > +++ linux/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> > > @@ -6005,6 +6005,16 @@ S:	Maintained
> > >  F:	Documentation/kbuild/kconfig-language.txt
> > >  F:	scripts/kconfig/
> > >  
> > > +KDBUS
> > > +M:	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git kdbus
> > > +S:	Supported
> > > +F:	ipc/kdbus/
> > > +F:	Documentation/kdbus/
> > > +F:	samples/kdbus/
> > > +F:	tools/testing/selftests/kdbus/
> > > +F:	include/uapi/linux/kdbus.h
> > 
> > What is this patch for?  I didn't send this or create this entry, why
> > did you?
> 
> Greg, that is necessary for the 0day email testing feature. After
> adding the above information, when someone posted patch modifying the
> listed kdbus files, the robot will know to apply the patch to your
> char-misc/kdbus branch for testing.

There is no char-misc/kdbus anymore, please do a 'git pull --prune' to
delete branches that get removed.  And even if there was, that old
branch did have a MAINTAINERS entry in it...

thanks,

greg k-h

Re: [PATCH 18/43] MAINTAINERS: add kdbus

[Fengguang Wu]

On Sat, Dec 19, 2015 at 04:23:51PM -0800, Greg KH wrote:
> On Sat, Dec 19, 2015 at 10:10:24AM +0800, Fengguang Wu wrote:
> > On Fri, Dec 18, 2015 at 07:33:36AM -0800, Greg KH wrote:
> > > On Fri, Dec 18, 2015 at 03:51:41PM +0800, Fengguang Wu wrote:
> > > > CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > > > ---
> > > >  MAINTAINERS |   10 ++++++++++
> > > >  1 file changed, 10 insertions(+)
> > > > 
> > > > --- linux.orig/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> > > > +++ linux/MAINTAINERS	2015-12-18 15:43:28.229016896 +0800
> > > > @@ -6005,6 +6005,16 @@ S:	Maintained
> > > >  F:	Documentation/kbuild/kconfig-language.txt
> > > >  F:	scripts/kconfig/
> > > >  
> > > > +KDBUS
> > > > +M:	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > > +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git kdbus
> > > > +S:	Supported
> > > > +F:	ipc/kdbus/
> > > > +F:	Documentation/kdbus/
> > > > +F:	samples/kdbus/
> > > > +F:	tools/testing/selftests/kdbus/
> > > > +F:	include/uapi/linux/kdbus.h
> > > 
> > > What is this patch for?  I didn't send this or create this entry, why
> > > did you?
> > 
> > Greg, that is necessary for the 0day email testing feature. After
> > adding the above information, when someone posted patch modifying the
> > listed kdbus files, the robot will know to apply the patch to your
> > char-misc/kdbus branch for testing.
> 
> There is no char-misc/kdbus anymore, please do a 'git pull --prune' to
> delete branches that get removed.  And even if there was, that old
> branch did have a MAINTAINERS entry in it...

Ah got it. Sorry, then this patch should be dropped.

Regards,
Fengguang

Re: [PATCH 27/43] MAINTAINERS: update git URL for PCMCIA

[Dominik Brodowski]

[-- Attachment #1: Type: text/plain, Size: 909 bytes --]

On Fri, Dec 18, 2015 at 03:51:50PM +0800, Fengguang Wu wrote:
> CC: linux-pcmcia@lists.infradead.org
> CC: Dominik Brodowski <linux@dominikbrodowski.net>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Acked-by: Dominik Brodowski <linux@dominikbrodowski.net>

Thanks!
	Dominik

> ---
>  MAINTAINERS |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux.orig/MAINTAINERS	2015-12-18 15:43:29.233016928 +0800
> +++ linux/MAINTAINERS	2015-12-18 15:43:29.229016928 +0800
> @@ -8285,7 +8285,7 @@ PCMCIA SUBSYSTEM
>  P:	Linux PCMCIA Team
>  L:	linux-pcmcia@lists.infradead.org
>  W:	http://lists.infradead.org/mailman/listinfo/linux-pcmcia
> -T:	git git://git.kernel.org/pub/scm/linux/kernel/git/brodo/pcmcia-2.6.git
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/brodo/pcmcia.git
>  S:	Maintained
>  F:	Documentation/pcmcia/
>  F:	drivers/pcmcia/
> 
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]