All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
@ 2015-01-20 10:22 Robby Cai
  2015-01-20 11:38   ` Shawn Guo
  0 siblings, 1 reply; 9+ messages in thread
From: Robby Cai @ 2015-01-20 10:22 UTC (permalink / raw)
  To: linux-pwm; +Cc: jg1.han, lee.jones, shawn.guo, thierry.reding

Hannstar XGA panel is used by default on i.MX6QDL Sabre Auto/Lite/SD
or derived board.

According to its data sheet [1],
a) the dimming duty cycle is 20% - 100%.
This patch changes the brightness-level from 20 to 100 with step 10 to
make it straight-forward.
b) the typical dimming frequency is 20KHz.
The previous setting is 200Hz. This patch corrects it.

[1]
http://boundarydevices.com/wp-content/uploads/2013/06/HSD100PXN1-A00-C11-1.0.pdf

Signed-off-by: Robby Cai <r63905@freescale.com>
---
 arch/arm/boot/dts/imx6qdl-gw52xx.dtsi     | 6 +++---
 arch/arm/boot/dts/imx6qdl-gw53xx.dtsi     | 6 +++---
 arch/arm/boot/dts/imx6qdl-gw54xx.dtsi     | 6 +++---
 arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 6 +++---
 arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 6 +++---
 arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 6 +++---
 arch/arm/boot/dts/imx6qdl-sabresd.dtsi    | 6 +++---
 7 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
index b5756c2..33484d8 100644
--- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
@@ -29,9 +29,9 @@
 
 	backlight {
 		compatible = "pwm-backlight";
-		pwms = <&pwm4 0 5000000>;
-		brightness-levels = <0 4 8 16 32 64 128 255>;
-		default-brightness-level = <7>;
+		pwms = <&pwm4 0 50000>;
+		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
+		default-brightness-level = <9>;
 	};
 
 	leds {
diff --git a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
index 86f03c1..0d01751 100644
--- a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
@@ -30,9 +30,9 @@
 
 	backlight {
 		compatible = "pwm-backlight";
-		pwms = <&pwm4 0 5000000>;
-		brightness-levels = <0 4 8 16 32 64 128 255>;
-		default-brightness-level = <7>;
+		pwms = <&pwm4 0 50000>;
+		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
+		default-brightness-level = <9>;
 	};
 
 	leds {
diff --git a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
index 4a8d97f..8973bf2 100644
--- a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
@@ -30,9 +30,9 @@
 
 	backlight {
 		compatible = "pwm-backlight";
-		pwms = <&pwm4 0 5000000>;
-		brightness-levels = <0 4 8 16 32 64 128 255>;
-		default-brightness-level = <7>;
+		pwms = <&pwm4 0 50000>;
+		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
+		default-brightness-level = <9>;
 	};
 
 	leds {
diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
index 0821812..5453cf0 100644
--- a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
@@ -124,9 +124,9 @@
 
 	backlight_lvds {
 		compatible = "pwm-backlight";
-		pwms = <&pwm4 0 5000000>;
-		brightness-levels = <0 4 8 16 32 64 128 255>;
-		default-brightness-level = <7>;
+		pwms = <&pwm4 0 50000>;
+		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
+		default-brightness-level = <9>;
 		power-supply = <&reg_3p3v>;
 		status = "okay";
 	};
diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
index 009abd6..2bfcf5a 100644
--- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
@@ -38,9 +38,9 @@
 
 	backlight {
 		compatible = "pwm-backlight";
-		pwms = <&pwm3 0 5000000>;
-		brightness-levels = <0 4 8 16 32 64 128 255>;
-		default-brightness-level = <7>;
+		pwms = <&pwm3 0 50000>;
+		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
+		default-brightness-level = <9>;
 		status = "okay";
 	};
 };
diff --git a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
index 0b28a9d..d1409c0 100644
--- a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
@@ -123,9 +123,9 @@
 
 	backlight_lvds {
 		compatible = "pwm-backlight";
-		pwms = <&pwm4 0 5000000>;
-		brightness-levels = <0 4 8 16 32 64 128 255>;
-		default-brightness-level = <7>;
+		pwms = <&pwm4 0 50000>;
+		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
+		default-brightness-level = <9>;
 		power-supply = <&reg_3p3v>;
 		status = "okay";
 	};
diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
index f1cd214..3efc363 100644
--- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
@@ -115,9 +115,9 @@
 
 	backlight {
 		compatible = "pwm-backlight";
-		pwms = <&pwm1 0 5000000>;
-		brightness-levels = <0 4 8 16 32 64 128 255>;
-		default-brightness-level = <7>;
+		pwms = <&pwm1 0 50000>;
+		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
+		default-brightness-level = <9>;
 		status = "okay";
 	};
 
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
  2015-01-20 10:22 [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel Robby Cai
@ 2015-01-20 11:38   ` Shawn Guo
  0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2015-01-20 11:38 UTC (permalink / raw)
  To: Robby Cai
  Cc: linux-pwm, jg1.han, lee.jones, thierry.reding, linux-arm-kernel,
	Tim Harvey, Eric Nelson, Troy Kisky, kernel

Add LAKML and a few interested people.

Shawn

On Tue, Jan 20, 2015 at 06:22:35PM +0800, Robby Cai wrote:
> Hannstar XGA panel is used by default on i.MX6QDL Sabre Auto/Lite/SD
> or derived board.
> 
> According to its data sheet [1],
> a) the dimming duty cycle is 20% - 100%.
> This patch changes the brightness-level from 20 to 100 with step 10 to
> make it straight-forward.
> b) the typical dimming frequency is 20KHz.
> The previous setting is 200Hz. This patch corrects it.
> 
> [1]
> http://boundarydevices.com/wp-content/uploads/2013/06/HSD100PXN1-A00-C11-1.0.pdf
> 
> Signed-off-by: Robby Cai <r63905@freescale.com>
> ---
>  arch/arm/boot/dts/imx6qdl-gw52xx.dtsi     | 6 +++---
>  arch/arm/boot/dts/imx6qdl-gw53xx.dtsi     | 6 +++---
>  arch/arm/boot/dts/imx6qdl-gw54xx.dtsi     | 6 +++---
>  arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 6 +++---
>  arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 6 +++---
>  arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 6 +++---
>  arch/arm/boot/dts/imx6qdl-sabresd.dtsi    | 6 +++---
>  7 files changed, 21 insertions(+), 21 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> index b5756c2..33484d8 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> @@ -29,9 +29,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  	};
>  
>  	leds {
> diff --git a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> index 86f03c1..0d01751 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> @@ -30,9 +30,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  	};
>  
>  	leds {
> diff --git a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> index 4a8d97f..8973bf2 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> @@ -30,9 +30,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  	};
>  
>  	leds {
> diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
> index 0821812..5453cf0 100644
> --- a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
> @@ -124,9 +124,9 @@
>  
>  	backlight_lvds {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		power-supply = <&reg_3p3v>;
>  		status = "okay";
>  	};
> diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> index 009abd6..2bfcf5a 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> @@ -38,9 +38,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm3 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm3 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		status = "okay";
>  	};
>  };
> diff --git a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
> index 0b28a9d..d1409c0 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
> @@ -123,9 +123,9 @@
>  
>  	backlight_lvds {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		power-supply = <&reg_3p3v>;
>  		status = "okay";
>  	};
> diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> index f1cd214..3efc363 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> @@ -115,9 +115,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm1 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm1 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		status = "okay";
>  	};
>  
> -- 
> 1.9.1
> 

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
@ 2015-01-20 11:38   ` Shawn Guo
  0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2015-01-20 11:38 UTC (permalink / raw)
  To: linux-arm-kernel

Add LAKML and a few interested people.

Shawn

On Tue, Jan 20, 2015 at 06:22:35PM +0800, Robby Cai wrote:
> Hannstar XGA panel is used by default on i.MX6QDL Sabre Auto/Lite/SD
> or derived board.
> 
> According to its data sheet [1],
> a) the dimming duty cycle is 20% - 100%.
> This patch changes the brightness-level from 20 to 100 with step 10 to
> make it straight-forward.
> b) the typical dimming frequency is 20KHz.
> The previous setting is 200Hz. This patch corrects it.
> 
> [1]
> http://boundarydevices.com/wp-content/uploads/2013/06/HSD100PXN1-A00-C11-1.0.pdf
> 
> Signed-off-by: Robby Cai <r63905@freescale.com>
> ---
>  arch/arm/boot/dts/imx6qdl-gw52xx.dtsi     | 6 +++---
>  arch/arm/boot/dts/imx6qdl-gw53xx.dtsi     | 6 +++---
>  arch/arm/boot/dts/imx6qdl-gw54xx.dtsi     | 6 +++---
>  arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 6 +++---
>  arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 6 +++---
>  arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 6 +++---
>  arch/arm/boot/dts/imx6qdl-sabresd.dtsi    | 6 +++---
>  7 files changed, 21 insertions(+), 21 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> index b5756c2..33484d8 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> @@ -29,9 +29,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  	};
>  
>  	leds {
> diff --git a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> index 86f03c1..0d01751 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> @@ -30,9 +30,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  	};
>  
>  	leds {
> diff --git a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> index 4a8d97f..8973bf2 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> @@ -30,9 +30,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  	};
>  
>  	leds {
> diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
> index 0821812..5453cf0 100644
> --- a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi
> @@ -124,9 +124,9 @@
>  
>  	backlight_lvds {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		power-supply = <&reg_3p3v>;
>  		status = "okay";
>  	};
> diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> index 009abd6..2bfcf5a 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi
> @@ -38,9 +38,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm3 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm3 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		status = "okay";
>  	};
>  };
> diff --git a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
> index 0b28a9d..d1409c0 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabrelite.dtsi
> @@ -123,9 +123,9 @@
>  
>  	backlight_lvds {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm4 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm4 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		power-supply = <&reg_3p3v>;
>  		status = "okay";
>  	};
> diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> index f1cd214..3efc363 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> @@ -115,9 +115,9 @@
>  
>  	backlight {
>  		compatible = "pwm-backlight";
> -		pwms = <&pwm1 0 5000000>;
> -		brightness-levels = <0 4 8 16 32 64 128 255>;
> -		default-brightness-level = <7>;
> +		pwms = <&pwm1 0 50000>;
> +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> +		default-brightness-level = <9>;
>  		status = "okay";
>  	};
>  
> -- 
> 1.9.1
> 

^ permalink raw reply	[flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
  2015-01-20 11:38   ` Shawn Guo
@ 2015-01-22 15:38     ` Philipp Zabel
  -1 siblings, 0 replies; 9+ messages in thread
From: Philipp Zabel @ 2015-01-22 15:38 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Robby Cai, linux-pwm, Eric Nelson, Tim Harvey, jg1.han,
	Troy Kisky, thierry.reding, kernel, lee.jones, linux-arm-kernel

Am Dienstag, den 20.01.2015, 19:38 +0800 schrieb Shawn Guo:
> Add LAKML and a few interested people.
> 
> Shawn
> 
> On Tue, Jan 20, 2015 at 06:22:35PM +0800, Robby Cai wrote:
> > Hannstar XGA panel is used by default on i.MX6QDL Sabre Auto/Lite/SD
> > or derived board.
> > 
> > According to its data sheet [1],
> > a) the dimming duty cycle is 20% - 100%.
> > This patch changes the brightness-level from 20 to 100 with step 10 to
> > make it straight-forward.
> > b) the typical dimming frequency is 20KHz.
> > The previous setting is 200Hz. This patch corrects it.
> > 
> > [1]
> > http://boundarydevices.com/wp-content/uploads/2013/06/HSD100PXN1-A00-C11-1.0.pdf
> > 
> > Signed-off-by: Robby Cai <r63905@freescale.com>
> > ---
> >  arch/arm/boot/dts/imx6qdl-gw52xx.dtsi     | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-gw53xx.dtsi     | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-gw54xx.dtsi     | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-sabresd.dtsi    | 6 +++---
> >  7 files changed, 21 insertions(+), 21 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > index b5756c2..33484d8 100644
> > --- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > +++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > @@ -29,9 +29,9 @@
> >  
> >  	backlight {
> >  		compatible = "pwm-backlight";
> > -		pwms = <&pwm4 0 5000000>;
> > -		brightness-levels = <0 4 8 16 32 64 128 255>;
> > -		default-brightness-level = <7>;
> > +		pwms = <&pwm4 0 50000>;
> > +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> > +		default-brightness-level = <9>;

I notice that the intervals are changed from exponential to linear.
Is this really wanted here?
I expect that the new settings result in huge steps in perceived
brightness at low levels and a nearly imperceptible change at the
last step.

regards
Philipp


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
@ 2015-01-22 15:38     ` Philipp Zabel
  0 siblings, 0 replies; 9+ messages in thread
From: Philipp Zabel @ 2015-01-22 15:38 UTC (permalink / raw)
  To: linux-arm-kernel

Am Dienstag, den 20.01.2015, 19:38 +0800 schrieb Shawn Guo:
> Add LAKML and a few interested people.
> 
> Shawn
> 
> On Tue, Jan 20, 2015 at 06:22:35PM +0800, Robby Cai wrote:
> > Hannstar XGA panel is used by default on i.MX6QDL Sabre Auto/Lite/SD
> > or derived board.
> > 
> > According to its data sheet [1],
> > a) the dimming duty cycle is 20% - 100%.
> > This patch changes the brightness-level from 20 to 100 with step 10 to
> > make it straight-forward.
> > b) the typical dimming frequency is 20KHz.
> > The previous setting is 200Hz. This patch corrects it.
> > 
> > [1]
> > http://boundarydevices.com/wp-content/uploads/2013/06/HSD100PXN1-A00-C11-1.0.pdf
> > 
> > Signed-off-by: Robby Cai <r63905@freescale.com>
> > ---
> >  arch/arm/boot/dts/imx6qdl-gw52xx.dtsi     | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-gw53xx.dtsi     | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-gw54xx.dtsi     | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 6 +++---
> >  arch/arm/boot/dts/imx6qdl-sabresd.dtsi    | 6 +++---
> >  7 files changed, 21 insertions(+), 21 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > index b5756c2..33484d8 100644
> > --- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > +++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > @@ -29,9 +29,9 @@
> >  
> >  	backlight {
> >  		compatible = "pwm-backlight";
> > -		pwms = <&pwm4 0 5000000>;
> > -		brightness-levels = <0 4 8 16 32 64 128 255>;
> > -		default-brightness-level = <7>;
> > +		pwms = <&pwm4 0 50000>;
> > +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> > +		default-brightness-level = <9>;

I notice that the intervals are changed from exponential to linear.
Is this really wanted here?
I expect that the new settings result in huge steps in perceived
brightness at low levels and a nearly imperceptible change at the
last step.

regards
Philipp

^ permalink raw reply	[flat|nested] 9+ messages in thread
* RE: [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
  2015-01-22 15:38     ` Philipp Zabel
@ 2015-01-26 14:44       ` Robby.Cai at freescale.com
  -1 siblings, 0 replies; 9+ messages in thread
From: Robby.Cai @ 2015-01-26 14:44 UTC (permalink / raw)
  To: Philipp Zabel, Shawn Guo
  Cc: linux-pwm@vger.kernel.org, Eric Nelson, Tim Harvey,
	jg1.han@samsung.com, Troy Kisky, thierry.reding@gmail.com,
	kernel@pengutronix.de, lee.jones@linaro.org,
	linux-arm-kernel@lists.infradead.org



> -----Original Message-----
> From: Philipp Zabel [mailto:p.zabel@pengutronix.de]
> Sent: Thursday, January 22, 2015 11:38 PM
> To: Shawn Guo
> Cc: Cai Robby-R63905; linux-pwm@vger.kernel.org; Eric Nelson; Tim Harvey;
> jg1.han@samsung.com; Troy Kisky; thierry.reding@gmail.com;
> kernel@pengutronix.de; lee.jones@linaro.org; linux-arm-
> kernel@lists.infradead.org
> Subject: Re: [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for
> LVDS panel
> 
> Am Dienstag, den 20.01.2015, 19:38 +0800 schrieb Shawn Guo:
> > Add LAKML and a few interested people.
> >
> > Shawn
> >
> > On Tue, Jan 20, 2015 at 06:22:35PM +0800, Robby Cai wrote:
> > > Hannstar XGA panel is used by default on i.MX6QDL Sabre Auto/Lite/SD
> > > or derived board.
> > >
> > > According to its data sheet [1],
> > > a) the dimming duty cycle is 20% - 100%.
> > > This patch changes the brightness-level from 20 to 100 with step 10
> > > to make it straight-forward.
> > > b) the typical dimming frequency is 20KHz.
> > > The previous setting is 200Hz. This patch corrects it.
> > >
> > > [1]
> > > http://boundarydevices.com/wp-content/uploads/2013/06/HSD100PXN1-A00
> > > -C11-1.0.pdf
> > >
> > > Signed-off-by: Robby Cai <r63905@freescale.com>
> > > ---
> > >  arch/arm/boot/dts/imx6qdl-gw52xx.dtsi     | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-gw53xx.dtsi     | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-gw54xx.dtsi     | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 6 +++---
> > > arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 6 +++---
> > > arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-sabresd.dtsi    | 6 +++---
> > >  7 files changed, 21 insertions(+), 21 deletions(-)
> > >
> > > diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > index b5756c2..33484d8 100644
> > > --- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > +++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > @@ -29,9 +29,9 @@
> > >
> > >  	backlight {
> > >  		compatible = "pwm-backlight";
> > > -		pwms = <&pwm4 0 5000000>;
> > > -		brightness-levels = <0 4 8 16 32 64 128 255>;
> > > -		default-brightness-level = <7>;
> > > +		pwms = <&pwm4 0 50000>;
> > > +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> > > +		default-brightness-level = <9>;
> 
> I notice that the intervals are changed from exponential to linear.
> Is this really wanted here?
> I expect that the new settings result in huge steps in perceived
> brightness at low levels and a nearly imperceptible change at the last
> step.
> 
> regards
> Philipp


No particular reason here. I understand your concern but seems there's no obvious rule.
How about add some interpolation, such as <0 52 64 96 128 160 192 224 255>
or even more, <0 52 64 80 96 112 128 144 160 176 192 208 224 240 255>

Regards,
Robby

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
@ 2015-01-26 14:44       ` Robby.Cai at freescale.com
  0 siblings, 0 replies; 9+ messages in thread
From: Robby.Cai at freescale.com @ 2015-01-26 14:44 UTC (permalink / raw)
  To: linux-arm-kernel



> -----Original Message-----
> From: Philipp Zabel [mailto:p.zabel at pengutronix.de]
> Sent: Thursday, January 22, 2015 11:38 PM
> To: Shawn Guo
> Cc: Cai Robby-R63905; linux-pwm at vger.kernel.org; Eric Nelson; Tim Harvey;
> jg1.han at samsung.com; Troy Kisky; thierry.reding at gmail.com;
> kernel at pengutronix.de; lee.jones at linaro.org; linux-arm-
> kernel at lists.infradead.org
> Subject: Re: [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for
> LVDS panel
> 
> Am Dienstag, den 20.01.2015, 19:38 +0800 schrieb Shawn Guo:
> > Add LAKML and a few interested people.
> >
> > Shawn
> >
> > On Tue, Jan 20, 2015 at 06:22:35PM +0800, Robby Cai wrote:
> > > Hannstar XGA panel is used by default on i.MX6QDL Sabre Auto/Lite/SD
> > > or derived board.
> > >
> > > According to its data sheet [1],
> > > a) the dimming duty cycle is 20% - 100%.
> > > This patch changes the brightness-level from 20 to 100 with step 10
> > > to make it straight-forward.
> > > b) the typical dimming frequency is 20KHz.
> > > The previous setting is 200Hz. This patch corrects it.
> > >
> > > [1]
> > > http://boundarydevices.com/wp-content/uploads/2013/06/HSD100PXN1-A00
> > > -C11-1.0.pdf
> > >
> > > Signed-off-by: Robby Cai <r63905@freescale.com>
> > > ---
> > >  arch/arm/boot/dts/imx6qdl-gw52xx.dtsi     | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-gw53xx.dtsi     | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-gw54xx.dtsi     | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 6 +++---
> > > arch/arm/boot/dts/imx6qdl-sabreauto.dtsi  | 6 +++---
> > > arch/arm/boot/dts/imx6qdl-sabrelite.dtsi  | 6 +++---
> > >  arch/arm/boot/dts/imx6qdl-sabresd.dtsi    | 6 +++---
> > >  7 files changed, 21 insertions(+), 21 deletions(-)
> > >
> > > diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > index b5756c2..33484d8 100644
> > > --- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > +++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> > > @@ -29,9 +29,9 @@
> > >
> > >  	backlight {
> > >  		compatible = "pwm-backlight";
> > > -		pwms = <&pwm4 0 5000000>;
> > > -		brightness-levels = <0 4 8 16 32 64 128 255>;
> > > -		default-brightness-level = <7>;
> > > +		pwms = <&pwm4 0 50000>;
> > > +		brightness-levels = <0 20 30 40 50 60 70 80 90 100>;
> > > +		default-brightness-level = <9>;
> 
> I notice that the intervals are changed from exponential to linear.
> Is this really wanted here?
> I expect that the new settings result in huge steps in perceived
> brightness at low levels and a nearly imperceptible change at the last
> step.
> 
> regards
> Philipp


No particular reason here. I understand your concern but seems there's no obvious rule.
How about add some interpolation, such as <0 52 64 96 128 160 192 224 255>
or even more, <0 52 64 80 96 112 128 144 160 176 192 208 224 240 255>

Regards,
Robby

^ permalink raw reply	[flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
  2015-01-26 14:44       ` Robby.Cai at freescale.com
@ 2015-01-27 15:21         ` Philipp Zabel
  -1 siblings, 0 replies; 9+ messages in thread
From: Philipp Zabel @ 2015-01-27 15:21 UTC (permalink / raw)
  To: Robby.Cai@freescale.com
  Cc: Shawn Guo, linux-pwm@vger.kernel.org, Eric Nelson, Tim Harvey,
	jg1.han@samsung.com, Troy Kisky, thierry.reding@gmail.com,
	kernel@pengutronix.de, lee.jones@linaro.org,
	linux-arm-kernel@lists.infradead.org

Am Montag, den 26.01.2015, 14:44 +0000 schrieb Robby.Cai@freescale.com:
> > I notice that the intervals are changed from exponential to linear.
> > Is this really wanted here?
> > I expect that the new settings result in huge steps in perceived
> > brightness at low levels and a nearly imperceptible change at the last
> > step.
> > 
> > regards
> > Philipp
> 
> 
> No particular reason here. I understand your concern but seems there's no obvious rule.
> How about add some interpolation, such as <0 52 64 96 128 160 192 224 255>
> or even more, <0 52 64 80 96 112 128 144 160 176 192 208 224 240 255>

I just wanted to point out the possible issue. Maybe somebody with the
actual panel could sit down and experiment and come up with some
visually pleasing steps.

regards
Philipp


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel
@ 2015-01-27 15:21         ` Philipp Zabel
  0 siblings, 0 replies; 9+ messages in thread
From: Philipp Zabel @ 2015-01-27 15:21 UTC (permalink / raw)
  To: linux-arm-kernel

Am Montag, den 26.01.2015, 14:44 +0000 schrieb Robby.Cai at freescale.com:
> > I notice that the intervals are changed from exponential to linear.
> > Is this really wanted here?
> > I expect that the new settings result in huge steps in perceived
> > brightness at low levels and a nearly imperceptible change at the last
> > step.
> > 
> > regards
> > Philipp
> 
> 
> No particular reason here. I understand your concern but seems there's no obvious rule.
> How about add some interpolation, such as <0 52 64 96 128 160 192 224 255>
> or even more, <0 52 64 80 96 112 128 144 160 176 192 208 224 240 255>

I just wanted to point out the possible issue. Maybe somebody with the
actual panel could sit down and experiment and come up with some
visually pleasing steps.

regards
Philipp

^ permalink raw reply	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-01-27 15:22 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-20 10:22 [PATCH] ARM: dts: imx6qdl: Adjust the brightness setting for LVDS panel Robby Cai
2015-01-20 11:38 ` Shawn Guo
2015-01-20 11:38   ` Shawn Guo
2015-01-22 15:38   ` Philipp Zabel
2015-01-22 15:38     ` Philipp Zabel
2015-01-26 14:44     ` Robby.Cai
2015-01-26 14:44       ` Robby.Cai at freescale.com
2015-01-27 15:21       ` Philipp Zabel
2015-01-27 15:21         ` Philipp Zabel

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.