Hi Rob, Thank you for the patch. On Wed, May 25, 2022 at 03:56:26PM -0500, Rob Herring wrote: > While 'ddc-i2c-bus' is a common property, it should be in a connector > node rather than the HDMI bridge node as the I2C bus goes to a > connector and not the HDMI block. Drop it from the example. > > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > .../devicetree/bindings/display/bridge/ingenic,jz4780-hdmi.yaml | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/display/bridge/ingenic,jz4780-hdmi.yaml b/Documentation/devicetree/bindings/display/bridge/ingenic,jz4780-hdmi.yaml > index b8219eab4475..89490fdffeb0 100644 > --- a/Documentation/devicetree/bindings/display/bridge/ingenic,jz4780-hdmi.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/ingenic,jz4780-hdmi.yaml > @@ -55,7 +55,6 @@ examples: > compatible = "ingenic,jz4780-dw-hdmi"; > reg = <0x10180000 0x8000>; > reg-io-width = <4>; > - ddc-i2c-bus = <&i2c4>; > interrupt-parent = <&intc>; > interrupts = <3>; > clocks = <&cgu JZ4780_CLK_AHB0>, <&cgu JZ4780_CLK_HDMI>; -- Regards, Laurent Pinchart
