diff mbox series

ARM: dts: aspeed: Rainier remove PSU gpio-keys

Message ID 20210623230401.3050076-1-bjwyman@gmail.com
State New
Headers show
Series ARM: dts: aspeed: Rainier remove PSU gpio-keys | expand

Commit Message

Brandon Wyman June 23, 2021, 11:04 p.m. UTC 
From: "B. J. Wyman" <bjwyman@gmail.com>

Remove the gpio-keys entries for the power supply presence lines from
the Rainier device tree. The user space applications are going to change
from using libevdev to libgpiod.

Signed-off-by: B. J. Wyman <bjwyman@gmail.com>
---
 arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 28 --------------------
 1 file changed, 28 deletions(-)

Comments

Andrew Jeffery June 23, 2021, 11:08 p.m. UTC | #1 
On Thu, 24 Jun 2021, at 08:34, Brandon Wyman wrote:
> From: "B. J. Wyman" <bjwyman@gmail.com>
> 
> Remove the gpio-keys entries for the power supply presence lines from
> the Rainier device tree. The user space applications are going to change
> from using libevdev to libgpiod.
> 

Have they already added support for libgpiod? We should support both at 
least while we're switching over.

Andrew
Brandon Wyman June 24, 2021, 6:01 p.m. UTC | #2 
On 2021-06-23 18:08, Andrew Jeffery wrote:
>
> On Thu, 24 Jun 2021, at 08:34, Brandon Wyman wrote:
>> From: "B. J. Wyman" <bjwyman@gmail.com>
>>
>> Remove the gpio-keys entries for the power supply presence lines from
>> the Rainier device tree. The user space applications are going to change
>> from using libevdev to libgpiod.
>>
> Have they already added support for libgpiod? We should support both at
> least while we're switching over.
There is a change, hopefully nearing the end of review, that will 
support both.
>
> Andrew
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
index 941c0489479a..d6ed583cfb1b 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
@@ -116,34 +116,6 @@  vga_memory: region@bf000000 {
 		};
 	};
 
-	gpio-keys {
-		compatible = "gpio-keys";
-
-		ps0-presence {
-			label = "ps0-presence";
-			gpios = <&gpio0 ASPEED_GPIO(S, 0) GPIO_ACTIVE_LOW>;
-			linux,code = <ASPEED_GPIO(S, 0)>;
-		};
-
-		ps1-presence {
-			label = "ps1-presence";
-			gpios = <&gpio0 ASPEED_GPIO(S, 1) GPIO_ACTIVE_LOW>;
-			linux,code = <ASPEED_GPIO(S, 1)>;
-		};
-
-		ps2-presence {
-			label = "ps2-presence";
-			gpios = <&gpio0 ASPEED_GPIO(S, 2) GPIO_ACTIVE_LOW>;
-			linux,code = <ASPEED_GPIO(S, 2)>;
-		};
-
-		ps3-presence {
-			label = "ps3-presence";
-			gpios = <&gpio0 ASPEED_GPIO(S, 3) GPIO_ACTIVE_LOW>;
-			linux,code = <ASPEED_GPIO(S, 3)>;
-		};
-	};
-
 	i2c2mux: i2cmux {
 		compatible = "i2c-mux-gpio";
 		#address-cells = <1>;