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

[Abel Vesa]

On 26-09-02 17:50:07, Konrad Dybcio wrote:
> 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: 09d77be56093 ("arm64: dts: qcom: Add support for X1-based Surface Laptop 7 devices")
> > Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
> > ---
> >  arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi
> > index 13d0f09e0c2a..bc684c295b89 100644
> > --- a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi
> > @@ -1324,6 +1324,7 @@ wcd_tx: codec@0,3 {
> >  
> >  &tlmm {
> >  	gpio-reserved-ranges = <44 4>, /* SPI (TPM) */
> > +			       <65 1>, /* EC reset */
> >  			       <238 1>; /* UFS Reset */
> 
> On this device, GPIO_65 is an enable pin for the touchpad regulator
> (or something else that is a pre-requisite of the touchpad working)

OK, so I'll drop all changes for Microsoft devices then.