diff mbox

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

Message ID 20161017163329.11821-2-marex@denx.de (mailing list archive)
State New, archived
Headers show

Commit Message

Marek Vasut Oct. 17, 2016, 4:33 p.m. UTC
Add DT bindings for ITE IT6251 LVDS-to-eDP bridge.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: devicetree@vger.kernel.org
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Rob Herring <robh@kernel.org>
Cc: Sean Cross <xobs@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

Comments

Rob Herring (Arm) Oct. 18, 2016, 4:30 p.m. UTC | #1
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@denx.de>
> Cc: devicetree@vger.kernel.org
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Rob Herring <robh@kernel.org>
> Cc: Sean Cross <xobs@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"?

> +- 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
>
Marek Vasut Oct. 18, 2016, 4:35 p.m. UTC | #2
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@denx.de>
>> Cc: devicetree@vger.kernel.org
>> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
>> Cc: Rob Herring <robh@kernel.org>
>> Cc: Sean Cross <xobs@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@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
>>
diff mbox

Patch

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
+- 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>;
+				};
+			};
+		};
+	};
+