All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree
@ 2014-10-26 14:11 Laurent Pinchart
  2015-12-24 19:45 ` Sergei Shtylyov
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Laurent Pinchart @ 2014-10-26 14:11 UTC (permalink / raw)
  To: linux-sh

Add the DU device with a disabled state. Boards that want to enable the
DU need to specify the output topology

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
 arch/arm/boot/dts/r8a7791.dtsi | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index 74f18d89a35f..6b06a7665fae 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -695,6 +695,36 @@
 		renesas,#wpf = <4>;
 	};
 
+	du: display@feb00000 {
+		compatible = "renesas,du-r8a7791";
+		reg = <0 0xfeb00000 0 0x40000>,
+		      <0 0xfeb90000 0 0x1c>;
+		reg-names = "du", "lvds.0";
+		interrupts = <0 256 IRQ_TYPE_LEVEL_HIGH>,
+			     <0 268 IRQ_TYPE_LEVEL_HIGH>;
+		clocks = <&mstp7_clks R8A7791_CLK_DU0>,
+			 <&mstp7_clks R8A7791_CLK_DU1>,
+			 <&mstp7_clks R8A7791_CLK_LVDS0>;
+		clock-names = "du.0", "du.1", "lvds.0";
+		status = "disabled";
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			port@0 {
+				reg = <0>;
+				du_out_rgb: endpoint {
+				};
+			};
+			port@1 {
+				reg = <1>;
+				du_out_lvds0: endpoint {
+				};
+			};
+		};
+	};
+
 	clocks {
 		#address-cells = <2>;
 		#size-cells = <2>;
-- 
2.0.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree
  2014-10-26 14:11 [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree Laurent Pinchart
@ 2015-12-24 19:45 ` Sergei Shtylyov
  2015-12-27  0:07 ` Laurent Pinchart
  2015-12-30 22:06 ` Sergei Shtylyov
  2 siblings, 0 replies; 4+ messages in thread
From: Sergei Shtylyov @ 2015-12-24 19:45 UTC (permalink / raw)
  To: linux-sh

Hello.

On 10/26/2014 05:11 PM, Laurent Pinchart wrote:

> Add the DU device with a disabled state. Boards that want to enable the
> DU need to specify the output topology
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
>   arch/arm/boot/dts/r8a7791.dtsi | 30 ++++++++++++++++++++++++++++++
>   1 file changed, 30 insertions(+)
>
> diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
> index 74f18d89a35f..6b06a7665fae 100644
> --- a/arch/arm/boot/dts/r8a7791.dtsi
> +++ b/arch/arm/boot/dts/r8a7791.dtsi
> @@ -695,6 +695,36 @@
>   		renesas,#wpf = <4>;
>   	};
>
> +	du: display@feb00000 {
> +		compatible = "renesas,du-r8a7791";
> +		reg = <0 0xfeb00000 0 0x40000>,
> +		      <0 0xfeb90000 0 0x1c>;
> +		reg-names = "du", "lvds.0";
> +		interrupts = <0 256 IRQ_TYPE_LEVEL_HIGH>,
> +			     <0 268 IRQ_TYPE_LEVEL_HIGH>;
> +		clocks = <&mstp7_clks R8A7791_CLK_DU0>,
> +			 <&mstp7_clks R8A7791_CLK_DU1>,
> +			 <&mstp7_clks R8A7791_CLK_LVDS0>;
> +		clock-names = "du.0", "du.1", "lvds.0";
> +		status = "disabled";
> +
> +		ports {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			port@0 {
> +				reg = <0>;
> +				du_out_rgb: endpoint {
> +				};
> +			};
> +			port@1 {
> +				reg = <1>;
> +				du_out_lvds0: endpoint {
> +				};

    Hm, my manual tell me LVDS is supported on DU0 only for R8A7791/3...

[...]

MBR, Sergei


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree
  2014-10-26 14:11 [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree Laurent Pinchart
  2015-12-24 19:45 ` Sergei Shtylyov
@ 2015-12-27  0:07 ` Laurent Pinchart
  2015-12-30 22:06 ` Sergei Shtylyov
  2 siblings, 0 replies; 4+ messages in thread
From: Laurent Pinchart @ 2015-12-27  0:07 UTC (permalink / raw)
  To: linux-sh

Hi Sergei,

On Thursday 24 December 2015 22:45:33 Sergei Shtylyov wrote:
> On 10/26/2014 05:11 PM, Laurent Pinchart wrote:
> > Add the DU device with a disabled state. Boards that want to enable the
> > DU need to specify the output topology
> > 
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@ideasonboard.com>
> > ---
> > 
> >   arch/arm/boot/dts/r8a7791.dtsi | 30 ++++++++++++++++++++++++++++++
> >   1 file changed, 30 insertions(+)
> > 
> > diff --git a/arch/arm/boot/dts/r8a7791.dtsi
> > b/arch/arm/boot/dts/r8a7791.dtsi index 74f18d89a35f..6b06a7665fae 100644
> > --- a/arch/arm/boot/dts/r8a7791.dtsi
> > +++ b/arch/arm/boot/dts/r8a7791.dtsi
> > @@ -695,6 +695,36 @@
> >   		renesas,#wpf = <4>;
> >   	};
> > 
> > +	du: display@feb00000 {
> > +		compatible = "renesas,du-r8a7791";
> > +		reg = <0 0xfeb00000 0 0x40000>,
> > +		      <0 0xfeb90000 0 0x1c>;
> > +		reg-names = "du", "lvds.0";
> > +		interrupts = <0 256 IRQ_TYPE_LEVEL_HIGH>,
> > +			     <0 268 IRQ_TYPE_LEVEL_HIGH>;
> > +		clocks = <&mstp7_clks R8A7791_CLK_DU0>,
> > +			 <&mstp7_clks R8A7791_CLK_DU1>,
> > +			 <&mstp7_clks R8A7791_CLK_LVDS0>;
> > +		clock-names = "du.0", "du.1", "lvds.0";
> > +		status = "disabled";
> > +
> > +		ports {
> > +			#address-cells = <1>;
> > +			#size-cells = <0>;
> > +
> > +			port@0 {
> > +				reg = <0>;
> > +				du_out_rgb: endpoint {
> > +				};
> > +			};
> > +			port@1 {
> > +				reg = <1>;
> > +				du_out_lvds0: endpoint {
> > +				};
> 
> Hm, my manual tell me LVDS is supported on DU0 only for R8A7791/3...

Yes, but note that the port number doesn't correspond to the DU number. A 
given port can be driven by multiple DU channels on some of the R-Car SoCs. 
The routing tables are defined in the per-SoC device info in 
drivers/gpu/drm/rcar-du/rcar_du_drv.c.

-- 
Regards,

Laurent Pinchart


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree
  2014-10-26 14:11 [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree Laurent Pinchart
  2015-12-24 19:45 ` Sergei Shtylyov
  2015-12-27  0:07 ` Laurent Pinchart
@ 2015-12-30 22:06 ` Sergei Shtylyov
  2 siblings, 0 replies; 4+ messages in thread
From: Sergei Shtylyov @ 2015-12-30 22:06 UTC (permalink / raw)
  To: linux-sh

On 12/27/2015 03:07 AM, Laurent Pinchart wrote:

>>> Add the DU device with a disabled state. Boards that want to enable the
>>> DU need to specify the output topology
>>>
>>> Signed-off-by: Laurent Pinchart
>>> <laurent.pinchart+renesas@ideasonboard.com>
>>> ---
>>>
>>>    arch/arm/boot/dts/r8a7791.dtsi | 30 ++++++++++++++++++++++++++++++
>>>    1 file changed, 30 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/r8a7791.dtsi
>>> b/arch/arm/boot/dts/r8a7791.dtsi index 74f18d89a35f..6b06a7665fae 100644
>>> --- a/arch/arm/boot/dts/r8a7791.dtsi
>>> +++ b/arch/arm/boot/dts/r8a7791.dtsi
>>> @@ -695,6 +695,36 @@
>>>    		renesas,#wpf = <4>;
>>>    	};
>>>
>>> +	du: display@feb00000 {
>>> +		compatible = "renesas,du-r8a7791";
>>> +		reg = <0 0xfeb00000 0 0x40000>,
>>> +		      <0 0xfeb90000 0 0x1c>;
>>> +		reg-names = "du", "lvds.0";
>>> +		interrupts = <0 256 IRQ_TYPE_LEVEL_HIGH>,
>>> +			     <0 268 IRQ_TYPE_LEVEL_HIGH>;
>>> +		clocks = <&mstp7_clks R8A7791_CLK_DU0>,
>>> +			 <&mstp7_clks R8A7791_CLK_DU1>,
>>> +			 <&mstp7_clks R8A7791_CLK_LVDS0>;
>>> +		clock-names = "du.0", "du.1", "lvds.0";
>>> +		status = "disabled";
>>> +
>>> +		ports {
>>> +			#address-cells = <1>;
>>> +			#size-cells = <0>;
>>> +
>>> +			port@0 {
>>> +				reg = <0>;
>>> +				du_out_rgb: endpoint {
>>> +				};
>>> +			};
>>> +			port@1 {
>>> +				reg = <1>;
>>> +				du_out_lvds0: endpoint {
>>> +				};
>>
>> Hm, my manual tell me LVDS is supported on DU0 only for R8A7791/3...
>
> Yes, but note that the port number doesn't correspond to the DU number. A
> given port can be driven by multiple DU channels on some of the R-Car SoCs.

    Ah, I didn't dive into these matters deep enough...

> The routing tables are defined in the per-SoC device info in
> drivers/gpu/drm/rcar-du/rcar_du_drv.c.

    Thank you for the pointer!
    I've fixed up the Porter patch to hang HDMI deconder from the "correct" 
port 0 ISO port 1 and I lost the picture on my LVDS display after doing that. 
With the wrong Porter patch, both displays work...

MBR, Sergei


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-12-30 22:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-26 14:11 [PATCH v5 03/14] ARM: shmobile: r8a7791: Add DU node to device tree Laurent Pinchart
2015-12-24 19:45 ` Sergei Shtylyov
2015-12-27  0:07 ` Laurent Pinchart
2015-12-30 22:06 ` Sergei Shtylyov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.