On Tue, Feb 13, 2018 at 03:54:56PM +0200, Laurent Pinchart wrote: > 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+renesas@xxxxxxxxx> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> On Tue, Feb 13, 2018 at 03:04:31PM +0100, Niklas Söderlund wrote: > Hi Geert, > > Thanks for your patch. > > On 2018-02-13 14:40:45 +0100, 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+renesas@xxxxxxxxx> > > Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> Thanks, applied.