[PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399

Linux-Rockchip Archive on lore.kernel.org
 help / color / mirror / Atom feed

* [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399
@ 2020-04-03 16:15 Helen Koike
  2020-04-03 16:15 ` [PATCH v2 3/9] media: staging: dt-bindings: rkisp1: re-order properties Helen Koike
                   ` (5 more replies)
  0 siblings, 6 replies; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree, linux-media, linux-rockchip
  Cc: linux-kernel, devel, robh+dt, heiko, hverkuil-cisco, kernel,
	dafna.hirschfeld, ezequiel, mark.rutland, karthik.poduval,
	jbx6244, kishon

Move the bindings out of drivers/staging and place them in
Documentation/devicetree/bindings instead.

Also, add DT nodes for RK3399 and verify with make ARCH=arm64 dtbs_check
and make ARCH=arm64 dt_binding_check.

Tested by verifying images streamed from RockPi 4 board with imx219
module.

Changes in v2:
Add patches modifying bindings, as sugested by Johan Jonker in v1,
before moving them out of staging.

Johan, I added a Suggested-by tag in the commits, please let me know if
you prefer not having the tag there.

Helen Koike (7):
  media: staging: dt-bindings: rkisp1: add missing required nodes
  media: staging: dt-bindings: rkisp1: add required items in i2c example
  media: staging: dt-bindings: rkisp1: re-order properties
  media: staging: dt-bindings: phy-rockchip-dphy-rx0: remove non-used
    reg property
  dt-bindings: phy: phy-rockchip-dphy-rx0: move rockchip dphy rx0
    bindings out of staging
  dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging
  media: MAINTAINERS: rkisp1: add path to dt-bindings

Shunqian Zheng (2):
  arm64: dts: rockchip: add rx0 mipi-phy for rk3399
  arm64: dts: rockchip: add isp0 node for rk3399

 .../bindings/media/rockchip-isp1.yaml         | 45 ++++++++++++-------
 .../bindings/phy/rockchip-mipi-dphy-rx0.yaml  |  3 --
 MAINTAINERS                                   |  1 +
 arch/arm64/boot/dts/rockchip/rk3399.dtsi      | 38 ++++++++++++++++
 4 files changed, 68 insertions(+), 19 deletions(-)
 rename {drivers/staging/media/rkisp1/Documentation => Documentation}/devicetree/bindings/media/rockchip-isp1.yaml (91%)
 rename {drivers/staging/media/phy-rockchip-dphy-rx0/Documentation => Documentation}/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml (98%)

-- 
2.26.0

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v2 3/9] media: staging: dt-bindings: rkisp1: re-order properties
  2020-04-03 16:15 [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Helen Koike
@ 2020-04-03 16:15 ` Helen Koike
  2020-04-03 16:15 ` [PATCH v2 4/9] media: staging: dt-bindings: phy-rockchip-dphy-rx0: remove non-used reg property Helen Koike
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree, linux-media, linux-rockchip
  Cc: linux-kernel, devel, robh+dt, heiko, hverkuil-cisco, kernel,
	dafna.hirschfeld, ezequiel, mark.rutland, karthik.poduval,
	jbx6244, kishon

Organize properties order in dt-binbings to move it out of staging.

On top: compatible, reg and interrupts.
Then alphabetical order, then properties starting with '#'.

Suggested-by: Johan Jonker <jbx6244@gmail.com>
Signed-off-by: Helen Koike <helen.koike@collabora.com>
---

V2:
- this is a new patch in the series
---
 .../bindings/media/rockchip-isp1.yaml         | 32 +++++++++----------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml b/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
index 8c7904845788d..e5b9c0574e352 100644
--- a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
+++ b/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
@@ -23,19 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  iommus:
-    maxItems: 1
-
-  power-domains:
-    maxItems: 1
-
-  phys:
-    maxItems: 1
-    description: phandle for the PHY port
-
-  phy-names:
-    const: dphy
-
   clocks:
     items:
       - description: ISP clock
@@ -52,6 +39,19 @@ properties:
       - const: hclk_isp
       - const: hclk_isp_wrap
 
+  iommus:
+    maxItems: 1
+
+  phys:
+    maxItems: 1
+    description: phandle for the PHY port
+
+  phy-names:
+    const: dphy
+
+  power-domains:
+    maxItems: 1
+
   # See ./video-interfaces.txt for details
   ports:
     type: object
@@ -110,10 +110,10 @@ required:
   - interrupts
   - clocks
   - clock-names
-  - power-domains
   - iommus
   - phys
   - phy-names
+  - power-domains
   - ports
 
 additionalProperties: false
@@ -139,19 +139,19 @@ examples:
             clock-names = "clk_isp",
                           "aclk_isp", "aclk_isp_wrap",
                           "hclk_isp", "hclk_isp_wrap";
-            power-domains = <&power RK3399_PD_ISP0>;
             iommus = <&isp0_mmu>;
             phys = <&dphy>;
             phy-names = "dphy";
+            power-domains = <&power RK3399_PD_ISP0>;
 
             ports {
                 #address-cells = <1>;
                 #size-cells = <0>;
 
                 port@0 {
+                    reg = <0>;
                     #address-cells = <1>;
                     #size-cells = <0>;
-                    reg = <0>;
 
                     mipi_in_wcam: endpoint@0 {
                         reg = <0>;
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [PATCH v2 4/9] media: staging: dt-bindings: phy-rockchip-dphy-rx0: remove non-used reg property
  2020-04-03 16:15 [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Helen Koike
  2020-04-03 16:15 ` [PATCH v2 3/9] media: staging: dt-bindings: rkisp1: re-order properties Helen Koike
@ 2020-04-03 16:15 ` Helen Koike
  2020-04-03 16:15 ` [PATCH v2 5/9] dt-bindings: phy: phy-rockchip-dphy-rx0: move rockchip dphy rx0 bindings out of staging Helen Koike
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree, linux-media, linux-rockchip
  Cc: linux-kernel, devel, robh+dt, heiko, hverkuil-cisco, kernel,
	dafna.hirschfeld, ezequiel, mark.rutland, karthik.poduval,
	jbx6244, kishon

reg property is not used in Rockchip MIPI DPHY RX0 bindings, thus remove
it.

Suggested-by: Johan Jonker <jbx6244@gmail.com>
Signed-off-by: Helen Koike <helen.koike@collabora.com>
---

Changes in v2:
- this is a new patch in the series
---
 .../devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml        | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/staging/media/phy-rockchip-dphy-rx0/Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml b/drivers/staging/media/phy-rockchip-dphy-rx0/Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
index 5dacece35702b..7d888d3588239 100644
--- a/drivers/staging/media/phy-rockchip-dphy-rx0/Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
+++ b/drivers/staging/media/phy-rockchip-dphy-rx0/Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
@@ -18,9 +18,6 @@ properties:
   compatible:
     const: rockchip,rk3399-mipi-dphy-rx0
 
-  reg:
-    maxItems: 1
-
   clocks:
     items:
       - description: MIPI D-PHY ref clock
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [PATCH v2 5/9] dt-bindings: phy: phy-rockchip-dphy-rx0: move rockchip dphy rx0 bindings out of staging
  2020-04-03 16:15 [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Helen Koike
  2020-04-03 16:15 ` [PATCH v2 3/9] media: staging: dt-bindings: rkisp1: re-order properties Helen Koike
  2020-04-03 16:15 ` [PATCH v2 4/9] media: staging: dt-bindings: phy-rockchip-dphy-rx0: remove non-used reg property Helen Koike
@ 2020-04-03 16:15 ` Helen Koike
  2020-04-14 18:08   ` Rob Herring
  2020-04-03 16:15 ` [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings Helen Koike
                   ` (2 subsequent siblings)
  5 siblings, 1 reply; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree, linux-media, linux-rockchip
  Cc: linux-kernel, devel, robh+dt, heiko, hverkuil-cisco, kernel,
	dafna.hirschfeld, ezequiel, mark.rutland, karthik.poduval,
	jbx6244, kishon

Move phy-rockchip-dphy-rx0 bindings to Documentation/devicetree/bindings/phy

Verified with:
make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml

Signed-off-by: Helen Koike <helen.koike@collabora.com>
---

V2:
- no changes

 .../devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml           | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 rename {drivers/staging/media/phy-rockchip-dphy-rx0/Documentation => Documentation}/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml (100%)

diff --git a/drivers/staging/media/phy-rockchip-dphy-rx0/Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml b/Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
similarity index 100%
rename from drivers/staging/media/phy-rockchip-dphy-rx0/Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
rename to Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
-- 
2.26.0

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v2 5/9] dt-bindings: phy: phy-rockchip-dphy-rx0: move rockchip dphy rx0 bindings out of staging
  2020-04-03 16:15 ` [PATCH v2 5/9] dt-bindings: phy: phy-rockchip-dphy-rx0: move rockchip dphy rx0 bindings out of staging Helen Koike
@ 2020-04-14 18:08   ` Rob Herring
  0 siblings, 0 replies; 20+ messages in thread
From: Rob Herring @ 2020-04-14 18:08 UTC (permalink / raw)
  To: Helen Koike
  Cc: devicetree, linux-media, linux-rockchip, linux-kernel, devel,
	robh+dt, heiko, hverkuil-cisco, kernel, dafna.hirschfeld,
	ezequiel, mark.rutland, karthik.poduval, jbx6244, kishon

On Fri,  3 Apr 2020 13:15:34 -0300, Helen Koike wrote:
> Move phy-rockchip-dphy-rx0 bindings to Documentation/devicetree/bindings/phy
> 
> Verified with:
> make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
> 
> Signed-off-by: Helen Koike <helen.koike@collabora.com>
> ---
> 
> V2:
> - no changes
> 
>  .../devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml           | 0
>  1 file changed, 0 insertions(+), 0 deletions(-)
>  rename {drivers/staging/media/phy-rockchip-dphy-rx0/Documentation => Documentation}/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml (100%)
> 

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings
  2020-04-03 16:15 [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Helen Koike
                   ` (2 preceding siblings ...)
  2020-04-03 16:15 ` [PATCH v2 5/9] dt-bindings: phy: phy-rockchip-dphy-rx0: move rockchip dphy rx0 bindings out of staging Helen Koike
@ 2020-04-03 16:15 ` Helen Koike
       [not found]   ` <20200403161538.1375908-8-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  2020-04-03 16:15 ` [PATCH v2 8/9] arm64: dts: rockchip: add rx0 mipi-phy for rk3399 Helen Koike
       [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  5 siblings, 1 reply; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree, linux-media, linux-rockchip
  Cc: linux-kernel, devel, robh+dt, heiko, hverkuil-cisco, kernel,
	dafna.hirschfeld, ezequiel, mark.rutland, karthik.poduval,
	jbx6244, kishon

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

Suggested-by: Johan Jonker <jbx6244@gmail.com>
Signed-off-by: Helen Koike <helen.koike@collabora.com>
---

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 <helen.koike@collabora.com>
 L:	linux-media@vger.kernel.org
 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 <jacob-chen@iotwrt.com>
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 20+ messages in thread

[parent not found: <20200403161538.1375908-8-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>]

* Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings
       [not found]   ` <20200403161538.1375908-8-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
@ 2020-04-17  7:18     ` Hans Verkuil
       [not found]       ` <9a95d227-9592-2c5c-fe6d-dff9b84f4292-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
  2020-04-23 12:48     ` Johan Jonker
  1 sibling, 1 reply; 20+ messages in thread
From: Hans Verkuil @ 2020-04-17  7:18 UTC (permalink / raw)
  To: Helen Koike, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

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 <jbx6244-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> ---
> 
> 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 <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
>  L:	linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>  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 <jacob-chen-fyOeoxGR3m/QT0dZR+AlfA@public.gmane.org>
> 

^ permalink raw reply	[flat|nested] 20+ messages in thread

[parent not found: <9a95d227-9592-2c5c-fe6d-dff9b84f4292-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>]

* Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings
       [not found]       ` <9a95d227-9592-2c5c-fe6d-dff9b84f4292-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
@ 2020-04-17  7:30         ` Joe Perches
  2020-05-09 12:51         ` Ezequiel Garcia
  1 sibling, 0 replies; 20+ messages in thread
From: Joe Perches @ 2020-04-17  7:30 UTC (permalink / raw)
  To: Hans Verkuil, Helen Koike, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

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 <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> >  L:	linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> >  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 <jacob-chen-fyOeoxGR3m/QT0dZR+AlfA@public.gmane.org>
> > 

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings
       [not found]       ` <9a95d227-9592-2c5c-fe6d-dff9b84f4292-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
  2020-04-17  7:30         ` Joe Perches
@ 2020-05-09 12:51         ` Ezequiel Garcia
  1 sibling, 0 replies; 20+ messages in thread
From: Ezequiel Garcia @ 2020-05-09 12:51 UTC (permalink / raw)
  To: Hans Verkuil, Helen Koike, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

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 <jbx6244-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> > Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> > ---
> > 
> > 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 <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> >  L:	linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> >  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 <jacob-chen-fyOeoxGR3m/QT0dZR+AlfA@public.gmane.org>
> > 
> 
> 

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings
       [not found]   ` <20200403161538.1375908-8-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  2020-04-17  7:18     ` Hans Verkuil
@ 2020-04-23 12:48     ` Johan Jonker
  1 sibling, 0 replies; 20+ messages in thread
From: Johan Jonker @ 2020-04-23 12:48 UTC (permalink / raw)
  To: Helen Koike, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

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 <jbx6244-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> ---
> 
> 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 <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
>  L:	linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

L:	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org

>  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 <jacob-chen-fyOeoxGR3m/QT0dZR+AlfA@public.gmane.org>
> 

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v2 8/9] arm64: dts: rockchip: add rx0 mipi-phy for rk3399
  2020-04-03 16:15 [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Helen Koike
                   ` (3 preceding siblings ...)
  2020-04-03 16:15 ` [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings Helen Koike
@ 2020-04-03 16:15 ` Helen Koike
       [not found]   ` <20200403161538.1375908-9-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
       [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  5 siblings, 1 reply; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree, linux-media, linux-rockchip
  Cc: linux-kernel, devel, robh+dt, heiko, hverkuil-cisco, kernel,
	dafna.hirschfeld, ezequiel, mark.rutland, karthik.poduval,
	jbx6244, kishon

From: Shunqian Zheng <zhengsq@rock-chips.com>

Designware MIPI D-PHY, used for ISP0 in rk3399.

Verified with:
make ARCH=arm64 dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml

Signed-off-by: Shunqian Zheng <zhengsq@rock-chips.com>
Signed-off-by: Jacob Chen <jacob2.chen@rock-chips.com>
Signed-off-by: Helen Koike <helen.koike@collabora.com>

---

Changes in v2:
- fix alignment of clocks

V1:
This patchset came from the original ISP series from Rockchip:

    https://patchwork.kernel.org/patch/10267409/

The only difference is:
- add phy-cells
- update compatible to "rockchip,rk3399-mipi-dphy-rx0"
- commit message
---
 arch/arm64/boot/dts/rockchip/rk3399.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
index 33cc21fcf4c10..6b3380b10e596 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
@@ -1394,6 +1394,17 @@ io_domains: io-domains {
 			status = "disabled";
 		};
 
+		mipi_dphy_rx0: mipi-dphy-rx0 {
+			compatible = "rockchip,rk3399-mipi-dphy-rx0";
+			clocks = <&cru SCLK_MIPIDPHY_REF>,
+				 <&cru SCLK_DPHY_RX0_CFG>,
+				 <&cru PCLK_VIO_GRF>;
+			clock-names = "dphy-ref", "dphy-cfg", "grf";
+			power-domains = <&power RK3399_PD_VIO>;
+			#phy-cells = <0>;
+			status = "disabled";
+		};
+
 		u2phy0: usb2-phy@e450 {
 			compatible = "rockchip,rk3399-usb2phy";
 			reg = <0xe450 0x10>;
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 20+ messages in thread

[parent not found: <20200403161538.1375908-9-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>]

* Re: [PATCH v2 8/9] arm64: dts: rockchip: add rx0 mipi-phy for rk3399
       [not found]   ` <20200403161538.1375908-9-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
@ 2020-05-09 12:53     ` Ezequiel Garcia
  0 siblings, 0 replies; 20+ messages in thread
From: Ezequiel Garcia @ 2020-05-09 12:53 UTC (permalink / raw)
  To: Helen Koike, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

Hi Heiko,

On Fri, 2020-04-03 at 13:15 -0300, Helen Koike wrote:
> From: Shunqian Zheng <zhengsq-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
> 
> Designware MIPI D-PHY, used for ISP0 in rk3399.
> 
> Verified with:
> make ARCH=arm64 dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/rockchip-mipi-dphy-rx0.yaml
> 
> Signed-off-by: Shunqian Zheng <zhengsq-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
> Signed-off-by: Jacob Chen <jacob2.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
> Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> 
> ---
> 
> Changes in v2:
> - fix alignment of clocks
> 
> V1:
> This patchset came from the original ISP series from Rockchip:
> 
>     https://patchwork.kernel.org/patch/10267409/
> 

Can you take the devicetree changes (patches 8 and 9) ?

Thanks,
Ezequiel

> The only difference is:
> - add phy-cells
> - update compatible to "rockchip,rk3399-mipi-dphy-rx0"
> - commit message
> ---
>  arch/arm64/boot/dts/rockchip/rk3399.dtsi | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> index 33cc21fcf4c10..6b3380b10e596 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> @@ -1394,6 +1394,17 @@ io_domains: io-domains {
>  			status = "disabled";
>  		};
>  
> +		mipi_dphy_rx0: mipi-dphy-rx0 {
> +			compatible = "rockchip,rk3399-mipi-dphy-rx0";
> +			clocks = <&cru SCLK_MIPIDPHY_REF>,
> +				 <&cru SCLK_DPHY_RX0_CFG>,
> +				 <&cru PCLK_VIO_GRF>;
> +			clock-names = "dphy-ref", "dphy-cfg", "grf";
> +			power-domains = <&power RK3399_PD_VIO>;
> +			#phy-cells = <0>;
> +			status = "disabled";
> +		};
> +
>  		u2phy0: usb2-phy@e450 {
>  			compatible = "rockchip,rk3399-usb2phy";
>  			reg = <0xe450 0x10>;
> -- 
> 2.26.0
> 
> 

^ permalink raw reply	[flat|nested] 20+ messages in thread

[parent not found: <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>]

* [PATCH v2 1/9] media: staging: dt-bindings: rkisp1: add missing required nodes
       [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
@ 2020-04-03 16:15   ` Helen Koike
  2020-04-03 16:15   ` [PATCH v2 2/9] media: staging: dt-bindings: rkisp1: add required items in i2c example Helen Koike
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

Add missing required nodes in json-schema yaml file for
Rockchip ISP1 dt-bindings.

Suggested-by: Johan Jonker <jbx6244-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
---

Changes in v2:
- New patch in the series
---
 .../devicetree/bindings/media/rockchip-isp1.yaml          | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml b/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
index af246b71eac6b..a77b6ec500c95 100644
--- a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
+++ b/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
@@ -94,11 +94,19 @@ properties:
 
               remote-endpoint: true
 
+        required:
+          - reg
+          - "#address-cells"
+          - "#size-cells"
+
     required:
+      - "#address-cells"
+      - "#size-cells"
       - port@0
 
 required:
   - compatible
+  - reg
   - interrupts
   - clocks
   - clock-names
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [PATCH v2 2/9] media: staging: dt-bindings: rkisp1: add required items in i2c example
       [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  2020-04-03 16:15   ` [PATCH v2 1/9] media: staging: dt-bindings: rkisp1: add missing required nodes Helen Koike
@ 2020-04-03 16:15   ` Helen Koike
  2020-04-03 16:15   ` [PATCH v2 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging Helen Koike
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

Add missing required items in Rockchip ISP1 dt-bindings example for
a complete i2c node.

Suggested-by: Johan Jonker <jbx6244-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
---

Changes in v2:
- new patch in the series
---
 .../devicetree/bindings/media/rockchip-isp1.yaml             | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml b/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
index a77b6ec500c95..8c7904845788d 100644
--- a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
+++ b/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
@@ -169,6 +169,11 @@ examples:
         };
 
         i2c7: i2c@ff160000 {
+            compatible = "rockchip,rk3399-i2c";
+            reg = <0x0 0xff160000 0x0 0x1000>;
+            interrupts = <GIC_SPI 36 IRQ_TYPE_LEVEL_HIGH 0>;
+            clocks = <&cru SCLK_I2C7>, <&cru PCLK_I2C7>;
+            clock-names = "i2c", "pclk";
             clock-frequency = <400000>;
             #address-cells = <1>;
             #size-cells = <0>;
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [PATCH v2 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging
       [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  2020-04-03 16:15   ` [PATCH v2 1/9] media: staging: dt-bindings: rkisp1: add missing required nodes Helen Koike
  2020-04-03 16:15   ` [PATCH v2 2/9] media: staging: dt-bindings: rkisp1: add required items in i2c example Helen Koike
@ 2020-04-03 16:15   ` Helen Koike
       [not found]     ` <20200403161538.1375908-7-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  2020-04-03 16:15   ` [PATCH v2 9/9] arm64: dts: rockchip: add isp0 node for rk3399 Helen Koike
  2020-07-01 12:31   ` [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Heiko Stuebner
  4 siblings, 1 reply; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

Move rkisp1 bindings to Documentation/devicetree/bindings/media

Verified with:
make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml

Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
---

V2:
- no changes

 .../devicetree/bindings/media/rockchip-isp1.yaml                  | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 rename {drivers/staging/media/rkisp1/Documentation => Documentation}/devicetree/bindings/media/rockchip-isp1.yaml (100%)

diff --git a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml b/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
similarity index 100%
rename from drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
rename to Documentation/devicetree/bindings/media/rockchip-isp1.yaml
-- 
2.26.0

^ permalink raw reply	[flat|nested] 20+ messages in thread

[parent not found: <20200403161538.1375908-7-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>]

* Re: [PATCH v2 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging
       [not found]     ` <20200403161538.1375908-7-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
@ 2020-04-14 18:08       ` Rob Herring
  2020-04-23 12:36       ` Johan Jonker
  2020-04-23 13:33       ` Johan Jonker
  2 siblings, 0 replies; 20+ messages in thread
From: Rob Herring @ 2020-04-14 18:08 UTC (permalink / raw)
  To: Helen Koike
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

On Fri,  3 Apr 2020 13:15:35 -0300, Helen Koike wrote:
> Move rkisp1 bindings to Documentation/devicetree/bindings/media
> 
> Verified with:
> make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> 
> Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> ---
> 
> V2:
> - no changes
> 
>  .../devicetree/bindings/media/rockchip-isp1.yaml                  | 0
>  1 file changed, 0 insertions(+), 0 deletions(-)
>  rename {drivers/staging/media/rkisp1/Documentation => Documentation}/devicetree/bindings/media/rockchip-isp1.yaml (100%)
> 

Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v2 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging
       [not found]     ` <20200403161538.1375908-7-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  2020-04-14 18:08       ` Rob Herring
@ 2020-04-23 12:36       ` Johan Jonker
  2020-04-23 13:33       ` Johan Jonker
  2 siblings, 0 replies; 20+ messages in thread
From: Johan Jonker @ 2020-04-23 12:36 UTC (permalink / raw)
  To: Helen Koike, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

Hi Helen,

make ARCH=arm dt_binding_check
DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml
  SCHEMA  Documentation/devicetree/bindings/processed-schema-examples.yaml
  SCHEMA  Documentation/devicetree/bindings/processed-schema.yaml
  CHKDT   Documentation/devicetree/bindings/media/rockchip-isp1.yaml
  DTC
Documentation/devicetree/bindings/media/rockchip-isp1.example.dt.yaml
Documentation/devicetree/bindings/media/rockchip-isp1.example.dts:22.27-99.11:
Warning (unit_address_vs_reg): /example-0/parent@0: node has a unit
name, but no reg or ranges property
  CHECK
Documentation/devicetree/bindings/media/rockchip-isp1.example.dt.yaml

On 4/3/20 6:15 PM, Helen Koike wrote:
> Move rkisp1 bindings to Documentation/devicetree/bindings/media
> 
> Verified with:
> make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> 
> Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> ---
> 
> V2:
> - no changes
> 
>  .../devicetree/bindings/media/rockchip-isp1.yaml                  | 0
>  1 file changed, 0 insertions(+), 0 deletions(-)
>  rename {drivers/staging/media/rkisp1/Documentation => Documentation}/devicetree/bindings/media/rockchip-isp1.yaml (100%)
> 
> diff --git a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml b/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> similarity index 100%
> rename from drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> rename to Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> 

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v2 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging
       [not found]     ` <20200403161538.1375908-7-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
  2020-04-14 18:08       ` Rob Herring
  2020-04-23 12:36       ` Johan Jonker
@ 2020-04-23 13:33       ` Johan Jonker
  2 siblings, 0 replies; 20+ messages in thread
From: Johan Jonker @ 2020-04-23 13:33 UTC (permalink / raw)
  To: Helen Koike, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

Hi Helen,


>   clocks:
>     items:
>       - description: ISP clock
>       - description: ISP AXI clock clock
>       - description: ISP AXI clock  wrapper clock
>       - description: ISP AHB clock clock

Too many clocks here             ^
Too many spaces here                 ^

>       - description: ISP AHB wrapper clock

We can expect 4 to 5 clocks.
With 5 clocks and a different description this layout is maybe not so handy.

As first change:

  clocks:
    maxItems: 5
    description:
      rk3399 clocks
        ISP clock
        ISP AXI clock
        ISP AXI wrapper clock
        ISP AHB clock
        ISP AHB wrapper clock

And then later:

  clocks:
    maxItems: 5
    description:
      rk3288 clocks
        ISP clock
        ISP AXI clock
        ISP AHB clock
        ISP Pixel clock
        ISP JPEG source clock
      rk3399 clocks
        ISP clock
        ISP AXI clock
        ISP AXI wrapper clock
        ISP AHB clock
        ISP AHB wrapper clock

With 4 clocks:

  clocks:
    minItems: 4
    maxItems: 5
    description:
      rk1808 clocks
      rk3288 clocks
      rk3326 clocks
      rk3368 clocks
      rk3399 clocks
[..]


> 
>   clock-names:
>     items:
>       - const: clk_isp
>       - const: aclk_isp
>       - const: aclk_isp_wrap
>       - const: hclk_isp
>       - const: hclk_isp_wrap



On 4/3/20 6:15 PM, Helen Koike wrote:
> Move rkisp1 bindings to Documentation/devicetree/bindings/media
> 
> Verified with:
> make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> 
> Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
> ---
> 
> V2:
> - no changes
> 
>  .../devicetree/bindings/media/rockchip-isp1.yaml                  | 0
>  1 file changed, 0 insertions(+), 0 deletions(-)
>  rename {drivers/staging/media/rkisp1/Documentation => Documentation}/devicetree/bindings/media/rockchip-isp1.yaml (100%)
> 
> diff --git a/drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml b/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> similarity index 100%
> rename from drivers/staging/media/rkisp1/Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> rename to Documentation/devicetree/bindings/media/rockchip-isp1.yaml
> 

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v2 9/9] arm64: dts: rockchip: add isp0 node for rk3399
       [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
                     ` (2 preceding siblings ...)
  2020-04-03 16:15   ` [PATCH v2 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging Helen Koike
@ 2020-04-03 16:15   ` Helen Koike
  2020-07-01 12:31   ` [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Heiko Stuebner
  4 siblings, 0 replies; 20+ messages in thread
From: Helen Koike @ 2020-04-03 16:15 UTC (permalink / raw)
  To: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, mark.rutland-5wv7dgnIgG8,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	jbx6244-Re5JQEeQqe8AvxtiuMwx3w, kishon-l0cyMroinI0

From: Shunqian Zheng <zhengsq-TNX95d0MmH7DzftRWevZcw@public.gmane.org>

RK3399 has two ISPs, but only ISP0 was tested at present.
Add isp0 node in rk3399 dtsi

Verified with:
make ARCH=arm64 dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml

Signed-off-by: Shunqian Zheng <zhengsq-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
Signed-off-by: Jacob Chen <jacob2.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
Signed-off-by: Helen Koike <helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>

---
V2:
- re-order power-domains property

V1:
This patch was originally part of this patchset:

    https://patchwork.kernel.org/patch/10267431/

The only difference is:
- add phy properties
- add ports
---
 arch/arm64/boot/dts/rockchip/rk3399.dtsi | 27 ++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
index 6b3380b10e596..b1d62a87c5616 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
@@ -1718,6 +1718,33 @@ vopb_mmu: iommu@ff903f00 {
 		status = "disabled";
 	};
 
+	isp0: isp0@ff910000 {
+		compatible = "rockchip,rk3399-cif-isp";
+		reg = <0x0 0xff910000 0x0 0x4000>;
+		interrupts = <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH 0>;
+		clocks = <&cru SCLK_ISP0>,
+			 <&cru ACLK_ISP0>, <&cru ACLK_ISP0_WRAPPER>,
+			 <&cru HCLK_ISP0>, <&cru HCLK_ISP0_WRAPPER>;
+		clock-names = "clk_isp",
+			      "aclk_isp", "aclk_isp_wrap",
+			      "hclk_isp", "hclk_isp_wrap";
+		iommus = <&isp0_mmu>;
+		phys = <&mipi_dphy_rx0>;
+		phy-names = "dphy";
+		power-domains = <&power RK3399_PD_ISP0>;
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			port@0 {
+				reg = <0>;
+				#address-cells = <1>;
+				#size-cells = <0>;
+			};
+		};
+	};
+
 	isp0_mmu: iommu@ff914000 {
 		compatible = "rockchip,iommu";
 		reg = <0x0 0xff914000 0x0 0x100>, <0x0 0xff915000 0x0 0x100>;
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* Re: [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399
       [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
                     ` (3 preceding siblings ...)
  2020-04-03 16:15   ` [PATCH v2 9/9] arm64: dts: rockchip: add isp0 node for rk3399 Helen Koike
@ 2020-07-01 12:31   ` Heiko Stuebner
  4 siblings, 0 replies; 20+ messages in thread
From: Heiko Stuebner @ 2020-07-01 12:31 UTC (permalink / raw)
  To: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Helen Koike,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-media-u79uwXL29TY76Z2rM5mHXA
  Cc: Heiko Stuebner, jbx6244-Re5JQEeQqe8AvxtiuMwx3w,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	hverkuil-cisco-qWit8jRvyhVmR6Xm/wNWPw, mark.rutland-5wv7dgnIgG8,
	devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	ezequiel-ZGY8ohtN/8qB+jHODAdFcQ, kishon-l0cyMroinI0,
	kernel-ZGY8ohtN/8qB+jHODAdFcQ,
	karthik.poduval-Re5JQEeQqe8AvxtiuMwx3w,
	dafna.hirschfeld-ZGY8ohtN/8qB+jHODAdFcQ

On Fri, 3 Apr 2020 13:15:29 -0300, Helen Koike wrote:
> Move the bindings out of drivers/staging and place them in
> Documentation/devicetree/bindings instead.
> 
> Also, add DT nodes for RK3399 and verify with make ARCH=arm64 dtbs_check
> and make ARCH=arm64 dt_binding_check.
> 
> Tested by verifying images streamed from RockPi 4 board with imx219
> module.
> 
> [...]

Applied, thanks!

[1/1] arm64: dts: rockchip: add rx0 mipi-phy for rk3399
      commit: e4bfde13e323f9ee5f2f38aa5cac0676dd656f8e

Best regards,
-- 
Heiko Stuebner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>

^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2020-07-01 12:31 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-03 16:15 [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Helen Koike
2020-04-03 16:15 ` [PATCH v2 3/9] media: staging: dt-bindings: rkisp1: re-order properties Helen Koike
2020-04-03 16:15 ` [PATCH v2 4/9] media: staging: dt-bindings: phy-rockchip-dphy-rx0: remove non-used reg property Helen Koike
2020-04-03 16:15 ` [PATCH v2 5/9] dt-bindings: phy: phy-rockchip-dphy-rx0: move rockchip dphy rx0 bindings out of staging Helen Koike
2020-04-14 18:08   ` Rob Herring
2020-04-03 16:15 ` [PATCH v2 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings Helen Koike
     [not found]   ` <20200403161538.1375908-8-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
2020-04-17  7:18     ` Hans Verkuil
     [not found]       ` <9a95d227-9592-2c5c-fe6d-dff9b84f4292-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2020-04-17  7:30         ` Joe Perches
2020-05-09 12:51         ` Ezequiel Garcia
2020-04-23 12:48     ` Johan Jonker
2020-04-03 16:15 ` [PATCH v2 8/9] arm64: dts: rockchip: add rx0 mipi-phy for rk3399 Helen Koike
     [not found]   ` <20200403161538.1375908-9-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
2020-05-09 12:53     ` Ezequiel Garcia
     [not found] ` <20200403161538.1375908-1-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
2020-04-03 16:15   ` [PATCH v2 1/9] media: staging: dt-bindings: rkisp1: add missing required nodes Helen Koike
2020-04-03 16:15   ` [PATCH v2 2/9] media: staging: dt-bindings: rkisp1: add required items in i2c example Helen Koike
2020-04-03 16:15   ` [PATCH v2 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging Helen Koike
     [not found]     ` <20200403161538.1375908-7-helen.koike-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
2020-04-14 18:08       ` Rob Herring
2020-04-23 12:36       ` Johan Jonker
2020-04-23 13:33       ` Johan Jonker
2020-04-03 16:15   ` [PATCH v2 9/9] arm64: dts: rockchip: add isp0 node for rk3399 Helen Koike
2020-07-01 12:31   ` [PATCH v2 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399 Heiko Stuebner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox