On Mon, 2025-01-27 at 14:40 +0100, Christian Marangi wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>
>
> Move gpio-controller up to pinctrl node and rename label to "pio" to
> match the label used in upstream kernel linux.
>
> Update any DTS that reference the pinctrl to follow the new naming
> and
> structure.
>
> Signed-off-by: Christian Marangi <ansuels...@gmail.com>
> ---
> arch/arm/dts/mt7986.dtsi | 8 +++-----
> arch/arm/dts/mt7986a-bpi-r3-sd.dts | 12 ++++++------
> arch/arm/dts/mt7986a-rfb.dts | 4 ++--
> arch/arm/dts/mt7986a-sd-rfb.dts | 4 ++--
> arch/arm/dts/mt7986b-rfb.dts | 4 ++--
> arch/arm/dts/mt7986b-sd-rfb.dts | 4 ++--
> 6 files changed, 17 insertions(+), 19 deletions(-)
>
> diff --git a/arch/arm/dts/mt7986.dtsi b/arch/arm/dts/mt7986.dtsi
> index 64ce45a6530..b2c57098a5e 100644
> --- a/arch/arm/dts/mt7986.dtsi
> +++ b/arch/arm/dts/mt7986.dtsi
> @@ -122,7 +122,7 @@
> #clock-cells = <1>;
> };
>
> - pinctrl: pinctrl@1001f000 {
> + pio: pinctrl@1001f000 {
> compatible = "mediatek,mt7986-pinctrl";
> reg = <0x1001f000 0x1000>,
> <0x11c30000 0x1000>,
> @@ -135,10 +135,8 @@
> reg-names = "gpio", "iocfg_rt", "iocfg_rb",
> "iocfg_lt", "iocfg_lb", "iocfg_tr",
> "iocfg_tl", "eint";
> - gpio: gpio-controller {
> - gpio-controller;
> - #gpio-cells = <2>;
> - };
> + gpio-controller;
> + #gpio-cells = <2>;
> };
>
> pwm: pwm@10048000 {
> diff --git a/arch/arm/dts/mt7986a-bpi-r3-sd.dts
> b/arch/arm/dts/mt7986a-bpi-r3-sd.dts
> index c156a813634..e01ae4c3dc7 100644
> --- a/arch/arm/dts/mt7986a-bpi-r3-sd.dts
> +++ b/arch/arm/dts/mt7986a-bpi-r3-sd.dts
> @@ -38,12 +38,12 @@
>
> factory {
> label = "reset";
> - gpios = <&gpio 9 GPIO_ACTIVE_LOW>;
> + gpios = <&pio 9 GPIO_ACTIVE_LOW>;
> };
>
> wps {
> label = "wps";
> - gpios = <&gpio 10 GPIO_ACTIVE_LOW>;
> + gpios = <&pio 10 GPIO_ACTIVE_LOW>;
> };
> };
>
> @@ -52,12 +52,12 @@
>
> led_status_green: green {
> label = "green:status";
> - gpios = <&gpio 69 GPIO_ACTIVE_HIGH>;
> + gpios = <&pio 69 GPIO_ACTIVE_HIGH>;
> };
>
> led_status_blue: blue {
> label = "blue:status";
> - gpios = <&gpio 86 GPIO_ACTIVE_HIGH>;
> + gpios = <&pio 86 GPIO_ACTIVE_HIGH>;
> };
> };
>
> @@ -78,7 +78,7 @@
> mediatek,gmac-id = <0>;
> phy-mode = "2500base-x";
> mediatek,switch = "mt7531";
> - reset-gpios = <&gpio 5 GPIO_ACTIVE_HIGH>;
> + reset-gpios = <&pio 5 GPIO_ACTIVE_HIGH>;
>
> fixed-link {
> speed = <2500>;
> @@ -86,7 +86,7 @@
> };
> };
>
> -&pinctrl {
> +&pio {
> spic_pins: spi1-pins-func-1 {
> mux {
> function = "spi";
> diff --git a/arch/arm/dts/mt7986a-rfb.dts b/arch/arm/dts/mt7986a-
> rfb.dts
> index 67d14a99dae..d4bbb23204a 100644
> --- a/arch/arm/dts/mt7986a-rfb.dts
> +++ b/arch/arm/dts/mt7986a-rfb.dts
> @@ -57,7 +57,7 @@
> mediatek,gmac-id = <0>;
> phy-mode = "2500base-x";
> mediatek,switch = "mt7531";
> - reset-gpios = <&gpio 5 GPIO_ACTIVE_HIGH>;
> + reset-gpios = <&pio 5 GPIO_ACTIVE_HIGH>;
>
> fixed-link {
> speed = <2500>;
> @@ -65,7 +65,7 @@
> };
> };
>
> -&pinctrl {
> +&pio {
> spi_flash_pins: spi0-pins-func-1 {
> mux {
> function = "flash";
> diff --git a/arch/arm/dts/mt7986a-sd-rfb.dts b/arch/arm/dts/mt7986a-
> sd-rfb.dts
> index 4f8fa70ec99..11823e05bda 100644
> --- a/arch/arm/dts/mt7986a-sd-rfb.dts
> +++ b/arch/arm/dts/mt7986a-sd-rfb.dts
> @@ -49,7 +49,7 @@
> mediatek,gmac-id = <0>;
> phy-mode = "2500base-x";
> mediatek,switch = "mt7531";
> - reset-gpios = <&gpio 5 GPIO_ACTIVE_HIGH>;
> + reset-gpios = <&pio 5 GPIO_ACTIVE_HIGH>;
>
> fixed-link {
> speed = <2500>;
> @@ -57,7 +57,7 @@
> };
> };
>
> -&pinctrl {
> +&pio {
> spi_flash_pins: spi0-pins-func-1 {
> mux {
> function = "flash";
> diff --git a/arch/arm/dts/mt7986b-rfb.dts b/arch/arm/dts/mt7986b-
> rfb.dts
> index f98b04ab140..0a731fe87fd 100644
> --- a/arch/arm/dts/mt7986b-rfb.dts
> +++ b/arch/arm/dts/mt7986b-rfb.dts
> @@ -48,7 +48,7 @@
> mediatek,gmac-id = <0>;
> phy-mode = "2500base-x";
> mediatek,switch = "mt7531";
> - reset-gpios = <&gpio 5 GPIO_ACTIVE_HIGH>;
> + reset-gpios = <&pio 5 GPIO_ACTIVE_HIGH>;
>
> fixed-link {
> speed = <2500>;
> @@ -56,7 +56,7 @@
> };
> };
>
> -&pinctrl {
> +&pio {
> spi_flash_pins: spi0-pins-func-1 {
> mux {
> function = "flash";
> diff --git a/arch/arm/dts/mt7986b-sd-rfb.dts b/arch/arm/dts/mt7986b-
> sd-rfb.dts
> index ec80a2fb710..e5fb4d73e70 100644
> --- a/arch/arm/dts/mt7986b-sd-rfb.dts
> +++ b/arch/arm/dts/mt7986b-sd-rfb.dts
> @@ -49,7 +49,7 @@
> mediatek,gmac-id = <0>;
> phy-mode = "2500base-x";
> mediatek,switch = "mt7531";
> - reset-gpios = <&gpio 5 GPIO_ACTIVE_HIGH>;
> + reset-gpios = <&pio 5 GPIO_ACTIVE_HIGH>;
>
> fixed-link {
> speed = <2500>;
> @@ -57,7 +57,7 @@
> };
> };
>
> -&pinctrl {
> +&pio {
> spi_flash_pins: spi0-pins-func-1 {
> mux {
> function = "flash";
> --
> 2.47.1
>
Thanks!
Reviewed-by: Weijie Gao <weijie....@mediatek.com>
Tested-by: Weijie Gao <weijie....@mediatek.com>
