Re: [PATCH v2 16/16] arm64: dts: qcom: x1p42100-microsoft-sp12in: Mark the EC reset gpio as reserved

[Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>]

On 9/1/26 10:07 AM, Abel Vesa wrote:
> 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: 447f8053b184 ("arm64: dts: qcom: purwa: Add Microsoft Surface Pro 12in")
> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> ---
>  arch/arm64/boot/dts/qcom/x1p42100-microsoft-sp12in.dts | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/x1p42100-microsoft-sp12in.dts b/arch/arm64/boot/dts/qcom/x1p42100-microsoft-sp12in.dts
> index 0314b2246cdb..7f7245773e6d 100644
> --- a/arch/arm64/boot/dts/qcom/x1p42100-microsoft-sp12in.dts
> +++ b/arch/arm64/boot/dts/qcom/x1p42100-microsoft-sp12in.dts
> @@ -970,7 +970,8 @@ right_spkr: speaker@0,1 {
>  
>  &tlmm {
>  	gpio-reserved-ranges = <34 2>, /* Unused */
> -			       <44 4>; /* SPI (TPM) */
> +			       <44 4>, /* SPI (TPM) */
> +			       <65 1>; /* EC reset */

Same comment as the other Surface

Konrad