Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1540954 > unrolled thread

[PATCH v7 0/5] ARM: dts: da850: tilcdc related DT changes

Started byBartosz Golaszewski <bgolaszewski@baylibre.com>
First post2016-12-13 11:10 +0100
Last post2016-12-14 06:10 +0100
Articles 16 — 4 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v7 0/5] ARM: dts: da850: tilcdc related DT changes Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-13 11:10 +0100
    [PATCH v7 5/5] ARM: dts: da850: specify the maximum pixel clock rate for tilcdc Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-13 11:20 +0100
      Re: [PATCH v7 5/5] ARM: dts: da850: specify the maximum pixel clock  rate for tilcdc Sekhar Nori <nsekhar@ti.com> - 2017-01-02 12:40 +0100
    [PATCH v7 3/5] drm: bridge: add support for TI ths8135 Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-13 11:20 +0100
      Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135 Archit Taneja <architt@codeaurora.org> - 2016-12-14 06:10 +0100
        Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135 Sekhar Nori <nsekhar@ti.com> - 2017-01-02 12:10 +0100
          Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135 Archit Taneja <architt@codeaurora.org> - 2017-01-03 06:30 +0100
            Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135 Sekhar Nori <nsekhar@ti.com> - 2017-01-03 06:40 +0100
    [PATCH v7 1/5] ARM: dts: da850: rename the display node label Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-13 11:20 +0100
      Re: [PATCH v7 1/5] ARM: dts: da850: rename the display node label Tomi Valkeinen <tomi.valkeinen@ti.com> - 2016-12-14 11:00 +0100
        Re: [PATCH v7 1/5] ARM: dts: da850: rename the display node label Sekhar Nori <nsekhar@ti.com> - 2017-01-02 12:10 +0100
    [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-13 11:20 +0100
      Re: [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node Tomi Valkeinen <tomi.valkeinen@ti.com> - 2016-12-14 11:00 +0100
        Re: [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node Sekhar Nori <nsekhar@ti.com> - 2017-01-02 12:20 +0100
    [PATCH v7 2/5] drm: bridge: add DT bindings for TI ths8135 Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2016-12-13 11:20 +0100
      Re: [PATCH v7 2/5] drm: bridge: add DT bindings for TI ths8135 Archit Taneja <architt@codeaurora.org> - 2016-12-14 06:10 +0100

#1540954 — [PATCH v7 0/5] ARM: dts: da850: tilcdc related DT changes

FromBartosz Golaszewski <bgolaszewski@baylibre.com>
Date2016-12-13 11:10 +0100
Subject[PATCH v7 0/5] ARM: dts: da850: tilcdc related DT changes
Message-ID<sNSc1-8lZ-17@gated-at.bofh.it>
This series contains the last DT changes required for LCDC support
on da850-lcdk. The first one adds the dumb-vga-dac nodes, the second
limits the maximum pixel clock rate.

v1 -> v2:
- drop patch 3/3 (already merged)
- use max-pixelclock instead of max-bandwidth for display mode limiting

v2 -> v3:
- make the commit message in patch [2/2] more detailed
- move the max-pixelclock property to da850.dtsi as the limit
  affects all da850-based boards

v3 -> v4:
- remove the input port from the display node
- move the display ports node to da850-lcdk.dts
- rename the vga_bridge node to vga-bridge
- move the LCDC pins to the LCDC node (from the vga bridge node)

v4 -> v5:
- rename the display label to lcdc
- instead of using the 'dumb-vga-dac' compatible, add bindings for
  ti,ths8135 and use it as the vga-bridge node compatible

v5 -> v6:
- drop the endpoint numbers for nodes with single endpoints
- split patch 2/4 from v5 into two separate patches: one adding DT
  bindings and one adding actual support for ths8135

v6 -> v7:
- simplified patch 4/5 by removing unnecessary properties from ports
  and endpoints nodes

Bartosz Golaszewski (5):
  ARM: dts: da850: rename the display node label
  drm: bridge: add DT bindings for TI ths8135
  drm: bridge: add support for TI ths8135
  ARM: dts: da850-lcdk: add the vga-bridge node
  ARM: dts: da850: specify the maximum pixel clock rate for tilcdc

 .../bindings/display/bridge/ti,ths8135.txt         | 46 +++++++++++++++++++
 arch/arm/boot/dts/da850-lcdk.dts                   | 51 ++++++++++++++++++++++
 arch/arm/boot/dts/da850.dtsi                       |  3 +-
 drivers/gpu/drm/bridge/dumb-vga-dac.c              |  1 +
 4 files changed, 100 insertions(+), 1 deletion(-)
 create mode 100644 Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt

-- 
2.9.3

[toc] | [next] | [standalone]


#1540963 — [PATCH v7 5/5] ARM: dts: da850: specify the maximum pixel clock rate for tilcdc

FromBartosz Golaszewski <bgolaszewski@baylibre.com>
Date2016-12-13 11:20 +0100
Subject[PATCH v7 5/5] ARM: dts: da850: specify the maximum pixel clock rate for tilcdc
Message-ID<sNSlH-8pl-23@gated-at.bofh.it>
In reply to#1540954
At maximum CPU frequency of 300 MHz the maximum pixel clock frequency
is 37.5 MHz[1]. We must filter out any mode for which the calculated
pixel clock rate would exceed this value.

Specify the max-pixelclock property for the display node for
da850-lcdk.

[1] http://processors.wiki.ti.com/index.php/OMAP-L1x/C674x/AM1x_LCD_Controller_(LCDC)_Throughput_and_Optimization_Techniques

Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
 arch/arm/boot/dts/da850.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 6b0ef3d..58b1566 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -462,6 +462,7 @@
 			compatible = "ti,da850-tilcdc";
 			reg = <0x213000 0x1000>;
 			interrupts = <52>;
+			max-pixelclock = <37500>;
 			status = "disabled";
 		};
 	};
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]


#1549121 — Re: [PATCH v7 5/5] ARM: dts: da850: specify the maximum pixel clock rate for tilcdc

FromSekhar Nori <nsekhar@ti.com>
Date2017-01-02 12:40 +0100
SubjectRe: [PATCH v7 5/5] ARM: dts: da850: specify the maximum pixel clock rate for tilcdc
Message-ID<sV985-6ct-1@gated-at.bofh.it>
In reply to#1540963
On Tuesday 13 December 2016 03:39 PM, Bartosz Golaszewski wrote:
> At maximum CPU frequency of 300 MHz the maximum pixel clock frequency
> is 37.5 MHz[1]. We must filter out any mode for which the calculated
> pixel clock rate would exceed this value.
> 
> Specify the max-pixelclock property for the display node for
> da850-lcdk.
> 
> [1] http://processors.wiki.ti.com/index.php/OMAP-L1x/C674x/AM1x_LCD_Controller_(LCDC)_Throughput_and_Optimization_Techniques

This wiki page is not really a TI datasheet and can change without
notice. I changed this to refer to
http://www.ti.com/lit/ds/symlink/am1808.pdf (SPRS653E, revised March
2014), table 6-110.

Applied to v4.11/dt

Thanks,
Sekhar

[toc] | [prev] | [next] | [standalone]


#1540966 — [PATCH v7 3/5] drm: bridge: add support for TI ths8135

FromBartosz Golaszewski <bgolaszewski@baylibre.com>
Date2016-12-13 11:20 +0100
Subject[PATCH v7 3/5] drm: bridge: add support for TI ths8135
Message-ID<sNSlI-8pl-37@gated-at.bofh.it>
In reply to#1540954
THS8135 is a configurable video DAC, but no configuration is actually
necessary to make it work.

For now use the dumb-vga-dac driver to support it.

Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 drivers/gpu/drm/bridge/dumb-vga-dac.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c b/drivers/gpu/drm/bridge/dumb-vga-dac.c
index e570698..86e9f9c 100644
--- a/drivers/gpu/drm/bridge/dumb-vga-dac.c
+++ b/drivers/gpu/drm/bridge/dumb-vga-dac.c
@@ -237,6 +237,7 @@ static int dumb_vga_remove(struct platform_device *pdev)
 
 static const struct of_device_id dumb_vga_match[] = {
 	{ .compatible = "dumb-vga-dac" },
+	{ .compatible = "ti,ths8135" },
 	{},
 };
 MODULE_DEVICE_TABLE(of, dumb_vga_match);
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]


#1541701 — Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135

FromArchit Taneja <architt@codeaurora.org>
Date2016-12-14 06:10 +0100
SubjectRe: [PATCH v7 3/5] drm: bridge: add support for TI ths8135
Message-ID<sO9Zg-2jA-11@gated-at.bofh.it>
In reply to#1540966

On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote:
> THS8135 is a configurable video DAC, but no configuration is actually
> necessary to make it work.
>
> For now use the dumb-vga-dac driver to support it.

Queued to drm-misc-next

Archit

>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>  drivers/gpu/drm/bridge/dumb-vga-dac.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c b/drivers/gpu/drm/bridge/dumb-vga-dac.c
> index e570698..86e9f9c 100644
> --- a/drivers/gpu/drm/bridge/dumb-vga-dac.c
> +++ b/drivers/gpu/drm/bridge/dumb-vga-dac.c
> @@ -237,6 +237,7 @@ static int dumb_vga_remove(struct platform_device *pdev)
>
>  static const struct of_device_id dumb_vga_match[] = {
>  	{ .compatible = "dumb-vga-dac" },
> +	{ .compatible = "ti,ths8135" },
>  	{},
>  };
>  MODULE_DEVICE_TABLE(of, dumb_vga_match);
>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[toc] | [prev] | [next] | [standalone]


#1549113 — Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135

FromSekhar Nori <nsekhar@ti.com>
Date2017-01-02 12:10 +0100
SubjectRe: [PATCH v7 3/5] drm: bridge: add support for TI ths8135
Message-ID<sV8F3-5YN-1@gated-at.bofh.it>
In reply to#1541701
Hi Archit,

On Wednesday 14 December 2016 10:35 AM, Archit Taneja wrote:
> 
> 
> On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote:
>> THS8135 is a configurable video DAC, but no configuration is actually
>> necessary to make it work.
>>
>> For now use the dumb-vga-dac driver to support it.
> 
> Queued to drm-misc-next

This patch and 2/5 are not in v4.10 kernel. Did you mean to queue them
to v4.10?

Thanks,
Sekhar

[toc] | [prev] | [next] | [standalone]


#1549536 — Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135

FromArchit Taneja <architt@codeaurora.org>
Date2017-01-03 06:30 +0100
SubjectRe: [PATCH v7 3/5] drm: bridge: add support for TI ths8135
Message-ID<sVpPz-1tt-1@gated-at.bofh.it>
In reply to#1549113
Hi Sekhar,

On 1/2/2017 4:38 PM, Sekhar Nori wrote:
> Hi Archit,
>
> On Wednesday 14 December 2016 10:35 AM, Archit Taneja wrote:
>>
>>
>> On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote:
>>> THS8135 is a configurable video DAC, but no configuration is actually
>>> necessary to make it work.
>>>
>>> For now use the dumb-vga-dac driver to support it.
>>
>> Queued to drm-misc-next
>
> This patch and 2/5 are not in v4.10 kernel. Did you mean to queue them
> to v4.10?

These missed out on 4.10 because the drm related pull requests were
already sent by then. These 2 patches are queued for 4.11.

Thanks,
Archit

>
> Thanks,
> Sekhar
>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[toc] | [prev] | [next] | [standalone]


#1549539 — Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135

FromSekhar Nori <nsekhar@ti.com>
Date2017-01-03 06:40 +0100
SubjectRe: [PATCH v7 3/5] drm: bridge: add support for TI ths8135
Message-ID<sVpZf-1B7-1@gated-at.bofh.it>
In reply to#1549536
On Tuesday 03 January 2017 10:56 AM, Archit Taneja wrote:
> Hi Sekhar,
> 
> On 1/2/2017 4:38 PM, Sekhar Nori wrote:
>> Hi Archit,
>>
>> On Wednesday 14 December 2016 10:35 AM, Archit Taneja wrote:
>>>
>>>
>>> On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote:
>>>> THS8135 is a configurable video DAC, but no configuration is actually
>>>> necessary to make it work.
>>>>
>>>> For now use the dumb-vga-dac driver to support it.
>>>
>>> Queued to drm-misc-next
>>
>> This patch and 2/5 are not in v4.10 kernel. Did you mean to queue them
>> to v4.10?
> 
> These missed out on 4.10 because the drm related pull requests were
> already sent by then. These 2 patches are queued for 4.11.

Alright, thanks for the update.

Regards,
Sekhar

[toc] | [prev] | [next] | [standalone]


#1540967 — [PATCH v7 1/5] ARM: dts: da850: rename the display node label

FromBartosz Golaszewski <bgolaszewski@baylibre.com>
Date2016-12-13 11:20 +0100
Subject[PATCH v7 1/5] ARM: dts: da850: rename the display node label
Message-ID<sNSlI-8pl-33@gated-at.bofh.it>
In reply to#1540954
The tilcdc node name is 'display' as per the ePAPR 1.1 recommendation.
The label is also 'display', but change it to 'lcdc' to make it clear
what the underlying hardware is.

Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
 arch/arm/boot/dts/da850.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 104155d..6b0ef3d 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -458,7 +458,7 @@
 			dma-names = "tx", "rx";
 		};
 
-		display: display@213000 {
+		lcdc: display@213000 {
 			compatible = "ti,da850-tilcdc";
 			reg = <0x213000 0x1000>;
 			interrupts = <52>;
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]


#1541795 — Re: [PATCH v7 1/5] ARM: dts: da850: rename the display node label

FromTomi Valkeinen <tomi.valkeinen@ti.com>
Date2016-12-14 11:00 +0100
SubjectRe: [PATCH v7 1/5] ARM: dts: da850: rename the display node label
Message-ID<sOevU-4RP-7@gated-at.bofh.it>
In reply to#1540967

[Multipart message — attachments visible in raw view] — view raw

On 13/12/16 12:09, Bartosz Golaszewski wrote:
> The tilcdc node name is 'display' as per the ePAPR 1.1 recommendation.
> The label is also 'display', but change it to 'lcdc' to make it clear
> what the underlying hardware is.
> 
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
>  arch/arm/boot/dts/da850.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
> index 104155d..6b0ef3d 100644
> --- a/arch/arm/boot/dts/da850.dtsi
> +++ b/arch/arm/boot/dts/da850.dtsi
> @@ -458,7 +458,7 @@
>  			dma-names = "tx", "rx";
>  		};
>  
> -		display: display@213000 {
> +		lcdc: display@213000 {
>  			compatible = "ti,da850-tilcdc";
>  			reg = <0x213000 0x1000>;
>  			interrupts = <52>;
> 

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

 Tomi

[toc] | [prev] | [next] | [standalone]


#1549114 — Re: [PATCH v7 1/5] ARM: dts: da850: rename the display node label

FromSekhar Nori <nsekhar@ti.com>
Date2017-01-02 12:10 +0100
SubjectRe: [PATCH v7 1/5] ARM: dts: da850: rename the display node label
Message-ID<sV8F4-5YN-15@gated-at.bofh.it>
In reply to#1541795
On Wednesday 14 December 2016 03:27 PM, Tomi Valkeinen wrote:
> On 13/12/16 12:09, Bartosz Golaszewski wrote:
>> The tilcdc node name is 'display' as per the ePAPR 1.1 recommendation.
>> The label is also 'display', but change it to 'lcdc' to make it clear
>> what the underlying hardware is.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> ---
>>  arch/arm/boot/dts/da850.dtsi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
>> index 104155d..6b0ef3d 100644
>> --- a/arch/arm/boot/dts/da850.dtsi
>> +++ b/arch/arm/boot/dts/da850.dtsi
>> @@ -458,7 +458,7 @@
>>  			dma-names = "tx", "rx";
>>  		};
>>  
>> -		display: display@213000 {
>> +		lcdc: display@213000 {
>>  			compatible = "ti,da850-tilcdc";
>>  			reg = <0x213000 0x1000>;
>>  			interrupts = <52>;
>>
> 
> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

Applied with Tomi's reviewed-by.

Thanks,
Sekhar

[toc] | [prev] | [next] | [standalone]


#1540968 — [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node

FromBartosz Golaszewski <bgolaszewski@baylibre.com>
Date2016-12-13 11:20 +0100
Subject[PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node
Message-ID<sNSlI-8pl-39@gated-at.bofh.it>
In reply to#1540954
Add the vga-bridge node to the board DT together with corresponding
ports and vga connector. This allows to retrieve the edid info from
the display automatically.

Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 arch/arm/boot/dts/da850-lcdk.dts | 51 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 51 insertions(+)

diff --git a/arch/arm/boot/dts/da850-lcdk.dts b/arch/arm/boot/dts/da850-lcdk.dts
index afcb482..1ae2260 100644
--- a/arch/arm/boot/dts/da850-lcdk.dts
+++ b/arch/arm/boot/dts/da850-lcdk.dts
@@ -51,6 +51,45 @@
 			system-clock-frequency = <24576000>;
 		};
 	};
+
+	vga-bridge {
+		compatible = "ti,ths8135";
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			port@0 {
+				reg = <0>;
+
+				vga_bridge_in: endpoint {
+					remote-endpoint = <&lcdc_out_vga>;
+				};
+			};
+
+			port@1 {
+				reg = <1>;
+
+				vga_bridge_out: endpoint {
+					remote-endpoint = <&vga_con_in>;
+				};
+			};
+		};
+	};
+
+	vga {
+		compatible = "vga-connector";
+
+		ddc-i2c-bus = <&i2c0>;
+
+		port {
+			vga_con_in: endpoint {
+				remote-endpoint = <&vga_bridge_out>;
+			};
+		};
+	};
 };
 
 &pmx_core {
@@ -236,3 +275,15 @@
 &memctrl {
 	status = "okay";
 };
+
+&lcdc {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&lcd_pins>;
+
+	port {
+		lcdc_out_vga: endpoint {
+			remote-endpoint = <&vga_bridge_in>;
+		};
+	};
+};
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]


#1541803 — Re: [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node

FromTomi Valkeinen <tomi.valkeinen@ti.com>
Date2016-12-14 11:00 +0100
SubjectRe: [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node
Message-ID<sOevU-4RP-37@gated-at.bofh.it>
In reply to#1540968

[Multipart message — attachments visible in raw view] — view raw

On 13/12/16 12:09, Bartosz Golaszewski wrote:
> Add the vga-bridge node to the board DT together with corresponding
> ports and vga connector. This allows to retrieve the edid info from
> the display automatically.
> 
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>  arch/arm/boot/dts/da850-lcdk.dts | 51 ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 51 insertions(+)

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

 Tomi

[toc] | [prev] | [next] | [standalone]


#1549119 — Re: [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node

FromSekhar Nori <nsekhar@ti.com>
Date2017-01-02 12:20 +0100
SubjectRe: [PATCH v7 4/5] ARM: dts: da850-lcdk: add the vga-bridge node
Message-ID<sV8OJ-64t-17@gated-at.bofh.it>
In reply to#1541803
On Wednesday 14 December 2016 03:24 PM, Tomi Valkeinen wrote:
> On 13/12/16 12:09, Bartosz Golaszewski wrote:
>> Add the vga-bridge node to the board DT together with corresponding
>> ports and vga connector. This allows to retrieve the edid info from
>> the display automatically.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>> ---
>>  arch/arm/boot/dts/da850-lcdk.dts | 51 ++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 51 insertions(+)
> 
> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

Applied to v4.11/dt with Tomi's reviewed-by.

Thanks,
Sekhar

[toc] | [prev] | [next] | [standalone]


#1540969 — [PATCH v7 2/5] drm: bridge: add DT bindings for TI ths8135

FromBartosz Golaszewski <bgolaszewski@baylibre.com>
Date2016-12-13 11:20 +0100
Subject[PATCH v7 2/5] drm: bridge: add DT bindings for TI ths8135
Message-ID<sNSlI-8pl-49@gated-at.bofh.it>
In reply to#1540954
THS8135 is a configurable video DAC. Add DT bindings for this chip.

Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Rob Herring <robh@kernel.org>
---
 .../bindings/display/bridge/ti,ths8135.txt         | 46 ++++++++++++++++++++++
 1 file changed, 46 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt

diff --git a/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt b/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt
new file mode 100644
index 0000000..6ec1a88
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt
@@ -0,0 +1,46 @@
+THS8135 Video DAC
+-----------------
+
+This is the binding for Texas Instruments THS8135 Video DAC bridge.
+
+Required properties:
+
+- compatible: Must be "ti,ths8135"
+
+Required nodes:
+
+This device has two video ports. Their connections are modelled using the OF
+graph bindings specified in Documentation/devicetree/bindings/graph.txt.
+
+- Video port 0 for RGB input
+- Video port 1 for VGA output
+
+Example
+-------
+
+vga-bridge {
+	compatible = "ti,ths8135";
+	#address-cells = <1>;
+	#size-cells = <0>;
+
+	ports {
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		port@0 {
+			reg = <0>;
+
+			vga_bridge_in: endpoint {
+				remote-endpoint = <&lcdc_out_vga>;
+			};
+		};
+
+		port@1 {
+			reg = <1>;
+
+			vga_bridge_out: endpoint {
+				remote-endpoint = <&vga_con_in>;
+			};
+		};
+	};
+};
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]


#1541698 — Re: [PATCH v7 2/5] drm: bridge: add DT bindings for TI ths8135

FromArchit Taneja <architt@codeaurora.org>
Date2016-12-14 06:10 +0100
SubjectRe: [PATCH v7 2/5] drm: bridge: add DT bindings for TI ths8135
Message-ID<sO9Zf-2jA-3@gated-at.bofh.it>
In reply to#1540969
Hi,

On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote:
> THS8135 is a configurable video DAC. Add DT bindings for this chip.

Queued to drm-misc-next
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Acked-by: Rob Herring <robh@kernel.org>
> ---
>  .../bindings/display/bridge/ti,ths8135.txt         | 46 ++++++++++++++++++++++
>  1 file changed, 46 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt b/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt
> new file mode 100644
> index 0000000..6ec1a88
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt
> @@ -0,0 +1,46 @@
> +THS8135 Video DAC
> +-----------------
> +
> +This is the binding for Texas Instruments THS8135 Video DAC bridge.
> +
> +Required properties:
> +
> +- compatible: Must be "ti,ths8135"
> +
> +Required nodes:
> +
> +This device has two video ports. Their connections are modelled using the OF
> +graph bindings specified in Documentation/devicetree/bindings/graph.txt.
> +
> +- Video port 0 for RGB input
> +- Video port 1 for VGA output
> +
> +Example
> +-------
> +
> +vga-bridge {
> +	compatible = "ti,ths8135";
> +	#address-cells = <1>;
> +	#size-cells = <0>;
> +
> +	ports {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		port@0 {
> +			reg = <0>;
> +
> +			vga_bridge_in: endpoint {
> +				remote-endpoint = <&lcdc_out_vga>;
> +			};
> +		};
> +
> +		port@1 {
> +			reg = <1>;
> +
> +			vga_bridge_out: endpoint {
> +				remote-endpoint = <&vga_con_in>;
> +			};
> +		};
> +	};
> +};
>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web