linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: Update Zynq maintainer record
@ 2014-10-21 21:31 Soren Brinkmann
  2014-10-22  6:29 ` Michal Simek
  0 siblings, 1 reply; 4+ messages in thread
From: Soren Brinkmann @ 2014-10-21 21:31 UTC (permalink / raw)
  To: linux-arm-kernel

 - add myself as reviewer for Zynq
 - add entry to cover Zynq clock drivers
 - add entry to cover Xilinx DMA drivers

Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
---
 MAINTAINERS | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index a20df9bf8ab0..07eecdddbe8b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1543,6 +1543,7 @@ F:	arch/arm/mach-pxa/include/mach/z2.h
 
 ARM/ZYNQ ARCHITECTURE
 M:	Michal Simek <michal.simek@xilinx.com>
+R:	S?ren Brinkmann <soren.brinkmann@xilinx.com>
 L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
 W:	http://wiki.xilinx.com
 T:	git git://git.xilinx.com/linux-xlnx.git
@@ -1552,7 +1553,9 @@ F:	drivers/cpuidle/cpuidle-zynq.c
 F:	drivers/block/xsysace.c
 N:	zynq
 N:	xilinx
+F:	drivers/clk/zynq/
 F:	drivers/clocksource/cadence_ttc_timer.c
+F:	drivers/dma/xilinx/
 F:	drivers/i2c/busses/i2c-cadence.c
 F:	drivers/mmc/host/sdhci-of-arasan.c
 
-- 
2.1.2.1.g5e69ed6

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH] MAINTAINERS: Update Zynq maintainer record
  2014-10-21 21:31 [PATCH] MAINTAINERS: Update Zynq maintainer record Soren Brinkmann
@ 2014-10-22  6:29 ` Michal Simek
  2014-10-22  6:54   ` Joe Perches
  0 siblings, 1 reply; 4+ messages in thread
From: Michal Simek @ 2014-10-22  6:29 UTC (permalink / raw)
  To: linux-arm-kernel

On 10/21/2014 11:31 PM, Soren Brinkmann wrote:
>  - add myself as reviewer for Zynq
>  - add entry to cover Zynq clock drivers
>  - add entry to cover Xilinx DMA drivers
> 
> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
> ---
>  MAINTAINERS | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a20df9bf8ab0..07eecdddbe8b 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1543,6 +1543,7 @@ F:	arch/arm/mach-pxa/include/mach/z2.h
>  
>  ARM/ZYNQ ARCHITECTURE
>  M:	Michal Simek <michal.simek@xilinx.com>
> +R:	S?ren Brinkmann <soren.brinkmann@xilinx.com>

this is fine.

>  L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
>  W:	http://wiki.xilinx.com
>  T:	git git://git.xilinx.com/linux-xlnx.git
> @@ -1552,7 +1553,9 @@ F:	drivers/cpuidle/cpuidle-zynq.c
>  F:	drivers/block/xsysace.c
>  N:	zynq
>  N:	xilinx
> +F:	drivers/clk/zynq/

this is handled by N: zynq

>  F:	drivers/clocksource/cadence_ttc_timer.c
> +F:	drivers/dma/xilinx/

ditto.

>  F:	drivers/i2c/busses/i2c-cadence.c
>  F:	drivers/mmc/host/sdhci-of-arasan.c
>  

Thanks,
Michal

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] MAINTAINERS: Update Zynq maintainer record
  2014-10-22  6:29 ` Michal Simek
@ 2014-10-22  6:54   ` Joe Perches
  2014-10-22  7:36     ` Michal Simek
  0 siblings, 1 reply; 4+ messages in thread
From: Joe Perches @ 2014-10-22  6:54 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, 2014-10-22 at 08:29 +0200, Michal Simek wrote:
> On 10/21/2014 11:31 PM, Soren Brinkmann wrote:
> >  - add myself as reviewer for Zynq
> >  - add entry to cover Zynq clock drivers
> >  - add entry to cover Xilinx DMA drivers
> > 
> > Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
> > ---
> >  MAINTAINERS | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index a20df9bf8ab0..07eecdddbe8b 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1543,6 +1543,7 @@ F:	arch/arm/mach-pxa/include/mach/z2.h
> >  
> >  ARM/ZYNQ ARCHITECTURE
> >  M:	Michal Simek <michal.simek@xilinx.com>
> > +R:	S?ren Brinkmann <soren.brinkmann@xilinx.com>
> 
> this is fine.
> 
> >  L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
> >  W:	http://wiki.xilinx.com
> >  T:	git git://git.xilinx.com/linux-xlnx.git
> > @@ -1552,7 +1553,9 @@ F:	drivers/cpuidle/cpuidle-zynq.c
> >  F:	drivers/block/xsysace.c
> >  N:	zynq
> >  N:	xilinx
> > +F:	drivers/clk/zynq/
> 
> this is handled by N: zynq
> 
> >  F:	drivers/clocksource/cadence_ttc_timer.c
> > +F:	drivers/dma/xilinx/
> 
> ditto.

F: specific paths are better than N: file names
because get_maintainers doesn't add the patches
submitters for N: entries.

> >  F:	drivers/i2c/busses/i2c-cadence.c
> >  F:	drivers/mmc/host/sdhci-of-arasan.c
> >  
> 
> Thanks,
> Michal
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] MAINTAINERS: Update Zynq maintainer record
  2014-10-22  6:54   ` Joe Perches
@ 2014-10-22  7:36     ` Michal Simek
  0 siblings, 0 replies; 4+ messages in thread
From: Michal Simek @ 2014-10-22  7:36 UTC (permalink / raw)
  To: linux-arm-kernel

On 10/22/2014 08:54 AM, Joe Perches wrote:
> On Wed, 2014-10-22 at 08:29 +0200, Michal Simek wrote:
>> On 10/21/2014 11:31 PM, Soren Brinkmann wrote:
>>>  - add myself as reviewer for Zynq
>>>  - add entry to cover Zynq clock drivers
>>>  - add entry to cover Xilinx DMA drivers
>>>
>>> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>
>>> ---
>>>  MAINTAINERS | 3 +++
>>>  1 file changed, 3 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index a20df9bf8ab0..07eecdddbe8b 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -1543,6 +1543,7 @@ F:	arch/arm/mach-pxa/include/mach/z2.h
>>>  
>>>  ARM/ZYNQ ARCHITECTURE
>>>  M:	Michal Simek <michal.simek@xilinx.com>
>>> +R:	S?ren Brinkmann <soren.brinkmann@xilinx.com>
>>
>> this is fine.
>>
>>>  L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
>>>  W:	http://wiki.xilinx.com
>>>  T:	git git://git.xilinx.com/linux-xlnx.git
>>> @@ -1552,7 +1553,9 @@ F:	drivers/cpuidle/cpuidle-zynq.c
>>>  F:	drivers/block/xsysace.c
>>>  N:	zynq
>>>  N:	xilinx
>>> +F:	drivers/clk/zynq/
>>
>> this is handled by N: zynq
>>
>>>  F:	drivers/clocksource/cadence_ttc_timer.c
>>> +F:	drivers/dma/xilinx/
>>
>> ditto.
> 
> F: specific paths are better than N: file names
> because get_maintainers doesn't add the patches
> submitters for N: entries.

I expect that you are referring to this description

        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.
           scripts/get_maintainer.pl has different behavior for files that
           match F: pattern and matches of N: patterns.  By default,
           get_maintainer will not look at git log history when an F: pattern
           match occurs.  When an N: match occurs, git log history is used
           to also notify the people that have git commit signatures.

The purpose of this patch was that Soren is aware about changes
around xilinx drivers not to sign dma drivers to me + CC him.

Also this fragment informing me that something is happening around these drivers
and also driver submitter is informed that's why using git log history is
at least for me preferred way.

Thanks,
Michal

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-10-22  7:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-21 21:31 [PATCH] MAINTAINERS: Update Zynq maintainer record Soren Brinkmann
2014-10-22  6:29 ` Michal Simek
2014-10-22  6:54   ` Joe Perches
2014-10-22  7:36     ` Michal Simek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).