Skip to contentSkip to conversation

arm64: dts: qcom: x1e80100-crd: add support for volume-up key

PATCH Applied Johan Hovold 3 participants 5 messages Mar 7, 2025 – Mar 14, 2025

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

cover arm64: dts: qcom: x1e80100-crd: add support for volume-up key

Add support for the volume-up key on the debug extension board, which can also be used as a wakeup source from deep idle states (CX shutdown).

Johan

Johan Hovold (2):
arm64: dts: qcom: x1e80100-crd: add support for volume-up key
arm64: dts: qcom: x1e80100-crd: add gpio-keys label for lid switch

 arch/arm64/boot/dts/qcom/x1-crd.dtsi | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)
Signature · 1 line
2.45.3
#1 lore raw

PATCH 1/2 arm64: dts: qcom: x1e80100-crd: add support for volume-up key

Add support for the keypad volume-up key on the debug extension board.

This is useful to have when testing PMIC interrupt handling, and the key can also be used to wake up from deep suspend states (CX shutdown).

---
 arch/arm64/boot/dts/qcom/x1-crd.dtsi | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)
arch/arm64/boot/dts/qcom/x1-crd.dtsi +18 -1
index 296b41409ad1..0b62fdc85478 100644
@@ -52,9 +52,16 @@ chosen {
 	gpio-keys {
 		compatible = "gpio-keys";
 
-		pinctrl-0 = <&hall_int_n_default>;
+		pinctrl-0 = <&hall_int_n_default>, <&kypd_vol_up_n>;
 		pinctrl-names = "default";
 
+		key-vol-up {
+			label = "volume_up";
+			gpios = <&pm8550_gpios 6 GPIO_ACTIVE_LOW>;
+			linux,code = <KEY_VOLUMEUP>;
+			wakeup-source;
+		};
+
 		switch-lid {
 			gpios = <&tlmm 92 GPIO_ACTIVE_LOW>;
 			linux,input-type = <EV_SW>;
@@ -876,6 +883,16 @@ &pcie6a_phy {
 	status = "okay";
 };
 
+&pm8550_gpios {
+	kypd_vol_up_n: kypd-vol-up-n-state {
+		pins = "gpio6";
+		function = "normal";
+		power-source = <1>; /* 1.8 V */
+		bias-pull-up;
+		input-enable;
+	};
+};
+
 &pm8550ve_8_gpios {
 	misc_3p3_reg_en: misc-3p3-reg-en-state {
 		pins = "gpio6";
Signature · 1 line
2.45.3
#2 lore raw

PATCH 2/2 arm64: dts: qcom: x1e80100-crd: add gpio-keys label for lid switch

In reply to Johan Hovold

Add a gpio-keys label for the lid-switch for consistency and to separate it from the volume-up key (e.g. in /proc/interrupts).

---
 arch/arm64/boot/dts/qcom/x1-crd.dtsi | 1 +
 1 file changed, 1 insertion(+)
arch/arm64/boot/dts/qcom/x1-crd.dtsi +1 -0
index 0b62fdc85478..fc4ea6be5fd1 100644
@@ -63,6 +63,7 @@ key-vol-up {
 		};
 
 		switch-lid {
+			label = "lid";
 			gpios = <&tlmm 92 GPIO_ACTIVE_LOW>;
 			linux,input-type = <EV_SW>;
 			linux,code = <SW_LID>;
Signature · 1 line
2.45.3
maintainer Applied
#4 lore raw
In reply to Johan Hovold
Quoted 11 lines · Johan Hovold

Add support for the volume-up key on the debug extension board, which can also be used as a wakeup source from deep idle states (CX shutdown).

Johan

Johan Hovold (2):
arm64: dts: qcom: x1e80100-crd: add support for volume-up key
arm64: dts: qcom: x1e80100-crd: add gpio-keys label for lid switch

[...]

Applied, thanks!

[1/2] arm64: dts: qcom: x1e80100-crd: add support for volume-up key

      commit: 9eca3fd5c336afc3b90804ec008f54ce59320aee
[2/2] arm64: dts: qcom: x1e80100-crd: add gpio-keys label for lid switch
      commit: ee95bcc58890e63f52fdb9ab096c3d7b9cb889cc

Best regards,

Signature · 1 line
Bjorn Andersson <[email protected]>