Re: [PATCH 06/10] ARM: dts: Augment panel setting for Versatile

2018-03-12 Thread Liviu Dudau
On Sun, Mar 11, 2018 at 02:53:03PM +0100, Linus Walleij wrote:
> This adds the actual VGA DAC bridge that is used in the
> Versatile AB, and sets the mode to 640x480 VGA.
> 
> The "clcd" clock was incorrectly named, the proper name
> (from bindings) is "clcdclk". So far drivers survived
> by just getting the first clock, but future drivers will
> use named clocks.
> 
> We add the panel connector to the
> "arm,versatile-tft-panel" as well, the signals actually
> fork on the board, reaching both the VGA DAC and the
> display connector.
> 
> Cc: Liviu Dudau 
> Cc: Mali DP Maintainers 
> Signed-off-by: Linus Walleij 

Reviewed-by: Liviu Dudau 

> ---
>  arch/arm/boot/dts/versatile-ab.dts | 83 
> +-
>  1 file changed, 81 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/versatile-ab.dts 
> b/arch/arm/boot/dts/versatile-ab.dts
> index 4a51612996bc..5f61d3609027 100644
> --- a/arch/arm/boot/dts/versatile-ab.dts
> +++ b/arch/arm/boot/dts/versatile-ab.dts
> @@ -30,6 +30,43 @@
>   clock-frequency = <2400>;
>   };
>  
> + bridge {
> + compatible = "ti,ths8134b", "ti,ths8134";
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@0 {
> + reg = <0>;
> +
> + vga_bridge_in: endpoint {
> + remote-endpoint = <&clcd_pads_vga_dac>;
> + };
> + };
> +
> + port@1 {
> + reg = <1>;
> +
> + vga_bridge_out: endpoint {
> + remote-endpoint = <&vga_con_in>;
> + };
> + };
> + };
> + };
> +
> + vga {
> + compatible = "vga-connector";
> +
> + port {
> + vga_con_in: endpoint {
> + remote-endpoint = <&vga_bridge_out>;
> + };
> + };
> + };
> +
>   core-module@1000 {
>   compatible = "arm,core-module-versatile", "syscon", 
> "simple-mfd";
>   reg = <0x1000 0x200>;
> @@ -230,7 +267,39 @@
>   reg = <0x1012 0x1000>;
>   interrupts = <16>;
>   clocks = <&osc1>, <&pclk>;
> - clock-names = "clcd", "apb_pclk";
> + clock-names = "clcdclk", "apb_pclk";
> + /* 800x600 16bpp @ 36MHz works fine */
> + max-memory-bandwidth = <5400>;
> +
> + /*
> +  * This port is routed through a PLD (Programmable
> +  * Logic Device) that routes the output from the CLCD
> +  * (after transformations) to the VGA DAC and also an
> +  * external panel connector. The PLD is essential for
> +  * supporting RGB565/BGR565.
> +  *
> +  * The signals from the port thus reaches two endpoints.
> +  * The PLD is managed through a few special bits in the
> +  * FPGA "sysreg".
> +  *
> +  * This arrangement can be clearly seen in
> +  * ARM DUI 0225D, page 3-41, figure 3-19.
> +  */

If I remember correctly, besides being able to flip outputs via "sysreg"
there was a debug menu on the microcontroller that had options to set
the MUX to the right output before OS boot.

That brings back a lot of memories that I thought I'll never have any
use of :)

Best regards,
Liviu

> + port@0 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + clcd_pads_panel: endpoint@0 {
> + reg = <0>;
> + remote-endpoint = <&panel_in>;
> + arm,pl11x,tft-r0g0b0-pads = <0 8 16>;
> + };
> + clcd_pads_vga_dac: endpoint@1 {
> + reg = <1>;
> + remote-endpoint = <&vga_bridge_in>;
> + arm,pl11x,tft-r0g0b0-pads = <0 8 16>;
> + };
> + };
>   };
>  
>   sctl@101e {
> @@ -319,8 +388,18 @@
>   ranges = <0 0x1000 0x1>;
>  
>   sysreg@0 {
> - compatible = "arm,versatile-sysreg", "syscon";
> + compatible = "arm,versatile-sysreg", "syscon", 
> "simp

[PATCH 06/10] ARM: dts: Augment panel setting for Versatile

2018-03-11 Thread Linus Walleij
This adds the actual VGA DAC bridge that is used in the
Versatile AB, and sets the mode to 640x480 VGA.

The "clcd" clock was incorrectly named, the proper name
(from bindings) is "clcdclk". So far drivers survived
by just getting the first clock, but future drivers will
use named clocks.

We add the panel connector to the
"arm,versatile-tft-panel" as well, the signals actually
fork on the board, reaching both the VGA DAC and the
display connector.

Cc: Liviu Dudau 
Cc: Mali DP Maintainers 
Signed-off-by: Linus Walleij 
---
 arch/arm/boot/dts/versatile-ab.dts | 83 +-
 1 file changed, 81 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/versatile-ab.dts 
b/arch/arm/boot/dts/versatile-ab.dts
index 4a51612996bc..5f61d3609027 100644
--- a/arch/arm/boot/dts/versatile-ab.dts
+++ b/arch/arm/boot/dts/versatile-ab.dts
@@ -30,6 +30,43 @@
clock-frequency = <2400>;
};
 
+   bridge {
+   compatible = "ti,ths8134b", "ti,ths8134";
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   ports {
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   port@0 {
+   reg = <0>;
+
+   vga_bridge_in: endpoint {
+   remote-endpoint = <&clcd_pads_vga_dac>;
+   };
+   };
+
+   port@1 {
+   reg = <1>;
+
+   vga_bridge_out: endpoint {
+   remote-endpoint = <&vga_con_in>;
+   };
+   };
+   };
+   };
+
+   vga {
+   compatible = "vga-connector";
+
+   port {
+   vga_con_in: endpoint {
+   remote-endpoint = <&vga_bridge_out>;
+   };
+   };
+   };
+
core-module@1000 {
compatible = "arm,core-module-versatile", "syscon", 
"simple-mfd";
reg = <0x1000 0x200>;
@@ -230,7 +267,39 @@
reg = <0x1012 0x1000>;
interrupts = <16>;
clocks = <&osc1>, <&pclk>;
-   clock-names = "clcd", "apb_pclk";
+   clock-names = "clcdclk", "apb_pclk";
+   /* 800x600 16bpp @ 36MHz works fine */
+   max-memory-bandwidth = <5400>;
+
+   /*
+* This port is routed through a PLD (Programmable
+* Logic Device) that routes the output from the CLCD
+* (after transformations) to the VGA DAC and also an
+* external panel connector. The PLD is essential for
+* supporting RGB565/BGR565.
+*
+* The signals from the port thus reaches two endpoints.
+* The PLD is managed through a few special bits in the
+* FPGA "sysreg".
+*
+* This arrangement can be clearly seen in
+* ARM DUI 0225D, page 3-41, figure 3-19.
+*/
+   port@0 {
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   clcd_pads_panel: endpoint@0 {
+   reg = <0>;
+   remote-endpoint = <&panel_in>;
+   arm,pl11x,tft-r0g0b0-pads = <0 8 16>;
+   };
+   clcd_pads_vga_dac: endpoint@1 {
+   reg = <1>;
+   remote-endpoint = <&vga_bridge_in>;
+   arm,pl11x,tft-r0g0b0-pads = <0 8 16>;
+   };
+   };
};
 
sctl@101e {
@@ -319,8 +388,18 @@
ranges = <0 0x1000 0x1>;
 
sysreg@0 {
-   compatible = "arm,versatile-sysreg", "syscon";
+   compatible = "arm,versatile-sysreg", "syscon", 
"simple-mfd";
reg = <0x0 0x1000>;
+
+   panel: display@0 {
+   compatible = "arm,versatile-tft-panel";
+
+   port {
+   panel_in: endpoint {
+   remote-endpoint = 
<&clcd_pads_panel>;
+   };
+