RE: [PATCH 2/2] ARM: dts: Update display clock frequency for Origen-4412

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sachin Kamat wrote:
> 
+ dt ml.

> 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

Should be 683?

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

> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx>
> ---
>  arch/arm/boot/dts/exynos4412-origen.dts |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/exynos4412-origen.dts
> b/arch/arm/boot/dts/exynos4412-origen.dts
> index 8768b03..923bcd7 100644
> --- a/arch/arm/boot/dts/exynos4412-origen.dts
> +++ b/arch/arm/boot/dts/exynos4412-origen.dts
> @@ -152,7 +152,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

Please make sure it works fine on both origen boards with your two fixes.

If so, let me pick them up into Samsung tree after squash into one patch.

Thanks,
Kukjin

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux