Re: [PATCH 1/2] media: ov2680: dt: Add bindings for OV2680

From: Rui Miguel Silva
Date: Fri Feb 23 2018 - 09:56:50 EST


Hi Sakari,
Thanks for the review.

On Thu 22 Feb 2018 at 10:59, Sakari Ailus wrote:
Hi Rui,

Thanks for the patchset.

Could you use "dt: bindings: " prefix in the subject?

Sure, no problem.


On Thu, Feb 22, 2018 at 10:23:37AM +0000, Rui Miguel Silva wrote:
Add device tree binding documentation for the OV5640 camera sensor.

CC: devicetree@xxxxxxxxxxxxxxx
Signed-off-by: Rui Miguel Silva <rui.silva@xxxxxxxxxx>
---
.../devicetree/bindings/media/i2c/ov2680.txt | 34 ++++++++++++++++++++++
1 file changed, 34 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/i2c/ov2680.txt

diff --git a/Documentation/devicetree/bindings/media/i2c/ov2680.txt b/Documentation/devicetree/bindings/media/i2c/ov2680.txt
new file mode 100644
index 000000000000..f9dc63ce5044
--- /dev/null
+++ b/Documentation/devicetree/bindings/media/i2c/ov2680.txt
@@ -0,0 +1,34 @@
+* Omnivision OV2680 MIPI CSI-2 sensor
+
+Required Properties:
+- compatible: should be "ovti,ov2680"
+- clocks: reference to the xvclk input clock.
+- clock-names: should be "xvclk".
+
+Optional Properties:
+- powerdown-gpios: reference to the GPIO connected to the powerdown pin,
+ if any. This is an active high signal to the OV2680.
+
+The device node must contain one 'port' child node for its digital output

Please add that the port contains a single endpoint as well.

Ack.


+video port, in accordance with the video interface bindings defined in
+Documentation/devicetree/bindings/media/video-interfaces.txt.

Please list required and optional endpoint properties as well.

OK.

---
Cheers,
Rui


+
+Example:
+
+&i2c2 {
+ ov2680: camera-sensor@36 {
+ compatible = "ovti,ov2680";
+ reg = <0x36>;
+ clocks = <&osc>;
+ clock-names = "xvclk";
+ powerdown-gpios = <&gpio1 3 GPIO_ACTIVE_HIGH>;
+
+ port {
+ ov2680_mipi_ep: endpoint {
+ remote-endpoint = <&mipi_sensor_ep>;
+ clock-lanes = <0>;
+ data-lanes = <1>;
+ };
+ };
+ };
+};