ARM: dts: exynos/i9100: Fix LCD screen's physical size

[PATCH] ARM: dts: exynos/i9100: Fix LCD screen's physical size

Posted by Paul Cercueil 2 years, 6 months ago

The previous values were completely bogus, and resulted in the computed
DPI ratio being much lower than reality, causing applications and UIs to
misbehave.

The new values were measured by myself with a ruler.

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Fixes: 8620cc2f99b7 ("ARM: dts: exynos: Add devicetree file for the Galaxy S2")
Cc: <stable@vger.kernel.org> # v5.8+
---
 arch/arm/boot/dts/samsung/exynos4210-i9100.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/samsung/exynos4210-i9100.dts b/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
index 37cd4dde53e4..a9ec1f6c1dea 100644
--- a/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
+++ b/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
@@ -207,8 +207,8 @@ lcd@0 {
 			power-on-delay = <10>;
 			reset-delay = <10>;
 
-			panel-width-mm = <90>;
-			panel-height-mm = <154>;
+			panel-width-mm = <56>;
+			panel-height-mm = <93>;
 
 			display-timings {
 				timing {
-- 
2.40.1

Re: [PATCH] ARM: dts: exynos/i9100: Fix LCD screen's physical size

Posted by Paul Cercueil 2 years, 6 months ago

Ooops, should have been V3.

V1/V2 was part of my "Galaxy S2 (i9100) panel updates" series. This one
patch did not change since V1 - except the ACK from Sam.

-Paul

Le vendredi 14 juillet 2023 à 17:37 +0200, Paul Cercueil a écrit :
> The previous values were completely bogus, and resulted in the
> computed
> DPI ratio being much lower than reality, causing applications and UIs
> to
> misbehave.
> 
> The new values were measured by myself with a ruler.
> 
> Signed-off-by: Paul Cercueil <paul@crapouillou.net>
> Acked-by: Sam Ravnborg <sam@ravnborg.org>
> Fixes: 8620cc2f99b7 ("ARM: dts: exynos: Add devicetree file for the
> Galaxy S2")
> Cc: <stable@vger.kernel.org> # v5.8+
> ---
>  arch/arm/boot/dts/samsung/exynos4210-i9100.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> b/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> index 37cd4dde53e4..a9ec1f6c1dea 100644
> --- a/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> +++ b/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> @@ -207,8 +207,8 @@ lcd@0 {
>                         power-on-delay = <10>;
>                         reset-delay = <10>;
>  
> -                       panel-width-mm = <90>;
> -                       panel-height-mm = <154>;
> +                       panel-width-mm = <56>;
> +                       panel-height-mm = <93>;
>  
>                         display-timings {
>                                 timing {

Re: [PATCH] ARM: dts: exynos/i9100: Fix LCD screen's physical size

Posted by Krzysztof Kozlowski 2 years, 6 months ago

On Fri, 14 Jul 2023 17:37:20 +0200, Paul Cercueil wrote:
> The previous values were completely bogus, and resulted in the computed
> DPI ratio being much lower than reality, causing applications and UIs to
> misbehave.
> 
> The new values were measured by myself with a ruler.
> 
> 
> [...]

Applied, thanks!

[1/1] ARM: dts: exynos/i9100: Fix LCD screen's physical size
      https://git.kernel.org/krzk/linux/c/b3f3fc32e5ff1e848555af8616318cc667457f90

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>