Hi Geert,

Thank you for the patch.

On Tuesday, 13 February 2018 15:40:45 EET Geert Uytterhoeven wrote:
> cec-clock is a fixed clock generator that is not controlled by i2c-12
> and thus should not be a child of the i2c-12 bus node. Rather, it should
> be a child of the root node of the DT.
> 
> Fixes: c5aa87977626e778 ("ARM: dts: lager: Add CEC clock for HDMI
> transmitter")
> Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be>

Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>

> ---
>  arch/arm/boot/dts/r8a7790-lager.dts | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> b/arch/arm/boot/dts/r8a7790-lager.dts index
> b42579ea7a50d47c..3c66366f7c550034 100644
> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -247,6 +247,12 @@
>               };
>       };
> 
> +     cec_clock: cec-clock {
> +             compatible = "fixed-clock";
> +             #clock-cells = <0>;
> +             clock-frequency = <12000000>;
> +     };
> +
>       hdmi-out {
>               compatible = "hdmi-connector";
>               type = "a";
> @@ -352,12 +358,6 @@
>                       };
>               };
> 
> -             cec_clock: cec-clock {
> -                     compatible = "fixed-clock";
> -                     #clock-cells = <0>;
> -                     clock-frequency = <12000000>;
> -             };
> -
>               hdmi@39 {
>                       compatible = "adi,adv7511w";
>                       reg = <0x39>;

-- 
Regards,

Laurent Pinchart

Reply via email to