Re: [PATCH v3 13/13] arm64: dts: mediatek: mt7988a-bpi-r4: configure switch phys and leds

AngeloGioacchino Del Regno
2025-06-11T09:33:40Z

Il 08/06/25 23:14, Frank Wunderlich ha scritto:
> From: Frank Wunderlich <frank-w@public-files.de>
>
> Assign pinctrl to switch phys and leds.
>
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>
> Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
> ---
> v2:
> - add labels and led-function and include after dropping from soc dtsi
> ---
> .../dts/mediatek/mt7988a-bananapi-bpi-r4.dtsi | 61 +++++++++++++++++++
> 1 file changed, 61 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4.dtsi b/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4.dtsi
> index d8b9cd794ee3..f10d3617dcac 100644
> --- a/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4.dtsi
> @@ -4,6 +4,7 @@
>
> #include <dt-bindings/gpio/gpio.h>
> #include <dt-bindings/regulator/richtek,rt5190a-regulator.h>
> +#include <dt-bindings/leds/common.h>
>
> #include "mt7988a.dtsi"
>
> @@ -151,6 +152,66 @@ &gmac2 {
> phy-mode = "usxgmii";
> };
>
> +&gsw_phy0 {
> + pinctrl-names = "gbe-led";
> + pinctrl-0 = <&gbe0_led0_pins>;

pinctrl-0
pinctrl-names (here and everywhere else)

> +};
> +
> +&gsw_phy0_led0 {

function
color
status

> + status = "okay";
> + function = LED_FUNCTION_WAN;
> + color = <LED_COLOR_ID_GREEN>;
> +};
> +
> +&gsw_port0 {
> + label = "wan";
> +};
> +
> +&gsw_phy1 {
> + pinctrl-names = "gbe-led";
> + pinctrl-0 = <&gbe1_led0_pins>;
> +};
> +
> +&gsw_phy1_led0 {
> + status = "okay";
> + function = LED_FUNCTION_LAN;
> + color = <LED_COLOR_ID_GREEN>;
> +};
> +
> +&gsw_port1 {
> + label = "lan1";
> +};
> +
> +&gsw_phy2 {
> + pinctrl-names = "gbe-led";
> + pinctrl-0 = <&gbe2_led0_pins>;
> +};
> +
> +&gsw_phy2_led0 {
> + status = "okay";
> + function = LED_FUNCTION_LAN;
> + color = <LED_COLOR_ID_GREEN>;
> +};
> +
> +&gsw_port2 {
> + label = "lan2";
> +};
> +
> +&gsw_phy3 {

function

pinctrl-0
pinctrl-names

> + pinctrl-names = "gbe-led";
> + function = LED_FUNCTION_LAN;
> + pinctrl-0 = <&gbe3_led0_pins>;
> +};
> +
> +&gsw_phy3_led0 {

color
status

> + status = "okay";
> + color = <LED_COLOR_ID_GREEN>;
> +};

...after which

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>