On Mon, Jan 16, 2023 at 02:29:11PM +0100, Ahmad Fatoum wrote: > When board support was first added, it used a version of the device tree > of the Linux kernel mailing list. The device tree is now upstream, so > replace the copy we have with the version already existing in dts/. > > Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> > --- > I don't have the hardware, Cc'ing Xogium, who has this board and may be > able to test this patch. > --- > arch/arm/dts/stm32mp157c-odyssey-som.dtsi | 294 ---------------------- > arch/arm/dts/stm32mp157c-odyssey.dts | 6 +- > arch/arm/dts/stm32mp157c-odyssey.dtsi | 72 ------ > 3 files changed, 1 insertion(+), 371 deletions(-) > delete mode 100644 arch/arm/dts/stm32mp157c-odyssey-som.dtsi > delete mode 100644 arch/arm/dts/stm32mp157c-odyssey.dtsi Applied, thanks Sascha > > diff --git a/arch/arm/dts/stm32mp157c-odyssey-som.dtsi b/arch/arm/dts/stm32mp157c-odyssey-som.dtsi > deleted file mode 100644 > index 1e5bd8bccb71..000000000000 > --- a/arch/arm/dts/stm32mp157c-odyssey-som.dtsi > +++ /dev/null > @@ -1,294 +0,0 @@ > -// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) > -/* > - * Copyright (C) 2020 Marcin Sloniewski <marcin.sloniewski@xxxxxxxxx>. > - */ > - > -/dts-v1/; > - > -#include <arm/stm32mp157.dtsi> > -#include <arm/stm32mp15xc.dtsi> > -#include <arm/stm32mp15-pinctrl.dtsi> > -#include <arm/stm32mp15xxac-pinctrl.dtsi> > -#include <dt-bindings/gpio/gpio.h> > -#include <dt-bindings/leds/common.h> > -#include <dt-bindings/mfd/st,stpmic1.h> > - > -/ { > - model = "Seeed Studio Odyssey-STM32MP157C SOM"; > - compatible = "seeed,stm32mp157c-odyssey-som", "st,stm32mp157"; > - > - memory@c0000000 { > - device_type = "memory"; > - reg = <0xc0000000 0x20000000>; > - }; > - > - reserved-memory { > - #address-cells = <1>; > - #size-cells = <1>; > - ranges; > - > - mcuram2: mcuram2@10000000 { > - compatible = "shared-dma-pool"; > - reg = <0x10000000 0x40000>; > - no-map; > - }; > - > - vdev0vring0: vdev0vring0@10040000 { > - compatible = "shared-dma-pool"; > - reg = <0x10040000 0x1000>; > - no-map; > - }; > - > - vdev0vring1: vdev0vring1@10041000 { > - compatible = "shared-dma-pool"; > - reg = <0x10041000 0x1000>; > - no-map; > - }; > - > - vdev0buffer: vdev0buffer@10042000 { > - compatible = "shared-dma-pool"; > - reg = <0x10042000 0x4000>; > - no-map; > - }; > - > - mcuram: mcuram@30000000 { > - compatible = "shared-dma-pool"; > - reg = <0x30000000 0x40000>; > - no-map; > - }; > - > - retram: retram@38000000 { > - compatible = "shared-dma-pool"; > - reg = <0x38000000 0x10000>; > - no-map; > - }; > - > - gpu_reserved: gpu@d4000000 { > - reg = <0xd4000000 0x4000000>; > - no-map; > - }; > - }; > - > - led { > - compatible = "gpio-leds"; > - led-blue { > - color = <LED_COLOR_ID_BLUE>; > - function = LED_FUNCTION_HEARTBEAT; > - gpios = <&gpiog 3 GPIO_ACTIVE_HIGH>; > - linux,default-trigger = "heartbeat"; > - }; > - }; > -}; > - > -&gpu { > - contiguous-area = <&gpu_reserved>; > - status = "okay"; > -}; > - > -&i2c2 { > - pinctrl-names = "default"; > - pinctrl-0 = <&i2c2_pins_a>; > - i2c-scl-rising-time-ns = <185>; > - i2c-scl-falling-time-ns = <20>; > - status = "okay"; > - /* spare dmas for other usage */ > - /delete-property/dmas; > - /delete-property/dma-names; > - > - pmic: stpmic@33 { > - compatible = "st,stpmic1"; > - reg = <0x33>; > - interrupts-extended = <&gpioa 0 IRQ_TYPE_EDGE_FALLING>; > - interrupt-controller; > - #interrupt-cells = <2>; > - > - regulators { > - compatible = "st,stpmic1-regulators"; > - ldo1-supply = <&v3v3>; > - ldo3-supply = <&vdd_ddr>; > - ldo6-supply = <&v3v3>; > - pwr_sw1-supply = <&bst_out>; > - pwr_sw2-supply = <&bst_out>; > - > - vddcore: buck1 { > - regulator-name = "vddcore"; > - regulator-min-microvolt = <800000>; > - regulator-max-microvolt = <1350000>; > - regulator-always-on; > - regulator-initial-mode = <0>; > - regulator-over-current-protection; > - }; > - > - vdd_ddr: buck2 { > - regulator-name = "vdd_ddr"; > - regulator-min-microvolt = <1350000>; > - regulator-max-microvolt = <1350000>; > - regulator-always-on; > - regulator-initial-mode = <0>; > - regulator-over-current-protection; > - }; > - > - vdd: buck3 { > - regulator-name = "vdd"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > - regulator-always-on; > - st,mask-reset; > - regulator-initial-mode = <0>; > - regulator-over-current-protection; > - }; > - > - v3v3: buck4 { > - regulator-name = "v3v3"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > - regulator-always-on; > - regulator-over-current-protection; > - regulator-initial-mode = <0>; > - }; > - > - v1v8_audio: ldo1 { > - regulator-name = "v1v8_audio"; > - regulator-min-microvolt = <1800000>; > - regulator-max-microvolt = <1800000>; > - regulator-always-on; > - interrupts = <IT_CURLIM_LDO1 0>; > - }; > - > - v3v3_hdmi: ldo2 { > - regulator-name = "v3v3_hdmi"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > - regulator-always-on; > - interrupts = <IT_CURLIM_LDO2 0>; > - }; > - > - vtt_ddr: ldo3 { > - regulator-name = "vtt_ddr"; > - regulator-min-microvolt = <500000>; > - regulator-max-microvolt = <750000>; > - regulator-always-on; > - regulator-over-current-protection; > - }; > - > - vdd_usb: ldo4 { > - regulator-name = "vdd_usb"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > - interrupts = <IT_CURLIM_LDO4 0>; > - }; > - > - vdda: ldo5 { > - regulator-name = "vdda"; > - regulator-min-microvolt = <2900000>; > - regulator-max-microvolt = <2900000>; > - interrupts = <IT_CURLIM_LDO5 0>; > - regulator-boot-on; > - }; > - > - v1v2_hdmi: ldo6 { > - regulator-name = "v1v2_hdmi"; > - regulator-min-microvolt = <1200000>; > - regulator-max-microvolt = <1200000>; > - regulator-always-on; > - interrupts = <IT_CURLIM_LDO6 0>; > - }; > - > - vref_ddr: vref_ddr { > - regulator-name = "vref_ddr"; > - regulator-always-on; > - regulator-over-current-protection; > - }; > - > - bst_out: boost { > - regulator-name = "bst_out"; > - interrupts = <IT_OCP_BOOST 0>; > - }; > - > - vbus_otg: pwr_sw1 { > - regulator-name = "vbus_otg"; > - interrupts = <IT_OCP_OTG 0>; > - }; > - > - vbus_sw: pwr_sw2 { > - regulator-name = "vbus_sw"; > - interrupts = <IT_OCP_SWOUT 0>; > - regulator-active-discharge; > - }; > - }; > - > - onkey { > - compatible = "st,stpmic1-onkey"; > - interrupts = <IT_PONKEY_F 0>, <IT_PONKEY_R 0>; > - interrupt-names = "onkey-falling", "onkey-rising"; > - power-off-time-sec = <10>; > - }; > - > - watchdog { > - compatible = "st,stpmic1-wdt"; > - status = "disabled"; > - }; > - }; > -}; > - > -&ipcc { > - status = "okay"; > -}; > - > -&iwdg2 { > - timeout-sec = <32>; > - status = "okay"; > -}; > - > -&m4_rproc { > - memory-region = <&retram>, <&mcuram>, <&mcuram2>, <&vdev0vring0>, > - <&vdev0vring1>, <&vdev0buffer>; > - mboxes = <&ipcc 0>, <&ipcc 1>, <&ipcc 2>; > - mbox-names = "vq0", "vq1", "shutdown"; > - interrupt-parent = <&exti>; > - interrupts = <68 1>; > - status = "okay"; > -}; > - > -&rng1 { > - status = "okay"; > -}; > - > -&rtc { > - status = "okay"; > -}; > - > -&sdmmc2_d47_pins_a { > - pins { > - pinmux = <STM32_PINMUX('A', 8, AF9)>, /* SDMMC2_D4 */ > - <STM32_PINMUX('A', 9, AF10)>, /* SDMMC2_D5 */ > - <STM32_PINMUX('E', 5, AF9)>, /* SDMMC2_D6 */ > - <STM32_PINMUX('C', 7, AF10)>; /* SDMMC2_D7 */ > - }; > -}; > - > -&sdmmc2_d47_sleep_pins_a { > - pins { > - pinmux = <STM32_PINMUX('A', 8, ANALOG)>, /* SDMMC2_D4 */ > - <STM32_PINMUX('A', 9, ANALOG)>, /* SDMMC2_D5 */ > - <STM32_PINMUX('E', 5, ANALOG)>, /* SDMMC2_D6 */ > - <STM32_PINMUX('C', 7, ANALOG)>; /* SDMMC2_D7 */ > - }; > -}; > - > -&sdmmc2 { > - pinctrl-names = "default", "opendrain", "sleep"; > - pinctrl-0 = <&sdmmc2_b4_pins_a &sdmmc2_d47_pins_a>; > - pinctrl-1 = <&sdmmc2_b4_od_pins_a &sdmmc2_d47_pins_a>; > - pinctrl-2 = <&sdmmc2_b4_sleep_pins_a &sdmmc2_d47_sleep_pins_a>; > - non-removable; > - no-sd; > - no-sdio; > - st,neg-edge; > - bus-width = <8>; > - vmmc-supply = <&v3v3>; > - vqmmc-supply = <&v3v3>; > - mmc-ddr-3_3v; > - status = "okay"; > -}; > - > diff --git a/arch/arm/dts/stm32mp157c-odyssey.dts b/arch/arm/dts/stm32mp157c-odyssey.dts > index 0e395bdec961..6ced6363eb0f 100644 > --- a/arch/arm/dts/stm32mp157c-odyssey.dts > +++ b/arch/arm/dts/stm32mp157c-odyssey.dts > @@ -3,7 +3,7 @@ > * Copyright (C) 2020 Ahmad Fatoum, Pengutronix > */ > > -#include "stm32mp157c-odyssey.dtsi" > +#include <arm/stm32mp157c-odyssey.dts> > #include "stm32mp151.dtsi" > > / { > @@ -21,7 +21,3 @@ > }; > }; > }; > - > -&phy0 { > - reset-gpios = <&gpiog 0 GPIO_ACTIVE_LOW>; > -}; > diff --git a/arch/arm/dts/stm32mp157c-odyssey.dtsi b/arch/arm/dts/stm32mp157c-odyssey.dtsi > deleted file mode 100644 > index 85a4f313ae33..000000000000 > --- a/arch/arm/dts/stm32mp157c-odyssey.dtsi > +++ /dev/null > @@ -1,72 +0,0 @@ > -// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) > -/* > - * Copyright (C) 2020 Marcin Sloniewski <marcin.sloniewski@xxxxxxxxx>. > - */ > - > -/dts-v1/; > - > -#include "stm32mp157c-odyssey-som.dtsi" > - > -/ { > - model = "Seeed Studio Odyssey-STM32MP157C Board"; > - compatible = "seeed,stm32mp157c-odyssey", > - "seeed,stm32mp157c-odyssey-som", "st,stm32mp157"; > - > - aliases { > - ethernet0 = ðernet0; > - serial0 = &uart4; > - }; > - > - chosen { > - stdout-path = "serial0:115200n8"; > - }; > -}; > - > -ðernet0 { > - status = "okay"; > - pinctrl-0 = <ðernet0_rgmii_pins_a>; > - pinctrl-1 = <ðernet0_rgmii_sleep_pins_a>; > - pinctrl-names = "default", "sleep"; > - phy-mode = "rgmii-id"; > - max-speed = <1000>; > - phy-handle = <&phy0>; > - > - mdio0 { > - #address-cells = <1>; > - #size-cells = <0>; > - compatible = "snps,dwmac-mdio"; > - phy0: ethernet-phy@7 { /* KSZ9031RN */ > - reg = <7>; > - }; > - }; > -}; > - > -&i2c1 { > - pinctrl-names = "default", "sleep"; > - pinctrl-0 = <&i2c1_pins_a>; > - pinctrl-1 = <&i2c1_sleep_pins_a>; > - i2c-scl-rising-time-ns = <100>; > - i2c-scl-falling-time-ns = <7>; > - status = "okay"; > - /delete-property/dmas; > - /delete-property/dma-names; > -}; > - > -&sdmmc1 { > - pinctrl-names = "default", "opendrain", "sleep"; > - pinctrl-0 = <&sdmmc1_b4_pins_a>; > - pinctrl-1 = <&sdmmc1_b4_od_pins_a>; > - pinctrl-2 = <&sdmmc1_b4_sleep_pins_a>; > - cd-gpios = <&gpiob 7 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>; > - disable-wp; > - st,neg-edge; > - bus-width = <4>; > - vmmc-supply = <&v3v3>; > - status = "okay"; > -}; > - > -&uart4 { > - pinctrl-names = "default"; > - pinctrl-0 = <&uart4_pins_a>; > - status = "okay"; > -}; > -- > 2.30.2 > > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |