On 10/18/2016 06:30 PM, Rob Herring wrote: > On Mon, Oct 17, 2016 at 06:33:28PM +0200, Marek Vasut wrote: >> Add DT bindings for ITE IT6251 LVDS-to-eDP bridge. >> >> Signed-off-by: Marek Vasut <marex at denx.de> >> Cc: devicetree at vger.kernel.org >> Cc: Daniel Vetter <daniel.vetter at ffwll.ch> >> Cc: Rob Herring <robh at kernel.org> >> Cc: Sean Cross <xobs at kosagi.com> >> --- >> .../bindings/display/bridge/ite,it6251.txt | 35 >> ++++++++++++++++++++++ >> 1 file changed, 35 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/display/bridge/ite,it6251.txt >> >> diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6251.txt >> b/Documentation/devicetree/bindings/display/bridge/ite,it6251.txt >> new file mode 100644 >> index 0000000..c5db82c >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/bridge/ite,it6251.txt >> @@ -0,0 +1,35 @@ >> +ITE IT6251 LVDS-to-eDP bridge bindings >> + >> +Required properties: >> +- compatible: Should be "ite,it6251" >> +- reg: i2c address of the bridge, i2c address of the >> LVDS part >> +- reg-names: Should be "bridge", "lvds" >> +- power-supply: Regulator to provide the supply voltage > > The input pin is called "power"?
The chip has about 7 power rails, but the design I have only has an upstream regulator which toggles them all. I wonder, shall we model all the rails ? >> +- video interfaces: Device node can contain video interface port nodes >> + for panel according to [1]. >> + >> +[1]: Documentation/devicetree/bindings/media/video-interfaces.txt >> + >> +Example: >> + >> + it6251 at 5c { >> + compatible = "ite,it6251"; >> + reg = <0x5c>, <0x5e>; >> + reg-names = "bridge", "lvds"; >> + power-supply = <®_display>; >> + >> + ports { >> + port at 0 { >> + bridge_out_edp0: endpoint { >> + remote-endpoint = <&panel_in_edp0>; >> + }; >> + }; >> + >> + port at 1 { >> + bridge_in_lvds0: endpoint { >> + remote-endpoint = <&lvds0_out>; >> + }; >> + }; >> + }; >> + }; >> + >> -- >> 2.9.3 >> -- Best regards, Marek Vasut