media: dt-bindings: ov2685: Correct data-lanes attribute

[PATCH] media: dt-bindings: ov2685: Correct data-lanes attribute

Posted by Luca Weiss 2 years, 8 months ago

When adapting the original doc conversion to support 2 lanes, minItems
should've been added as well since the sensor supports either 1 or 2
lanes. Add minItems to make the validation happy again.

Fixes: 8d561d78aeab ("media: dt-bindings: ov2685: convert to dtschema")
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
 Documentation/devicetree/bindings/media/i2c/ovti,ov2685.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov2685.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov2685.yaml
index 8b389314c352..e2ffe0a9c26b 100644
--- a/Documentation/devicetree/bindings/media/i2c/ovti,ov2685.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov2685.yaml
@@ -49,6 +49,7 @@ properties:
 
         properties:
           data-lanes:
+            minItems: 1
             maxItems: 2
 
         required:

---
base-commit: 4aa1da8d99724f6c0b762b58a71cee7c5e2e109b
change-id: 20230419-ov2685-dtschema-fixup-b04549453978

Best regards,
-- 
Luca Weiss <luca@z3ntu.xyz>

Re: [PATCH] media: dt-bindings: ov2685: Correct data-lanes attribute

Posted by Krzysztof Kozlowski 2 years, 7 months ago

On Wed, 19 Apr 2023 17:58:27 +0200, Luca Weiss wrote:
> When adapting the original doc conversion to support 2 lanes, minItems
> should've been added as well since the sensor supports either 1 or 2
> lanes. Add minItems to make the validation happy again.
> 
> 

Applied, thanks!

[1/1] media: dt-bindings: ov2685: Correct data-lanes attribute
      https://git.kernel.org/krzk/linux-dt/c/048bce15da19e46ce5e866a48338929c76ca4152

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Re: [PATCH] media: dt-bindings: ov2685: Correct data-lanes attribute

Posted by Rob Herring 2 years, 7 months ago

On Wed, 19 Apr 2023 17:58:27 +0200, Luca Weiss wrote:
> When adapting the original doc conversion to support 2 lanes, minItems
> should've been added as well since the sensor supports either 1 or 2
> lanes. Add minItems to make the validation happy again.
> 
> Fixes: 8d561d78aeab ("media: dt-bindings: ov2685: convert to dtschema")
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
>  Documentation/devicetree/bindings/media/i2c/ovti,ov2685.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

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