Hi Laurent,

On Wed, Oct 18, 2017 at 2:16 PM, Laurent Pinchart
<[email protected]> wrote:
> On Wednesday, 18 October 2017 12:33:47 EEST Biju Das wrote:
>> From: Fabrizio Castro <[email protected]>
>>
>> Although there is a HDMI connector on the q7 carrier board it is not
>> connected to the RZ/G1M SoC. One must use the HDMI connector on the
>> camera daughter board.
>>
>> This patch adds support for this connector.
>>
>> Signed-off-by: Fabrizio Castro <[email protected]>
>> Reviewed-by: Biju Das <[email protected]>

>> --- a/arch/arm/boot/dts/iwg20d-q7-dbcm-ca.dtsi
>> +++ b/arch/arm/boot/dts/iwg20d-q7-dbcm-ca.dtsi

>> @@ -23,7 +48,57 @@
>>       status = "okay";
>>  };
>>
>> +&i2c5 {
>> +     status = "okay";
>> +     clock-frequency = <400000>;
>> +
>> +     cec_clock: cec-clock {
>> +             compatible = "fixed-clock";
>> +             #clock-cells = <0>;
>> +             clock-frequency = <12000000>;
>> +     };
>
> I assume this describes a fixed clock generator that is not I2C-controlled.
> The node should thus not be a child of the i2c5 bus node, but should be moved
> to the root node of the DT.

Oops, that's something we should fix in r8a7791-koelsch.dts, too
(CC Hans)

> The rest looks good to me. With this fixed,
>
> Reviewed-by: Laurent Pinchart <[email protected]>
>
> However I can't find arch/arm/boot/dts/iwg20d-q7-dbcm-ca.dtsi in linux-next.
> Which tree would you like to get this series merged through ?

It's been in next as of next-20171012.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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