Hi Hans, On 11/02/2021 12:37, Hans Verkuil wrote: > Add cec clock to the dra7 and omap5 device trees. > > Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> > --- > arch/arm/boot/dts/dra7.dtsi | 5 +++-- > arch/arm/boot/dts/omap5.dtsi | 5 +++-- > 2 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi > index ce1194744f84..efe579ddb324 100644 > --- a/arch/arm/boot/dts/dra7.dtsi > +++ b/arch/arm/boot/dts/dra7.dtsi > @@ -879,8 +879,9 @@ hdmi: encoder@0 { > interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>; > status = "disabled"; > clocks = <&dss_clkctrl DRA7_DSS_DSS_CORE_CLKCTRL 9>, > - <&dss_clkctrl DRA7_DSS_DSS_CORE_CLKCTRL 10>; > - clock-names = "fck", "sys_clk"; > + <&dss_clkctrl DRA7_DSS_DSS_CORE_CLKCTRL 10>, > + <&dss_clkctrl DRA7_DSS_DSS_CORE_CLKCTRL 11>; > + clock-names = "fck", "sys_clk", "cec"; > dmas = <&sdma_xbar 76>; > dma-names = "audio_tx"; > }; > diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi > index 5f1a8bd13880..2bb1000aeae9 100644 > --- a/arch/arm/boot/dts/omap5.dtsi > +++ b/arch/arm/boot/dts/omap5.dtsi > @@ -580,8 +580,9 @@ hdmi: encoder@0 { > interrupts = <GIC_SPI 101 IRQ_TYPE_LEVEL_HIGH>; > status = "disabled"; > clocks = <&dss_clkctrl OMAP5_DSS_CORE_CLKCTRL 9>, > - <&dss_clkctrl OMAP5_DSS_CORE_CLKCTRL 10>; > - clock-names = "fck", "sys_clk"; > + <&dss_clkctrl OMAP5_DSS_CORE_CLKCTRL 10>, > + <&dss_clkctrl OMAP5_DSS_CORE_CLKCTRL 11>; > + clock-names = "fck", "sys_clk", "cec"; > dmas = <&sdma 76>; > dma-names = "audio_tx"; > }; > If I read right, the cec clock is mandatory after adding the CEC support in the previous patch. So please move this patch before the patch 4, and also update the bindings in ti,omap5-dss.txt to list the cec clock. Tomi