2021-07-12 09:10:12

by Stefan Riedmüller

[permalink] [raw]
Subject: [PATCH v5 6/6] media: dt-bindings: mt9p031: Add missing required properties

Add missing required clocks and supply regulator properties for the
sensor input clock and vdd, vdd_io and vaa supply regulators.

Signed-off-by: Stefan Riedmueller <[email protected]>
---
.../bindings/media/i2c/aptina,mt9p031.yaml | 22 +++++++++++++++++++
1 file changed, 22 insertions(+)

diff --git a/Documentation/devicetree/bindings/media/i2c/aptina,mt9p031.yaml b/Documentation/devicetree/bindings/media/i2c/aptina,mt9p031.yaml
index ad9a2db73d86..487a3facfcbc 100644
--- a/Documentation/devicetree/bindings/media/i2c/aptina,mt9p031.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/aptina,mt9p031.yaml
@@ -24,6 +24,18 @@ properties:
description: I2C device address
maxItems: 1

+ clocks:
+ maxItems: 1
+
+ vdd-supply:
+ description: Digital supply voltage, 1.8 V
+
+ vdd_io-supply:
+ description: I/O supply voltage, 1.8 or 2.8 V
+
+ vaa-supply:
+ description: Analog supply voltage, 2.8 V
+
reset-gpios:
maxItems: 1
description: Chip reset GPIO
@@ -48,6 +60,10 @@ properties:
required:
- compatible
- reg
+ - clocks
+ - vdd-supply
+ - vdd_io-supply
+ - vaa-supply
- port

additionalProperties: false
@@ -63,6 +79,12 @@ examples:
reg = <0x5d>;
reset-gpios = <&gpio_sensor 0 0>;

+ clocks = <&sensor_clk>;
+
+ vdd-supply = <&reg_vdd>;
+ vdd_io-supply = <&reg_vdd_io>;
+ vaa-supply = <&reg_vaa>;
+
port {
mt9p031_1: endpoint {
input-clock-frequency = <6000000>;
--
2.25.1


2021-07-14 02:22:02

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH v5 6/6] media: dt-bindings: mt9p031: Add missing required properties

On Mon, 12 Jul 2021 10:55:35 +0200, Stefan Riedmueller wrote:
> Add missing required clocks and supply regulator properties for the
> sensor input clock and vdd, vdd_io and vaa supply regulators.
>
> Signed-off-by: Stefan Riedmueller <[email protected]>
> ---
> .../bindings/media/i2c/aptina,mt9p031.yaml | 22 +++++++++++++++++++
> 1 file changed, 22 insertions(+)
>

Reviewed-by: Rob Herring <[email protected]>