Re: [RFT][PATCH 1/3] ARM: dts: exynos: fix UART3 pins configuration in Exynos5250

[Marek Szyprowski]

On 30.12.2021 20:53, Krzysztof Kozlowski wrote:
> The gpa1-4 pin was put twice in UART3 pin configuration of Exynos5250,
> instead of proper pin gpa1-5.
>
> Fixes: f8bfe2b050f3 ("ARM: dts: add pin state information in client nodes for Exynos5 platforms")
> Cc: <stable@xxxxxxxxxxxxxxx>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx>

Well, the uart3_data node is not referenced anywhere, so this change is 
not really relevant to any board, but for the completeness, feel free to 
add:

Tested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>


> ---
>   arch/arm/boot/dts/exynos5250-pinctrl.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos5250-pinctrl.dtsi b/arch/arm/boot/dts/exynos5250-pinctrl.dtsi
> index d31a68672bfa..d7d756614edd 100644
> --- a/arch/arm/boot/dts/exynos5250-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/exynos5250-pinctrl.dtsi
> @@ -260,7 +260,7 @@ i2c3_hs_bus: i2c3-hs-bus {
>   	};
>   
>   	uart3_data: uart3-data {
> -		samsung,pins = "gpa1-4", "gpa1-4";
> +		samsung,pins = "gpa1-4", "gpa1-5";
>   		samsung,pin-function = <EXYNOS_PIN_FUNC_2>;
>   		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;
>   		samsung,pin-drv = <EXYNOS4_PIN_DRV_LV1>;

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland