2020-06-28 18:06:23

by Lukas Bulwahn

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: remove obsolete entry after file renaming

Commit f16861b12fa0 ("regulator: rename da903x to da903x-regulator") missed
to adjust the DIALOG SEMICONDUCTOR DRIVERS section in MAINTAINERS.

Hence, ./scripts/get_maintainer.pl --self-test=patterns complains:

warning: no file matches F: drivers/regulator/da903x.c

The da903x-regulator.c file is already covered by the pattern
drivers/regulator/da9???-regulator.[ch] in the section.

So, simply remove the non-matching file entry in MAINTAINERS.

Signed-off-by: Lukas Bulwahn <[email protected]>
---
applies cleanly on next-20200626

Seth, please ack.
Mark, please pick this minor non-urgent patch into your -next tree.

MAINTAINERS | 1 -
1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 04fceaee5200..970136e262c2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5021,7 +5021,6 @@ F: drivers/mfd/da91??-*.c
F: drivers/pinctrl/pinctrl-da90??.c
F: drivers/power/supply/da9052-battery.c
F: drivers/power/supply/da91??-*.c
-F: drivers/regulator/da903x.c
F: drivers/regulator/da9???-regulator.[ch]
F: drivers/regulator/slg51000-regulator.[ch]
F: drivers/rtc/rtc-da90??.c
--
2.17.1


2020-06-29 22:01:52

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: remove obsolete entry after file renaming

On Sun, 28 Jun 2020 20:02:29 +0200, Lukas Bulwahn wrote:
> Commit f16861b12fa0 ("regulator: rename da903x to da903x-regulator") missed
> to adjust the DIALOG SEMICONDUCTOR DRIVERS section in MAINTAINERS.
>
> Hence, ./scripts/get_maintainer.pl --self-test=patterns complains:
>
> warning: no file matches F: drivers/regulator/da903x.c
>
> [...]

Applied to

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next

Thanks!

[1/1] MAINTAINERS: remove obsolete entry after file renaming
commit: 412847fb47336c58885ac6231a5f34ac7ac862a0

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

2020-06-29 22:44:09

by Seth Forshee

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: remove obsolete entry after file renaming

On Sun, Jun 28, 2020 at 08:02:29PM +0200, Lukas Bulwahn wrote:
> Commit f16861b12fa0 ("regulator: rename da903x to da903x-regulator") missed
> to adjust the DIALOG SEMICONDUCTOR DRIVERS section in MAINTAINERS.
>
> Hence, ./scripts/get_maintainer.pl --self-test=patterns complains:
>
> warning: no file matches F: drivers/regulator/da903x.c
>
> The da903x-regulator.c file is already covered by the pattern
> drivers/regulator/da9???-regulator.[ch] in the section.
>
> So, simply remove the non-matching file entry in MAINTAINERS.
>
> Signed-off-by: Lukas Bulwahn <[email protected]>

I didn't think to check MAINTAINERS when renaming the file. This makes
sense to me.

Acked-by: Seth Forshee <[email protected]>

> ---
> applies cleanly on next-20200626
>
> Seth, please ack.
> Mark, please pick this minor non-urgent patch into your -next tree.
>
> MAINTAINERS | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 04fceaee5200..970136e262c2 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5021,7 +5021,6 @@ F: drivers/mfd/da91??-*.c
> F: drivers/pinctrl/pinctrl-da90??.c
> F: drivers/power/supply/da9052-battery.c
> F: drivers/power/supply/da91??-*.c
> -F: drivers/regulator/da903x.c
> F: drivers/regulator/da9???-regulator.[ch]
> F: drivers/regulator/slg51000-regulator.[ch]
> F: drivers/rtc/rtc-da90??.c
> --
> 2.17.1
>