2018-09-28 21:57:53

by Joe Perches

[permalink] [raw]
Subject: Bad MAINTAINERS pattern in section 'MICROCHIP ISI DRIVER'

Please fix this defect appropriately.

linux-next MAINTAINERS section:

9550 MICROCHIP ISI DRIVER
9551 M: Eugen Hristev <[email protected]>
9552 L: [email protected]
9553 S: Supported
9554 F: drivers/media/platform/atmel/atmel-isi.c
--> 9555 F: include/media/atmel-isi.h

Commit that introduced this:

commit 92de0f8845adcd55f37f581ddc6c09f1127e217a
Author: Nicolas Ferre <[email protected]>
Date: Wed Aug 29 16:31:46 2018 +0200

MAINTAINERS: move former ATMEL entries to proper MICROCHIP location

Standardize the Microchip / Atmel entries with the same form and move them
so that they are all located at the same place, under the newer MICROCHIP
banner.
Only modifications to the titles of the entries are done in this patch.

Signed-off-by: Nicolas Ferre <[email protected]>
Signed-off-by: Alexandre Belloni <[email protected]>

MAINTAINERS | 154 ++++++++++++++++++++++++++++++------------------------------
1 file changed, 77 insertions(+), 77 deletions(-)

Last commit with include/media/atmel-isi.h

commit 40a78f36fc92bb156872468fb829984a9d946df7
Author: Laurent Pinchart <[email protected]>
Date: Sat Aug 1 06:22:54 2015 -0300

[media] v4l: atmel-isi: Remove support for platform data

All in-tree users have migrated to DT, remove support for platform data.

Signed-off-by: Laurent Pinchart <[email protected]>
[[email protected]: squash the commit to remove the unused variable: dev]
Signed-off-by: Josh Wu <[email protected]>
Signed-off-by: Guennadi Liakhovetski <[email protected]>

Signed-off-by: Mauro Carvalho Chehab <[email protected]>

drivers/media/platform/soc_camera/atmel-isi.c | 24 ++++++----------------
.../media/platform/soc_camera}/atmel-isi.h | 0
2 files changed, 6 insertions(+), 18 deletions(-)


2018-09-30 06:57:48

by Laurent Pinchart

[permalink] [raw]
Subject: Re: Bad MAINTAINERS pattern in section 'MICROCHIP ISI DRIVER'

Hi Joe,

Thank you for the report.

On Saturday, 29 September 2018 00:57:21 EEST Joe Perches wrote:
> Please fix this defect appropriately.

I've just submitted a patch named "MAINTAINERS: Remove stale file entry for
the Atmel ISI driver".

> linux-next MAINTAINERS section:
>
> 9550 MICROCHIP ISI DRIVER
> 9551 M: Eugen Hristev <[email protected]>
> 9552 L: [email protected]
> 9553 S: Supported
> 9554 F: drivers/media/platform/atmel/atmel-isi.c
> --> 9555 F: include/media/atmel-isi.h
>
> Commit that introduced this:
>
> commit 92de0f8845adcd55f37f581ddc6c09f1127e217a
> Author: Nicolas Ferre <[email protected]>
> Date: Wed Aug 29 16:31:46 2018 +0200
>
> MAINTAINERS: move former ATMEL entries to proper MICROCHIP location
>
> Standardize the Microchip / Atmel entries with the same form and move
> them so that they are all located at the same place, under the newer
> MICROCHIP banner.
> Only modifications to the titles of the entries are done in this patch.
>
> Signed-off-by: Nicolas Ferre <[email protected]>
> Signed-off-by: Alexandre Belloni <[email protected]>
>
> MAINTAINERS | 154 ++++++++++++++++++++++++++++----------------------------
> 1 file changed, 77 insertions(+), 77 deletions(-)
>
> Last commit with include/media/atmel-isi.h
>
> commit 40a78f36fc92bb156872468fb829984a9d946df7
> Author: Laurent Pinchart <[email protected]>
> Date: Sat Aug 1 06:22:54 2015 -0300
>
> [media] v4l: atmel-isi: Remove support for platform data
>
> All in-tree users have migrated to DT, remove support for platform data.
>
> Signed-off-by: Laurent Pinchart <[email protected]>
> [[email protected]: squash the commit to remove the unused variable:
> dev] Signed-off-by: Josh Wu <[email protected]>
> Signed-off-by: Guennadi Liakhovetski <[email protected]>
>
> Signed-off-by: Mauro Carvalho Chehab <[email protected]>
>
> drivers/media/platform/soc_camera/atmel-isi.c | 24 +++++--------------
> .../media/platform/soc_camera}/atmel-isi.h | 0
> 2 files changed, 6 insertions(+), 18 deletions(-)

--
Regards,

Laurent Pinchart