All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jacopo Mondi <jacopo+renesas@jmondi.org>
To: robh+dt@kernel.org, devicetree@vger.kernel.org,
	linux-media@vger.kernel.org
Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>,
	mchehab@kernel.org, sakari.ailus@linux.intel.com,
	hverkuil-cisco@xs4all.nl, laurent.pinchart@ideasonboard.com,
	linux-renesas-soc@vger.kernel.org,
	Todor Tomov <todor.too@gmail.com>,
	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Subject: [PATCH 2/8] dt-bindings: media: ov5645: Convert to json-schema
Date: Wed, 15 Jul 2020 16:09:45 +0200	[thread overview]
Message-ID: <20200715140951.90753-3-jacopo+renesas@jmondi.org> (raw)
In-Reply-To: <20200715140951.90753-1-jacopo+renesas@jmondi.org>

Convert the ov5645 bindings document to json-schema and update
the MAINTAINERS file accordingly, as the entry was not documented.

Add myself as maintainer for odd fixes only, as I don't have the
sensor to test with.

Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---

Hello Todor, Manivannan
   I've added in this patch an entry for ov5645 in the MAINTAINERS
file, and I've added myself as maintainers for Odd Fixes only.

As you seem to be the author and committers of this bindings respectively,
would you be interested in maintaining this driver ?

---
 .../devicetree/bindings/media/i2c/ov5645.txt  |  54 --------
 .../devicetree/bindings/media/i2c/ov5645.yaml | 123 ++++++++++++++++++
 MAINTAINERS                                   |   8 ++
 3 files changed, 131 insertions(+), 54 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov5645.txt
 create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5645.yaml

diff --git a/Documentation/devicetree/bindings/media/i2c/ov5645.txt b/Documentation/devicetree/bindings/media/i2c/ov5645.txt
deleted file mode 100644
index 72ad992f77be..000000000000
--- a/Documentation/devicetree/bindings/media/i2c/ov5645.txt
+++ /dev/null
@@ -1,54 +0,0 @@
-* Omnivision 1/4-Inch 5Mp CMOS Digital Image Sensor
-
-The Omnivision OV5645 is a 1/4-Inch CMOS active pixel digital image sensor with
-an active array size of 2592H x 1944V. It is programmable through a serial I2C
-interface.
-
-Required Properties:
-- compatible: Value should be "ovti,ov5645".
-- clocks: Reference to the xclk clock.
-- clock-names: Should be "xclk".
-- clock-frequency: Frequency of the xclk clock.
-- enable-gpios: Chip enable GPIO. Polarity is GPIO_ACTIVE_HIGH. This corresponds
-  to the hardware pin PWDNB which is physically active low.
-- reset-gpios: Chip reset GPIO. Polarity is GPIO_ACTIVE_LOW. This corresponds to
-  the hardware pin RESETB.
-- vdddo-supply: Chip digital IO regulator.
-- vdda-supply: Chip analog regulator.
-- vddd-supply: Chip digital core regulator.
-
-The device node must contain one 'port' child node for its digital output
-video port, in accordance with the video interface bindings defined in
-Documentation/devicetree/bindings/media/video-interfaces.txt.
-
-Example:
-
-	&i2c1 {
-		...
-
-		ov5645: ov5645@3c {
-			compatible = "ovti,ov5645";
-			reg = <0x3c>;
-
-			enable-gpios = <&gpio1 6 GPIO_ACTIVE_HIGH>;
-			reset-gpios = <&gpio5 20 GPIO_ACTIVE_LOW>;
-			pinctrl-names = "default";
-			pinctrl-0 = <&camera_rear_default>;
-
-			clocks = <&clks 200>;
-			clock-names = "xclk";
-			clock-frequency = <24000000>;
-
-			vdddo-supply = <&camera_dovdd_1v8>;
-			vdda-supply = <&camera_avdd_2v8>;
-			vddd-supply = <&camera_dvdd_1v2>;
-
-			port {
-				ov5645_ep: endpoint {
-					clock-lanes = <1>;
-					data-lanes = <0 2>;
-					remote-endpoint = <&csi0_ep>;
-				};
-			};
-		};
-	};
diff --git a/Documentation/devicetree/bindings/media/i2c/ov5645.yaml b/Documentation/devicetree/bindings/media/i2c/ov5645.yaml
new file mode 100644
index 000000000000..ff52d0fffb74
--- /dev/null
+++ b/Documentation/devicetree/bindings/media/i2c/ov5645.yaml
@@ -0,0 +1,123 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/media/i2c/ov5645.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Omnivision OV5645 1/4 Inch 5Mp CMOS Digital Image Sensor
+
+maintainers:
+  - Jacopo Mondi <jacopo@jmondi.org>
+
+description: -|
+  The Omnivision OV5645 is a 1/4-Inch CMOS active pixel digital image sensor
+  with an active array size of 2592H x 1944V. It is programmable through a
+  serial I2C interface.
+
+properties:
+  compatible:
+    const: ovti,ov5645
+
+  reg:
+    description: I2C device address
+    maxItems: 1
+
+  clocks:
+    description: Reference to the xclk clock.
+    maxItems: 1
+
+  clock-names:
+    description: Should be "xclk".
+    maxItems: 1
+
+  clock-frequency:
+    description: Frequency of the xclk clock.
+
+  enable-gpios:
+    description: |
+      Chip enable GPIO. Polarity is GPIO_ACTIVE_HIGH. This corresponds
+      to the hardware pin PWDNB which is physically active low.
+    maxItems: 1
+
+  reset-gpios:
+    description: |
+      Chip reset GPIO. Polarity is GPIO_ACTIVE_LOW. This corresponds to
+      the hardware pin RESETB.
+    maxItems: 1
+
+  vdddo-supply:
+    description: Chip digital IO regulator.
+    maxItems: 1
+
+  vdda-supply:
+    description: Chip analog regulator.
+    maxItems: 1
+
+  vddd-supply:
+    description: Chip digital core regulator.
+    maxItems: 1
+
+  port:
+    type: object
+    description: |
+      The device node must contain one 'port' child node for its digital output
+      video port, in accordance with the video interface bindings defined in
+      Documentation/devicetree/bindings/media/video-interfaces.txt.
+
+    properties:
+      endpoint:
+        type: object
+        properties:
+          remote-endpoint:
+            description: A phandle to the bus receiver's endpoint node.
+
+        required:
+          - remote-endpoint
+
+    additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - clock-names
+  - clock-frequency
+  - vdddo-supply
+  - vdda-supply
+  - vddd-supply
+  - port
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+
+    i2c0 {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        camera@3c {
+            compatible = "ovti,ov5645";
+            reg = <0x3c>;
+
+            enable-gpios = <&gpio1 6 GPIO_ACTIVE_HIGH>;
+            reset-gpios = <&gpio5 20 GPIO_ACTIVE_LOW>;
+
+            clocks = <&clks 200>;
+            clock-names = "xclk";
+            clock-frequency = <24000000>;
+
+            vdddo-supply = <&camera_dovdd_1v8>;
+            vdda-supply = <&camera_avdd_2v8>;
+            vddd-supply = <&camera_dvdd_1v2>;
+
+            port {
+                ov5645_ep: endpoint {
+                    clock-lanes = <1>;
+                    data-lanes = <0 2>;
+                    remote-endpoint = <&csi0_ep>;
+                };
+            };
+        };
+    };
+
+...
diff --git a/MAINTAINERS b/MAINTAINERS
index 0160d7567ed3..bf8902ebbe30 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12607,6 +12607,14 @@ T:	git git://linuxtv.org/media_tree.git
 F:	drivers/media/i2c/ov5640.c
 F:	Documentation/devicetree/bindings/media/i2c/ov5640.yaml

+OMNIVISION OV5645 SENSOR DRIVER
+M:	Jacopo Mondi <jacopo@jmondi.org>
+L:	linux-media@vger.kernel.org
+S:	Odd Fixes
+T:	git git://linuxtv.org/media_tree.git
+F:	drivers/media/i2c/ov5645.c
+F:	Documentation/devicetree/bindings/media/i2c/ov5645.yaml
+
 OMNIVISION OV5647 SENSOR DRIVER
 M:	Jacopo Mondi <jacopo@jmondi.org>
 M:	Dave Stevenson <dave.stevenson@raspberrypi.com>
--
2.27.0


  parent reply	other threads:[~2020-07-15 14:06 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-15 14:09 [PATCH 0/8] dt-bindings: media: i2c: Convert to json-schema Jacopo Mondi
2020-07-15 14:09 ` [PATCH 1/8] dt-bindings: media: ov5640: " Jacopo Mondi
2020-07-15 17:04   ` Steve Longerbeam
2020-07-16  7:06     ` Jacopo Mondi
2020-07-16  1:53   ` Laurent Pinchart
2020-07-16  7:04     ` Jacopo Mondi
2020-07-16 13:08       ` Laurent Pinchart
2020-07-16  7:52     ` Geert Uytterhoeven
2020-07-16 14:51   ` Rob Herring
2020-07-15 14:09 ` Jacopo Mondi [this message]
2020-07-15 15:31   ` [PATCH 2/8] dt-bindings: media: ov5645: " Manivannan Sadhasivam
2020-07-15 14:09 ` [PATCH 3/8] dt-bindings: media: mt9v111: " Jacopo Mondi
2020-07-15 14:09 ` [PATCH 4/8] dt-bindings: media: imx290: " Jacopo Mondi
2020-07-15 14:09 ` [PATCH 5/8] dt-bindings: media: imx274: " Jacopo Mondi
2020-07-15 14:09 ` [PATCH 6/8] dt-bindings: media: imx214: " Jacopo Mondi
2020-07-15 14:09 ` [PATCH 7/8] dt-bindings: media: ov772x: " Jacopo Mondi
2020-07-15 14:09 ` [PATCH 8/8] dt-bindings: media: i2c: Add prefix to yaml bindings Jacopo Mondi
2020-08-17 15:12 ` [PATCH 0/8] dt-bindings: media: i2c: Convert to json-schema Jacopo Mondi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200715140951.90753-3-jacopo+renesas@jmondi.org \
    --to=jacopo+renesas@jmondi.org \
    --cc=devicetree@vger.kernel.org \
    --cc=hverkuil-cisco@xs4all.nl \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=manivannan.sadhasivam@linaro.org \
    --cc=mchehab@kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    --cc=todor.too@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.