Since the GPIO 3 controls both reset and powerdown, we rename it to reset. Fix the polarity of this, as it is a low active signal and not an high active. As at it, rename the mipi endpoint in the example to be coherent with the a real case that will be introduce for imx7 media driver. Signed-off-by: Rui Miguel Silva <rui.silva@xxxxxxxxxx> --- Documentation/devicetree/bindings/media/i2c/ov2680.txt | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/media/i2c/ov2680.txt b/Documentation/devicetree/bindings/media/i2c/ov2680.txt index e5d8f130309d..11e925ed9dad 100644 --- a/Documentation/devicetree/bindings/media/i2c/ov2680.txt +++ b/Documentation/devicetree/bindings/media/i2c/ov2680.txt @@ -9,8 +9,8 @@ Required Properties: - AVDD-supply: Analog voltage supply. Optional Properties: -- powerdown-gpios: reference to the GPIO connected to the powerdown pin, - if any. This is an active high signal to the OV2680. +- reset-gpios: reference to the GPIO connected to the powerdown/reset pin, + if any. This is an active low signal to the OV2680. The device node must contain one 'port' child node for its digital output video port, and this port must have a single endpoint in accordance with @@ -30,14 +30,14 @@ Example: reg = <0x36>; clocks = <&osc>; clock-names = "xvclk"; - powerdown-gpios = <&gpio1 3 GPIO_ACTIVE_HIGH>; + reset-gpios = <&gpio1 3 GPIO_ACTIVE_LOW>; DOVDD-supply = <&sw2_reg>; DVDD-supply = <&sw2_reg>; AVDD-supply = <®_peri_3p15v>; port { - ov2680_mipi_ep: endpoint { - remote-endpoint = <&mipi_sensor_ep>; + ov2680_to_mipi: endpoint { + remote-endpoint = <&mipi_from_sensor>; clock-lanes = <0>; data-lanes = <1>; }; -- 2.17.0 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html