diff mbox

[V3] ARM: dts: Add DT changes for display on peach_pi

Message ID 1407145638-3803-1-git-send-email-ajaykumar.rs@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ajay Kumar Aug. 4, 2014, 9:47 a.m. UTC
Add DT nodes for panel-simple "auo,b133htn01" panel.
Add backlight enable pin and backlight power supply for pwm-backlight.
Also, add panel phandle needed by dp to enable display on peach_pi.

Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
---
Changes since V1:
Remove "simple-panel" compatible string and use only ""auo,b133htn01".

Changes since V2:
As suggested by Andreas,
Use GPIO_ACTIVE_HIGH for backlight enable-gpios instead of using 0.
Change panel node naming from "panel-simple" to "panel".

 arch/arm/boot/dts/exynos5800-peach-pi.dts |   27 ++++++++++-----------------
 1 file changed, 10 insertions(+), 17 deletions(-)

Comments

Ajay kumar Aug. 25, 2014, 6:13 a.m. UTC | #1
On Mon, Aug 4, 2014 at 3:17 PM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote:
> Add DT nodes for panel-simple "auo,b133htn01" panel.
> Add backlight enable pin and backlight power supply for pwm-backlight.
> Also, add panel phandle needed by dp to enable display on peach_pi.
>
> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
> ---
> Changes since V1:
> Remove "simple-panel" compatible string and use only ""auo,b133htn01".
>
> Changes since V2:
> As suggested by Andreas,
> Use GPIO_ACTIVE_HIGH for backlight enable-gpios instead of using 0.
> Change panel node naming from "panel-simple" to "panel".
>
>  arch/arm/boot/dts/exynos5800-peach-pi.dts |   27 ++++++++++-----------------
>  1 file changed, 10 insertions(+), 17 deletions(-)
>
> diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
> index f3ee48b..f8c2e61 100644
> --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
> +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
> @@ -28,11 +28,13 @@
>                 i2c20 = "/spi@12d40000/cros-ec@0/i2c-tunnel";
>         };
>
> -       backlight {
> +       backlight: backlight {
>                 compatible = "pwm-backlight";
>                 pwms = <&pwm 0 1000000 0>;
>                 brightness-levels = <0 100 500 1000 1500 2000 2500 2800>;
>                 default-brightness-level = <7>;
> +               enable-gpios = <&gpx2 2 GPIO_ACTIVE_HIGH>;
> +               power-supply = <&tps65090_fet1>;
>                 pinctrl-0 = <&pwm0_out>;
>                 pinctrl-names = "default";
>         };
> @@ -98,6 +100,12 @@
>                 regulator-boot-on;
>                 regulator-always-on;
>         };
> +
> +       panel: panel {
> +               compatible = "auo,b133htn01";
> +               power-supply = <&tps65090_fet6>;
> +               backlight = <&backlight>;
> +       };
>  };
>
>  &dp {
> @@ -111,22 +119,7 @@
>         samsung,link-rate = <0x0a>;
>         samsung,lane-count = <2>;
>         samsung,hpd-gpio = <&gpx2 6 0>;
> -
> -       display-timings {
> -               native-mode = <&timing1>;
> -
> -               timing1: timing@1 {
> -                       clock-frequency = <150660000>;
> -                       hactive = <1920>;
> -                       vactive = <1080>;
> -                       hfront-porch = <60>;
> -                       hback-porch = <172>;
> -                       hsync-len = <80>;
> -                       vback-porch = <25>;
> -                       vfront-porch = <10>;
> -                       vsync-len = <10>;
> -               };
> -       };
> +       panel = <&panel>;
>  };
>
>  &fimd {
> --
> 1.7.9.5
>
Kim Kukjin Sept. 14, 2014, 12:23 a.m. UTC | #2
On 08/25/14 15:13, Ajay kumar wrote:
> On Mon, Aug 4, 2014 at 3:17 PM, Ajay Kumar<ajaykumar.rs@samsung.com>  wrote:
>> Add DT nodes for panel-simple "auo,b133htn01" panel.
>> Add backlight enable pin and backlight power supply for pwm-backlight.
>> Also, add panel phandle needed by dp to enable display on peach_pi.
>>
>> Signed-off-by: Ajay Kumar<ajaykumar.rs@samsung.com>
>> ---
>> Changes since V1:
>> Remove "simple-panel" compatible string and use only ""auo,b133htn01".
>>
>> Changes since V2:
>> As suggested by Andreas,
>> Use GPIO_ACTIVE_HIGH for backlight enable-gpios instead of using 0.
>> Change panel node naming from "panel-simple" to "panel".
>>
>>   arch/arm/boot/dts/exynos5800-peach-pi.dts |   27 ++++++++++-----------------
>>   1 file changed, 10 insertions(+), 17 deletions(-)
>>
Applied, thanks.

- Kukjin
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
index f3ee48b..f8c2e61 100644
--- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
+++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
@@ -28,11 +28,13 @@ 
 		i2c20 = "/spi@12d40000/cros-ec@0/i2c-tunnel";
 	};
 
-	backlight {
+	backlight: backlight {
 		compatible = "pwm-backlight";
 		pwms = <&pwm 0 1000000 0>;
 		brightness-levels = <0 100 500 1000 1500 2000 2500 2800>;
 		default-brightness-level = <7>;
+		enable-gpios = <&gpx2 2 GPIO_ACTIVE_HIGH>;
+		power-supply = <&tps65090_fet1>;
 		pinctrl-0 = <&pwm0_out>;
 		pinctrl-names = "default";
 	};
@@ -98,6 +100,12 @@ 
 		regulator-boot-on;
 		regulator-always-on;
 	};
+
+	panel: panel {
+		compatible = "auo,b133htn01";
+		power-supply = <&tps65090_fet6>;
+		backlight = <&backlight>;
+	};
 };
 
 &dp {
@@ -111,22 +119,7 @@ 
 	samsung,link-rate = <0x0a>;
 	samsung,lane-count = <2>;
 	samsung,hpd-gpio = <&gpx2 6 0>;
-
-	display-timings {
-		native-mode = <&timing1>;
-
-		timing1: timing@1 {
-			clock-frequency = <150660000>;
-			hactive = <1920>;
-			vactive = <1080>;
-			hfront-porch = <60>;
-			hback-porch = <172>;
-			hsync-len = <80>;
-			vback-porch = <25>;
-			vfront-porch = <10>;
-			vsync-len = <10>;
-		};
-	};
+	panel = <&panel>;
 };
 
 &fimd {