diff mbox

[1/2] ARM: dts: Update display clock frequency for Origen-4210

Message ID 1380171018-24229-1-git-send-email-sachin.kamat@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sachin Kamat Sept. 26, 2013, 4:50 a.m. UTC
From: Tushar Behera <tushar.behera@linaro.org>

As per the timing information for supported panel, the value should
be between 47.2 MHz to 47.9 MHz for 60Hz refresh rate.

Total horizontal pixels = 1024 (x-res) + 80 (margin) + 48 (hsync) = 1152
Total vertical pixels = 600 (y-res) + 80 (margin) + 3 (vsync) = 653

Target pixel clock rate for refresh rate @60 Hz
	= 1152 * 653 * 60 = 47208960 Hz ~ 47.5 MHz

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 arch/arm/boot/dts/exynos4210-origen.dts |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sachin Kamat Oct. 11, 2013, 3:59 a.m. UTC | #1
Kukjin,

Without these patches display is broken on these boards. Please add them.


On 26 September 2013 10:20, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> From: Tushar Behera <tushar.behera@linaro.org>
>
> As per the timing information for supported panel, the value should
> be between 47.2 MHz to 47.9 MHz for 60Hz refresh rate.
>
> Total horizontal pixels = 1024 (x-res) + 80 (margin) + 48 (hsync) = 1152
> Total vertical pixels = 600 (y-res) + 80 (margin) + 3 (vsync) = 653
>
> Target pixel clock rate for refresh rate @60 Hz
>         = 1152 * 653 * 60 = 47208960 Hz ~ 47.5 MHz
>
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  arch/arm/boot/dts/exynos4210-origen.dts |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos4210-origen.dts b/arch/arm/boot/dts/exynos4210-origen.dts
> index 382d8c7..6906e8a 100644
> --- a/arch/arm/boot/dts/exynos4210-origen.dts
> +++ b/arch/arm/boot/dts/exynos4210-origen.dts
> @@ -301,7 +301,7 @@
>         display-timings {
>                 native-mode = <&timing0>;
>                 timing0: timing {
> -                       clock-frequency = <50000>;
> +                       clock-frequency = <47500000>;
>                         hactive = <1024>;
>                         vactive = <600>;
>                         hfront-porch = <64>;
> --
> 1.7.9.5
>
Sachin Kamat Oct. 21, 2013, 3:04 a.m. UTC | #2
Hi Kukjin,

On 26 September 2013 10:20, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> From: Tushar Behera <tushar.behera@linaro.org>
>
> As per the timing information for supported panel, the value should
> be between 47.2 MHz to 47.9 MHz for 60Hz refresh rate.
>
> Total horizontal pixels = 1024 (x-res) + 80 (margin) + 48 (hsync) = 1152
> Total vertical pixels = 600 (y-res) + 80 (margin) + 3 (vsync) = 653
>
> Target pixel clock rate for refresh rate @60 Hz
>         = 1152 * 653 * 60 = 47208960 Hz ~ 47.5 MHz
>
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  arch/arm/boot/dts/exynos4210-origen.dts |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos4210-origen.dts b/arch/arm/boot/dts/exynos4210-origen.dts
> index 382d8c7..6906e8a 100644
> --- a/arch/arm/boot/dts/exynos4210-origen.dts
> +++ b/arch/arm/boot/dts/exynos4210-origen.dts
> @@ -301,7 +301,7 @@
>         display-timings {
>                 native-mode = <&timing0>;
>                 timing0: timing {
> -                       clock-frequency = <50000>;
> +                       clock-frequency = <47500000>;
>                         hactive = <1024>;
>                         vactive = <600>;
>                         hfront-porch = <64>;
> --
> 1.7.9.5
>

Since these patches are essential fixes, can you please apply these.
Sachin Kamat Oct. 31, 2013, 8:45 a.m. UTC | #3
On 26 September 2013 10:20, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> From: Tushar Behera <tushar.behera@linaro.org>
>
> As per the timing information for supported panel, the value should
> be between 47.2 MHz to 47.9 MHz for 60Hz refresh rate.
>
> Total horizontal pixels = 1024 (x-res) + 80 (margin) + 48 (hsync) = 1152
> Total vertical pixels = 600 (y-res) + 80 (margin) + 3 (vsync) = 653
>
> Target pixel clock rate for refresh rate @60 Hz
>         = 1152 * 653 * 60 = 47208960 Hz ~ 47.5 MHz
>
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  arch/arm/boot/dts/exynos4210-origen.dts |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos4210-origen.dts b/arch/arm/boot/dts/exynos4210-origen.dts
> index 382d8c7..6906e8a 100644
> --- a/arch/arm/boot/dts/exynos4210-origen.dts
> +++ b/arch/arm/boot/dts/exynos4210-origen.dts
> @@ -301,7 +301,7 @@
>         display-timings {
>                 native-mode = <&timing0>;
>                 timing0: timing {
> -                       clock-frequency = <50000>;
> +                       clock-frequency = <47500000>;
>                         hactive = <1024>;
>                         vactive = <600>;
>                         hfront-porch = <64>;
> --
> 1.7.9.5
>

Ping Kukjin.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos4210-origen.dts b/arch/arm/boot/dts/exynos4210-origen.dts
index 382d8c7..6906e8a 100644
--- a/arch/arm/boot/dts/exynos4210-origen.dts
+++ b/arch/arm/boot/dts/exynos4210-origen.dts
@@ -301,7 +301,7 @@ 
 	display-timings {
 		native-mode = <&timing0>;
 		timing0: timing {
-			clock-frequency = <50000>;
+			clock-frequency = <47500000>;
 			hactive = <1024>;
 			vactive = <600>;
 			hfront-porch = <64>;