[PATCH] arm64: dts: qcom: sm8150-microsoft-surface-duo: fix da7280 pinctrl state name
From: Ivan Verbovoy via B4 Relay
Date: Tue Nov 03 2026 - 14:27:48 EST
From: Ivan Verbovoy <ivanverbovoy@xxxxxxxxx>
The pinctrl core applies a device's "default" state by itself and the
da7280 driver requests no state of its own, so the state named
"da7280_default" is never selected: gpio42 stays as the firmware left it
and its pull-up is not applied. The interrupt line is open-drain and
level-low; after a suspend/resume cycle it stays low and the threaded
handler runs continuously (about 2900 times a second) with no event in
the chip, which slows the whole system down.
Name the state "default" so it is applied at probe.
Fixes: d1f781db47a8 ("arm64: dts: qcom: add initial device-tree for Microsoft Surface Duo")
Signed-off-by: Ivan Verbovoy <ivanverbovoy@xxxxxxxxx>
---
arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
index f37e11bde067..4078a7521a2e 100644
--- a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
+++ b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
@@ -372,7 +372,7 @@ da7280@4a {
compatible = "dlg,da7280";
reg = <0x4a>;
interrupts-extended = <&tlmm 42 IRQ_TYPE_LEVEL_LOW>;
- pinctrl-names = "da7280_default";
+ pinctrl-names = "default";
pinctrl-0 = <&da7280_intr_default>;
dlg,actuator-type = "LRA";
---
base-commit: d6bcfc94eb4da6e089c5473970cd96727292fd97
change-id: 20261103-a1-duo-da7280-pinctrl-beddacac8298
Best regards,
--
Ivan Verbovoy <ivanverbovoy@xxxxxxxxx>