realtek: move RTL8231 definitions to board files

The RTL8231 is an external chip, and not part of the SoC. That means
it is more appropriate to define it in the board specific (base) files,
instead of the DT include for the SoC itself.

Moving the RTL8231 definition also ensures that boards with no GPIO
expander, or an alternative one, don't have a useless gpio1 node label
defined.

Tested on a Netgear GS110TPPv1.

Signed-off-by: Sander Vanheule <sander@svanheule.net>
This commit is contained in:
Sander Vanheule 2022-02-20 16:25:42 +01:00 committed by Daniel Golle
parent 004d4d66c8
commit 4b31717fb0
4 changed files with 23 additions and 21 deletions

View File

@ -28,12 +28,13 @@
gpios = <&gpio0 13 GPIO_ACTIVE_LOW>; gpios = <&gpio0 13 GPIO_ACTIVE_LOW>;
open-source; open-source;
}; };
};
&gpio1 { gpio1: rtl8231-gpio {
status = "okay"; compatible = "realtek,rtl8231-gpio";
#gpio-cells = <2>;
indirect-access-bus-id = <31>; gpio-controller;
indirect-access-bus-id = <31>;
};
}; };
&ethernet0 { &ethernet0 {

View File

@ -39,15 +39,18 @@
gpios = <&gpio0 0 GPIO_ACTIVE_HIGH>; gpios = <&gpio0 0 GPIO_ACTIVE_HIGH>;
}; };
}; };
};
&gpio1 { gpio1: rtl8231-gpio {
status = "okay"; compatible = "realtek,rtl8231-gpio";
#gpio-cells = <2>;
gpio-controller;
indirect-access-bus-id = <0>;
poe_enable { poe_enable {
gpio-hog; gpio-hog;
gpios = <13 0>; gpios = <13 GPIO_ACTIVE_HIGH>;
output-high; output-high;
};
}; };
}; };

View File

@ -44,6 +44,13 @@
linux,code = <KEY_RESTART>; linux,code = <KEY_RESTART>;
};*/ };*/
}; };
gpio1: rtl8231-gpio {
compatible = "realtek,rtl8231-gpio";
#gpio-cells = <2>;
gpio-controller;
indirect-access-bus-id = <0>;
};
}; };

View File

@ -169,15 +169,6 @@
}; };
}; };
gpio1: rtl8231-gpio {
compatible = "realtek,rtl8231-gpio";
#gpio-cells = <2>;
indirect-access-bus-id = <0>;
gpio-controller;
status = "disabled";
};
pinmux: pinmux@1b001000 { pinmux: pinmux@1b001000 {
compatible = "pinctrl-single"; compatible = "pinctrl-single";
reg = <0x1b001000 0x4>; reg = <0x1b001000 0x4>;