* [PATCH] ARM: dts: intel: Drop pxa2xx @ 2025-12-12 20:32 Rob Herring (Arm) 2025-12-12 21:58 ` Robert Jarzmik 0 siblings, 1 reply; 7+ messages in thread From: Rob Herring (Arm) @ 2025-12-12 20:32 UTC (permalink / raw) To: Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski, Conor Dooley, Daniel Mack, Haojian Zhuang, Robert Jarzmik, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth Cc: linux-renesas-soc, devicetree, linux-kernel, linux-arm-kernel These .dtsi files are not included anywhere in the tree and can't be tested. They have not been touched since 2018 other than clean-ups. Signed-off-by: Rob Herring (Arm) <robh@kernel.org> --- arch/arm/boot/dts/intel/pxa/pxa25x.dtsi | 118 ----------- arch/arm/boot/dts/intel/pxa/pxa27x.dtsi | 188 ------------------ arch/arm/boot/dts/marvell/armada-380.dtsi | 148 -------------- arch/arm64/boot/dts/marvell/armada-7020.dtsi | 10 - arch/arm64/boot/dts/marvell/armada-8020.dtsi | 20 -- .../boot/dts/marvell/armada-ap806-dual.dtsi | 60 ------ .../dts/marvell/cn9130-db-comexpress.dtsi | 96 --------- 7 files changed, 640 deletions(-) delete mode 100644 arch/arm/boot/dts/intel/pxa/pxa25x.dtsi delete mode 100644 arch/arm/boot/dts/intel/pxa/pxa27x.dtsi delete mode 100644 arch/arm/boot/dts/marvell/armada-380.dtsi delete mode 100644 arch/arm64/boot/dts/marvell/armada-7020.dtsi delete mode 100644 arch/arm64/boot/dts/marvell/armada-8020.dtsi delete mode 100644 arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi delete mode 100644 arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi diff --git a/arch/arm/boot/dts/intel/pxa/pxa25x.dtsi b/arch/arm/boot/dts/intel/pxa/pxa25x.dtsi deleted file mode 100644 index 5f8300e356ad..000000000000 --- a/arch/arm/boot/dts/intel/pxa/pxa25x.dtsi +++ /dev/null @@ -1,118 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0-or-later -/* - * Copyright (C) 2016 Robert Jarzmik <robert.jarzmik@free.fr> - */ -#include "pxa2xx.dtsi" -#include "dt-bindings/clock/pxa-clock.h" - -/ { - model = "Marvell PXA25x family SoC"; - compatible = "marvell,pxa250"; - - clocks { - /* - * The muxing of external clocks/internal dividers for osc* clock - * sources has been hidden under the carpet by now. - */ - #address-cells = <1>; - #size-cells = <1>; - ranges; - - clks: pxa2xx_clks@41300004 { - compatible = "marvell,pxa250-core-clocks"; - #clock-cells = <1>; - status = "okay"; - }; - - /* timer oscillator */ - clktimer: oscillator { - compatible = "fixed-clock"; - #clock-cells = <0>; - clock-frequency = <3686400>; - clock-output-names = "ostimer"; - }; - }; - - pxabus { - pdma: dma-controller@40000000 { - compatible = "marvell,pdma-1.0"; - reg = <0x40000000 0x10000>; - interrupts = <25>; - #dma-cells = <2>; - /* For backwards compatibility: */ - #dma-channels = <16>; - dma-channels = <16>; - #dma-requests = <40>; - dma-requests = <40>; - status = "okay"; - }; - - pxairq: interrupt-controller@40d00000 { - marvell,intc-priority; - marvell,intc-nr-irqs = <32>; - }; - - pinctrl: pinctrl@40e00000 { - reg = <0x40e00054 0x20 0x40e0000c 0xc 0x40e0010c 4 - 0x40f00020 0x10>; - compatible = "marvell,pxa25x-pinctrl"; - }; - - gpio: gpio@40e00000 { - compatible = "intel,pxa25x-gpio"; - gpio-ranges = <&pinctrl 0 0 84>; - clocks = <&clks CLK_NONE>; - }; - - pwm0: pwm@40b00000 { - compatible = "marvell,pxa250-pwm"; - reg = <0x40b00000 0x10>; - #pwm-cells = <1>; - clocks = <&clks CLK_PWM0>; - }; - - pwm1: pwm@40b00010 { - compatible = "marvell,pxa250-pwm"; - reg = <0x40b00010 0x10>; - #pwm-cells = <1>; - clocks = <&clks CLK_PWM1>; - }; - - rtc@40900000 { - clocks = <&clks CLK_OSC32k768>; - }; - }; - - timer@40a00000 { - compatible = "marvell,pxa-timer"; - reg = <0x40a00000 0x20>; - interrupts = <26>; - clocks = <&clktimer>; - status = "okay"; - }; - - pxa250_opp_table: opp_table0 { - compatible = "operating-points-v2"; - - opp-99532800 { - opp-hz = /bits/ 64 <99532800>; - opp-microvolt = <1000000 950000 1650000>; - clock-latency-ns = <20>; - }; - opp-199065600 { - opp-hz = /bits/ 64 <199065600>; - opp-microvolt = <1000000 950000 1650000>; - clock-latency-ns = <20>; - }; - opp-298598400 { - opp-hz = /bits/ 64 <298598400>; - opp-microvolt = <1100000 1045000 1650000>; - clock-latency-ns = <20>; - }; - opp-398131200 { - opp-hz = /bits/ 64 <398131200>; - opp-microvolt = <1300000 1235000 1650000>; - clock-latency-ns = <20>; - }; - }; -}; diff --git a/arch/arm/boot/dts/intel/pxa/pxa27x.dtsi b/arch/arm/boot/dts/intel/pxa/pxa27x.dtsi deleted file mode 100644 index a2cbfb3be609..000000000000 --- a/arch/arm/boot/dts/intel/pxa/pxa27x.dtsi +++ /dev/null @@ -1,188 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 -/* The pxa3xx skeleton simply augments the 2xx version */ -#include "pxa2xx.dtsi" -#include "dt-bindings/clock/pxa-clock.h" - -/ { - model = "Marvell PXA27x familiy SoC"; - compatible = "marvell,pxa27x"; - - pxabus { - pdma: dma-controller@40000000 { - compatible = "marvell,pdma-1.0"; - reg = <0x40000000 0x10000>; - interrupts = <25>; - #dma-cells = <2>; - /* For backwards compatibility: */ - #dma-channels = <32>; - dma-channels = <32>; - #dma-requests = <75>; - dma-requests = <75>; - status = "okay"; - }; - - pxairq: interrupt-controller@40d00000 { - marvell,intc-priority; - marvell,intc-nr-irqs = <34>; - }; - - pinctrl: pinctrl@40e00000 { - reg = <0x40e00054 0x20 0x40e0000c 0xc 0x40e0010c 4 - 0x40f00020 0x10>; - compatible = "marvell,pxa27x-pinctrl"; - }; - - gpio: gpio@40e00000 { - compatible = "intel,pxa27x-gpio"; - gpio-ranges = <&pinctrl 0 0 128>; - clocks = <&clks CLK_NONE>; - }; - - usb0: usb@4c000000 { - compatible = "marvell,pxa-ohci"; - reg = <0x4c000000 0x10000>; - interrupts = <3>; - clocks = <&clks CLK_USBHOST>; - status = "disabled"; - }; - - pwm0: pwm@40b00000 { - compatible = "marvell,pxa270-pwm", "marvell,pxa250-pwm"; - reg = <0x40b00000 0x10>; - #pwm-cells = <1>; - clocks = <&clks CLK_PWM0>; - }; - - pwm1: pwm@40b00010 { - compatible = "marvell,pxa270-pwm", "marvell,pxa250-pwm"; - reg = <0x40b00010 0x10>; - #pwm-cells = <1>; - clocks = <&clks CLK_PWM1>; - }; - - pwm2: pwm@40c00000 { - compatible = "marvell,pxa270-pwm", "marvell,pxa250-pwm"; - reg = <0x40c00000 0x10>; - #pwm-cells = <1>; - clocks = <&clks CLK_PWM0>; - }; - - pwm3: pwm@40c00010 { - compatible = "marvell,pxa270-pwm", "marvell,pxa250-pwm"; - reg = <0x40c00010 0x10>; - #pwm-cells = <1>; - clocks = <&clks CLK_PWM1>; - }; - - pwri2c: i2c@40f00180 { - compatible = "mrvl,pxa-i2c"; - reg = <0x40f00180 0x24>; - interrupts = <6>; - clocks = <&clks CLK_PWRI2C>; - #address-cells = <0x1>; - #size-cells = <0>; - status = "disabled"; - }; - - pxa27x_udc: udc@40600000 { - compatible = "marvell,pxa270-udc"; - reg = <0x40600000 0x10000>; - interrupts = <11>; - clocks = <&clks CLK_USB>; - status = "disabled"; - }; - - keypad: keypad@41500000 { - compatible = "marvell,pxa27x-keypad"; - reg = <0x41500000 0x4c>; - interrupts = <4>; - clocks = <&clks CLK_KEYPAD>; - status = "disabled"; - }; - - pxa_camera: imaging@50000000 { - compatible = "marvell,pxa270-qci"; - reg = <0x50000000 0x1000>; - interrupts = <33>; - dmas = <&pdma 68 0 /* Y channel */ - &pdma 69 0 /* U channel */ - &pdma 70 0>; /* V channel */ - dma-names = "CI_Y", "CI_U", "CI_V"; - - clocks = <&clks CLK_CAMERA>; - clock-names = "ciclk"; - clock-frequency = <5000000>; - clock-output-names = "qci_mclk"; - - status = "disabled"; - }; - - rtc@40900000 { - clocks = <&clks CLK_OSC32k768>; - }; - }; - - clocks { - /* - * The muxing of external clocks/internal dividers for osc* clock - * sources has been hidden under the carpet by now. - */ - #address-cells = <1>; - #size-cells = <1>; - ranges; - - clks: pxa2xx_clks@41300004 { - compatible = "marvell,pxa270-clocks"; - #clock-cells = <1>; - status = "okay"; - }; - }; - - timer@40a00000 { - compatible = "marvell,pxa-timer"; - reg = <0x40a00000 0x20>; - interrupts = <26>; - clocks = <&clks CLK_OSTIMER>; - status = "okay"; - }; - - pxa270_opp_table: opp_table0 { - compatible = "operating-points-v2"; - - opp-104000000 { - opp-hz = /bits/ 64 <104000000>; - opp-microvolt = <900000 900000 1705000>; - clock-latency-ns = <20>; - }; - opp-156000000 { - opp-hz = /bits/ 64 <156000000>; - opp-microvolt = <1000000 1000000 1705000>; - clock-latency-ns = <20>; - }; - opp-208000000 { - opp-hz = /bits/ 64 <208000000>; - opp-microvolt = <1180000 1180000 1705000>; - clock-latency-ns = <20>; - }; - opp-312000000 { - opp-hz = /bits/ 64 <312000000>; - opp-microvolt = <1250000 1250000 1705000>; - clock-latency-ns = <20>; - }; - opp-416000000 { - opp-hz = /bits/ 64 <416000000>; - opp-microvolt = <1350000 1350000 1705000>; - clock-latency-ns = <20>; - }; - opp-520000000 { - opp-hz = /bits/ 64 <520000000>; - opp-microvolt = <1450000 1450000 1705000>; - clock-latency-ns = <20>; - }; - opp-624000000 { - opp-hz = /bits/ 64 <624000000>; - opp-microvolt = <1550000 1550000 1705000>; - clock-latency-ns = <20>; - }; - }; -}; diff --git a/arch/arm/boot/dts/marvell/armada-380.dtsi b/arch/arm/boot/dts/marvell/armada-380.dtsi deleted file mode 100644 index e94f22b0e9b5..000000000000 --- a/arch/arm/boot/dts/marvell/armada-380.dtsi +++ /dev/null @@ -1,148 +0,0 @@ -// SPDX-License-Identifier: (GPL-2.0+ OR MIT) -/* - * Device Tree Include file for Marvell Armada 380 SoC. - * - * Copyright (C) 2014 Marvell - * - * Lior Amsalem <alior@marvell.com> - * Gregory CLEMENT <gregory.clement@free-electrons.com> - * Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - */ - -#include "armada-38x.dtsi" - -/ { - model = "Marvell Armada 380 family SoC"; - compatible = "marvell,armada380"; - - cpus { - #address-cells = <1>; - #size-cells = <0>; - enable-method = "marvell,armada-380-smp"; - - cpu@0 { - device_type = "cpu"; - compatible = "arm,cortex-a9"; - reg = <0>; - }; - }; - - soc { - internal-regs { - pinctrl@18000 { - compatible = "marvell,mv88f6810-pinctrl"; - }; - }; - - pcie { - compatible = "marvell,armada-370-pcie"; - status = "disabled"; - device_type = "pci"; - - #address-cells = <3>; - #size-cells = <2>; - - msi-parent = <&mpic>; - bus-range = <0x00 0xff>; - - ranges = - <0x82000000 0 0x80000 MBUS_ID(0xf0, 0x01) 0x80000 0 0x00002000 - 0x82000000 0 0x40000 MBUS_ID(0xf0, 0x01) 0x40000 0 0x00002000 - 0x82000000 0 0x44000 MBUS_ID(0xf0, 0x01) 0x44000 0 0x00002000 - 0x82000000 0 0x48000 MBUS_ID(0xf0, 0x01) 0x48000 0 0x00002000 - 0x82000000 0x1 0 MBUS_ID(0x08, 0xe8) 0 1 0 /* Port 0 MEM */ - 0x81000000 0x1 0 MBUS_ID(0x08, 0xe0) 0 1 0 /* Port 0 IO */ - 0x82000000 0x2 0 MBUS_ID(0x04, 0xe8) 0 1 0 /* Port 1 MEM */ - 0x81000000 0x2 0 MBUS_ID(0x04, 0xe0) 0 1 0 /* Port 1 IO */ - 0x82000000 0x3 0 MBUS_ID(0x04, 0xd8) 0 1 0 /* Port 2 MEM */ - 0x81000000 0x3 0 MBUS_ID(0x04, 0xd0) 0 1 0 /* Port 2 IO */>; - - /* x1 port */ - pcie@1,0 { - device_type = "pci"; - assigned-addresses = <0x82000800 0 0x80000 0 0x2000>; - reg = <0x0800 0 0 0 0>; - #address-cells = <3>; - #size-cells = <2>; - interrupt-names = "intx"; - interrupts-extended = <&gic GIC_SPI 29 IRQ_TYPE_LEVEL_HIGH>; - #interrupt-cells = <1>; - ranges = <0x82000000 0 0 0x82000000 0x1 0 1 0 - 0x81000000 0 0 0x81000000 0x1 0 1 0>; - bus-range = <0x00 0xff>; - interrupt-map-mask = <0 0 0 7>; - interrupt-map = <0 0 0 1 &pcie1_intc 0>, - <0 0 0 2 &pcie1_intc 1>, - <0 0 0 3 &pcie1_intc 2>, - <0 0 0 4 &pcie1_intc 3>; - marvell,pcie-port = <0>; - marvell,pcie-lane = <0>; - clocks = <&gateclk 8>; - status = "disabled"; - - pcie1_intc: interrupt-controller { - interrupt-controller; - #interrupt-cells = <1>; - }; - }; - - /* x1 port */ - pcie@2,0 { - device_type = "pci"; - assigned-addresses = <0x82001000 0 0x40000 0 0x2000>; - reg = <0x1000 0 0 0 0>; - #address-cells = <3>; - #size-cells = <2>; - interrupt-names = "intx"; - interrupts-extended = <&gic GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; - #interrupt-cells = <1>; - ranges = <0x82000000 0 0 0x82000000 0x2 0 1 0 - 0x81000000 0 0 0x81000000 0x2 0 1 0>; - bus-range = <0x00 0xff>; - interrupt-map-mask = <0 0 0 7>; - interrupt-map = <0 0 0 1 &pcie2_intc 0>, - <0 0 0 2 &pcie2_intc 1>, - <0 0 0 3 &pcie2_intc 2>, - <0 0 0 4 &pcie2_intc 3>; - marvell,pcie-port = <1>; - marvell,pcie-lane = <0>; - clocks = <&gateclk 5>; - status = "disabled"; - - pcie2_intc: interrupt-controller { - interrupt-controller; - #interrupt-cells = <1>; - }; - }; - - /* x1 port */ - pcie@3,0 { - device_type = "pci"; - assigned-addresses = <0x82001800 0 0x44000 0 0x2000>; - reg = <0x1800 0 0 0 0>; - #address-cells = <3>; - #size-cells = <2>; - interrupt-names = "intx"; - interrupts-extended = <&gic GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH>; - #interrupt-cells = <1>; - ranges = <0x82000000 0 0 0x82000000 0x3 0 1 0 - 0x81000000 0 0 0x81000000 0x3 0 1 0>; - bus-range = <0x00 0xff>; - interrupt-map-mask = <0 0 0 7>; - interrupt-map = <0 0 0 1 &pcie3_intc 0>, - <0 0 0 2 &pcie3_intc 1>, - <0 0 0 3 &pcie3_intc 2>, - <0 0 0 4 &pcie3_intc 3>; - marvell,pcie-port = <2>; - marvell,pcie-lane = <0>; - clocks = <&gateclk 6>; - status = "disabled"; - - pcie3_intc: interrupt-controller { - interrupt-controller; - #interrupt-cells = <1>; - }; - }; - }; - }; -}; diff --git a/arch/arm64/boot/dts/marvell/armada-7020.dtsi b/arch/arm64/boot/dts/marvell/armada-7020.dtsi deleted file mode 100644 index 570f901b4f4a..000000000000 --- a/arch/arm64/boot/dts/marvell/armada-7020.dtsi +++ /dev/null @@ -1,10 +0,0 @@ -// SPDX-License-Identifier: (GPL-2.0+ OR MIT) -/* - * Copyright (C) 2016 Marvell Technology Group Ltd. - * - * Device Tree file for the Armada 7020 SoC, made of an AP806 Dual and - * one CP110. - */ - -#include "armada-ap806-dual.dtsi" -#include "armada-70x0.dtsi" diff --git a/arch/arm64/boot/dts/marvell/armada-8020.dtsi b/arch/arm64/boot/dts/marvell/armada-8020.dtsi deleted file mode 100644 index b6fc18876093..000000000000 --- a/arch/arm64/boot/dts/marvell/armada-8020.dtsi +++ /dev/null @@ -1,20 +0,0 @@ -// SPDX-License-Identifier: (GPL-2.0+ OR MIT) -/* - * Copyright (C) 2016 Marvell Technology Group Ltd. - * - * Device Tree file for the Armada 8020 SoC, made of an AP806 Dual and - * two CP110. - */ - -#include "armada-ap806-dual.dtsi" -#include "armada-80x0.dtsi" - -/* The RTC requires external oscillator. But on Aramda 80x0, the RTC clock - * in CP master is not connected (by package) to the oscillator. So - * disable it. However, the RTC clock in CP slave is connected to the - * oscillator so this one is let enabled. - */ - -&cp0_rtc { - status = "disabled"; -}; diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi deleted file mode 100644 index 82f4dedfc25e..000000000000 --- a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi +++ /dev/null @@ -1,60 +0,0 @@ -// SPDX-License-Identifier: (GPL-2.0+ OR MIT) -/* - * Copyright (C) 2016 Marvell Technology Group Ltd. - * - * Device Tree file for Marvell Armada AP806. - */ - -#include "armada-ap806.dtsi" - -/ { - cpus { - #address-cells = <1>; - #size-cells = <0>; - - cpu0: cpu@0 { - device_type = "cpu"; - compatible = "arm,cortex-a72"; - reg = <0x000>; - enable-method = "psci"; - #cooling-cells = <2>; - clocks = <&cpu_clk 0>; - i-cache-size = <0xc000>; - i-cache-line-size = <64>; - i-cache-sets = <256>; - d-cache-size = <0x8000>; - d-cache-line-size = <64>; - d-cache-sets = <256>; - next-level-cache = <&l2>; - }; - cpu1: cpu@1 { - device_type = "cpu"; - compatible = "arm,cortex-a72"; - reg = <0x001>; - enable-method = "psci"; - #cooling-cells = <2>; - clocks = <&cpu_clk 0>; - i-cache-size = <0xc000>; - i-cache-line-size = <64>; - i-cache-sets = <256>; - d-cache-size = <0x8000>; - d-cache-line-size = <64>; - d-cache-sets = <256>; - next-level-cache = <&l2>; - }; - - l2: l2-cache { - compatible = "cache"; - cache-size = <0x80000>; - cache-line-size = <64>; - cache-sets = <512>; - cache-level = <2>; - cache-unified; - }; - }; - - thermal-zones { - /delete-node/ ap-thermal-cpu2; - /delete-node/ ap-thermal-cpu3; - }; -}; diff --git a/arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi b/arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi deleted file mode 100644 index 028496ebc473..000000000000 --- a/arch/arm64/boot/dts/marvell/cn9130-db-comexpress.dtsi +++ /dev/null @@ -1,96 +0,0 @@ -// SPDX-License-Identifier: (GPL-2.0+ OR MIT) -/* - * Copyright (C) 2023 Marvell International Ltd. - * - * Device tree for the CN9130-DB Com Express CPU module board. - */ - -#include "cn9130-db.dtsi" - -/ { - model = "Marvell Armada CN9130-DB COM EXPRESS type 7 CPU module board"; - compatible = "marvell,cn9130-cpu-module", "marvell,cn9130", - "marvell,armada-ap807-quad", "marvell,armada-ap807"; - -}; - -&ap0_reg_sd_vccq { - regulator-max-microvolt = <1800000>; - states = <1800000 0x1 1800000 0x0>; - /delete-property/ gpios; -}; - -&cp0_reg_usb3_vbus0 { - /delete-property/ gpio; -}; - -&cp0_reg_usb3_vbus1 { - /delete-property/ gpio; -}; - -&cp0_reg_sd_vcc { - status = "disabled"; -}; - -&cp0_reg_sd_vccq { - status = "disabled"; -}; - -&cp0_sdhci0 { - status = "disabled"; -}; - -&cp0_eth0 { - status = "disabled"; -}; - -&cp0_eth1 { - status = "okay"; - phy = <&phy0>; - phy-mode = "rgmii-id"; -}; - -&cp0_eth2 { - status = "disabled"; -}; - -&cp0_mdio { - status = "okay"; - pinctrl-0 = <&cp0_ge_mdio_pins>; - phy0: ethernet-phy@0 { - status = "okay"; - }; -}; - -&cp0_syscon0 { - cp0_pinctrl: pinctrl { - compatible = "marvell,cp115-standalone-pinctrl"; - - cp0_ge_mdio_pins: ge-mdio-pins { - marvell,pins = "mpp40", "mpp41"; - marvell,function = "ge"; - }; - }; -}; - -&cp0_sdhci0 { - status = "disabled"; -}; - -&cp0_spi1 { - status = "okay"; -}; - -&cp0_usb3_0 { - status = "okay"; - usb-phy = <&cp0_usb3_0_phy0>; - phy-names = "usb"; - /delete-property/ phys; -}; - -&cp0_usb3_1 { - status = "okay"; - usb-phy = <&cp0_usb3_0_phy1>; - phy-names = "usb"; - /delete-property/ phys; -}; -- 2.51.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: intel: Drop pxa2xx 2025-12-12 20:32 [PATCH] ARM: dts: intel: Drop pxa2xx Rob Herring (Arm) @ 2025-12-12 21:58 ` Robert Jarzmik 2025-12-17 16:04 ` Arnd Bergmann 0 siblings, 1 reply; 7+ messages in thread From: Robert Jarzmik @ 2025-12-12 21:58 UTC (permalink / raw) To: Rob Herring (Arm) Cc: Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski, Conor Dooley, Daniel Mack, Haojian Zhuang, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, linux-renesas-soc, devicetree, linux-kernel, linux-arm-kernel, Arnd Bergmann "Rob Herring (Arm)" <robh@kernel.org> writes: > These .dtsi files are not included anywhere in the tree and > can't be > tested. They have not been touched since 2018 other than > clean-ups. > And yet, there are used by people using pxa2xx board with an DT support (like the mioa701 for which a board file was never merged). If you remove pxa25x.dtsi and pxa27x.dtsi, you might as well remove all support for this architecture from the kernel, as these are the building blocks needed to make it work. That might be what should be done, I'll let Arnd and Daniel comment on the future of PXA in the kernel. Cheers. -- Robert -- Formal Signature Emacs 25, org-mode 9, mu4e 1.0 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: intel: Drop pxa2xx 2025-12-12 21:58 ` Robert Jarzmik @ 2025-12-17 16:04 ` Arnd Bergmann 2025-12-18 8:21 ` Daniel 2025-12-28 11:11 ` Robert Jarzmik 0 siblings, 2 replies; 7+ messages in thread From: Arnd Bergmann @ 2025-12-17 16:04 UTC (permalink / raw) To: Robert Jarzmik, Rob Herring Cc: Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski, Conor Dooley, Daniel Mack, Haojian Zhuang, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Linux-Renesas, devicetree, linux-kernel, linux-arm-kernel On Fri, Dec 12, 2025, at 22:58, Robert Jarzmik wrote: > "Rob Herring (Arm)" <robh@kernel.org> writes: > >> These .dtsi files are not included anywhere in the tree and >> can't be tested. They have not been touched since 2018 other than >> clean-ups. >> > And yet, there are used by people using pxa2xx board with an DT > support (like the mioa701 for which a board file was never merged). > > If you remove pxa25x.dtsi and pxa27x.dtsi, you might as well > remove all support for this architecture from the kernel, as > these are the building blocks needed to make it work. > > That might be what should be done, I'll let Arnd and Daniel > comment on the future of PXA in the kernel. I agree with Rob that at the very minimum, any dtsi files in the kernel should be build-tested along with the rest, so either we add some dts files using them soon, or we can remove pxa2xx (and maybe pxa3xx) completely. I already plan to remove the remaining board files for both sa1100 and pxa soon, as nobody has picked up the task to do any further DT conversion after we removed the majority of the files back in 2022 and left only the ones that someone said they'd be interested in keeping. Robert, let me know if you or someone else is able to spend some time on sending (warning-free) dts files for pxa2xx machines soon. If not, I'd plan to remove whatever is there along with the board files and drivers. I also checked the pxa3xx Raumfeld devicetree support that Daniel added in linux-3.7. The manufacturer's firmware download page contains kernel images based on this work, using a 4.19 kernel but built in 2022. If we remove pxa2xx DT support, it would be good to find out if anyone still plans to use something newer on that hardware, which may happen because of CRA requirements. Arnd ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: intel: Drop pxa2xx 2025-12-17 16:04 ` Arnd Bergmann @ 2025-12-18 8:21 ` Daniel 2025-12-22 22:04 ` Arnd Bergmann 2025-12-28 11:11 ` Robert Jarzmik 1 sibling, 1 reply; 7+ messages in thread From: Daniel @ 2025-12-18 8:21 UTC (permalink / raw) To: Arnd Bergmann, Robert Jarzmik, Rob Herring Cc: Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski, Conor Dooley, Haojian Zhuang, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Linux-Renesas, devicetree, linux-kernel, linux-arm-kernel Hi, On 12/17/25 17:04, Arnd Bergmann wrote: > On Fri, Dec 12, 2025, at 22:58, Robert Jarzmik wrote: >> "Rob Herring (Arm)" <robh@kernel.org> writes: >> >>> These .dtsi files are not included anywhere in the tree and >>> can't be tested. They have not been touched since 2018 other than >>> clean-ups. >>> >> And yet, there are used by people using pxa2xx board with an DT >> support (like the mioa701 for which a board file was never merged). >> >> If you remove pxa25x.dtsi and pxa27x.dtsi, you might as well >> remove all support for this architecture from the kernel, as >> these are the building blocks needed to make it work. >> >> That might be what should be done, I'll let Arnd and Daniel >> comment on the future of PXA in the kernel. > > I agree with Rob that at the very minimum, any dtsi files in the > kernel should be build-tested along with the rest, so either > we add some dts files using them soon, or we can remove pxa2xx > (and maybe pxa3xx) completely. PXA3xx is in use by the Raumfeld boards, and I would really like to keep them around. For PXA2xx I'm torn. I don't have the capacity and hardware to work on those, but I don't know how many users are out there with out-of-tree dts files. > I already plan to remove the remaining board files for both > sa1100 and pxa soon, as nobody has picked up the task to do > any further DT conversion after we removed the majority of > the files back in 2022 and left only the ones that someone > said they'd be interested in keeping. > > Robert, let me know if you or someone else is able to spend > some time on sending (warning-free) dts files for pxa2xx > machines soon. If not, I'd plan to remove whatever is there > along with the board files and drivers. > > I also checked the pxa3xx Raumfeld devicetree support that > Daniel added in linux-3.7. The manufacturer's firmware > download page contains kernel images based on this work, > using a 4.19 kernel but built in 2022. If we remove pxa2xx > DT support, it would be good to find out if anyone still > plans to use something newer on that hardware, which may > happen because of CRA requirements. As I said, I would like to not close the door on this one if possible. Thanks, Daniel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: intel: Drop pxa2xx 2025-12-18 8:21 ` Daniel @ 2025-12-22 22:04 ` Arnd Bergmann 0 siblings, 0 replies; 7+ messages in thread From: Arnd Bergmann @ 2025-12-22 22:04 UTC (permalink / raw) To: Daniel Mack, Robert Jarzmik, Rob Herring Cc: Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski, Conor Dooley, Haojian Zhuang, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Linux-Renesas, devicetree, linux-kernel, linux-arm-kernel On Thu, Dec 18, 2025, at 09:21, Daniel wrote: > On 12/17/25 17:04, Arnd Bergmann wrote: >> On Fri, Dec 12, 2025, at 22:58, Robert Jarzmik wrote: >>> "Rob Herring (Arm)" <robh@kernel.org> writes: >>> >>>> These .dtsi files are not included anywhere in the tree and >>>> can't be tested. They have not been touched since 2018 other than >>>> clean-ups. >>>> >>> And yet, there are used by people using pxa2xx board with an DT >>> support (like the mioa701 for which a board file was never merged). >>> >>> If you remove pxa25x.dtsi and pxa27x.dtsi, you might as well >>> remove all support for this architecture from the kernel, as >>> these are the building blocks needed to make it work. >>> >>> That might be what should be done, I'll let Arnd and Daniel >>> comment on the future of PXA in the kernel. >> >> I agree with Rob that at the very minimum, any dtsi files in the >> kernel should be build-tested along with the rest, so either >> we add some dts files using them soon, or we can remove pxa2xx >> (and maybe pxa3xx) completely. > > PXA3xx is in use by the Raumfeld boards, and I would really like to keep > them around. Ok, no problem. If there are upstream users, we can always keep that. > For PXA2xx I'm torn. I don't have the capacity and hardware to work on > those, but I don't know how many users are out there with out-of-tree > dts files. It's clear that nobody has done anything upstream for a while on pxa2xx, you both initially put a lot of work into it but it has been incomplete for almost a decade at this point. I wouldn't even expect you to do any runtime testing, but if we want to keep it in mainline, it seems the minimum state we should get to is to complete the upstreaming to the degree that one machine can plausibly boot with a mainline that gets built by default, and not introduce new warnings in the process. At the moment, an empty dts file that includes pxa25x.dtsi doesn't even build because there is no pwri2c node: arch/arm/boot/dts/intel/pxa/pxa2xx.dtsi:36.10-43.4: ERROR (path_references): /aliases: Reference to non-existent node or label "pwri2c" The pxa27x.dtsi file does build but produces a couple of 'dtbs_check' W=1 and W=2 warnings: arch/arm/boot/dts/intel/pxa/pxa27x.dtsi:134.30-138.5: Warning (node_name_chars_strict): /clocks/pxa2xx_clks@41300004: Character '_' not recommended in node name arch/arm/boot/dts/intel/pxa/pxa27x.dtsi:149.31-187.4: Warning (node_name_chars_strict): /opp_table0: Character '_' not recommended in node name arch/arm/boot/dts/intel/pxa/pxa27x.dtsi:134.30-138.5: Warning (unit_address_vs_reg): /clocks/pxa2xx_clks@41300004: node has a unit name, but no reg or ranges property arch/arm/boot/dts/intel/pxa/pxa2xx.dtsi:154.33-160.5: Warning (simple_bus_reg): /pxabus/lcd-controller@40500000: simple-bus unit address format error, expected "44000000" arch/arm/boot/dts/intel/pxa/pxa27x.dtsi:29.29-33.5: Warning (simple_bus_reg): /pxabus/pinctrl@40e00000: simple-bus unit address format error, expected "40e00054" arch/arm/boot/dts/intel/pxa/pxa2xx.dtsi:67.23-94.5: Warning (unique_unit_address): /pxabus/gpio@40e00000: duplicate unit-address (also used in node /pxabus/pinctrl@40e00000) pxa270-minimal.dtb: /: failed to match any schema with compatible: ['marvell,pxa27x'] pxa270-minimal.dtb: cpus: '#address-cells' is a required property from schema $id: http://devicetree.org/schemas/cpus.yaml# pxa270-minimal.dtb: cpus: '#size-cells' is a required property from schema $id: http://devicetree.org/schemas/cpus.yaml# pxa270-minimal.dtb: /cpus/cpu: failed to match any schema with compatible: ['marvell,xscale'] pxa270-minimal.dtb: pxabus (simple-bus): $nodename:0: 'pxabus' does not match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$' from schema $id: http://devicetree.org/schemas/simple-bus.yaml# pxa270-minimal.dtb: /pxabus/interrupt-controller@40d00000: failed to match any schema with compatible: ['marvell,pxa-intc'] pxa270-minimal.dtb: interrupt-controller@40d00000 (marvell,pxa-intc): interrupt-parent: True is not of type 'array' from schema $id: http://devicetree.org/schemas/interrupts.yaml# pxa270-minimal.dtb: /pxabus/mmc@41100000: failed to match any schema with compatible: ['marvell,pxa-mmc'] pxa270-minimal.dtb: rtc@40900000 (marvell,pxa-rtc): Unevaluated properties are not allowed ('clocks' was unexpected) from schema $id: http://devicetree.org/schemas/rtc/marvell,pxa-rtc.yaml# pxa270-minimal.dtb: /pxabus/lcd-controller@44000000: failed to match any schema with compatible: ['marvell,pxa2xx-lcdc'] pxa270-minimal.dtb: /pxabus/pinctrl@40e00000: failed to match any schema with compatible: ['marvell,pxa27x-pinctrl'] pxa270-minimal.dtb: /pxabus/usb@4c000000: failed to match any schema with compatible: ['marvell,pxa-ohci'] pxa270-minimal.dtb: pwm@40b00000 (marvell,pxa270-pwm): compatible: 'oneOf' conditional failed, one must be fixed: ['marvell,pxa270-pwm', 'marvell,pxa250-pwm'] is too long 'spacemit,k1-pwm' was expected 'marvell,pxa910-pwm' was expected from schema $id: http://devicetree.org/schemas/pwm/marvell,pxa-pwm.yaml# pxa270-minimal.dtb: pwm@40b00010 (marvell,pxa270-pwm): compatible: 'oneOf' conditional failed, one must be fixed: ['marvell,pxa270-pwm', 'marvell,pxa250-pwm'] is too long 'spacemit,k1-pwm' was expected 'marvell,pxa910-pwm' was expected from schema $id: http://devicetree.org/schemas/pwm/marvell,pxa-pwm.yaml# pxa270-minimal.dtb: pwm@40c00000 (marvell,pxa270-pwm): compatible: 'oneOf' conditional failed, one must be fixed: ['marvell,pxa270-pwm', 'marvell,pxa250-pwm'] is too long 'spacemit,k1-pwm' was expected 'marvell,pxa910-pwm' was expected from schema $id: http://devicetree.org/schemas/pwm/marvell,pxa-pwm.yaml# pxa270-minimal.dtb: pwm@40c00010 (marvell,pxa270-pwm): compatible: 'oneOf' conditional failed, one must be fixed: ['marvell,pxa270-pwm', 'marvell,pxa250-pwm'] is too long 'spacemit,k1-pwm' was expected 'marvell,pxa910-pwm' was expected from schema $id: http://devicetree.org/schemas/pwm/marvell,pxa-pwm.yaml# pxa270-minimal.dtb: /pxabus/udc@40600000: failed to match any schema with compatible: ['marvell,pxa270-udc'] pxa270-minimal.dtb: /pxabus/keypad@41500000: failed to match any schema with compatible: ['marvell,pxa27x-keypad'] pxa270-minimal.dtb: /pxabus/imaging@50000000: failed to match any schema with compatible: ['marvell,pxa270-qci'] pxa270-minimal.dtb: imaging@50000000 (marvell,pxa270-qci): '#clock-cells' is a dependency of 'clock-output-names' from schema $id: http://devicetree.org/schemas/clock/clock.yaml# pxa270-minimal.dtb: /clocks/pxa2xx_clks@41300004: failed to match any schema with compatible: ['marvell,pxa270-clocks'] pxa270-minimal.dtb: /timer@40a00000: failed to match any schema with compatible: ['marvell,pxa-timer'] pxa270-minimal.dtb: opp_table0 (operating-points-v2): $nodename:0: 'opp_table0' does not match '^opp-table(-[a-z0-9]+)?$' from schema $id: http://devicetree.org/schemas/opp/opp-v2.yaml# pxa270-minimal.dtb: opp_table0 (operating-points-v2): Unevaluated properties are not allowed ('opp-104000000', 'opp-156000000', 'opp-208000000', 'opp-312000000', 'opp-416000000', 'opp-520000000', 'opp-624000000' were unexpected) from schema $id: http://devicetree.org/schemas/opp/opp-v2.yaml# Most of these have an obvious fix and just need someone to care a little bit. The Raumfeld files have a similar set of warnings: arch/arm/boot/dts/intel/pxa/pxa300-raumfeld-common.dtsi:17.9-20.4: Warning (unit_address_vs_reg): /memory: node has a reg or ranges property, but no unit name arch/arm/boot/dts/intel/pxa/pxa300-raumfeld-controller.dts:242.29-248.4: Warning (node_name_chars_strict): /pxabus/pinctrl@40e10000/charger_pins: Character '_' not recommended in node name arch/arm/boot/dts/intel/pxa/pxa300-raumfeld-controller.dts:250.37-256.4: Warning (node_name_chars_strict): /pxabus/pinctrl@40e10000/dock_detect_pins: Character '_' not recommended in node name arch/arm/boot/dts/intel/pxa/pxa300-raumfeld-speaker-one.dts:122.3-25: Warning (property_name_chars_strict): /pxabus/i2c@40301680/codec@1a:st,needs_esd_watchdog: Character '_' not recommended in property name pxa300-raumfeld-speaker-l.dtb: /: failed to match any schema with compatible: ['raumfeld,raumfeld-speaker-l-pxa303', 'marvell,pxa300'] pxa300-raumfeld-speaker-l.dtb: /: failed to match any schema with compatible: ['raumfeld,raumfeld-speaker-l-pxa303', 'marvell,pxa300'] pxa300-raumfeld-speaker-l.dtb: / (raumfeld,raumfeld-speaker-l-pxa303): memory: False schema does not allow {'device_type': ['memory'], 'reg': [[2684354560, 134217728]]} from schema $id: http://devicetree.org/schemas/root-node.yaml# pxa300-raumfeld-speaker-l.dtb: cpus: '#address-cells' is a required property from schema $id: http://devicetree.org/schemas/cpus.yaml# pxa300-raumfeld-speaker-l.dtb: cpus: '#size-cells' is a required property from schema $id: http://devicetree.org/schemas/cpus.yaml# pxa300-raumfeld-speaker-l.dtb: /cpus/cpu: failed to match any schema with compatible: ['marvell,xscale'] pxa300-raumfeld-speaker-l.dtb: pxabus (simple-bus): $nodename:0: 'pxabus' does not match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$' from schema $id: http://devicetree.org/schemas/simple-bus.yaml# pxa300-raumfeld-speaker-l.dtb: /pxabus/interrupt-controller@40d00000: failed to match any schema with compatible: ['marvell,pxa-intc'] pxa300-raumfeld-speaker-l.dtb: interrupt-controller@40d00000 (marvell,pxa-intc): interrupt-parent: True is not of type 'array' from schema $id: http://devicetree.org/schemas/interrupts.yaml# pxa300-raumfeld-speaker-l.dtb: /pxabus/mmc@41100000: failed to match any schema with compatible: ['marvell,pxa-mmc'] pxa300-raumfeld-speaker-l.dtb: /pxabus/lcd-controller@44000000: failed to match any schema with compatible: ['marvell,pxa2xx-lcdc'] pxa300-raumfeld-speaker-l.dtb: /pxabus/i2c@40f500c0: failed to match any schema with compatible: ['mrvl,pwri2c'] pxa300-raumfeld-speaker-l.dtb: regulator@34 (maxim,max8660): $nodename:0: 'regulator@34' does not match 'pmic@[0-9a-f]{1,2}' from schema $id: http://devicetree.org/schemas/regulator/max8660.yaml# pxa300-raumfeld-speaker-l.dtb: regulator@34 (maxim,max8660): regulators:regulator-v3: Unevaluated properties are not allowed ('regulator-compatible' was unexpected) from schema $id: http://devicetree.org/schemas/regulator/max8660.yaml# pxa300-raumfeld-speaker-l.dtb: regulator@34 (maxim,max8660): regulators:regulator-v4: Unevaluated properties are not allowed ('regulator-compatible' was unexpected) from schema $id: http://devicetree.org/schemas/regulator/max8660.yaml# pxa300-raumfeld-speaker-l.dtb: regulator@34 (maxim,max8660): regulators:regulator-v5: Unevaluated properties are not allowed ('regulator-compatible' was unexpected) from schema $id: http://devicetree.org/schemas/regulator/max8660.yaml# pxa300-raumfeld-speaker-l.dtb: regulator@34 (maxim,max8660): regulators:regulator-v6: Unevaluated properties are not allowed ('regulator-compatible' was unexpected) from schema $id: http://devicetree.org/schemas/regulator/max8660.yaml# pxa300-raumfeld-speaker-l.dtb: regulator@34 (maxim,max8660): regulators:regulator-v7: Unevaluated properties are not allowed ('regulator-compatible' was unexpected) from schema $id: http://devicetree.org/schemas/regulator/max8660.yaml# pxa300-raumfeld-speaker-l.dtb: pinctrl@40e10000 (pinconf-single): 'led-pins-a', 'led-pins-b' do not match any of the regexes: '-pins(-[0-9]+)?$|-pin$', '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/pinctrl/pinctrl-single.yaml# pxa300-raumfeld-speaker-l.dtb: /pxabus/mmc@42000000: failed to match any schema with compatible: ['marvell,pxa-mmc'] pxa300-raumfeld-speaker-l.dtb: /pxabus/mmc@42500000: failed to match any schema with compatible: ['marvell,pxa-mmc'] pxa300-raumfeld-speaker-l.dtb: /pxabus/usb@4c000000: failed to match any schema with compatible: ['marvell,pxa-ohci'] pxa300-raumfeld-speaker-l.dtb: ssp@41000000 (mrvl,pxa3xx-ssp): $nodename:0: 'ssp@41000000' does not match '^spi(@.*|-([0-9]|[1-9][0-9]+))?$' from schema $id: http://devicetree.org/schemas/spi/marvell,mmp2-ssp.yaml# pxa300-raumfeld-speaker-l.dtb: ssp@41700000 (mrvl,pxa3xx-ssp): $nodename:0: 'ssp@41700000' does not match '^spi(@.*|-([0-9]|[1-9][0-9]+))?$' from schema $id: http://devicetree.org/schemas/spi/marvell,mmp2-ssp.yaml# pxa300-raumfeld-speaker-l.dtb: ssp@41900000 (mrvl,pxa3xx-ssp): $nodename:0: 'ssp@41900000' does not match '^spi(@.*|-([0-9]|[1-9][0-9]+))?$' from schema $id: http://devicetree.org/schemas/spi/marvell,mmp2-ssp.yaml# pxa300-raumfeld-speaker-l.dtb: ssp@41a00000 (mrvl,pxa3xx-ssp): $nodename:0: 'ssp@41a00000' does not match '^spi(@.*|-([0-9]|[1-9][0-9]+))?$' from schema $id: http://devicetree.org/schemas/spi/marvell,mmp2-ssp.yaml# pxa300-raumfeld-speaker-l.dtb: /pxabus/timer@40a00000: failed to match any schema with compatible: ['marvell,pxa-timer'] pxa300-raumfeld-speaker-l.dtb: /pxabus/display-controller@54000000: failed to match any schema with compatible: ['marvell,pxa300-gcu'] pxa300-raumfeld-speaker-l.dtb: /clocks/clocks: failed to match any schema with compatible: ['marvell,pxa300-clocks'] pxa300-raumfeld-speaker-l.dtb: /ssp-dai0: failed to match any schema with compatible: ['mrvl,pxa-ssp-dai'] pxa300-raumfeld-speaker-l.dtb: /ssp-dai1: failed to match any schema with compatible: ['mrvl,pxa-ssp-dai'] pxa300-raumfeld-speaker-l.dtb: spi (spi-gpio): gpio-miso: False schema does not allow [8, 98, 0] from schema $id: http://devicetree.org/schemas/spi/spi-gpio.yaml# pxa300-raumfeld-speaker-l.dtb: spi (spi-gpio): gpio-mosi: False schema does not allow [8, 97, 0] from schema $id: http://devicetree.org/schemas/spi/spi-gpio.yaml# pxa300-raumfeld-speaker-l.dtb: spi (spi-gpio): gpio-sck: False schema does not allow [8, 95, 0] from schema $id: http://devicetree.org/schemas/spi/spi-gpio.yaml# pxa300-raumfeld-speaker-l.dtb: spi (spi-gpio): 'sck-gpios' is a required property from schema $id: http://devicetree.org/schemas/spi/spi-gpio.yaml# pxa300-raumfeld-speaker-l.dtb: spi (spi-gpio): Unevaluated properties are not allowed ('gpio-miso', 'gpio-mosi', 'gpio-sck' were unexpected) from schema $id: http://devicetree.org/schemas/spi/spi-gpio.yaml# pxa300-raumfeld-speaker-l.dtb: dac@2 (ti,dac7512): 'vcc-supply' does not match any of the regexes: '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/iio/dac/ti,dac7512.yaml# pxa300-raumfeld-speaker-l.dtb: gpio-keys (gpio-keys): 'on-off', 'rescue-boot', 'setup' do not match any of the regexes: '^(button|event|key|switch|(button|event|key|switch)-[a-z0-9-]+|[a-z0-9-]+-(button|event|key|switch))$', '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/input/gpio-keys.yaml# pxa300-raumfeld-speaker-l.dtb: leds (gpio-leds): 'left', 'right' do not match any of the regexes: '(^led-[0-9a-f]$|led)', '^pinctrl-[0-9]+$' from schema $id: http://devicetree.org/schemas/leds/leds-gpio.yaml#:%! It would be good to at least address the warnings for nodes that have an existing yaml binding. Converting the remaining others or writing new ones takes more time and I wouldn't stress out about those (yet). Arnd ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: intel: Drop pxa2xx 2025-12-17 16:04 ` Arnd Bergmann 2025-12-18 8:21 ` Daniel @ 2025-12-28 11:11 ` Robert Jarzmik 2025-12-31 0:02 ` Arnd Bergmann 1 sibling, 1 reply; 7+ messages in thread From: Robert Jarzmik @ 2025-12-28 11:11 UTC (permalink / raw) To: Arnd Bergmann Cc: Rob Herring, Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski, Conor Dooley, Daniel Mack, Haojian Zhuang, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Linux-Renesas, devicetree, linux-kernel, linux-arm-kernel "Arnd Bergmann" <arnd@arndb.de> writes: > Robert, let me know if you or someone else is able to spend > some time on sending (warning-free) dts files for pxa2xx > machines soon. If not, I'd plan to remove whatever is there > along with the board files and drivers. Here is one attached, the previously mentioned mioa701 board dts file I've been using for years (the date of the patch should be funny enough). It was first submitted here : https://lkml.org/lkml/2018/9/15/321 I'm not very sure pxa25x and pxa27x should survive though. Lately, the platform which I have left have 64MB of RAM, and cannot cope with recent binaries sizes. And to my best knowledge, pxa2xx architecture are not built anymore nor supported ... The drivers might still be useful (the DMA, the SPI for intel CPU, ...). Cheers. -- Robert -- >8 -- From a3cbbe846c4651d71edcf36d114f5e48f4455347 Mon Sep 17 00:00:00 2001 From: Robert Jarzmik <robert.jarzmik@free.fr> Date: Mon, 1 Sep 2014 13:26:56 +0200 Subject: [PATCH] arm: dts: add mioa701 board description Add device-tree description of the Mitac MIO A701 board. Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr> --- arch/arm/boot/dts/intel/pxa/Makefile | 1 + arch/arm/boot/dts/intel/pxa/mioa701.dts | 309 ++++++++++++++++++++++++ 2 files changed, 310 insertions(+) create mode 100644 arch/arm/boot/dts/intel/pxa/mioa701.dts diff --git a/arch/arm/boot/dts/intel/pxa/Makefile b/arch/arm/boot/dts/intel/pxa/Makefile index 24d5240f08e7..29670ec5861b 100644 --- a/arch/arm/boot/dts/intel/pxa/Makefile +++ b/arch/arm/boot/dts/intel/pxa/Makefile @@ -6,3 +6,4 @@ dtb-$(CONFIG_ARCH_PXA) += \ pxa300-raumfeld-speaker-m.dtb \ pxa300-raumfeld-speaker-one.dtb \ pxa300-raumfeld-speaker-s.dtb +dtb-$(CONFIG_ARCH_PXA) += mioa701.dtb diff --git a/arch/arm/boot/dts/intel/pxa/mioa701.dts b/arch/arm/boot/dts/intel/pxa/mioa701.dts new file mode 100644 index 000000000000..b8a27e6db2b2 --- /dev/null +++ b/arch/arm/boot/dts/intel/pxa/mioa701.dts @@ -0,0 +1,309 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * Copyright (C) Robert Jarzmik <robert.jarzmik@free.fr> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * publishhed by the Free Software Foundation. + */ + +/dts-v1/; +#include "pxa27x.dtsi" +#include "dt-bindings/gpio/gpio.h" + +#define PMGROUP(pin) #pin +#define PMMUX(func, pin, af) \ + mux- ## func { \ + groups = PMGROUP(P ## pin); \ + function = #af; \ + } +#define PMMUX_LPM_LOW(func, pin, af) \ + mux- ## func { \ + groups = PMGROUP(P ## pin); \ + function = #af; \ + low-power-disable; \ + } +#define PMMUX_LPM_HIGH(func, pin, af) \ + mux- ## func { \ + groups = PMGROUP(P ## pin); \ + function = #af; \ + low-power-enable; \ + } + +/ { + model = "Mitac Mio A701 Board"; + /* compatible = "mitac,mioa701"; */ + compatible = "marvell,pxa270"; + + chosen { + bootargs = "mtdparts=docg3.0:256k@3456k(barebox)ro,256k(barebox-logo),128k(barebox-env),4M(kernel),-(root) ubi.mtd=4 rootfstype=ubifs root=ubi0:linux_root ro"; + }; + + memory { + reg = <0xa0000000 0x04000000>; + + reserved-memory { + #address-cells = <1>; + #size-cells = <1>; + + pstore_region:region@a2000000 { + compatible = "linux,contiguous-memory-region"; + reg = <0xa2000000 1048576>; + }; + }; + }; + + cpus { + cpu { + cpu-supply = <&vcc_core>; + }; + }; + + pxabus { + pinctrl: pinctrl@40e00000 { + status = "okay"; + pinctrl_mmc_default: mmc-default { + PMMUX(sd-insert, 15, gpio_in); + PMMUX(mmclk, 32, MMCLK); + PMMUX(sd-ro, 78, gpio_in); + PMMUX(sd-enable, 91, gpio_out); + PMMUX(mmdat0, 92, MMDAT<0>); + PMMUX(mmdat1, 109, MMDAT<1>); + PMMUX(mmdat2, 110, MMDAT<2>); + PMMUX(mmdat3, 111, MMDAT<3>); + PMMUX(mmcmd, 112, MMCMD); + }; + pinctrl_leds_default: leds-default { + PMMUX(led-charging, 10, gpio_out); + PMMUX(led-vibra, 82, gpio_out); + PMMUX(led-blue, 97, gpio_out); + PMMUX_LPM_LOW(led-orange, 98, gpio_out); + PMMUX_LPM_HIGH(led-keyboard, 115, gpio_out); + }; + }; + + gpio: gpio@40e00000 { + status = "okay"; + }; + + uart@40100000 { + status = "okay"; + }; + + uart@40200000 { + status = "okay"; + }; + + uart@40700000 { + status = "okay"; + }; + + usb2phy: gpio-vbus@13 { + compatible = "usb-nop-xceiv"; + vbus-detect-gpio = <&gpio 13 GPIO_ACTIVE_LOW>; + #phy-cells = <0>; + wakeup; + }; + + pxa27x_udc: udc@40600000 { + status = "okay"; + gpios = <&gpio 22 0>; + phys = <&usb2phy>; + phys-names = "usb2phy"; + }; + + i2c@40f00180 { + status = "okay"; + + max1586@14 { + compatible = "maxim,max1586"; + reg = <0x14>; + #address-cells = <0x1>; + #size-cells = <0x1>; + v3-gain = <1000000>; + + regulators { + vcc_core: v3 { + regulator-name = "vcc_core"; + regulator-compatible = "Output_V3"; + regulator-min-microvolt = <1000000>; + regulator-max-microvolt = <1705000>; + regulator-always-on; + }; + }; + }; + }; + + pxai2c1: i2c@40301680 { + mrvl,i2c-fast-mode; + status = "okay"; + + mt9m111: camera@5d { + compatible = "micron,mt9m111"; + reg = <0x5d>; + gpios = <&gpio 56 GPIO_ACTIVE_HIGH>; + }; + }; + + keypad: keypad@41500000 { + status = "okay"; + + keypad,num-rows = <3>; + keypad,num-columns = <3>; + linux,keymap = < + 0x00000067 /* KEY_UP */ + 0x0001006a /* KEY_RIGHT */ + 0x000200e2 /* KEY_MEDIA */ + 0x0100006c /* KEY_DOWN */ + 0x0101001c /* KEY_ENTER */ + 0x010200da /* KEY_CONNECT */ + 0x02000069 /* KEY_LEFT */ + 0x020100a9 /* KEY_PHONE */ + 0x020200d4>; /* KEY_CAMERA */ + marvell,debounce-interval = <0>; + }; + + gpio-keys { + compatible = "gpio-keys"; + #address-cells = <1>; + #size-cells = <0>; + autorepeat; + status = "okay"; + + button@0 { + label = "GPIO Key Power"; + linux,code = <174>; + gpios = <&gpio 0 0>; + gpio-key,wakeup; + }; + button@12 { + label = "HP jack detect"; + linux,code = <211>; + gpios = <&gpio 12 0>; + }; + button@93 { + label = "Volume Up Key"; + linux,code = <115>; + gpios = <&gpio 93 0>; + }; + button@94 { + label = "Volume Down Key"; + linux,code = <114>; + gpios = <&gpio 94 0>; + }; + }; + + mmc0: mmc@41100000 { + vmmc-supply = <®_vmmc>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_mmc_default>; + status = "okay"; + }; + + pxa_camera: imaging@50000000 { + status = "okay"; + }; + + lcd-controller@40500000 { + status = "okay"; + port { + lcdc_out: endpoint { + remote-endpoint = <&panel_in>; + bus-width = <16>; + }; + }; + }; + }; + + regulators { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <0>; + + reg_vmmc: regulator@0 { + compatible = "regulator-fixed"; + regulator-name = "vmmc"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-always-on; + }; + }; + + backlight { + compatible = "pwm-backlight"; + pwms = <&pwm0 40960000>; + pwm-names = "backlight"; + + brightness-levels = <0 4 8 16 32 64 128 255>; + default-brightness-level = <2>; + }; + + docg3: flash@0 { + compatible = "m-systems,diskonchip-g3"; + reg = <0x0 0x2000>; + }; + + panel { + compatible = "toshiba,ltm0305a776"; + lcd-type = "color-tft"; + + port { + panel_in: endpoint { + remote-endpoint = <&lcdc_out>; + }; + }; + + display-timings { + native-mode = <&timing0>; + timing0: 240p { + /* 240x320p24 */ + clock-frequency = <4545000>; + hactive = <240>; + vactive = <320>; + hfront-porch = <4>; + hback-porch = <6>; + hsync-len = <4>; + vback-porch = <5>; + vfront-porch = <3>; + vsync-len = <2>; + }; + }; + }; + + leds { + compatible = "gpio-leds"; + + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_leds_default>; + + charger-led { + label = "mioa701:charging"; + gpios = <&gpio 10 GPIO_ACTIVE_LOW>; + default-state = "off"; + }; + + vibrator { + label = "mioa701:vibra"; + gpios = <&gpio 82 GPIO_ACTIVE_LOW>; + default-state = "off"; + }; + + bluetooth-led { + label = "mioa701:blue"; + gpios = <&gpio 97 GPIO_ACTIVE_LOW>; + default-state = "off"; + }; + + orange-led { + label = "mioa701:orange"; + gpios = <&gpio 98 GPIO_ACTIVE_LOW>; + default-state = "off"; + }; + + keyboard-led { + label = "mioa701:keyboard"; + gpios = <&gpio 115 GPIO_ACTIVE_LOW>; + default-state = "off"; + }; + }; +}; -- 2.50.1 (Apple Git-155) ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ARM: dts: intel: Drop pxa2xx 2025-12-28 11:11 ` Robert Jarzmik @ 2025-12-31 0:02 ` Arnd Bergmann 0 siblings, 0 replies; 7+ messages in thread From: Arnd Bergmann @ 2025-12-31 0:02 UTC (permalink / raw) To: Robert Jarzmik Cc: Rob Herring, Geert Uytterhoeven, Magnus Damm, Krzysztof Kozlowski, Conor Dooley, Daniel Mack, Haojian Zhuang, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Linux-Renesas, devicetree, linux-kernel, linux-arm-kernel On Sun, Dec 28, 2025, at 12:11, Robert Jarzmik wrote: > >> Robert, let me know if you or someone else is able to spend >> some time on sending (warning-free) dts files for pxa2xx >> machines soon. If not, I'd plan to remove whatever is there >> along with the board files and drivers. > > Here is one attached, the previously mentioned mioa701 board dts > file > I've been using for years (the date of the patch should be funny > enough). It was first submitted here : > https://lkml.org/lkml/2018/9/15/321 > > I'm not very sure pxa25x and pxa27x should survive though. Lately, > the platform which I have left have 64MB of RAM, and cannot cope with > recent binaries sizes. And to my best knowledge, pxa2xx architecture are > not built anymore nor supported ... Thanks for posting this one, that is helpful both for reference and in case anyone wants to keep using or bringing back pxa27x after we remove it. My feeling is that if neither you nor Daniel are planning on actually using PXA2xx on future kernels, we're better off removing both PXA25x and PXA27x along with the board files and only keep PXA3xx around for Raumfeld. Looking at when PXA2xx was used, I see a lot of handhelds from 2002 to 2007 but not much later, as smartphones then started moved to newer Marvell chips, or TI and Qualcomm alternatives. The last embedded PXA2xx boards were in 2010 (Colibri/Income, Voipac vpac270, Arcom Zeus), and were all discontinued by their manufacturers years ago, usually 10 years after the hardware introduction, and nobody complained when I removed the board files in 2022. I had left the gumstix boards in the source tree at the time because it had more memory than the others and was supported by qemu. The upstream qemu releases now removed that as well. If you or anyone else is still using PXA27x, we can always bring it back later, but I'd then put the burden on them to submit the dts file for inclusion without introducing DT warnings. > The drivers might still be useful (the DMA, the SPI for intel CPU, > ...). Right, most drivers have to stay as they are still used on newer chips. The ones that I think would become obsolete without pxa2xx are pinctrl, cpufreq, camera, mtd, udc and pcmcia. Especially the latter two have come up a few times during cleanups where they cause extra work from complexity that probably nobody is using any more. > + docg3: flash@0 { > + compatible = "m-systems,diskonchip-g3"; > + reg = <0x0 0x2000>; > + }; This currently has no upstream users I can see, so removing pxa27x would probably mean we should remove that as well. > + panel { > + compatible = "toshiba,ltm0305a776"; > + lcd-type = "color-tft"; I don't see a driver that matches the compatible string, does this one actually work? Arnd ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2025-12-31 0:03 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-12-12 20:32 [PATCH] ARM: dts: intel: Drop pxa2xx Rob Herring (Arm) 2025-12-12 21:58 ` Robert Jarzmik 2025-12-17 16:04 ` Arnd Bergmann 2025-12-18 8:21 ` Daniel 2025-12-22 22:04 ` Arnd Bergmann 2025-12-28 11:11 ` Robert Jarzmik 2025-12-31 0:02 ` Arnd Bergmann
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).