* [PATCH v4 0/3] dt-bindings: media: ov772x: Convert to json-schema
@ 2020-09-03 13:10 Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 1/3] " Jacopo Mondi
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Jacopo Mondi @ 2020-09-03 13:10 UTC (permalink / raw)
To: robh+dt, devicetree, linux-media, Lad, Prabhakar
Cc: Jacopo Mondi, mchehab, sakari.ailus, hverkuil-cisco,
laurent.pinchart, linux-renesas-soc
Convert the ov722x bindings to json-schema and add endpoint properties and
bus-type.
Compared to v3 in 1/3 I have not documented 'endpoint'
Then 2/3 introduces the 'bus-type' endpoint properties, so I had to document
'endpoint' and list 'remote-endpoint' as required (which makes me think I
could have kept it in 1/3, but for sake of consistency...)
3/3 adds endpoint properties, whose presence is conditional to the specified
bus-type.
Thanks
j
Jacopo Mondi (3):
dt-bindings: media: ov772x: Convert to json-schema
dt-bindings: media: ov772x: Make bus-type mandatory
dt-bindings: media: ov772x: Document endpoint props
.../devicetree/bindings/media/i2c/ov772x.txt | 40 -----
.../bindings/media/i2c/ovti,ov772x.yaml | 138 ++++++++++++++++++
MAINTAINERS | 2 +-
3 files changed, 139 insertions(+), 41 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov772x.txt
create mode 100644 Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
--
2.28.0
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v4 1/3] dt-bindings: media: ov772x: Convert to json-schema
2020-09-03 13:10 [PATCH v4 0/3] dt-bindings: media: ov772x: Convert to json-schema Jacopo Mondi
@ 2020-09-03 13:10 ` Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 3/3] dt-bindings: media: ov772x: Document endpoint props Jacopo Mondi
2 siblings, 0 replies; 8+ messages in thread
From: Jacopo Mondi @ 2020-09-03 13:10 UTC (permalink / raw)
To: robh+dt, devicetree, linux-media, Lad, Prabhakar
Cc: Jacopo Mondi, mchehab, sakari.ailus, hverkuil-cisco,
laurent.pinchart, linux-renesas-soc, Lad Prabhakar
Convert the ov772x binding document to json-schema and update
the MAINTAINERS file accordingly.
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
.../devicetree/bindings/media/i2c/ov772x.txt | 40 ----------
.../bindings/media/i2c/ovti,ov772x.yaml | 74 +++++++++++++++++++
MAINTAINERS | 2 +-
3 files changed, 75 insertions(+), 41 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov772x.txt
create mode 100644 Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
diff --git a/Documentation/devicetree/bindings/media/i2c/ov772x.txt b/Documentation/devicetree/bindings/media/i2c/ov772x.txt
deleted file mode 100644
index 0b3ede5b8e6a..000000000000
--- a/Documentation/devicetree/bindings/media/i2c/ov772x.txt
+++ /dev/null
@@ -1,40 +0,0 @@
-* Omnivision OV7720/OV7725 CMOS sensor
-
-The Omnivision OV7720/OV7725 sensor supports multiple resolutions output,
-such as VGA, QVGA, and any size scaling down from CIF to 40x30. It also can
-support the YUV422, RGB565/555/444, GRB422 or raw RGB output formats.
-
-Required Properties:
-- compatible: shall be one of
- "ovti,ov7720"
- "ovti,ov7725"
-- clocks: reference to the xclk input clock.
-
-Optional Properties:
-- reset-gpios: reference to the GPIO connected to the RSTB pin which is
- active low, if any.
-- powerdown-gpios: reference to the GPIO connected to the PWDN pin which is
- active high, if any.
-
-The device node shall contain one 'port' child node with one child 'endpoint'
-subnode for its digital output video port, in accordance with the video
-interface bindings defined in Documentation/devicetree/bindings/media/
-video-interfaces.txt.
-
-Example:
-
-&i2c0 {
- ov772x: camera@21 {
- compatible = "ovti,ov7725";
- reg = <0x21>;
- reset-gpios = <&axi_gpio_0 0 GPIO_ACTIVE_LOW>;
- powerdown-gpios = <&axi_gpio_0 1 GPIO_ACTIVE_LOW>;
- clocks = <&xclk>;
-
- port {
- ov772x_0: endpoint {
- remote-endpoint = <&vcap1_in0>;
- };
- };
- };
-};
diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
new file mode 100644
index 000000000000..e7e2d31fcc23
--- /dev/null
+++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
@@ -0,0 +1,74 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/media/i2c/ovti,ov772x.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Omnivision OV7720/OV7725 CMOS sensor
+
+maintainers:
+ - Jacopo Mondi <jacopo@jmondi.org>
+
+description: |
+ The Omnivision OV7720/OV7725 sensor supports multiple output resolutions,
+ such as VGA, QVGA, and any size scaling down from CIF to 40x30. It also
+ supports YUV422, RGB565/555/444, GRB422 or raw RGB output formats.
+
+properties:
+ compatible:
+ enum:
+ - ovti,ov7720
+ - ovti,ov7725
+
+ reg:
+ maxItems: 1
+
+ clocks:
+ maxItems: 1
+
+ reset-gpios:
+ description: |
+ Reference to the GPIO connected to the RSTB pin which is active low.
+ maxItems: 1
+
+ powerdown-gpios:
+ description: |
+ Reference to the GPIO connected to the PWDN pin which is active high.
+ maxItems: 1
+
+ port:
+ type: object
+ description: |
+ Video output port. See ../video-interfaces.txt.
+
+required:
+ - compatible
+ - reg
+ - clocks
+ - port
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+
+ i2c0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ ov772x: camera@21 {
+ compatible = "ovti,ov7725";
+ reg = <0x21>;
+ reset-gpios = <&axi_gpio_0 0 GPIO_ACTIVE_LOW>;
+ powerdown-gpios = <&axi_gpio_0 1 GPIO_ACTIVE_LOW>;
+ clocks = <&xclk>;
+
+ port {
+ ov772x_0: endpoint {
+ remote-endpoint = <&vcap1_in0>;
+ };
+ };
+ };
+ };
+
+...
diff --git a/MAINTAINERS b/MAINTAINERS
index b706598d764b..0d83a1304b2e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12800,7 +12800,7 @@ M: Jacopo Mondi <jacopo@jmondi.org>
L: linux-media@vger.kernel.org
S: Odd fixes
T: git git://linuxtv.org/media_tree.git
-F: Documentation/devicetree/bindings/media/i2c/ov772x.txt
+F: Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
F: drivers/media/i2c/ov772x.c
F: include/media/i2c/ov772x.h
--
2.28.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory
2020-09-03 13:10 [PATCH v4 0/3] dt-bindings: media: ov772x: Convert to json-schema Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 1/3] " Jacopo Mondi
@ 2020-09-03 13:10 ` Jacopo Mondi
2020-09-04 8:05 ` Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 3/3] dt-bindings: media: ov772x: Document endpoint props Jacopo Mondi
2 siblings, 1 reply; 8+ messages in thread
From: Jacopo Mondi @ 2020-09-03 13:10 UTC (permalink / raw)
To: robh+dt, devicetree, linux-media, Lad, Prabhakar
Cc: Jacopo Mondi, mchehab, sakari.ailus, hverkuil-cisco,
laurent.pinchart, linux-renesas-soc, Rob Herring, Lad Prabhakar
In order to establish required properties based on the selected
bus type, make the 'bus-type' property mandatory. As this change
documents an endpoint property, also document the 'remote-endpoint'
one now that the 'endpoint' schema has been expanded.
Binary compatibility with existing DTB is kept as the driver does not
enforce the property to be present, and shall fall-back to default
parallel bus configuration, which was the only supported bus type, if
the property is not specified.
Reviewed-by: Rob Herring <robh@kernel.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
.../bindings/media/i2c/ovti,ov772x.yaml | 20 +++++++++++++++++++
1 file changed, 20 insertions(+)
diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
index e7e2d31fcc23..406e9cd463a2 100644
--- a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
@@ -41,6 +41,25 @@ properties:
description: |
Video output port. See ../video-interfaces.txt.
+ properties:
+ endpoint:
+ type: object
+
+ properties:
+ remote-endpoint:
+ description: See ../video-interfaces.txt
+
+ bus-type:
+ enum: [5, 6]
+
+ required:
+ - remote-endpoint
+ - bus-type
+
+ additionalProperties: false
+
+ additionalProperties: false
+
required:
- compatible
- reg
@@ -65,6 +84,7 @@ examples:
port {
ov772x_0: endpoint {
+ bus-type = <5>;
remote-endpoint = <&vcap1_in0>;
};
};
--
2.28.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v4 3/3] dt-bindings: media: ov772x: Document endpoint props
2020-09-03 13:10 [PATCH v4 0/3] dt-bindings: media: ov772x: Convert to json-schema Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 1/3] " Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory Jacopo Mondi
@ 2020-09-03 13:10 ` Jacopo Mondi
2020-09-03 21:33 ` Laurent Pinchart
2 siblings, 1 reply; 8+ messages in thread
From: Jacopo Mondi @ 2020-09-03 13:10 UTC (permalink / raw)
To: robh+dt, devicetree, linux-media, Lad, Prabhakar
Cc: Jacopo Mondi, mchehab, sakari.ailus, hverkuil-cisco,
laurent.pinchart, linux-renesas-soc, Lad Prabhakar
Document endpoint properties for the parallel bus type and
add them to the example.
Specify a few constraints:
- If the bus type is BT.656 no hsync or vsycn polarities can be
specified.
- If the bus width is 10 bits, not data-shift can be applied.
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
.../bindings/media/i2c/ovti,ov772x.yaml | 44 +++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
index 406e9cd463a2..c596cbd1e92d 100644
--- a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
@@ -52,6 +52,45 @@ properties:
bus-type:
enum: [5, 6]
+ bus-width:
+ enum: [8, 10]
+ default: 10
+
+ data-shift:
+ enum: [0, 2]
+ default: 0
+
+ hsync-active:
+ enum: [0, 1]
+ default: 1
+
+ vsync-active:
+ enum: [0, 1]
+ default: 1
+
+ pclk-sample:
+ enum: [0, 1]
+ default: 1
+
+ allOf:
+ - if:
+ properties:
+ bus-type:
+ const: 6
+ then:
+ properties:
+ hsync-active: false
+ vsync-active: false
+
+ - if:
+ properties:
+ bus-width:
+ const: 10
+ then:
+ properties:
+ data-shift:
+ const: 0
+
required:
- remote-endpoint
- bus-type
@@ -85,6 +124,11 @@ examples:
port {
ov772x_0: endpoint {
bus-type = <5>;
+ vsync-active = <0>;
+ hsync-active = <0>;
+ pclk-sample = <0>;
+ bus-width = <8>;
+ data-shift = <0>;
remote-endpoint = <&vcap1_in0>;
};
};
--
2.28.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v4 3/3] dt-bindings: media: ov772x: Document endpoint props
2020-09-03 13:10 ` [PATCH v4 3/3] dt-bindings: media: ov772x: Document endpoint props Jacopo Mondi
@ 2020-09-03 21:33 ` Laurent Pinchart
0 siblings, 0 replies; 8+ messages in thread
From: Laurent Pinchart @ 2020-09-03 21:33 UTC (permalink / raw)
To: Jacopo Mondi
Cc: robh+dt, devicetree, linux-media, Lad, Prabhakar, mchehab,
sakari.ailus, hverkuil-cisco, linux-renesas-soc, Lad Prabhakar
Hi Jacopo,
Thank you for the patch.
On Thu, Sep 03, 2020 at 03:10:29PM +0200, Jacopo Mondi wrote:
> Document endpoint properties for the parallel bus type and
> add them to the example.
>
> Specify a few constraints:
> - If the bus type is BT.656 no hsync or vsycn polarities can be
s/vsycn/vsync/
> specified.
> - If the bus width is 10 bits, not data-shift can be applied.
>
> Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
> ---
> .../bindings/media/i2c/ovti,ov772x.yaml | 44 +++++++++++++++++++
> 1 file changed, 44 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> index 406e9cd463a2..c596cbd1e92d 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> @@ -52,6 +52,45 @@ properties:
> bus-type:
> enum: [5, 6]
>
> + bus-width:
> + enum: [8, 10]
> + default: 10
> +
> + data-shift:
> + enum: [0, 2]
> + default: 0
> +
> + hsync-active:
> + enum: [0, 1]
> + default: 1
> +
> + vsync-active:
> + enum: [0, 1]
> + default: 1
> +
> + pclk-sample:
> + enum: [0, 1]
> + default: 1
> +
> + allOf:
> + - if:
> + properties:
> + bus-type:
> + const: 6
> + then:
> + properties:
> + hsync-active: false
> + vsync-active: false
> +
This should eventually end up in video-interfaces.yaml. I don't mind
keeping it here in the meantime. Same for the hsync-active, vsync-active
and pclk-sample enum, they should end up in video-interfaces.yaml.
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> + - if:
> + properties:
> + bus-width:
> + const: 10
> + then:
> + properties:
> + data-shift:
> + const: 0
> +
> required:
> - remote-endpoint
> - bus-type
> @@ -85,6 +124,11 @@ examples:
> port {
> ov772x_0: endpoint {
> bus-type = <5>;
> + vsync-active = <0>;
> + hsync-active = <0>;
> + pclk-sample = <0>;
> + bus-width = <8>;
> + data-shift = <0>;
> remote-endpoint = <&vcap1_in0>;
> };
> };
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory
2020-09-03 13:10 ` [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory Jacopo Mondi
@ 2020-09-04 8:05 ` Jacopo Mondi
2020-09-04 15:06 ` Laurent Pinchart
0 siblings, 1 reply; 8+ messages in thread
From: Jacopo Mondi @ 2020-09-04 8:05 UTC (permalink / raw)
To: Laurent Pinchart
Cc: robh+dt, devicetree, linux-media, Lad, Prabhakar, mchehab,
sakari.ailus, hverkuil-cisco, laurent.pinchart,
linux-renesas-soc, Rob Herring, Lad Prabhakar
Hi Laurent,
On Thu, Sep 03, 2020 at 03:10:28PM +0200, Jacopo Mondi wrote:
> In order to establish required properties based on the selected
> bus type, make the 'bus-type' property mandatory. As this change
> documents an endpoint property, also document the 'remote-endpoint'
> one now that the 'endpoint' schema has been expanded.
>
> Binary compatibility with existing DTB is kept as the driver does not
> enforce the property to be present, and shall fall-back to default
> parallel bus configuration, which was the only supported bus type, if
> the property is not specified.
>
> Reviewed-by: Rob Herring <robh@kernel.org>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
> ---
> .../bindings/media/i2c/ovti,ov772x.yaml | 20 +++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> index e7e2d31fcc23..406e9cd463a2 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> @@ -41,6 +41,25 @@ properties:
> description: |
> Video output port. See ../video-interfaces.txt.
>
> + properties:
> + endpoint:
> + type: object
> +
> + properties:
> + remote-endpoint:
> + description: See ../video-interfaces.txt
So I guess the same as you suggested for imx214 applies here (drop
remote-endpoint)
> +
> + bus-type:
> + enum: [5, 6]
> +
> + required:
> + - remote-endpoint
> + - bus-type
> +
> + additionalProperties: false
> +
> + additionalProperties: false
> +
> required:
> - compatible
> - reg
> @@ -65,6 +84,7 @@ examples:
>
> port {
> ov772x_0: endpoint {
> + bus-type = <5>;
> remote-endpoint = <&vcap1_in0>;
> };
> };
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory
2020-09-04 8:05 ` Jacopo Mondi
@ 2020-09-04 15:06 ` Laurent Pinchart
2020-09-04 15:16 ` Jacopo Mondi
0 siblings, 1 reply; 8+ messages in thread
From: Laurent Pinchart @ 2020-09-04 15:06 UTC (permalink / raw)
To: Jacopo Mondi
Cc: robh+dt, devicetree, linux-media, Lad, Prabhakar, mchehab,
sakari.ailus, hverkuil-cisco, linux-renesas-soc, Rob Herring,
Lad Prabhakar
Hi Jacopo,
On Fri, Sep 04, 2020 at 10:05:35AM +0200, Jacopo Mondi wrote:
> On Thu, Sep 03, 2020 at 03:10:28PM +0200, Jacopo Mondi wrote:
> > In order to establish required properties based on the selected
> > bus type, make the 'bus-type' property mandatory. As this change
> > documents an endpoint property, also document the 'remote-endpoint'
> > one now that the 'endpoint' schema has been expanded.
> >
> > Binary compatibility with existing DTB is kept as the driver does not
> > enforce the property to be present, and shall fall-back to default
> > parallel bus configuration, which was the only supported bus type, if
> > the property is not specified.
> >
> > Reviewed-by: Rob Herring <robh@kernel.org>
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
> > ---
> > .../bindings/media/i2c/ovti,ov772x.yaml | 20 +++++++++++++++++++
> > 1 file changed, 20 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> > index e7e2d31fcc23..406e9cd463a2 100644
> > --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> > +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> > @@ -41,6 +41,25 @@ properties:
> > description: |
> > Video output port. See ../video-interfaces.txt.
> >
> > + properties:
> > + endpoint:
> > + type: object
> > +
> > + properties:
> > + remote-endpoint:
> > + description: See ../video-interfaces.txt
>
> So I guess the same as you suggested for imx214 applies here (drop
> remote-endpoint)
Correct. And you should then replace additionalProperties: false with
unevaluatedProperties: false.
> > +
> > + bus-type:
> > + enum: [5, 6]
> > +
> > + required:
> > + - remote-endpoint
> > + - bus-type
> > +
> > + additionalProperties: false
> > +
> > + additionalProperties: false
> > +
> > required:
> > - compatible
> > - reg
> > @@ -65,6 +84,7 @@ examples:
> >
> > port {
> > ov772x_0: endpoint {
> > + bus-type = <5>;
> > remote-endpoint = <&vcap1_in0>;
> > };
> > };
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory
2020-09-04 15:06 ` Laurent Pinchart
@ 2020-09-04 15:16 ` Jacopo Mondi
0 siblings, 0 replies; 8+ messages in thread
From: Jacopo Mondi @ 2020-09-04 15:16 UTC (permalink / raw)
To: Laurent Pinchart
Cc: robh+dt, devicetree, linux-media, Lad, Prabhakar, mchehab,
sakari.ailus, hverkuil-cisco, linux-renesas-soc, Rob Herring,
Lad Prabhakar
Hi Laurent,
On Fri, Sep 04, 2020 at 06:06:50PM +0300, Laurent Pinchart wrote:
> Hi Jacopo,
>
> On Fri, Sep 04, 2020 at 10:05:35AM +0200, Jacopo Mondi wrote:
> > On Thu, Sep 03, 2020 at 03:10:28PM +0200, Jacopo Mondi wrote:
> > > In order to establish required properties based on the selected
> > > bus type, make the 'bus-type' property mandatory. As this change
> > > documents an endpoint property, also document the 'remote-endpoint'
> > > one now that the 'endpoint' schema has been expanded.
> > >
> > > Binary compatibility with existing DTB is kept as the driver does not
> > > enforce the property to be present, and shall fall-back to default
> > > parallel bus configuration, which was the only supported bus type, if
> > > the property is not specified.
> > >
> > > Reviewed-by: Rob Herring <robh@kernel.org>
> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
> > > ---
> > > .../bindings/media/i2c/ovti,ov772x.yaml | 20 +++++++++++++++++++
> > > 1 file changed, 20 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> > > index e7e2d31fcc23..406e9cd463a2 100644
> > > --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> > > +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov772x.yaml
> > > @@ -41,6 +41,25 @@ properties:
> > > description: |
> > > Video output port. See ../video-interfaces.txt.
> > >
> > > + properties:
> > > + endpoint:
> > > + type: object
> > > +
> > > + properties:
> > > + remote-endpoint:
> > > + description: See ../video-interfaces.txt
> >
> > So I guess the same as you suggested for imx214 applies here (drop
> > remote-endpoint)
>
> Correct. And you should then replace additionalProperties: false with
> unevaluatedProperties: false.
>
Ah thanks, I would have just dropped additionalProperties, I didn't
know about unevaluatedProperties.
Thanks
j
> > > +
> > > + bus-type:
> > > + enum: [5, 6]
> > > +
> > > + required:
> > > + - remote-endpoint
> > > + - bus-type
> > > +
> > > + additionalProperties: false
> > > +
> > > + additionalProperties: false
> > > +
> > > required:
> > > - compatible
> > > - reg
> > > @@ -65,6 +84,7 @@ examples:
> > >
> > > port {
> > > ov772x_0: endpoint {
> > > + bus-type = <5>;
> > > remote-endpoint = <&vcap1_in0>;
> > > };
> > > };
>
> --
> Regards,
>
> Laurent Pinchart
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-09-04 15:12 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-03 13:10 [PATCH v4 0/3] dt-bindings: media: ov772x: Convert to json-schema Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 1/3] " Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 2/3] dt-bindings: media: ov772x: Make bus-type mandatory Jacopo Mondi
2020-09-04 8:05 ` Jacopo Mondi
2020-09-04 15:06 ` Laurent Pinchart
2020-09-04 15:16 ` Jacopo Mondi
2020-09-03 13:10 ` [PATCH v4 3/3] dt-bindings: media: ov772x: Document endpoint props Jacopo Mondi
2020-09-03 21:33 ` Laurent Pinchart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).