2020-04-22 11:17:05

by Ezequiel Garcia

[permalink] [raw]
Subject: [PATCH] media: MAINTAINERS: Fix Hantro, Rga and Rkvdec entries

It seems recent merges introduced a couple issues
here, so let's fix them all. Also, reorder Rockchip
video decoder as per parse-maintainers.pl script
and add linux-rockchip mailing list.

Reported-by: Johan Jonker <[email protected]>
Signed-off-by: Ezequiel Garcia <[email protected]>
---
MAINTAINERS | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 09e0137df61d..4d5b06c6fe0f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7460,7 +7460,7 @@ L: [email protected]
L: [email protected]
S: Maintained
F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
-F: Documentation/devicetree/bindings/media/rockchip-vpu.txt
+F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
F: drivers/staging/media/hantro/

HARD DRIVE ACTIVE PROTECTION SYSTEM (HDAPS) DRIVER
@@ -14465,13 +14465,6 @@ F: Documentation/ABI/*/sysfs-driver-hid-roccat*
F: drivers/hid/hid-roccat*
F: include/linux/hid-roccat*

-ROCKCHIP VIDEO DECODER DRIVER
-M: Ezequiel Garcia <[email protected]>
-L: [email protected]
-S: Maintained
-F: drivers/staging/media/rkvdec/
-F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
-
ROCKCHIP ISP V1 DRIVER
M: Helen Koike <[email protected]>
L: [email protected]
@@ -14483,12 +14476,19 @@ ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
M: Jacob Chen <[email protected]>
M: Ezequiel Garcia <[email protected]>
L: [email protected]
+L: [email protected]
S: Maintained
-F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
F: Documentation/devicetree/bindings/media/rockchip-rga.yaml
-F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
F: drivers/media/platform/rockchip/rga/

+ROCKCHIP VIDEO DECODER DRIVER
+M: Ezequiel Garcia <[email protected]>
+L: [email protected]
+L: [email protected]
+S: Maintained
+F: drivers/staging/media/rkvdec/
+F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
+
ROCKER DRIVER
M: Jiri Pirko <[email protected]>
L: [email protected]
--
2.26.0.rc2


2020-04-22 12:03:59

by Johan Jonker

[permalink] [raw]
Subject: Re: [PATCH] media: MAINTAINERS: Fix Hantro, Rga and Rkvdec entries

Hi Ezequiel,

On 4/22/20 1:14 PM, Ezequiel Garcia wrote:
> It seems recent merges introduced a couple issues
> here, so let's fix them all. Also, reorder Rockchip
> video decoder as per parse-maintainers.pl script
> and add linux-rockchip mailing list.
>
> Reported-by: Johan Jonker <[email protected]>
> Signed-off-by: Ezequiel Garcia <[email protected]>
> ---
> MAINTAINERS | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 09e0137df61d..4d5b06c6fe0f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7460,7 +7460,7 @@ L: [email protected]
> L: [email protected]
> S: Maintained
> F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
> -F: Documentation/devicetree/bindings/media/rockchip-vpu.txt
> +F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
> F: drivers/staging/media/hantro/
>
> HARD DRIVE ACTIVE PROTECTION SYSTEM (HDAPS) DRIVER
> @@ -14465,13 +14465,6 @@ F: Documentation/ABI/*/sysfs-driver-hid-roccat*
> F: drivers/hid/hid-roccat*
> F: include/linux/hid-roccat*
>
> -ROCKCHIP VIDEO DECODER DRIVER
> -M: Ezequiel Garcia <[email protected]>
> -L: [email protected]
> -S: Maintained
> -F: drivers/staging/media/rkvdec/
> -F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
> -
> ROCKCHIP ISP V1 DRIVER
> M: Helen Koike <[email protected]>
> L: [email protected]

L: [email protected]

> @@ -14483,12 +14476,19 @@ ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
> M: Jacob Chen <[email protected]>
> M: Ezequiel Garcia <[email protected]>
> L: [email protected]
> +L: [email protected]
> S: Maintained
> -F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
> F: Documentation/devicetree/bindings/media/rockchip-rga.yaml
> -F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
> F: drivers/media/platform/rockchip/rga/
>
> +ROCKCHIP VIDEO DECODER DRIVER
> +M: Ezequiel Garcia <[email protected]>
> +L: [email protected]
> +L: [email protected]
> +S: Maintained

> +F: drivers/staging/media/rkvdec/
> +F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml

When I look at the other entries 'drivers/' is sort below 'Documentation/'.

./scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS
--order

> +
> ROCKER DRIVER
> M: Jiri Pirko <[email protected]>
> L: [email protected]
>

2020-04-22 14:09:02

by Mauro Carvalho Chehab

[permalink] [raw]
Subject: Re: [PATCH] media: MAINTAINERS: Fix Hantro, Rga and Rkvdec entries

Em Wed, 22 Apr 2020 14:01:41 +0200
Johan Jonker <[email protected]> escreveu:

> Hi Ezequiel,
>
> On 4/22/20 1:14 PM, Ezequiel Garcia wrote:
> > It seems recent merges introduced a couple issues
> > here, so let's fix them all. Also, reorder Rockchip
> > video decoder as per parse-maintainers.pl script
> > and add linux-rockchip mailing list.
> >
> > Reported-by: Johan Jonker <[email protected]>
> > Signed-off-by: Ezequiel Garcia <[email protected]>
> > ---
> > MAINTAINERS | 20 ++++++++++----------
> > 1 file changed, 10 insertions(+), 10 deletions(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 09e0137df61d..4d5b06c6fe0f 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -7460,7 +7460,7 @@ L: [email protected]
> > L: [email protected]
> > S: Maintained
> > F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
> > -F: Documentation/devicetree/bindings/media/rockchip-vpu.txt
> > +F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
> > F: drivers/staging/media/hantro/
> >
> > HARD DRIVE ACTIVE PROTECTION SYSTEM (HDAPS) DRIVER
> > @@ -14465,13 +14465,6 @@ F: Documentation/ABI/*/sysfs-driver-hid-roccat*
> > F: drivers/hid/hid-roccat*
> > F: include/linux/hid-roccat*
> >
> > -ROCKCHIP VIDEO DECODER DRIVER
> > -M: Ezequiel Garcia <[email protected]>
> > -L: [email protected]
> > -S: Maintained
> > -F: drivers/staging/media/rkvdec/
> > -F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
> > -
> > ROCKCHIP ISP V1 DRIVER
> > M: Helen Koike <[email protected]>
> > L: [email protected]
>
> L: [email protected]
>
> > @@ -14483,12 +14476,19 @@ ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
> > M: Jacob Chen <[email protected]>
> > M: Ezequiel Garcia <[email protected]>
> > L: [email protected]
> > +L: [email protected]
> > S: Maintained
> > -F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
> > F: Documentation/devicetree/bindings/media/rockchip-rga.yaml
> > -F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
> > F: drivers/media/platform/rockchip/rga/
> >
> > +ROCKCHIP VIDEO DECODER DRIVER
> > +M: Ezequiel Garcia <[email protected]>
> > +L: [email protected]
> > +L: [email protected]
> > +S: Maintained
>
> > +F: drivers/staging/media/rkvdec/
> > +F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
>
> When I look at the other entries 'drivers/' is sort below 'Documentation/'.
>
> ./scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS
> --order

Good point. It would be cool if checkpatch.pl could call it in order
to verify if MAKEFILE changes are at the right order.

As I noticed other fields that are out of the order on media,
due to some changes that didn't arrive upstream yet, I'll fix
them all on a single patch.

Thanks,
Mauro

2020-04-22 14:21:37

by Ezequiel Garcia

[permalink] [raw]
Subject: Re: [PATCH] media: MAINTAINERS: Fix Hantro, Rga and Rkvdec entries

On Wed, 2020-04-22 at 16:06 +0200, Mauro Carvalho Chehab wrote:
> Em Wed, 22 Apr 2020 14:01:41 +0200
> Johan Jonker <[email protected]> escreveu:
>
> > Hi Ezequiel,
> >
> > On 4/22/20 1:14 PM, Ezequiel Garcia wrote:
> > > It seems recent merges introduced a couple issues
> > > here, so let's fix them all. Also, reorder Rockchip
> > > video decoder as per parse-maintainers.pl script
> > > and add linux-rockchip mailing list.
> > >
> > > Reported-by: Johan Jonker <[email protected]>
> > > Signed-off-by: Ezequiel Garcia <[email protected]>
> > > ---
> > > MAINTAINERS | 20 ++++++++++----------
> > > 1 file changed, 10 insertions(+), 10 deletions(-)
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 09e0137df61d..4d5b06c6fe0f 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -7460,7 +7460,7 @@ L: [email protected]
> > > L: [email protected]
> > > S: Maintained
> > > F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
> > > -F: Documentation/devicetree/bindings/media/rockchip-vpu.txt
> > > +F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
> > > F: drivers/staging/media/hantro/
> > >
> > > HARD DRIVE ACTIVE PROTECTION SYSTEM (HDAPS) DRIVER
> > > @@ -14465,13 +14465,6 @@ F: Documentation/ABI/*/sysfs-driver-hid-roccat*
> > > F: drivers/hid/hid-roccat*
> > > F: include/linux/hid-roccat*
> > >
> > > -ROCKCHIP VIDEO DECODER DRIVER
> > > -M: Ezequiel Garcia <[email protected]>
> > > -L: [email protected]
> > > -S: Maintained
> > > -F: drivers/staging/media/rkvdec/
> > > -F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
> > > -
> > > ROCKCHIP ISP V1 DRIVER
> > > M: Helen Koike <[email protected]>
> > > L: [email protected]
> >
> > L: [email protected]
> >
> > > @@ -14483,12 +14476,19 @@ ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
> > > M: Jacob Chen <[email protected]>
> > > M: Ezequiel Garcia <[email protected]>
> > > L: [email protected]
> > > +L: [email protected]
> > > S: Maintained
> > > -F: Documentation/devicetree/bindings/media/nxp,imx8mq-vpu.yaml
> > > F: Documentation/devicetree/bindings/media/rockchip-rga.yaml
> > > -F: Documentation/devicetree/bindings/media/rockchip-vpu.yaml
> > > F: drivers/media/platform/rockchip/rga/
> > >
> > > +ROCKCHIP VIDEO DECODER DRIVER
> > > +M: Ezequiel Garcia <[email protected]>
> > > +L: [email protected]
> > > +L: [email protected]
> > > +S: Maintained
> > > +F: drivers/staging/media/rkvdec/
> > > +F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
> >
> > When I look at the other entries 'drivers/' is sort below 'Documentation/'.
> >
> > ./scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS
> > --order
>
> Good point. It would be cool if checkpatch.pl could call it in order
> to verify if MAKEFILE changes are at the right order.
>
> As I noticed other fields that are out of the order on media,
> due to some changes that didn't arrive upstream yet, I'll fix
> them all on a single patch.
>

Thanks for taking care of this.

Ezequiel

2020-04-22 14:49:27

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] media: MAINTAINERS: Fix Hantro, Rga and Rkvdec entries

On Wed, 2020-04-22 at 08:14 -0300, Ezequiel Garcia wrote:
> It seems recent merges introduced a couple issues
> here, so let's fix them all. Also, reorder Rockchip
> video decoder as per parse-maintainers.pl script
> and add linux-rockchip mailing list.
[]
> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> +ROCKCHIP VIDEO DECODER DRIVER
> +M: Ezequiel Garcia <[email protected]>
> +L: [email protected]
> +L: [email protected]
> +S: Maintained
> +F: drivers/staging/media/rkvdec/
> +F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml

alphabetic case sensitive order please.

+F: Documentation/devicetree/bindings/media/rockchip,vdec.yaml
+F: drivers/staging/media/rkvdec/