Re: [PATCH 2/3] dt-bindings: it6251: add bindings for IT6251 LVDS-to-eDP bridge

[Marek Vasut <marex@xxxxxxx>]

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@xxxxxxx>
>> Cc: devicetree@xxxxxxxxxxxxxxx
>> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
>> Cc: Rob Herring <robh@xxxxxxxxxx>
>> Cc: Sean Cross <xobs@xxxxxxxxxx>
>> ---
>>  .../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@5c {
>> +		compatible = "ite,it6251";
>> +		reg = <0x5c>, <0x5e>;
>> +		reg-names = "bridge", "lvds";
>> +		power-supply = <&reg_display>;
>> +
>> +		ports {
>> +			port@0 {
>> +				bridge_out_edp0: endpoint {
>> +					remote-endpoint = <&panel_in_edp0>;
>> +				};
>> +			};
>> +
>> +			port@1 {
>> +				bridge_in_lvds0: endpoint {
>> +					remote-endpoint = <&lvds0_out>;
>> +				};
>> +			};
>> +		};
>> +	};
>> +
>> -- 
>> 2.9.3
>>


-- 
Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html