[PATCH v4 1/2] ARM: tegra: paz00: Add host1x support

[PATCH v4 1/2] ARM: tegra: paz00: Add host1x support

[Marc Dietrich]

This adds host1x support which includes LVDS support. HDMI is disabled
for now as the tegra drm cannot handle two different display timings
at the same time.

Signed-off-by: Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>
---
V4: rebase against for-3.8/DT
V3: move host1x up, revert hdmi ddc clock change

 arch/arm/boot/dts/tegra20-paz00.dts |   11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts
index 6a93d14..8a14e1d 100644
--- a/arch/arm/boot/dts/tegra20-paz00.dts
+++ b/arch/arm/boot/dts/tegra20-paz00.dts
@@ -10,6 +10,15 @@
 		reg = <0x00000000 0x20000000>;
 	};
 
+	host1x {
+		dc@54200000 {
+			rgb {
+				status = "okay";
+				nvidia,ddc-i2c-bus = <&lvds_ddc>;
+			};
+		};
+	};
+
 	pinmux {
 		pinctrl-names = "default";
 		pinctrl-0 = <&state_default>;
@@ -240,7 +249,7 @@
 		clock-frequency = <216000000>;
 	};
 
-	i2c@7000c000 {
+	lvds_ddc: i2c@7000c000 {
 		status = "okay";
 		clock-frequency = <400000>;
 
-- 
1.7.9.5

[PATCH v4 2/2] ARM: tegra: paz00: enable backlight

[Marc Dietrich]

This enables backlight using power sequences.

Signed-off-by: Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>
---
 arch/arm/boot/dts/tegra20-paz00.dts |   72 +++++++++++++++++++++++++++++++++++
 1 file changed, 72 insertions(+)

diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts
index 8a14e1d..7349aa3 100644
--- a/arch/arm/boot/dts/tegra20-paz00.dts
+++ b/arch/arm/boot/dts/tegra20-paz00.dts
@@ -15,6 +15,10 @@
 			rgb {
 				status = "okay";
 				nvidia,ddc-i2c-bus = <&lvds_ddc>;
+
+				display {
+					nvidia,backlight = <&backlight>;
+				};
 			};
 		};
 	};
@@ -461,6 +465,74 @@
 		};
 	};
 
+	backlight: backlight {
+		compatible = "pwm-backlight";
+		brightness-levels = <0 16 32 48 64 80 96 112 128 144 160 176 192 208 224 240 255>;
+		default-brightness-level = <12>;
+
+		/* resources used by the power sequences */
+		pwms = <&pwm 0 5000000>;
+		pwm-names = "backlight";
+
+		power-sequences {
+			power-on {
+				step0 {
+					type = "pwm";
+					id = "backlight";
+					enable;
+				};
+				step1 {
+					type = "gpio";
+					gpio = <&gpio 4 0>;   /* gpio PA4, en_vdd_pnl  */
+					value = <1>;
+				};
+				step2 {
+					type = "gpio";
+					gpio = <&gpio 176 0>; /* gpio PW0, bl_vdd */
+					value = <1>;
+				};
+				step3 {
+					type = "gpio";
+					gpio = <&gpio 102 0>; /* gpio PM6, lvds_shdn */
+					value = <1>;
+				};
+				step4 {
+					type = "gpio";
+					gpio = <&gpio 164 0>; /* gpio PU4, bl_enb */
+					value = <1>;
+				};
+			};
+
+			power-off {
+				step4 {
+					type = "gpio";
+					gpio = <&gpio 164 0>; /* gpio PU4, bl_enb */
+					value = <0>;
+				};
+				step0 {
+					type = "gpio";
+					gpio = <&gpio 102 0>; /* gpio PM6, lvds_shdn */
+					value = <0>;
+				};
+				step1 {
+					type = "gpio";
+					gpio = <&gpio 176 0>; /* gpio PW0, bl_vdd */
+					value = <0>;
+				};
+				step2 {
+					type = "gpio";
+					gpio = <&gpio 4 0>;   /* gpio PA4, en_vdd_pnl  */
+					value = <0>;
+				};
+				step3 {
+					type = "pwm";
+					id = "backlight";
+					disable;
+				};
+			};
+		};
+	};
+
 	regulators {
 		compatible = "simple-bus";
 		#address-cells = <1>;
-- 
1.7.9.5

Re: [PATCH v4 2/2] ARM: tegra: paz00: enable backlight

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 400 bytes --]

On Sat, Nov 17, 2012 at 12:20:32AM +0100, Marc Dietrich wrote:
[...]
> +				step1 {
> +					type = "gpio";
> +					gpio = <&gpio 4 0>;   /* gpio PA4, en_vdd_pnl  */
> +					value = <1>;
> +				};
> +				step2 {
> +					type = "gpio";
> +					gpio = <&gpio 176 0>; /* gpio PW0, bl_vdd */
> +					value = <1>;
> +				};

These look like regulators, so maybe they should be abstracted as such?

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH v4 2/2] ARM: tegra: paz00: enable backlight

[Marc Dietrich]

On Saturday 17 November 2012 08:35:34 Thierry Reding wrote:
> On Sat, Nov 17, 2012 at 12:20:32AM +0100, Marc Dietrich wrote:
> [...]
> 
> > +				step1 {
> > +					type = "gpio";
> > +					gpio = <&gpio 4 0>;   /* gpio PA4, en_vdd_pnl  */
> > +					value = <1>;
> > +				};
> > +				step2 {
> > +					type = "gpio";
> > +					gpio = <&gpio 176 0>; /* gpio PW0, bl_vdd */
> > +					value = <1>;
> > +				};
> 
> These look like regulators, so maybe they should be abstracted as such?

Honstly, I never understood why we need to redefine a gpio as a regulator. 
IMHO, gpios don't regulate, they switch 0/1. But yes, these gpios just go to 
some FET which switches on some supply voltage.

Marc

Re: [PATCH v4 2/2] ARM: tegra: paz00: enable backlight

[Marc Dietrich]

Hi,

since backlight will only come in 3.9, I can add some remarks myself, so I 
don't forget them next time.

On Saturday 17 November 2012 00:20:32 Marc Dietrich wrote:
> This enables backlight using power sequences.
> 
> Signed-off-by: Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>
> ---
>  arch/arm/boot/dts/tegra20-paz00.dts |   72
> +++++++++++++++++++++++++++++++++++ 1 file changed, 72 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/tegra20-paz00.dts
> b/arch/arm/boot/dts/tegra20-paz00.dts index 8a14e1d..7349aa3 100644
> --- a/arch/arm/boot/dts/tegra20-paz00.dts
> +++ b/arch/arm/boot/dts/tegra20-paz00.dts
> @@ -15,6 +15,10 @@
>  			rgb {
>  				status = "okay";
>  				nvidia,ddc-i2c-bus = <&lvds_ddc>;
> +
> +				display {
> +					nvidia,backlight = <&backlight>;
> +				};
>  			};
>  		};
>  	};
> @@ -461,6 +465,74 @@
>  		};
>  	};
> 
> +	backlight: backlight {
> +		compatible = "pwm-backlight";
> +		brightness-levels = <0 16 32 48 64 80 96 112 128 144 160 176 192 208 
224
> 240 255>; +		default-brightness-level = <12>;
> +
> +		/* resources used by the power sequences */
> +		pwms = <&pwm 0 5000000>;
> +		pwm-names = "backlight";
> +
> +		power-sequences {
> +			power-on {
> +				step0 {
> +					type = "pwm";
> +					id = "backlight";
> +					enable;
> +				};
> +				step1 {
> +					type = "gpio";
> +					gpio = <&gpio 4 0>;   /* gpio PA4, en_vdd_pnl  */
> +					value = <1>;
> +				};
> +				step2 {
> +					type = "gpio";
> +					gpio = <&gpio 176 0>; /* gpio PW0, bl_vdd */

this one needs to be removed as it is not used. There is bl_pwm (gpio U3) 
however which I cannot sort in here.

> +					value = <1>;
> +				};
> +				step3 {
> +					type = "gpio";
> +					gpio = <&gpio 102 0>; /* gpio PM6, lvds_shdn */
> +					value = <1>;
> +				};
> +				step4 {
> +					type = "gpio";
> +					gpio = <&gpio 164 0>; /* gpio PU4, bl_enb */
> +					value = <1>;
> +				};
> +			};
> +
> +			power-off {
> +				step4 {
> +					type = "gpio";
> +					gpio = <&gpio 164 0>; /* gpio PU4, bl_enb */
> +					value = <0>;

and this is totally bogus.

/me notes: better do not send patches after midnight!

Marc