Hi Patrick On 2/24/21 11:00 AM, Patrick Delaunay wrote: > Remove duplicated uart nodes introduced with commit 62f95af92a3f > ("ARM: dts: stm32mp1: DT alignment with Linux kernel v5.9-rc4"), > because the uart nodes wasn't correctly ordered in alphabetic order. > > Only cosmetic: the generated device tree don't change. > > Signed-off-by: Patrick Delaunay <patrick.delau...@foss.st.com> > --- > > arch/arm/dts/stm32mp15-pinctrl.dtsi | 80 ----------------------------- > 1 file changed, 80 deletions(-) > > diff --git a/arch/arm/dts/stm32mp15-pinctrl.dtsi > b/arch/arm/dts/stm32mp15-pinctrl.dtsi > index dd4bd1e554..20a59e8f7a 100644 > --- a/arch/arm/dts/stm32mp15-pinctrl.dtsi > +++ b/arch/arm/dts/stm32mp15-pinctrl.dtsi > @@ -1971,86 +1971,6 @@ > }; > }; > > - uart4_pins_a: uart4-0 { > - pins1 { > - pinmux = <STM32_PINMUX('G', 11, AF6)>; /* UART4_TX */ > - bias-disable; > - drive-push-pull; > - slew-rate = <0>; > - }; > - pins2 { > - pinmux = <STM32_PINMUX('B', 2, AF8)>; /* UART4_RX */ > - bias-disable; > - }; > - }; > - > - uart4_pins_b: uart4-1 { > - pins1 { > - pinmux = <STM32_PINMUX('D', 1, AF8)>; /* UART4_TX */ > - bias-disable; > - drive-push-pull; > - slew-rate = <0>; > - }; > - pins2 { > - pinmux = <STM32_PINMUX('B', 2, AF8)>; /* UART4_RX */ > - bias-disable; > - }; > - }; > - > - uart4_pins_c: uart4-2 { > - pins1 { > - pinmux = <STM32_PINMUX('G', 11, AF6)>; /* UART4_TX */ > - bias-disable; > - drive-push-pull; > - slew-rate = <0>; > - }; > - pins2 { > - pinmux = <STM32_PINMUX('B', 2, AF8)>; /* UART4_RX */ > - bias-disable; > - }; > - }; > - > - uart7_pins_a: uart7-0 { > - pins1 { > - pinmux = <STM32_PINMUX('E', 8, AF7)>; /* UART4_TX */ > - bias-disable; > - drive-push-pull; > - slew-rate = <0>; > - }; > - pins2 { > - pinmux = <STM32_PINMUX('E', 7, AF7)>, /* UART4_RX */ > - <STM32_PINMUX('E', 10, AF7)>, /* UART4_CTS */ > - <STM32_PINMUX('E', 9, AF7)>; /* UART4_RTS */ > - bias-disable; > - }; > - }; > - > - uart7_pins_b: uart7-1 { > - pins1 { > - pinmux = <STM32_PINMUX('F', 7, AF7)>; /* UART7_TX */ > - bias-disable; > - drive-push-pull; > - slew-rate = <0>; > - }; > - pins2 { > - pinmux = <STM32_PINMUX('F', 6, AF7)>; /* UART7_RX */ > - bias-disable; > - }; > - }; > - > - uart8_pins_a: uart8-0 { > - pins1 { > - pinmux = <STM32_PINMUX('E', 1, AF8)>; /* UART8_TX */ > - bias-disable; > - drive-push-pull; > - slew-rate = <0>; > - }; > - pins2 { > - pinmux = <STM32_PINMUX('E', 0, AF8)>; /* UART8_RX */ > - bias-disable; > - }; > - }; > - > usbotg_hs_pins_a: usbotg-hs-0 { > pins { > pinmux = <STM32_PINMUX('A', 10, ANALOG)>; /* OTG_ID */ >
Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com> Thanks Patrice