2022-09-16 13:58:32

by Lad, Prabhakar

[permalink] [raw]
Subject: [PATCH] media: dt-bindings: i2c: ovti,ov02a10: Drop ref to video-interface-devices.yaml

From: Lad Prabhakar <[email protected]>

video-interface-devices.yaml isn't used so just drop it from the
DT binding doc.

Signed-off-by: Lad Prabhakar <[email protected]>
---
Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml | 3 ---
1 file changed, 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
index 63a040944f3d..bd1914cf6c33 100644
--- a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
@@ -17,9 +17,6 @@ description: |-
@ 1600x1200 (UXGA) resolution transferred over a 1-lane MIPI interface. The
sensor output is available via CSI-2 serial data output.

-allOf:
- - $ref: /schemas/media/video-interface-devices.yaml#
-
properties:
compatible:
const: ovti,ov02a10
--
2.25.1


2022-09-17 17:39:49

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] media: dt-bindings: i2c: ovti,ov02a10: Drop ref to video-interface-devices.yaml

On 16/09/2022 14:37, Prabhakar wrote:
> From: Lad Prabhakar <[email protected]>
>
> video-interface-devices.yaml isn't used so just drop it from the
> DT binding doc.

Can you explain what do you mean by "it is not used"? I see it being
used in the code you remove... Since it is there - it is used...

>
> Signed-off-by: Lad Prabhakar <[email protected]>
> ---
> Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> index 63a040944f3d..bd1914cf6c33 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> @@ -17,9 +17,6 @@ description: |-
> @ 1600x1200 (UXGA) resolution transferred over a 1-lane MIPI interface. The
> sensor output is available via CSI-2 serial data output.
>
> -allOf:
> - - $ref: /schemas/media/video-interface-devices.yaml#


Best regards,
Krzysztof

2022-09-17 23:45:11

by Laurent Pinchart

[permalink] [raw]
Subject: Re: [PATCH] media: dt-bindings: i2c: ovti,ov02a10: Drop ref to video-interface-devices.yaml

Hi Prabhakar,

Thank you for the patch.

On Fri, Sep 16, 2022 at 02:37:28PM +0100, Prabhakar wrote:
> From: Lad Prabhakar <[email protected]>
>
> video-interface-devices.yaml isn't used so just drop it from the
> DT binding doc.
>
> Signed-off-by: Lad Prabhakar <[email protected]>
> ---
> Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> index 63a040944f3d..bd1914cf6c33 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> @@ -17,9 +17,6 @@ description: |-
> @ 1600x1200 (UXGA) resolution transferred over a 1-lane MIPI interface. The
> sensor output is available via CSI-2 serial data output.
>
> -allOf:
> - - $ref: /schemas/media/video-interface-devices.yaml#
> -

Same comment as for your ovti,ov5640.yaml patch.

> properties:
> compatible:
> const: ovti,ov02a10

--
Regards,

Laurent Pinchart