From 6b8e7144c8863686a8ddef6f4fcebb3b8cf85f36 Mon Sep 17 00:00:00 2001
From: INAGAKI Hiroshi <musashino.open@gmail.com>
Date: Sun, 27 Aug 2023 14:49:41 +0900
Subject: [PATCH] mediatek: update LED/Key bindings for Buffalo WSR-2533DHP2

Update LED and key nodes with newer DeviceTree bindings for WSR-2533DHP2.

- LED
  - use led-[0-9] for node name of LEDs
  - add "color" and "function" properties
  - drop default-state = "on" from green:power LED
    - this LED will be turned on by led-running alias

- key
  - drop unnecessary poll-interval property
  - use key-[0-9] for node name of keys

Signed-off-by: INAGAKI Hiroshi <musashino.open@gmail.com>
---
 .../dts/mt7622-buffalo-wsr-2533dhp2.dts       | 37 ++++++++++++-------
 1 file changed, 24 insertions(+), 13 deletions(-)

diff --git a/target/linux/mediatek/dts/mt7622-buffalo-wsr-2533dhp2.dts b/target/linux/mediatek/dts/mt7622-buffalo-wsr-2533dhp2.dts
index 6d77be532b..fec559c8af 100644
--- a/target/linux/mediatek/dts/mt7622-buffalo-wsr-2533dhp2.dts
+++ b/target/linux/mediatek/dts/mt7622-buffalo-wsr-2533dhp2.dts
@@ -2,6 +2,7 @@
 /dts-v1/;
 #include <dt-bindings/input/input.h>
 #include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/leds/common.h>
 
 #include "mt7622.dtsi"
 #include "mt6380.dtsi"
@@ -29,43 +30,53 @@
 	leds {
 		compatible = "gpio-leds";
 
-		wireless_amber {
+		led-0 {
 			label = "amber:wireless";
 			gpios = <&pio 2 GPIO_ACTIVE_HIGH>;
+			color = <LED_COLOR_ID_AMBER>;
+			function = LED_FUNCTION_WLAN;
 		};
 
-		power_amber: power_amber {
+		power_amber: led-1 {
 			label = "amber:power";
 			gpios = <&pio 3 GPIO_ACTIVE_LOW>;
+			color = <LED_COLOR_ID_AMBER>;
+			function = LED_FUNCTION_POWER;
 		};
 
-		power_green: power_green {
+		power_green: led-2 {
 			label = "green:power";
 			gpios = <&pio 4 GPIO_ACTIVE_LOW>;
-			default-state = "on";
+			color = <LED_COLOR_ID_GREEN>;
+			function = LED_FUNCTION_POWER;
 		};
 
-		wireless_green {
+		led-3 {
 			label = "green:wireless";
 			gpios = <&pio 15 GPIO_ACTIVE_HIGH>;
+			color = <LED_COLOR_ID_GREEN>;
+			function = LED_FUNCTION_WLAN;
 		};
 
-		internet {
+		led-4 {
 			label = "green:internet";
 			gpios = <&pio 19 GPIO_ACTIVE_HIGH>;
+			color = <LED_COLOR_ID_GREEN>;
+			function = LED_FUNCTION_WAN;
 		};
 
-		router {
+		led-5 {
 			label = "green:router";
 			gpios = <&pio 20 GPIO_ACTIVE_HIGH>;
+			color = <LED_COLOR_ID_GREEN>;
+			function = LED_FUNCTION_INDICATOR;
 		};
 	};
 
 	keys {
 		compatible = "gpio-keys";
-		poll-interval = <100>;
 
-		reset {
+		key-reset {
 			label = "reset";
 			gpios = <&pio 0 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_RESTART>;
@@ -74,14 +85,14 @@
 		/* GPIO 1 and 16 are a tri-state switch button with
 		 * ROUTER / AP / WB.
 		 */
-		router {
+		key-router {
 			label = "router";
 			gpios = <&pio 1 GPIO_ACTIVE_LOW>;
 			linux,code = <BTN_0>;
 			linux,input-type = <EV_SW>;
 		};
 
-		bridge {
+		key-bridge {
 			label = "wb";
 			gpios = <&pio 16 GPIO_ACTIVE_LOW>;
 			linux,code = <BTN_1>;
@@ -89,14 +100,14 @@
 		};
 
 		/* GPIO 18 is a switch button with AUTO / MANUAL. */
-		manual {
+		key-manual {
 			label = "manual";
 			gpios = <&pio 18 GPIO_ACTIVE_LOW>;
 			linux,code = <BTN_2>;
 			linux,input-type = <EV_SW>;
 		};
 
-		wps {
+		key-wps {
 			label = "wps";
 			gpios = <&pio 102 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_WPS_BUTTON>;
-- 
2.30.2