Hi Sakari,
On 18-11-27 15:13, Sakari Ailus wrote:
> Hi Marco,
>
> On Tue, Nov 27, 2018 at 11:02:52AM +0100, Marco Felsch wrote:
> > Add the pclk-sample property to the list of optional properties
> > for the mt9m111 camera sensor.
> >
> > Signed-off-by: Marco Felsch <m.fel...@pengutronix.de>
> > Reviewed-by: Rob Herring <r...@kernel.org>
> > ---
> > Documentation/devicetree/bindings/media/i2c/mt9m111.txt | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/media/i2c/mt9m111.txt
> > b/Documentation/devicetree/bindings/media/i2c/mt9m111.txt
> > index a431fb45704b..d0bed6fa901a 100644
> > --- a/Documentation/devicetree/bindings/media/i2c/mt9m111.txt
> > +++ b/Documentation/devicetree/bindings/media/i2c/mt9m111.txt
> > @@ -14,6 +14,10 @@ sub-node for its digital output video port, in
> > accordance with the video
> > interface bindings defined in:
> > Documentation/devicetree/bindings/media/video-interfaces.txt
> >
> > +Optional endpoint properties:
> > +- pclk-sample: For information see ../video-interfaces.txt. The value is
> > set to
> > + 0 if it isn't specified.
>
> How about the data-active, hsync-active and vsync-active properties? Does
> the hardware have a fixed configuration, or can this be set? It appears the
> driver assumes active high for all.
As I understood it correctly the data-active, hsync-active and vsync-active
I/O lines are always ACTIVE_HIGH.
Kind regards,
Marco
> If there's something to change, this should be a separate patch IMO.
>
> > +
> > Example:
> >
> > i2c_master {
> > @@ -26,6 +30,7 @@ Example:
> > port {
> > mt9m111_1: endpoint {
> > remote-endpoint = <&pxa_camera>;
> > + pclk-sample = <1>;
> > };
> > };
> > };
>
> --
> Sakari Ailus
> sakari.ai...@linux.intel.com
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
