diff mbox

MAINTAINERS: Update Zynq maintainer record

Message ID 1413927103-31358-1-git-send-email-soren.brinkmann@xilinx.com (mailing list archive)
State New, archived
Headers show

Commit Message

Soren Brinkmann Oct. 21, 2014, 9:31 p.m. UTC
- 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(+)

Comments

Michal Simek Oct. 22, 2014, 6:29 a.m. UTC | #1
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@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
Joe Perches Oct. 22, 2014, 6:54 a.m. UTC | #2
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@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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
Michal Simek Oct. 22, 2014, 7:36 a.m. UTC | #3
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@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
diff mbox

Patch

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@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