linux-kernel - Re: [PATCH] MAINTAINERS: Update Zynq maintainer record

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <7f1a322b4d7b481ea8b316736fe6b24a@BL2FFO11FD025.protection.gbl>
Date:	Wed, 22 Oct 2014 09:36:15 +0200
From:	Michal Simek <michal.simek@...inx.com>
To:	Joe Perches <joe@...ches.com>,
	Michal Simek <michal.simek@...inx.com>
CC:	Soren Brinkmann <soren.brinkmann@...inx.com>,
	<linux-kernel@...r.kernel.org>,
	<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] MAINTAINERS: Update Zynq maintainer record

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