bug at scripts/get_maintainer.pl with ARM KOMEDA DRM-KMS DRIVER

bug at scripts/get_maintainer.pl with ARM KOMEDA DRM-KMS DRIVER

[Mauro Carvalho Chehab]

Hi Joe,

While submitting today a patch series, I noticed that the
./scripts/get_maintainer.pl script is not handling well the
ARM KOMEDA DRM-KMS DRIVER open list entry at MAINTAINERS.

I'm using v5.13-rc1 here.

See, it gets an empty e-mail for the ML:

	$ git show a1c3be890440 --pretty=email|./scripts/get_maintainer.pl 
	"James (Qian) Wang" <james.qian.wang@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
	Liviu Dudau <liviu.dudau@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
	Mihail Atanassov <mihail.atanassov@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
	Brian Starkey <brian.starkey@arm.com> (supporter:ARM MALI-DP DRM DRIVER)
	David Airlie <airlied@linux.ie> (maintainer:DRM DRIVERS)
	Daniel Vetter <daniel@ffwll.ch> (maintainer:DRM DRIVERS)
	 (open list:ARM KOMEDA DRM-KMS DRIVER)
	dri-devel@lists.freedesktop.org (open list:DRM DRIVERS)
	linux-kernel@vger.kernel.org (open list)

The entry at MAINTAINERS apparently looks OK:

	ARM KOMEDA DRM-KMS DRIVER
	M:      James (Qian) Wang <james.qian.wang@arm.com>
	M:      Liviu Dudau <liviu.dudau@arm.com>
	M:      Mihail Atanassov <mihail.atanassov@arm.com>
	L:      Mali DP Maintainers <malidp@foss.arm.com>
	S:      Supported
	T:      git git://anongit.freedesktop.org/drm/drm-misc
	F:      Documentation/devicetree/bindings/display/arm,komeda.txt
	F:      Documentation/gpu/komeda-kms.rst
	F:      drivers/gpu/drm/arm/display/include/
	F:      drivers/gpu/drm/arm/display/komeda/

Thanks,
Mauro

Re: bug at scripts/get_maintainer.pl with ARM KOMEDA DRM-KMS DRIVER

[Joe Perches]

On Mon, 2021-05-10 at 12:34 +0200, Mauro Carvalho Chehab wrote:
> Hi Joe,
> 
> While submitting today a patch series, I noticed that the
> ./scripts/get_maintainer.pl script is not handling well the
> ARM KOMEDA DRM-KMS DRIVER open list entry at MAINTAINERS.
> 
> I'm using v5.13-rc1 here.
> 
> See, it gets an empty e-mail for the ML:
> 
> 	$ git show a1c3be890440 --pretty=email|./scripts/get_maintainer.pl 
> 	"James (Qian) Wang" <james.qian.wang@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
> 	Liviu Dudau <liviu.dudau@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
> 	Mihail Atanassov <mihail.atanassov@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
> 	Brian Starkey <brian.starkey@arm.com> (supporter:ARM MALI-DP DRM DRIVER)
> 	David Airlie <airlied@linux.ie> (maintainer:DRM DRIVERS)
> 	Daniel Vetter <daniel@ffwll.ch> (maintainer:DRM DRIVERS)
> 	 (open list:ARM KOMEDA DRM-KMS DRIVER)
> 	dri-devel@lists.freedesktop.org (open list:DRM DRIVERS)
> 	linux-kernel@vger.kernel.org (open list)
> 
> The entry at MAINTAINERS apparently looks OK:
> 
> 	ARM KOMEDA DRM-KMS DRIVER
> 	M:      James (Qian) Wang <james.qian.wang@arm.com>
> 	M:      Liviu Dudau <liviu.dudau@arm.com>
> 	M:      Mihail Atanassov <mihail.atanassov@arm.com>
> 	L:      Mali DP Maintainers <malidp@foss.arm.com>
> 	S:      Supported
> 	T:      git git://anongit.freedesktop.org/drm/drm-misc
> 	F:      Documentation/devicetree/bindings/display/arm,komeda.txt
> 	F:      Documentation/gpu/komeda-kms.rst
> 	F:      drivers/gpu/drm/arm/display/include/
> 	F:      drivers/gpu/drm/arm/display/komeda/

get_maintainer.pl does not support the L: using a "name" for the mailing list.
Only use the actual mailing list email address.

This should just be:

L:	malidp@foss.arm.com

so a possible patch is:
---
 MAINTAINERS | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index bd7aff0c120f..a41a26079806 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1383,7 +1383,7 @@ ARM KOMEDA DRM-KMS DRIVER
 M:	James (Qian) Wang <james.qian.wang@arm.com>
 M:	Liviu Dudau <liviu.dudau@arm.com>
 M:	Mihail Atanassov <mihail.atanassov@arm.com>
-L:	Mali DP Maintainers <malidp@foss.arm.com>
+L:	malidp@foss.arm.com
 S:	Supported
 T:	git git://anongit.freedesktop.org/drm/drm-misc
 F:	Documentation/devicetree/bindings/display/arm,komeda.txt
@@ -1405,7 +1405,7 @@ F:	include/uapi/drm/panfrost_drm.h
 ARM MALI-DP DRM DRIVER
 M:	Liviu Dudau <liviu.dudau@arm.com>
 M:	Brian Starkey <brian.starkey@arm.com>
-L:	Mali DP Maintainers <malidp@foss.arm.com>
+L:	malidp@foss.arm.com
 S:	Supported
 T:	git git://anongit.freedesktop.org/drm/drm-misc
 F:	Documentation/devicetree/bindings/display/arm,malidp.txt

Re: bug at scripts/get_maintainer.pl with ARM KOMEDA DRM-KMS DRIVER

[Joe Perches]

On Mon, 2021-05-10 at 12:34 +0200, Mauro Carvalho Chehab wrote:
> Hi Joe,
> 
> While submitting today a patch series, I noticed that the
> ./scripts/get_maintainer.pl script is not handling well the
> ARM KOMEDA DRM-KMS DRIVER open list entry at MAINTAINERS.
> 
> I'm using v5.13-rc1 here.
> 
> See, it gets an empty e-mail for the ML:
> 
> 	$ git show a1c3be890440 --pretty=email|./scripts/get_maintainer.pl 
> 	"James (Qian) Wang" <james.qian.wang@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
> 	Liviu Dudau <liviu.dudau@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
> 	Mihail Atanassov <mihail.atanassov@arm.com> (supporter:ARM KOMEDA DRM-KMS DRIVER)
> 	Brian Starkey <brian.starkey@arm.com> (supporter:ARM MALI-DP DRM DRIVER)
> 	David Airlie <airlied@linux.ie> (maintainer:DRM DRIVERS)
> 	Daniel Vetter <daniel@ffwll.ch> (maintainer:DRM DRIVERS)
> 	 (open list:ARM KOMEDA DRM-KMS DRIVER)
> 	dri-devel@lists.freedesktop.org (open list:DRM DRIVERS)
> 	linux-kernel@vger.kernel.org (open list)
> 
> The entry at MAINTAINERS apparently looks OK:
> 
> 	ARM KOMEDA DRM-KMS DRIVER
> 	M:      James (Qian) Wang <james.qian.wang@arm.com>
> 	M:      Liviu Dudau <liviu.dudau@arm.com>
> 	M:      Mihail Atanassov <mihail.atanassov@arm.com>
> 	L:      Mali DP Maintainers <malidp@foss.arm.com>

Perhaps these "L:" entries should use "M:" instead.

Is this actually a mailing list or an exploder?

If it's a mailing list, is there a public archive somewhere?
How does one subscribe?