2020-04-03 16:18:12

by Helen Koike

[permalink] [raw]
Subject: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings

The Rockchip ISP bindings was moved out of staging.
Update MAINTAINERS file with the new path.

Suggested-by: Johan Jonker <[email protected]>
Signed-off-by: Helen Koike <[email protected]>
---

V2:
- This is a new patch in the series
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index d66ac41ef5872..726044b84cf23 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14303,6 +14303,7 @@ M: Helen Koike <[email protected]>
L: [email protected]
S: Maintained
F: drivers/staging/media/rkisp1/
+F: Documentation/devicetree/bindings/media/rockchip-isp1.yaml

ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
M: Jacob Chen <[email protected]>
--
2.26.0


2020-04-17 07:20:06

by Hans Verkuil

[permalink] [raw]
Subject: Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings

On 03/04/2020 18:15, Helen Koike wrote:
> The Rockchip ISP bindings was moved out of staging.
> Update MAINTAINERS file with the new path.

Shouldn't there be a reference to Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
as well in MAINTAINERS?

Regards,

Hans

>
> Suggested-by: Johan Jonker <[email protected]>
> Signed-off-by: Helen Koike <[email protected]>
> ---
>
> V2:
> - This is a new patch in the series
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d66ac41ef5872..726044b84cf23 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14303,6 +14303,7 @@ M: Helen Koike <[email protected]>
> L: [email protected]
> S: Maintained
> F: drivers/staging/media/rkisp1/
> +F: Documentation/devicetree/bindings/media/rockchip-isp1.yaml
>
> ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
> M: Jacob Chen <[email protected]>
>

2020-04-17 07:36:44

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings

On Fri, 2020-04-17 at 09:18 +0200, Hans Verkuil wrote:
> On 03/04/2020 18:15, Helen Koike wrote:
> > The Rockchip ISP bindings was moved out of staging.
> > Update MAINTAINERS file with the new path.
>
> Shouldn't there be a reference to Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
> as well in MAINTAINERS?

And please keep the file references F: fields in
alphabetic order.

> > diff --git a/MAINTAINERS b/MAINTAINERS
[]
> > @@ -14303,6 +14303,7 @@ M: Helen Koike <[email protected]>
> > L: [email protected]
> > S: Maintained
> > F: drivers/staging/media/rkisp1/
> > +F: Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> >
> > ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
> > M: Jacob Chen <[email protected]>
> >

2020-04-23 14:08:48

by Johan Jonker

[permalink] [raw]
Subject: Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings

Hi Helen,

On 4/3/20 6:15 PM, Helen Koike wrote:
> The Rockchip ISP bindings was moved out of staging.
> Update MAINTAINERS file with the new path.
>
> Suggested-by: Johan Jonker <[email protected]>
> Signed-off-by: Helen Koike <[email protected]>
> ---
>
> V2:
> - This is a new patch in the series
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d66ac41ef5872..726044b84cf23 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14303,6 +14303,7 @@ M: Helen Koike <[email protected]>
> L: [email protected]

L: [email protected]

> S: Maintained

> F: drivers/staging/media/rkisp1/
> +F: Documentation/devicetree/bindings/media/rockchip-isp1.yaml

Rebase.
The MAINTAINERS document was recently sort.
Check entries with:

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

>
> ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
> M: Jacob Chen <[email protected]>
>

2020-05-09 12:55:21

by Ezequiel Garcia

[permalink] [raw]
Subject: Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings

Hi Hans,

On Fri, 2020-04-17 at 09:18 +0200, Hans Verkuil wrote:
> On 03/04/2020 18:15, Helen Koike wrote:
> > The Rockchip ISP bindings was moved out of staging.
> > Update MAINTAINERS file with the new path.
>
> Shouldn't there be a reference to Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
> as well in MAINTAINERS?
>

Indeed, and I can take care of that later. I was planning to move
this PHY driver out of staging, but was waiting on patches 4 and 5
of this series.

They seem fine, would you mind picking them?

Thanks!
Ezequiel

> Regards,
>
> Hans
>
> > Suggested-by: Johan Jonker <[email protected]>
> > Signed-off-by: Helen Koike <[email protected]>
> > ---
> >
> > V2:
> > - This is a new patch in the series
> > ---
> > MAINTAINERS | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index d66ac41ef5872..726044b84cf23 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -14303,6 +14303,7 @@ M: Helen Koike <[email protected]>
> > L: [email protected]
> > S: Maintained
> > F: drivers/staging/media/rkisp1/
> > +F: Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> >
> > ROCKCHIP RASTER 2D GRAPHIC ACCELERATION UNIT DRIVER
> > M: Jacob Chen <[email protected]>
> >
>
>