Browse Source

realtek: update GPIO bindings in the dts files in dts-5.10

this patch includes the following changes:

- adjust mapping for the new driver
  - GPIO 24 -> GPIO 0
  - GPIO 47 -> GPIO 0 (+ disabling system LED)

- disable pins in the invalid range
  (out of the range 0-31 of the new driver)
  - are these pins on the external RTL8231 (&gpio1)?
    - GPIO 67 (-> GPIO 3 on &gpio1?)
    - GPIO 94 (-> GPIO 30 on &gpio1?)

- drop "indirect-access-bus-id" property from gpio0 node in device dts
  files

Signed-off-by: INAGAKI Hiroshi <[email protected]>
INAGAKI Hiroshi 4 years ago
parent
commit
61a3d0075b

+ 1 - 1
target/linux/realtek/dts-5.10/rtl8380_netgear_gigabit.dtsi

@@ -23,7 +23,7 @@
 
 		mode {
 			label = "reset";
-			gpios = <&gpio0 24 GPIO_ACTIVE_LOW>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_RESTART>;
 		};
 	};

+ 13 - 11
target/linux/realtek/dts-5.10/rtl8380_zyxel_gs1900.dtsi

@@ -22,16 +22,6 @@
 		reg = <0x0 0x8000000>;
 	};
 
-	gpio1: rtl8231-gpio {
-		status = "okay";
-
-		poe_enable {
-			gpio-hog;
-			gpios = <13 0>;
-			output-high;
-		};
-	};
-
 	keys {
 		compatible = "gpio-keys-polled";
 		poll-interval = <20>;
@@ -44,15 +34,27 @@
 	};
 
 	leds {
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinmux_disable_sys_led>;
 		compatible = "gpio-leds";
 
 		led_sys: sys {
 			label = "green:sys";
-			gpios = <&gpio0 47 GPIO_ACTIVE_HIGH>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_HIGH>;
 		};
 	};
 };
 
+&gpio1 {
+	status = "okay";
+
+	poe_enable {
+		gpio-hog;
+		gpios = <13 0>;
+		output-high;
+	};
+};
+
 &spi0 {
 	status = "okay";
 

+ 6 - 7
target/linux/realtek/dts-5.10/rtl8382_allnet_all-sg8208m.dts

@@ -29,28 +29,27 @@
 		compatible = "gpio-keys-polled";
 		poll-interval = <20>;
 
-		reset {
+		/* is this pin 3 on the external RTL8231 (&gpio1)? */
+		/*reset {
 			label = "reset";
 			gpios = <&gpio0 67 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_RESTART>;
-		};
+		};*/
 	};
 
 	leds {
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinmux_disable_sys_led>;
 		compatible = "gpio-leds";
 
 		led_sys: sys {
 			label = "green:sys";
-			gpios = <&gpio0 47 GPIO_ACTIVE_HIGH>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_HIGH>;
 		};
 		// GPIO 25: power on/off all port leds
 	};
 };
 
-&gpio0 {
-	indirect-access-bus-id = <0>;
-};
-
 &spi0 {
 	status = "okay";
 

+ 7 - 8
target/linux/realtek/dts-5.10/rtl8382_d-link_dgs-1210-10p.dts

@@ -26,12 +26,14 @@
 	};
 
 	leds {
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinmux_disable_sys_led>;
 		compatible = "gpio-leds";
 
 		led_power: power {
-			// GPIO 24 seems to provide power to the leds
+			// GPIO 0 seems to provide power to the leds
 			label = "green:power";
-			gpios = <&gpio0 47 GPIO_ACTIVE_LOW>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_LOW>;
 		};
 	};
 
@@ -39,19 +41,16 @@
 		compatible = "gpio-keys-polled";
 		poll-interval = <20>;
 
-		mode {
+		/* is this pin 30 on the external RTL8231 (&gpio1)? */
+		/*mode {
 			label = "reset";
 			gpios = <&gpio0 94 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_RESTART>;
-		};
+		};*/
 	};
 };
 
 
-&gpio0 {
-	indirect-access-bus-id = <0>;
-};
-
 &spi0 {
 	status = "okay";
 	flash@0 {

+ 1 - 1
target/linux/realtek/dts-5.10/rtl8382_d-link_dgs-1210.dtsi

@@ -27,7 +27,7 @@
 
 		led_power: power {
 			label = "green:power";
-			gpios = <&gpio0 24 GPIO_ACTIVE_LOW>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_LOW>;
 		};
 	};
 };

+ 1 - 5
target/linux/realtek/dts-5.10/rtl8382_inaba_aml2-17gp.dts

@@ -24,16 +24,12 @@
 
 		reset {
 			label = "reset";
-			gpios = <&gpio0 24 GPIO_ACTIVE_LOW>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_RESTART>;
 		};
 	};
 };
 
-&gpio0 {
-	indirect-access-bus-id = <0>;
-};
-
 &spi0 {
 	status = "okay";