Re: [PATCH] MAINTAINERS: Update path to tcb_clksrc.c

Nicolas.Ferre
2019-08-13T08:11:30Z

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@microchip.com>

Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
But, while you're at it, I would add another line: see below...

> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: linux-arm-kernel@lists.infradead.org
> Fixes: a7aae768166e ("clocksource/drivers/tcb_clksrc: Rename the file for consistency")
> Signed-off-by: Denis Efremov <efremov@linux.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@microchip.com>

+M: Alexandre Belloni <alexandre.belloni@bootlin.com>

But Alexandre have to agree, of course.

> L: linux-arm-kernel@lists.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@microchip.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.

Best regards,
--
Nicolas Ferre