2020-04-18 10:12:18

by Lukas Bulwahn

[permalink] [raw]
Subject: [PATCH RESEND] MAINTAINERS: adjust to renaming physmap_of_versatile.c

Commit 6ca15cfa0788 ("mtd: maps: Rename physmap_of_{versatile, gemini}
into physmap-{versatile, gemini}") renamed physmap_of_versatile.c to
physmap-versatile.c, but did not adjust the MAINTAINERS entry.

Since then, ./scripts/get_maintainer.pl --self-test complains:

warning: no file matches F: drivers/mtd/maps/physmap_of_versatile.c

Rectify the ARM INTEGRATOR, VERSATILE AND REALVIEW SUPPORT entry and now
also cover drivers/mtd/maps/physmap-versatile.h while at it.

Co-developed-by: Sebastian Duda <[email protected]>
Signed-off-by: Sebastian Duda <[email protected]>
Signed-off-by: Lukas Bulwahn <[email protected]>
---
Boris, please pick or ack this patch.
applies cleanly on current master and next-20200417

v1: https://lore.kernel.org/lkml/[email protected]/
- was wrongly sent to old [email protected] address
- got Reviewed-by: Linus Walleij <[email protected]>

MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index c77f02282044..52bb8819230a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1342,7 +1342,7 @@ F: arch/arm/plat-versatile/
F: drivers/clk/versatile/
F: drivers/i2c/busses/i2c-versatile.c
F: drivers/irqchip/irq-versatile-fpga.c
-F: drivers/mtd/maps/physmap_of_versatile.c
+F: drivers/mtd/maps/physmap-versatile.*
F: drivers/power/reset/arm-versatile-reboot.c
F: drivers/soc/versatile/

--
2.17.1


2020-04-18 12:13:02

by Boris Brezillon

[permalink] [raw]
Subject: Re: [PATCH RESEND] MAINTAINERS: adjust to renaming physmap_of_versatile.c

+ MTD maintainers and the MTD ML

On Sat, 18 Apr 2020 12:09:33 +0200
Lukas Bulwahn <[email protected]> wrote:

> Commit 6ca15cfa0788 ("mtd: maps: Rename physmap_of_{versatile, gemini}
> into physmap-{versatile, gemini}") renamed physmap_of_versatile.c to
> physmap-versatile.c, but did not adjust the MAINTAINERS entry.
>
> Since then, ./scripts/get_maintainer.pl --self-test complains:
>
> warning: no file matches F: drivers/mtd/maps/physmap_of_versatile.c
>
> Rectify the ARM INTEGRATOR, VERSATILE AND REALVIEW SUPPORT entry and now
> also cover drivers/mtd/maps/physmap-versatile.h while at it.
>
> Co-developed-by: Sebastian Duda <[email protected]>
> Signed-off-by: Sebastian Duda <[email protected]>
> Signed-off-by: Lukas Bulwahn <[email protected]>
> ---
> Boris, please pick or ack this patch.
> applies cleanly on current master and next-20200417

Reviewed-by: Boris Brezillon <[email protected]>

>
> v1: https://lore.kernel.org/lkml/[email protected]/
> - was wrongly sent to old [email protected] address

Please Cc the MTD ML and maintainers next time.

> - got Reviewed-by: Linus Walleij <[email protected]>

Linus' reviewed is missing in this version.

>
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c77f02282044..52bb8819230a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1342,7 +1342,7 @@ F: arch/arm/plat-versatile/
> F: drivers/clk/versatile/
> F: drivers/i2c/busses/i2c-versatile.c
> F: drivers/irqchip/irq-versatile-fpga.c
> -F: drivers/mtd/maps/physmap_of_versatile.c
> +F: drivers/mtd/maps/physmap-versatile.*
> F: drivers/power/reset/arm-versatile-reboot.c
> F: drivers/soc/versatile/
>

2020-04-20 06:40:45

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH RESEND] MAINTAINERS: adjust to renaming physmap_of_versatile.c

On Sat, Apr 18, 2020 at 12:10 PM Lukas Bulwahn <[email protected]> wrote:

> Commit 6ca15cfa0788 ("mtd: maps: Rename physmap_of_{versatile, gemini}
> into physmap-{versatile, gemini}") renamed physmap_of_versatile.c to
> physmap-versatile.c, but did not adjust the MAINTAINERS entry.
>
> Since then, ./scripts/get_maintainer.pl --self-test complains:
>
> warning: no file matches F: drivers/mtd/maps/physmap_of_versatile.c
>
> Rectify the ARM INTEGRATOR, VERSATILE AND REALVIEW SUPPORT entry and now
> also cover drivers/mtd/maps/physmap-versatile.h while at it.
>
> Co-developed-by: Sebastian Duda <[email protected]>
> Signed-off-by: Sebastian Duda <[email protected]>
> Signed-off-by: Lukas Bulwahn <[email protected]>
> ---
> Boris, please pick or ack this patch.
> applies cleanly on current master and next-20200417

Patch applied to my Versatile tree.

Will send a pull request for this to ARM SoC soon.

Yours,
Linus Walleij