On Sun, Dec 23, 2018 at 02:12:44AM +0900, Akinobu Mita wrote:
> Add device tree binding documentation for the MT9M001 CMOS image sensor.
> 
> Cc: Rob Herring <r...@kernel.org>
> Cc: Guennadi Liakhovetski <g.liakhovet...@gmx.de>
> Cc: Sakari Ailus <sakari.ai...@linux.intel.com>
> Cc: Mauro Carvalho Chehab <mche...@s-opensource.com>
> Signed-off-by: Akinobu Mita <akinobu.m...@gmail.com>
> ---
>  .../devicetree/bindings/media/i2c/mt9m001.txt      | 37 
> ++++++++++++++++++++++
>  1 file changed, 37 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/media/i2c/mt9m001.txt
> 
> diff --git a/Documentation/devicetree/bindings/media/i2c/mt9m001.txt 
> b/Documentation/devicetree/bindings/media/i2c/mt9m001.txt
> new file mode 100644
> index 0000000..794b787
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/media/i2c/mt9m001.txt
> @@ -0,0 +1,37 @@
> +MT9M001: 1/2-Inch Megapixel Digital Image Sensor
> +
> +The MT9M001 is an SXGA-format with a 1/2-inch CMOS active-pixel digital
> +image sensor. It is programmable through a simple two-wire serial
> +interface.

I2C?

> +
> +Required Properties:
> +
> +- compatible: shall be "onnn,mt9m001".
> +- clocks: reference to the master clock into sensor
> +
> +Optional Properties:
> +
> +- reset-gpios: GPIO handle which is connected to the reset pin of the chip.
> +  Active low.
> +- standby-gpios: GPIO handle which is connected to the standby pin of the 
> chip.
> +  Active high.
> +
> +For further reading on port node refer to
> +Documentation/devicetree/bindings/media/video-interfaces.txt.

You still need to state how many ports/endpoints and what they are.

> +
> +Example:
> +
> +     &i2c1 {
> +             mt9m001@5d {

camera-sensor@5d

> +                     compatible = "onnn,mt9m001";
> +                     reg = <0x5d>;
> +                     reset-gpios = <&gpio0 0 GPIO_ACTIVE_LOW>;
> +                     standby-gpios = <&gpio0 1 GPIO_ACTIVE_HIGH>;
> +                     clocks = <&camera_clk>;
> +                     port {
> +                             mt9m001_out: endpoint {
> +                                     remote-endpoint = <&vcap_in>;
> +                             };
> +                     };
> +             };
> +     };
> -- 
> 2.7.4
> 

Reply via email to