* [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
@ 2026-09-01 8:06 ` Abel Vesa
2026-09-02 15:53 ` Konrad Dybcio
2026-09-01 8:06 ` [PATCH v2 02/16] arm64: dts: qcom: x1e001de-devkit: " Abel Vesa
` (14 subsequent siblings)
15 siblings, 1 reply; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:06 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: bd50b1f5b6f3 ("arm64: dts: qcom: x1e80100: Add Compute Reference Device")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1-crd.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1-crd.dtsi b/arch/arm64/boot/dts/qcom/x1-crd.dtsi
index 60db83b8985f..d1bed84725e8 100644
--- a/arch/arm64/boot/dts/qcom/x1-crd.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1-crd.dtsi
@@ -1531,6 +1531,7 @@ right_tweeter: speaker@0,1 {
&tlmm {
gpio-reserved-ranges = <34 2>, /* Unused */
<44 4>, /* SPI (TPM) */
+ <65 1>, /* EC reset */
<238 1>; /* UFS Reset */
ec_int_n_default: ec-int-n-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved
2026-09-01 8:06 ` [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved Abel Vesa
@ 2026-09-02 15:53 ` Konrad Dybcio
2026-09-03 8:34 ` Abel Vesa
0 siblings, 1 reply; 27+ messages in thread
From: Konrad Dybcio @ 2026-09-02 15:53 UTC (permalink / raw)
To: Abel Vesa, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Rajendra Nayak, Abel Vesa,
Sibi Sankar, Marc Zyngier, Aleksandrs Vinarskis,
Konstantin Shabanov, Srinivas Kandagatla, Ettore Chimenti,
Georg Gottleuber, Xilin Wu, Bryan O'Donoghue, Val Packett,
Laurentiu Tudor, Jens Glathe, Dale Whinham,
Jérôme de Bretagne, Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
On 9/1/26 10:06 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.
Have you confirmed that behavior on the CRD?
Konrad
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved
2026-09-02 15:53 ` Konrad Dybcio
@ 2026-09-03 8:34 ` Abel Vesa
2026-09-03 9:16 ` Konrad Dybcio
0 siblings, 1 reply; 27+ messages in thread
From: Abel Vesa @ 2026-09-03 8:34 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Harrison Vanderbyl,
linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
On 26-09-02 17:53:23, Konrad Dybcio wrote:
> On 9/1/26 10:06 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.
>
> Have you confirmed that behavior on the CRD?
Toggling it does nothing observable.
However, the documentation says this GPIO is actually tied to EC reset.
I still think we should reserve it, just to be safe.
What do you think?
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved
2026-09-03 8:34 ` Abel Vesa
@ 2026-09-03 9:16 ` Konrad Dybcio
2026-09-03 9:29 ` Abel Vesa
0 siblings, 1 reply; 27+ messages in thread
From: Konrad Dybcio @ 2026-09-03 9:16 UTC (permalink / raw)
To: Abel Vesa
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Harrison Vanderbyl,
linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
On 9/3/26 10:34 AM, Abel Vesa wrote:
> On 26-09-02 17:53:23, Konrad Dybcio wrote:
>> On 9/1/26 10:06 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.
>>
>> Have you confirmed that behavior on the CRD?
>
> Toggling it does nothing observable.
>
> However, the documentation says this GPIO is actually tied to EC reset.
>
> I still think we should reserve it, just to be safe.
>
> What do you think?
I believe that contradicts the reasoning you gave in the commit
message then..
Konrad
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved
2026-09-03 9:16 ` Konrad Dybcio
@ 2026-09-03 9:29 ` Abel Vesa
2026-09-03 9:37 ` Konrad Dybcio
0 siblings, 1 reply; 27+ messages in thread
From: Abel Vesa @ 2026-09-03 9:29 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Harrison Vanderbyl,
linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
On 26-09-03 11:16:20, Konrad Dybcio wrote:
> On 9/3/26 10:34 AM, Abel Vesa wrote:
> > On 26-09-02 17:53:23, Konrad Dybcio wrote:
> >> On 9/1/26 10:06 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.
> >>
> >> Have you confirmed that behavior on the CRD?
> >
> > Toggling it does nothing observable.
> >
> > However, the documentation says this GPIO is actually tied to EC reset.
> >
> > I still think we should reserve it, just to be safe.
> >
> > What do you think?
>
> I believe that contradicts the reasoning you gave in the commit
> message then..
Yep, will rephrase it for this platform as:
While toggling it from linux doesn't seem to have any observable impact,
since documentation does confrm it is connected to EC reset, lets reserve
it just to be on the safe side.
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved
2026-09-03 9:29 ` Abel Vesa
@ 2026-09-03 9:37 ` Konrad Dybcio
2026-09-03 10:28 ` Abel Vesa
0 siblings, 1 reply; 27+ messages in thread
From: Konrad Dybcio @ 2026-09-03 9:37 UTC (permalink / raw)
To: Abel Vesa
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Harrison Vanderbyl,
linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
On 9/3/26 11:29 AM, Abel Vesa wrote:
> On 26-09-03 11:16:20, Konrad Dybcio wrote:
>> On 9/3/26 10:34 AM, Abel Vesa wrote:
>>> On 26-09-02 17:53:23, Konrad Dybcio wrote:
>>>> On 9/1/26 10:06 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.
>>>>
>>>> Have you confirmed that behavior on the CRD?
>>>
>>> Toggling it does nothing observable.
>>>
>>> However, the documentation says this GPIO is actually tied to EC reset.
>>>
>>> I still think we should reserve it, just to be safe.
>>>
>>> What do you think?
>>
>> I believe that contradicts the reasoning you gave in the commit
>> message then..
>
> Yep, will rephrase it for this platform as:
>
> While toggling it from linux doesn't seem to have any observable impact,
> since documentation does confrm it is connected to EC reset, lets reserve
> it just to be on the safe side.
Have you toggled the pin into the correct polarity? The EC FW seems not
to be handling that pin at all, so I'm guessing it should be just a dumb
chip reset
Konrad
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved
2026-09-03 9:37 ` Konrad Dybcio
@ 2026-09-03 10:28 ` Abel Vesa
0 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-03 10:28 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Harrison Vanderbyl,
linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
On 26-09-03 11:37:09, Konrad Dybcio wrote:
> On 9/3/26 11:29 AM, Abel Vesa wrote:
> > On 26-09-03 11:16:20, Konrad Dybcio wrote:
> >> On 9/3/26 10:34 AM, Abel Vesa wrote:
> >>> On 26-09-02 17:53:23, Konrad Dybcio wrote:
> >>>> On 9/1/26 10:06 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.
> >>>>
> >>>> Have you confirmed that behavior on the CRD?
> >>>
> >>> Toggling it does nothing observable.
> >>>
> >>> However, the documentation says this GPIO is actually tied to EC reset.
> >>>
> >>> I still think we should reserve it, just to be safe.
> >>>
> >>> What do you think?
> >>
> >> I believe that contradicts the reasoning you gave in the commit
> >> message then..
> >
> > Yep, will rephrase it for this platform as:
> >
> > While toggling it from linux doesn't seem to have any observable impact,
> > since documentation does confrm it is connected to EC reset, lets reserve
> > it just to be on the safe side.
>
> Have you toggled the pin into the correct polarity? The EC FW seems not
> to be handling that pin at all, so I'm guessing it should be just a dumb
> chip reset
Hm. Good catch. It is actually active high, and it renders keyboard
broken after EC reset. Also fans start spinning full speed during EC
reset.
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v2 02/16] arm64: dts: qcom: x1e001de-devkit: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
2026-09-01 8:06 ` [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved Abel Vesa
@ 2026-09-01 8:06 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 03/16] arm64: dts: qcom: x1e78100-t14s: " Abel Vesa
` (13 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:06 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: 7b8a31e82b87 ("arm64: dts: qcom: Add X1E001DE Snapdragon Devkit for Windows")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1e001de-devkit.dts | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts b/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts
index be1a8ac8c699..3ba45a7884a2 100644
--- a/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts
+++ b/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts
@@ -1194,7 +1194,8 @@ wcd_tx: codec@0,3 {
};
&tlmm {
- gpio-reserved-ranges = <44 4>; /* SPI (TPM) */
+ gpio-reserved-ranges = <44 4>, /* SPI (TPM) */
+ <65 1>; /* EC reset */
eusb3_reset_n: eusb3-reset-n-state {
pins = "gpio6";
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 03/16] arm64: dts: qcom: x1e78100-t14s: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
2026-09-01 8:06 ` [PATCH v2 01/16] arm64: dts: qcom: x1-crd: Mark the EC reset gpio as reserved Abel Vesa
2026-09-01 8:06 ` [PATCH v2 02/16] arm64: dts: qcom: x1e001de-devkit: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 04/16] arm64: dts: qcom: x1e80100-dell-xps13-9345: " Abel Vesa
` (12 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: 7d1cbe2f4985 ("arm64: dts: qcom: Add X1E78100 ThinkPad T14s Gen 6")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi b/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi
index 0bee2a3d6e7a..8f7b01617461 100644
--- a/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dtsi
@@ -1385,6 +1385,7 @@ wcd_tx: codec@0,3 {
&tlmm {
gpio-reserved-ranges = <34 2>, /* Unused */
<44 4>, /* SPI (TPM) */
+ <65 1>, /* EC reset */
<72 2>, /* Secure EC I2C connection (?) */
<238 1>; /* UFS Reset */
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 04/16] arm64: dts: qcom: x1e80100-dell-xps13-9345: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (2 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 03/16] arm64: dts: qcom: x1e78100-t14s: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 05/16] arm64: dts: qcom: honor-magicbook-art-14: " Abel Vesa
` (11 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: f5b788d0e8cd ("arm64: dts: qcom: Add support for X1-based Dell XPS 13 9345")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts b/arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts
index 2be5a37c00cc..b795b65d604e 100644
--- a/arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts
+++ b/arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts
@@ -1072,6 +1072,7 @@ &smb2360_1_eusb2_repeater {
&tlmm {
gpio-reserved-ranges = <44 4>, /* SPI11 (TPM) */
<76 4>, /* SPI19 (TZ Protected) */
+ <65 1>, /* EC reset */
<238 1>; /* UFS Reset */
cam_indicator_en: cam-indicator-en-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 05/16] arm64: dts: qcom: honor-magicbook-art-14: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (3 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 04/16] arm64: dts: qcom: x1e80100-dell-xps13-9345: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 06/16] arm64: dts: qcom: x1e80100-yoga-slim7x: " Abel Vesa
` (10 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: 21b15fa5f8f4 ("arm64: dts: qcom: Add HONOR MagicBook Art 14 device tree")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1e80100-honor-magicbook-art-14.dts | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/x1e80100-honor-magicbook-art-14.dts b/arch/arm64/boot/dts/qcom/x1e80100-honor-magicbook-art-14.dts
index b70c1e094bbf..a14f0d0632f0 100644
--- a/arch/arm64/boot/dts/qcom/x1e80100-honor-magicbook-art-14.dts
+++ b/arch/arm64/boot/dts/qcom/x1e80100-honor-magicbook-art-14.dts
@@ -1068,7 +1068,8 @@ right_tweeter: speaker@0,1 {
&tlmm {
gpio-reserved-ranges = <34 2>, /* Unused */
- <44 4>; /* SPI (TPM) */
+ <44 4>, /* SPI (TPM) */
+ <65 1>; /* EC reset */
tpad_default: tpad-default-state {
pins = "gpio3";
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 06/16] arm64: dts: qcom: x1e80100-yoga-slim7x: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (4 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 05/16] arm64: dts: qcom: honor-magicbook-art-14: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 07/16] arm64: dts: qcom: medion-sprchrgd-14-s1: " Abel Vesa
` (9 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: 45247fe17db2 ("arm64: dts: qcom: x1e80100: add Lenovo Thinkpad Yoga slim 7x devicetree")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
index 9eb2dfd99f20..3e4ab58d2f6f 100644
--- a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
+++ b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
@@ -1350,6 +1350,7 @@ right_tweeter: speaker@0,1 {
&tlmm {
gpio-reserved-ranges = <34 2>, /* Unused */
<44 4>, /* SPI (TPM) */
+ <65 1>, /* EC reset */
<238 1>; /* UFS Reset */
edp_reg_en: edp-reg-en-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 07/16] arm64: dts: qcom: medion-sprchrgd-14-s1: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (5 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 06/16] arm64: dts: qcom: x1e80100-yoga-slim7x: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 08/16] arm64: dts: qcom: microsoft-romulus: " Abel Vesa
` (8 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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@oss.qualcomm.com>
---
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 e4a0b86847cd..4b45f293e167 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
@@ -1251,6 +1251,7 @@ right_tweeter: speaker@0,1 {
&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 {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 08/16] arm64: dts: qcom: microsoft-romulus: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (6 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 07/16] arm64: dts: qcom: medion-sprchrgd-14-s1: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-02 15:50 ` Konrad Dybcio
2026-09-01 8:07 ` [PATCH v2 09/16] arm64: dts: qcom: x1e80100-qcp: " Abel Vesa
` (7 subsequent siblings)
15 siblings, 1 reply; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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@oss.qualcomm.com>
---
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 */
hall_int_n_default: hall-int-n-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 08/16] arm64: dts: qcom: microsoft-romulus: Mark the EC reset gpio as reserved
2026-09-01 8:07 ` [PATCH v2 08/16] arm64: dts: qcom: microsoft-romulus: " Abel Vesa
@ 2026-09-02 15:50 ` Konrad Dybcio
2026-09-03 8:35 ` Abel Vesa
0 siblings, 1 reply; 27+ messages in thread
From: Konrad Dybcio @ 2026-09-02 15:50 UTC (permalink / raw)
To: Abel Vesa, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Rajendra Nayak, Abel Vesa,
Sibi Sankar, Marc Zyngier, Aleksandrs Vinarskis,
Konstantin Shabanov, Srinivas Kandagatla, Ettore Chimenti,
Georg Gottleuber, Xilin Wu, Bryan O'Donoghue, Val Packett,
Laurentiu Tudor, Jens Glathe, Dale Whinham,
Jérôme de Bretagne, Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
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@oss.qualcomm.com>
> ---
> 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)
Konrad
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH v2 08/16] arm64: dts: qcom: microsoft-romulus: Mark the EC reset gpio as reserved
2026-09-02 15:50 ` Konrad Dybcio
@ 2026-09-03 8:35 ` Abel Vesa
0 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-03 8:35 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Harrison Vanderbyl,
linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
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@oss.qualcomm.com>
> > ---
> > 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.
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v2 09/16] arm64: dts: qcom: x1e80100-qcp: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (7 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 08/16] arm64: dts: qcom: microsoft-romulus: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 10/16] arm64: dts: qcom: x1-asus-vivobook-s15: " Abel Vesa
` (6 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: af16b00578a7 ("arm64: dts: qcom: Add base X1E80100 dtsi and the QCP dts")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1e80100-qcp.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1e80100-qcp.dts b/arch/arm64/boot/dts/qcom/x1e80100-qcp.dts
index 78380fec8839..19ea1add016c 100644
--- a/arch/arm64/boot/dts/qcom/x1e80100-qcp.dts
+++ b/arch/arm64/boot/dts/qcom/x1e80100-qcp.dts
@@ -1180,6 +1180,7 @@ wcd_tx: codec@0,3 {
&tlmm {
gpio-reserved-ranges = <33 3>, /* Unused */
<44 4>, /* SPI (TPM) */
+ <65 1>, /* EC reset */
<238 1>; /* UFS Reset */
edp_reg_en: edp-reg-en-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 10/16] arm64: dts: qcom: x1-asus-vivobook-s15: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (8 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 09/16] arm64: dts: qcom: x1e80100-qcp: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 11/16] arm64: dts: qcom: x1-asus-zenbook-a14: " Abel Vesa
` (5 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: d0e2f8f62dff ("arm64: dts: qcom: Add device tree for ASUS Vivobook S 15")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1-asus-vivobook-s15.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1-asus-vivobook-s15.dtsi b/arch/arm64/boot/dts/qcom/x1-asus-vivobook-s15.dtsi
index 0dc7c1b5a14a..27fab4db24d5 100644
--- a/arch/arm64/boot/dts/qcom/x1-asus-vivobook-s15.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1-asus-vivobook-s15.dtsi
@@ -1039,6 +1039,7 @@ &smb2360_1_eusb2_repeater {
&tlmm {
gpio-reserved-ranges = <34 2>, /* Unused */
<44 4>, /* SPI (TPM) */
+ <65 1>, /* EC reset */
<238 1>; /* UFS Reset */
edp_reg_en: edp-reg-en-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 11/16] arm64: dts: qcom: x1-asus-zenbook-a14: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (9 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 10/16] arm64: dts: qcom: x1-asus-vivobook-s15: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 12/16] arm64: dts: qcom: x1-dell-thena: " Abel Vesa
` (4 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: 6516961352a1 ("arm64: dts: qcom: Add support for X1-based Asus Zenbook A14")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi b/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi
index 59cbbfbce490..8eb4c045b58a 100644
--- a/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1-asus-zenbook-a14.dtsi
@@ -1274,6 +1274,7 @@ wcd_tx: codec@0,3 {
&tlmm {
gpio-reserved-ranges = <44 4>, /* SPI11, TZ Protected */
+ <65 1>, /* EC reset */
<90 1>; /* Unknown, TZ Protected */
cam_indicator_en: cam-indicator-en-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 12/16] arm64: dts: qcom: x1-dell-thena: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (10 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 11/16] arm64: dts: qcom: x1-asus-zenbook-a14: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 13/16] arm64: dts: qcom: x1-hp-omnibook-x14: " Abel Vesa
` (3 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: e7733b42111c ("arm64: dts: qcom: Add support for Dell Inspiron 7441 / Latitude 7455")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi b/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
index 4cda708455d8..b367eef085ae 100644
--- a/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1-dell-thena.dtsi
@@ -1303,6 +1303,7 @@ right_tweeter: speaker@0,1 {
&tlmm {
gpio-reserved-ranges = <44 4>, /* SPI11 (TPM) */
+ <65 1>, /* EC reset */
<76 4>, /* SPI19 (TZ Protected) */
<80 2>, /* I2C20 (Battery SMBus) */
<238 1>; /* UFS Reset */
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 13/16] arm64: dts: qcom: x1-hp-omnibook-x14: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (11 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 12/16] arm64: dts: qcom: x1-dell-thena: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 14/16] arm64: dts: qcom: x1-microsoft-denali: " Abel Vesa
` (2 subsequent siblings)
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: 6f18b8d4142c ("arm64: dts: qcom: x1e80100-hp-x14: dt for HP Omnibook X Laptop 14")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1-hp-omnibook-x14.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1-hp-omnibook-x14.dtsi b/arch/arm64/boot/dts/qcom/x1-hp-omnibook-x14.dtsi
index dba8cee43e29..bcd85cbdcb93 100644
--- a/arch/arm64/boot/dts/qcom/x1-hp-omnibook-x14.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1-hp-omnibook-x14.dtsi
@@ -1287,6 +1287,7 @@ wcd_tx: codec@0,3 {
&tlmm {
gpio-reserved-ranges = <34 2>, /* Unused */
<44 4>, /* SPI (TPM) */
+ <65 1>, /* EC reset */
<72 2>, /* Secure EC I2C connection (?) */
<238 1>; /* UFS Reset */
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 14/16] arm64: dts: qcom: x1-microsoft-denali: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (12 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 13/16] arm64: dts: qcom: x1-hp-omnibook-x14: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-02 15:54 ` Konrad Dybcio
2026-09-01 8:07 ` [PATCH v2 15/16] arm64: dts: qcom: x1p42100-thinkbook-16: " Abel Vesa
2026-09-01 8:07 ` [PATCH v2 16/16] arm64: dts: qcom: x1p42100-microsoft-sp12in: " Abel Vesa
15 siblings, 1 reply; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: 0d72ccaa1e84 ("arm64: dts: qcom: Add support for X1-based Surface Pro 11")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi b/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi
index ce7d958ca687..ef2684e23a09 100644
--- a/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi
+++ b/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi
@@ -1109,6 +1109,7 @@ right_spkr: speaker@0,1 {
&tlmm {
gpio-reserved-ranges = <44 4>, /* SPI (TPM) */
+ <65 1>, /* EC reset */
<238 1>; /* UFS Reset */
hall_int_n_default: hall-int-n-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 14/16] arm64: dts: qcom: x1-microsoft-denali: Mark the EC reset gpio as reserved
2026-09-01 8:07 ` [PATCH v2 14/16] arm64: dts: qcom: x1-microsoft-denali: " Abel Vesa
@ 2026-09-02 15:54 ` Konrad Dybcio
0 siblings, 0 replies; 27+ messages in thread
From: Konrad Dybcio @ 2026-09-02 15:54 UTC (permalink / raw)
To: Abel Vesa, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Rajendra Nayak, Abel Vesa,
Sibi Sankar, Marc Zyngier, Aleksandrs Vinarskis,
Konstantin Shabanov, Srinivas Kandagatla, Ettore Chimenti,
Georg Gottleuber, Xilin Wu, Bryan O'Donoghue, Val Packett,
Laurentiu Tudor, Jens Glathe, Dale Whinham,
Jérôme de Bretagne, Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
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: 0d72ccaa1e84 ("arm64: dts: qcom: Add support for X1-based Surface Pro 11")
> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi b/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi
> index ce7d958ca687..ef2684e23a09 100644
> --- a/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi
> +++ b/arch/arm64/boot/dts/qcom/x1-microsoft-denali.dtsi
> @@ -1109,6 +1109,7 @@ right_spkr: speaker@0,1 {
>
> &tlmm {
> gpio-reserved-ranges = <44 4>, /* SPI (TPM) */
> + <65 1>, /* EC reset */
Because it uses the same EC as Romulus, I would guesstimate this
isn't right for this device too
Konrad
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v2 15/16] arm64: dts: qcom: x1p42100-thinkbook-16: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (13 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 14/16] arm64: dts: qcom: x1-microsoft-denali: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-01 8:07 ` [PATCH v2 16/16] arm64: dts: qcom: x1p42100-microsoft-sp12in: " Abel Vesa
15 siblings, 0 replies; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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: d3f600dc452d ("arm64: dts: qcom: Add Lenovo ThinkBook 16 G7 QOY device tree")
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/x1p42100-lenovo-thinkbook-16.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/qcom/x1p42100-lenovo-thinkbook-16.dts b/arch/arm64/boot/dts/qcom/x1p42100-lenovo-thinkbook-16.dts
index c2f21fcb1b09..1e295741e676 100644
--- a/arch/arm64/boot/dts/qcom/x1p42100-lenovo-thinkbook-16.dts
+++ b/arch/arm64/boot/dts/qcom/x1p42100-lenovo-thinkbook-16.dts
@@ -1327,6 +1327,7 @@ wcd_tx: codec@0,3 {
&tlmm {
gpio-reserved-ranges = <34 2>, /* Unused */
+ <65 1>, /* EC reset */
<72 2>; /* Secure EC I2C connection (?) */
edp_hpd_default: edp-hpd-default-state {
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 16/16] arm64: dts: qcom: x1p42100-microsoft-sp12in: Mark the EC reset gpio as reserved
2026-09-01 8:06 [PATCH v2 00/16] arm64: dts: qcom: hamoa: Mark the EC reset gpio as reserved on all boards Abel Vesa
` (14 preceding siblings ...)
2026-09-01 8:07 ` [PATCH v2 15/16] arm64: dts: qcom: x1p42100-thinkbook-16: " Abel Vesa
@ 2026-09-01 8:07 ` Abel Vesa
2026-09-02 15:54 ` Konrad Dybcio
15 siblings, 1 reply; 27+ messages in thread
From: Abel Vesa @ 2026-09-01 8:07 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Rajendra Nayak, Abel Vesa, Sibi Sankar,
Marc Zyngier, Aleksandrs Vinarskis, Konstantin Shabanov,
Srinivas Kandagatla, Ettore Chimenti, Georg Gottleuber, Xilin Wu,
Bryan O'Donoghue, Val Packett, Laurentiu Tudor, Jens Glathe,
Dale Whinham, Jérôme de Bretagne, Abel Vesa,
Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio,
Konrad Dybcio
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 */
edp_reg_en: edp-reg-en-state {
pins = "gpio70";
--
2.54.0
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 16/16] arm64: dts: qcom: x1p42100-microsoft-sp12in: Mark the EC reset gpio as reserved
2026-09-01 8:07 ` [PATCH v2 16/16] arm64: dts: qcom: x1p42100-microsoft-sp12in: " Abel Vesa
@ 2026-09-02 15:54 ` Konrad Dybcio
0 siblings, 0 replies; 27+ messages in thread
From: Konrad Dybcio @ 2026-09-02 15:54 UTC (permalink / raw)
To: Abel Vesa, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Rajendra Nayak, Abel Vesa,
Sibi Sankar, Marc Zyngier, Aleksandrs Vinarskis,
Konstantin Shabanov, Srinivas Kandagatla, Ettore Chimenti,
Georg Gottleuber, Xilin Wu, Bryan O'Donoghue, Val Packett,
Laurentiu Tudor, Jens Glathe, Dale Whinham,
Jérôme de Bretagne, Harrison Vanderbyl
Cc: linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
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
^ permalink raw reply [flat|nested] 27+ messages in thread