Hi Jacopo,

On Mon, Jun 19, 2017 at 7:04 PM, Jacopo Mondi <jacopo+rene...@jmondi.org> wrote:
> Add camera module to Salvator-X M3W device tree.
> The camera module sits on a i2c-gpio interface and it connected to VIN

is connected

> channel #4 parallel video input port.
>
> Signed-off-by: Jacopo Mondi <jacopo+rene...@jmondi.org>
> ---
>  arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 35 
> ++++++++++++++++++++++
>  1 file changed, 35 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts 
> b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> index ca4eb98..41c94c3 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> @@ -103,6 +103,41 @@
>                 states = <3300000 1
>                           1800000 0>;
>         };
> +
> +       camera_module {
> +               compatible = "i2c-gpio";
> +               gpios = <&gpio1 9 GPIO_ACTIVE_HIGH
> +                        &gpio1 11 GPIO_ACTIVE_HIGH
> +                       >;
> +
> +               i2c-gpio,delay-us = <4>;
> +
> +               #address-cells = <1>;
> +               #size-cells = <0>;
> +
> +               mclk: xclk {
> +                       compatible = "fixed-clock";
> +                       #clock-cells = <1>;
> +                       clock-frequency  = <27000000>;
> +                       clock-output-names = "mclk";

Please the drop clock-output-names for clock with a single output, as it is
deprecated.  The clock name will then be taken from the node name, so you
may want to adjust that.

> +               };

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Reply via email to