Re: [PATCH] MAINTAINERS: Update Zynq maintainer record

From: Michal Simek
Date: Wed Oct 22 2014 - 03:36:29 EST


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@xxxxxxxxxx>
>>> ---
>>> 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@xxxxxxxxxx>
>>> +R: Sören Brinkmann <soren.brinkmann@xxxxxxxxxx>
>>
>> this is fine.
>>
>>> L: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/