2018-04-23 18:17 GMT+09:00 Laurent Pinchart <laurent.pinch...@ideasonboard.com>: > Hi Mita-san, > > On Sunday, 22 April 2018 18:56:07 EEST Akinobu Mita wrote: >> This adds a device tree binding documentation for OV7720/OV7725 sensor. >> >> Cc: Jacopo Mondi <jacopo+rene...@jmondi.org> >> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> >> Cc: Hans Verkuil <hans.verk...@cisco.com> >> Cc: Sakari Ailus <sakari.ai...@linux.intel.com> >> Cc: Mauro Carvalho Chehab <mche...@s-opensource.com> >> Cc: Rob Herring <robh...@kernel.org> >> Reviewed-by: Rob Herring <r...@kernel.org> >> Reviewed-by: Jacopo Mondi <jac...@jmondi.org> >> Signed-off-by: Akinobu Mita <akinobu.m...@gmail.com> >> --- >> * v3 >> - Add Reviewed-by: lines >> >> .../devicetree/bindings/media/i2c/ov772x.txt | 42 +++++++++++++++++++ >> MAINTAINERS | 1 + >> 2 files changed, 43 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/media/i2c/ov772x.txt >> >> diff --git a/Documentation/devicetree/bindings/media/i2c/ov772x.txt >> b/Documentation/devicetree/bindings/media/i2c/ov772x.txt new file mode >> 100644 >> index 0000000..b045503 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/media/i2c/ov772x.txt >> @@ -0,0 +1,42 @@ >> +* Omnivision OV7720/OV7725 CMOS sensor >> + >> +The Omnivision OV7720/OV7725 sensor supports multiple resolutions output, >> +such as VGA, QVGA, and any size scaling down from CIF to 40x30. It also can >> +support the YUV422, RGB565/555/444, GRB422 or raw RGB output formats. >> + >> +Required Properties: >> +- compatible: shall be one of >> + "ovti,ov7720" >> + "ovti,ov7725" >> +- clocks: reference to the xclk input clock. >> +- clock-names: shall be "xclk". > > As there's a single clock we could omit clock-names, couldn't we ?
Sounds good. I'll prepare another patch that replaces the clock consumer ID argument of clk_get() from "xclk" to NULL, and remove the above line in this bindings.