linux-kernel - Re: [PATCH] MAINTAINERS: Update path to tcb

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <8aba2175-f61c-e624-aee0-e724e1ac04da@microchip.com>
Date:   Fri, 23 Aug 2019 08:20:58 +0000
From:   <Nicolas.Ferre@...rochip.com>
To:     <alexandre.belloni@...tlin.com>
CC:     <efremov@...ux.com>, <linux-kernel@...r.kernel.org>,
        <joe@...ches.com>, <linux-arm-kernel@...ts.infradead.org>,
        <Ludovic.Desroches@...rochip.com>
Subject: Re: [PATCH] MAINTAINERS: Update path to tcb_clksrc.c

On 14/08/2019 at 11:08, Alexandre Belloni wrote:
> External E-Mail
> 
> 
> On 13/08/2019 08:11:23+0000, Nicolas Ferre wrote:
>> On 13/08/2019 at 08:10, Denis Efremov wrote:
>>> Update MAINTAINERS record to reflect the filename change
>>> from tcb_clksrc.c to timer-atmel-tcb.c
>>>
>>> Cc: Nicolas Ferre <nicolas.ferre@...rochip.com>
>>
>> Acked-by: Nicolas Ferre <nicolas.ferre@...rochip.com>
>> But, while you're at it, I would add another line: see below...
>>
>>> Cc: Alexandre Belloni <alexandre.belloni@...tlin.com>
>>> Cc: linux-arm-kernel@...ts.infradead.org
>>> Fixes: a7aae768166e ("clocksource/drivers/tcb_clksrc: Rename the file for consistency")
>>> Signed-off-by: Denis Efremov <efremov@...ux.com>
>>> ---
>>>    MAINTAINERS | 2 +-
>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index c9ad38a9414f..3ec8154e4630 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -10637,7 +10637,7 @@ M:	Nicolas Ferre <nicolas.ferre@...rochip.com>
>>
>> +M:      Alexandre Belloni <alexandre.belloni@...tlin.com>
>>
>> But Alexandre have to agree, of course.
>>
>>>    L:	linux-arm-kernel@...ts.infradead.org (moderated for non-subscribers)
>>>    S:	Supported
>>>    F:	drivers/misc/atmel_tclib.c
>>> -F:	drivers/clocksource/tcb_clksrc.c
>>> +F:	drivers/clocksource/timer-atmel-tcb.c
>>>    
>>>    MICROCHIP USBA UDC DRIVER
>>>    M:	Cristian Birsan <cristian.birsan@...rochip.com>
>>
>> We could also remove this entry and mix it with:
>> "ARM/Microchip (AT91) SoC support"
>>
>> But I prefer to keep it separated like this for various reasons.
>>
> 
> I would simply remove this entry because all the files are already
> matching the SoC entry (it has N: atmel) and atmel_tclib will go away (I
> have a series to do that).

All right: let's remove it.

Thanks Denis for the heads-up!

Best regards,
   Nicolas

> If you want to keep a separate entry, maybe we should then add the
> system timer and pit drivers.
> 
> 


-- 
Nicolas Ferre