diff mbox

ARM: dts: exynos: correct trats2 panel reset line

Message ID 20171121112424.GA6650@lineageos.org (mailing list archive)
State Accepted
Headers show

Commit Message

Simon Shields Nov. 21, 2017, 11:24 a.m. UTC
trats2 uses gpf2-1 as the panel reset GPIO. gpy4-5 was only used
on early revisions of the board.

Fixes: 420ae8451a22 ("ARM: dts: exynos4412-trats2: add panel node")
Signed-off-by: Simon Shields <simon@lineageos.org>
---
 arch/arm/boot/dts/exynos4412-trats2.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marek Szyprowski Nov. 21, 2017, 3:06 p.m. UTC | #1
Hi Simon,

On 2017-11-21 12:24, Simon Shields wrote:
> trats2 uses gpf2-1 as the panel reset GPIO. gpy4-5 was only used
> on early revisions of the board.
>
> Fixes: 420ae8451a22 ("ARM: dts: exynos4412-trats2: add panel node")
> Signed-off-by: Simon Shields <simon@lineageos.org>

Acked-by: Marek Szyprowski <m.szyprowski@samsung.com>
Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>

It looks that this has been accidentally copy/pasted from the previous
dts used for early version of the development board. I've checked the
Trats2 schematic and indeed gpf2-1 is the correct pin for panel reset
line.

> ---
>   arch/arm/boot/dts/exynos4412-trats2.dts | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts
> index 220cdf109405..9f4672ba9943 100644
> --- a/arch/arm/boot/dts/exynos4412-trats2.dts
> +++ b/arch/arm/boot/dts/exynos4412-trats2.dts
> @@ -454,7 +454,7 @@
>   		reg = <0>;
>   		vdd3-supply = <&lcd_vdd3_reg>;
>   		vci-supply = <&ldo25_reg>;
> -		reset-gpios = <&gpy4 5 GPIO_ACTIVE_HIGH>;
> +		reset-gpios = <&gpf2 1 GPIO_ACTIVE_HIGH>;
>   		power-on-delay= <50>;
>   		reset-delay = <100>;
>   		init-delay = <100>;

Best regards
Krzysztof Kozlowski Nov. 27, 2017, 6:12 p.m. UTC | #2
On Tue, Nov 21, 2017 at 04:06:38PM +0100, Marek Szyprowski wrote:
> Hi Simon,
> 
> On 2017-11-21 12:24, Simon Shields wrote:
> > trats2 uses gpf2-1 as the panel reset GPIO. gpy4-5 was only used
> > on early revisions of the board.
> > 
> > Fixes: 420ae8451a22 ("ARM: dts: exynos4412-trats2: add panel node")
> > Signed-off-by: Simon Shields <simon@lineageos.org>
> 
> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com>
> Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
> 
> It looks that this has been accidentally copy/pasted from the previous
> dts used for early version of the development board. I've checked the
> Trats2 schematic and indeed gpf2-1 is the correct pin for panel reset
> line.

Thanks for testing.

Thanks, applied.

Best regards,
Krzysztof

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

Patch

diff --git a/arch/arm/boot/dts/exynos4412-trats2.dts b/arch/arm/boot/dts/exynos4412-trats2.dts
index 220cdf109405..9f4672ba9943 100644
--- a/arch/arm/boot/dts/exynos4412-trats2.dts
+++ b/arch/arm/boot/dts/exynos4412-trats2.dts
@@ -454,7 +454,7 @@ 
 		reg = <0>;
 		vdd3-supply = <&lcd_vdd3_reg>;
 		vci-supply = <&ldo25_reg>;
-		reset-gpios = <&gpy4 5 GPIO_ACTIVE_HIGH>;
+		reset-gpios = <&gpf2 1 GPIO_ACTIVE_HIGH>;
 		power-on-delay= <50>;
 		reset-delay = <100>;
 		init-delay = <100>;