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 = <&reg_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

Reply via email to