Re: [PATCH v3 07/13] arm64: dts: qcom: medion-sprchrgd-14-s1: Mark the EC reset gpio as reserved

[Georg Gottleuber]

Am 22.09.26 um 09:51 schrieb Abel Vesa:
> GPIO 65 from TLMM is used by the embedded controller as its reset line.
> 
> On some devices, toggling the embedded controller reset line makes the
> device not power up at all, unless the battery is unplugged.
> 
> So reserve it in order to prevent Linux from requesting or
> reconfiguring it.
> 
> Fixes: 8cf65490cdb0 ("arm64: dts: qcom: Add dts for Medion SPRCHRGD 14 S1")
> Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/x1e80100-medion-sprchrgd-14-s1.dts | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/x1e80100-medion-sprchrgd-14-s1.dts b/arch/arm64/boot/dts/qcom/x1e80100-medion-sprchrgd-14-s1.dts
> index 6d26d73c18fc..9b599917c15a 100644
> --- a/arch/arm64/boot/dts/qcom/x1e80100-medion-sprchrgd-14-s1.dts
> +++ b/arch/arm64/boot/dts/qcom/x1e80100-medion-sprchrgd-14-s1.dts
> @@ -1268,6 +1268,7 @@ &tcsr {
>  &tlmm {
>  	gpio-reserved-ranges = <28 4>, /* Unused */
>  			       <44 4>, /* SPI (TPM) */
> +			       <65 1>, /* EC reset */
>  			       <238 1>; /* UFS Reset */
>  
>  	edp_reg_en: edp-reg-en-state {
> 

Looks good to me.

Reviewed-by: Georg Gottleuber <ggo@xxxxxxxxxxxxxxxxxxx>