2014/10/27 21:13 "Laurent Pinchart" <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>>: > > Hi Simon, > > On Monday 27 October 2014 09:38:29 Simon Horman wrote: > > On Wed, Sep 24, 2014 at 11:04:32PM +0300, Laurent Pinchart wrote: > > > Add DT nodes for the ADV7511 HDMI encoder and its HDMI output connector > > > and configure the DISP pin group that drives the HDMI transmitter DE > > > pin. > > > > > > Signed-off-by: Laurent Pinchart > > > <<a href="mailto:laurent.pinchart%2Brenesas@ideasonboard.com">laurent.pinchart+renesas@ideasonboard.com</a>> > > > > Acked-by: Simon Horman <<a href="mailto:horms%2Brenesas@verge.net.au">horms+renesas@verge.net.au</a>> > > > > Please be careful of any conflicts that may arise if this patch > > doesn't go through my renesas tree. > > I think it would be best if the patch went through your tree. There's no > compile time or runtime dependency on the DU HDMI code, so as soon as the > ADV7511 DT bindings get accepted I plan to ask you to merge this patch. Sure, that sounds like a good plan. > > > --- > > > > > > arch/arm/boot/dts/r8a7791-koelsch.dts | 50 +++++++++++++++++++++++++++++- > > > 1 file changed, 49 insertions(+), 1 deletion(-) > > > > > > diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts > > > b/arch/arm/boot/dts/r8a7791-koelsch.dts index 9051e01..b7d0c41 100644 > > > --- a/arch/arm/boot/dts/r8a7791-koelsch.dts > > > +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts > > > @@ -209,6 +209,17 @@ > > > states = <3300000 1 > > > 1800000 0>; > > > }; > > > + > > > + hdmi-out { > > > + compatible = "hdmi-connector"; > > > + type = "a"; > > > + > > > + port { > > > + hdmi_con: endpoint { > > > + remote-endpoint = <&adv7511_out>; > > > + }; > > > + }; > > > + }; > > > }; > > > > > > &du { > > > @@ -217,6 +228,11 @@ > > > status = "okay"; > > > > > > ports { > > > + port@0 { > > > + endpoint { > > > + remote-endpoint = <&adv7511_in>; > > > + }; > > > + }; > > > port@1 { > > > lvds_connector: endpoint { > > > }; > > > @@ -235,7 +251,7 @@ > > > }; > > > > > > du_pins: du { > > > - renesas,groups = "du_rgb666", "du_sync", "du_clk_out_0"; > > > + renesas,groups = "du_rgb666", "du_sync", "du_disp", > > > "du_clk_out_0"; > > > renesas,function = "du"; > > > }; > > > > > > @@ -439,6 +455,38 @@ > > > }; > > > }; > > > > > > + hdmi@39 { > > > + compatible = "adi,adv7511w"; > > > + reg = <0x39>; > > > + interrupt-parent = <&gpio3>; > > > + interrupts = <29 IRQ_TYPE_EDGE_FALLING>; > > > + > > > + adi,input-depth = <8>; > > > + adi,input-colorspace = "rgb"; > > > + adi,input-clock = "1x"; > > > + adi,input-style = <1>; > > > + adi,input-justification = "evenly"; > > > + > > > + ports { > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + > > > + port@0 { > > > + reg = <0>; > > > + adv7511_in: endpoint { > > > + remote-endpoint = <&du_out_rgb>; > > > + }; > > > + }; > > > + > > > + port@1 { > > > + reg = <1>; > > > + adv7511_out: endpoint { > > > + remote-endpoint = <&hdmi_con>; > > > + }; > > > + }; > > > + }; > > > + }; > > > + > > > eeprom@50 { > > > compatible = "renesas,24c02"; > > > reg = <0x50>; > > -- > Regards, > > Laurent Pinchart >