linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 0/3] STi device-tree cleanup
@ 2025-07-14 13:49 Raphael Gallais-Pou
  2025-07-14 13:49 ` [PATCH v2 1/3] ARM: sti: drop B2120 board support Raphael Gallais-Pou
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Raphael Gallais-Pou @ 2025-07-14 13:49 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Patrice Chotard,
	Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel, Alain Volmat,
	Krzysztof Kozlowski

This serie aims to cleanup and rework the STi device-tree.

STiH415/STIh416 SoCs are not supported anymore and B2120 boards are
internal boards.  Update the documentation and drop the related DT
files.

Patch [2/3] has already been sent by Alain several months ago, but got
no response [1].

[1] https://lore.kernel.org/all/20230416185939.18497-1-avolmat@me.com

Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
---
Changes in v2:
- Fix typo in cover letter s/[4/6]/[2/3]
- Reorder patchset to be bisectable as Krzysztof mentioned
- Add Krzysztof's Acked-by where relevant
- Link to v1: https://lore.kernel.org/r/20250713-sti-rework-v1-0-6716a09e538d@gmail.com

---
Alain Volmat (1):
      ARM: sti: removal of stih415/stih416 related entries

Raphael Gallais-Pou (2):
      ARM: sti: drop B2120 board support
      dt-bindings: arm: sti: drop B2120 board support

 Documentation/devicetree/bindings/arm/sti.yaml |   4 -
 arch/arm/boot/dts/st/Makefile                  |   2 -
 arch/arm/boot/dts/st/stih407-b2120.dts         |  27 ----
 arch/arm/boot/dts/st/stih407.dtsi              | 145 -----------------
 arch/arm/boot/dts/st/stih410-b2120.dts         |  66 --------
 arch/arm/boot/dts/st/stihxxx-b2120.dtsi        | 206 -------------------------
 arch/arm/mach-sti/Kconfig                      |  20 +--
 arch/arm/mach-sti/board-dt.c                   |   2 -
 8 files changed, 1 insertion(+), 471 deletions(-)
---
base-commit: a6cfa4c8833944f8912c1fa7f95795753f6376ea
change-id: 20250401-sti-rework-b009551a362c

Best regards,
-- 
Raphael Gallais-Pou <rgallaispou@gmail.com>



^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v2 1/3] ARM: sti: drop B2120 board support
  2025-07-14 13:49 [PATCH v2 0/3] STi device-tree cleanup Raphael Gallais-Pou
@ 2025-07-14 13:49 ` Raphael Gallais-Pou
  2025-07-28  9:25   ` Patrice CHOTARD
  2025-07-14 13:49 ` [PATCH v2 2/3] ARM: sti: removal of stih415/stih416 related entries Raphael Gallais-Pou
  2025-07-14 13:49 ` [PATCH v2 3/3] dt-bindings: arm: sti: drop B2120 board support Raphael Gallais-Pou
  2 siblings, 1 reply; 10+ messages in thread
From: Raphael Gallais-Pou @ 2025-07-14 13:49 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Patrice Chotard,
	Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel

B2120 boards are internal boards which never were commercialised.

Drop them.

Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
---
 arch/arm/boot/dts/st/Makefile           |   2 -
 arch/arm/boot/dts/st/stih407-b2120.dts  |  27 -----
 arch/arm/boot/dts/st/stih407.dtsi       | 145 ----------------------
 arch/arm/boot/dts/st/stih410-b2120.dts  |  66 ----------
 arch/arm/boot/dts/st/stihxxx-b2120.dtsi | 206 --------------------------------
 5 files changed, 446 deletions(-)

diff --git a/arch/arm/boot/dts/st/Makefile b/arch/arm/boot/dts/st/Makefile
index cc9948b9870f7f73629573149bfd342af75b07da..f57fc6dc48a00c9a9323b4508e5e4e161b197c47 100644
--- a/arch/arm/boot/dts/st/Makefile
+++ b/arch/arm/boot/dts/st/Makefile
@@ -13,8 +13,6 @@ dtb-$(CONFIG_ARCH_SPEAR3XX) += \
 dtb-$(CONFIG_ARCH_SPEAR6XX) += \
 	spear600-evb.dtb
 dtb-$(CONFIG_ARCH_STI) += \
-	stih407-b2120.dtb \
-	stih410-b2120.dtb \
 	stih410-b2260.dtb \
 	stih418-b2199.dtb \
 	stih418-b2264.dtb
diff --git a/arch/arm/boot/dts/st/stih407-b2120.dts b/arch/arm/boot/dts/st/stih407-b2120.dts
deleted file mode 100644
index 9c79982ee7ba8fadb1a2a92e732bf7f652b74c38..0000000000000000000000000000000000000000
--- a/arch/arm/boot/dts/st/stih407-b2120.dts
+++ /dev/null
@@ -1,27 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-only
-/*
- * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
- * Author: Giuseppe Cavallaro <peppe.cavallaro@st.com>
- */
-/dts-v1/;
-#include "stih407.dtsi"
-#include "stihxxx-b2120.dtsi"
-/ {
-	model = "STiH407 B2120";
-	compatible = "st,stih407-b2120", "st,stih407";
-
-	chosen {
-		stdout-path = &sbc_serial0;
-	};
-
-	memory@40000000 {
-		device_type = "memory";
-		reg = <0x40000000 0x80000000>;
-	};
-
-	aliases {
-		serial0 = &sbc_serial0;
-		ethernet0 = &ethernet0;
-	};
-
-};
diff --git a/arch/arm/boot/dts/st/stih407.dtsi b/arch/arm/boot/dts/st/stih407.dtsi
deleted file mode 100644
index aca43d2bdaad44ef2a0e8a120c679c217709af44..0000000000000000000000000000000000000000
--- a/arch/arm/boot/dts/st/stih407.dtsi
+++ /dev/null
@@ -1,145 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-only
-/*
- * Copyright (C) 2015 STMicroelectronics Limited.
- * Author: Gabriel Fernandez <gabriel.fernandez@linaro.org>
- */
-#include "stih407-clock.dtsi"
-#include "stih407-family.dtsi"
-#include <dt-bindings/gpio/gpio.h>
-/ {
-	soc {
-		sti-display-subsystem@0 {
-			compatible = "st,sti-display-subsystem";
-			#address-cells = <1>;
-			#size-cells = <1>;
-			reg = <0 0>;
-			assigned-clocks = <&clk_s_d2_quadfs 0>,
-					  <&clk_s_d2_quadfs 1>,
-					  <&clk_s_c0_pll1 0>,
-					  <&clk_s_c0_flexgen CLK_COMPO_DVP>,
-					  <&clk_s_c0_flexgen CLK_MAIN_DISP>,
-					  <&clk_s_d2_flexgen CLK_PIX_MAIN_DISP>,
-					  <&clk_s_d2_flexgen CLK_PIX_AUX_DISP>,
-					  <&clk_s_d2_flexgen CLK_PIX_GDP1>,
-					  <&clk_s_d2_flexgen CLK_PIX_GDP2>,
-					  <&clk_s_d2_flexgen CLK_PIX_GDP3>,
-					  <&clk_s_d2_flexgen CLK_PIX_GDP4>;
-
-			assigned-clock-parents = <0>,
-						 <0>,
-						 <0>,
-						 <&clk_s_c0_pll1 0>,
-						 <&clk_s_c0_pll1 0>,
-						 <&clk_s_d2_quadfs 0>,
-						 <&clk_s_d2_quadfs 1>,
-						 <&clk_s_d2_quadfs 0>,
-						 <&clk_s_d2_quadfs 0>,
-						 <&clk_s_d2_quadfs 0>,
-						 <&clk_s_d2_quadfs 0>;
-
-			assigned-clock-rates = <297000000>,
-					       <108000000>,
-					       <0>,
-					       <400000000>,
-					       <400000000>;
-
-			ranges;
-
-			sti-compositor@9d11000 {
-				compatible = "st,stih407-compositor";
-				reg = <0x9d11000 0x1000>;
-
-				clock-names = "compo_main",
-					      "compo_aux",
-					      "pix_main",
-					      "pix_aux",
-					      "pix_gdp1",
-					      "pix_gdp2",
-					      "pix_gdp3",
-					      "pix_gdp4",
-					      "main_parent",
-					      "aux_parent";
-
-				clocks = <&clk_s_c0_flexgen CLK_COMPO_DVP>,
-					 <&clk_s_c0_flexgen CLK_COMPO_DVP>,
-					 <&clk_s_d2_flexgen CLK_PIX_MAIN_DISP>,
-					 <&clk_s_d2_flexgen CLK_PIX_AUX_DISP>,
-					 <&clk_s_d2_flexgen CLK_PIX_GDP1>,
-					 <&clk_s_d2_flexgen CLK_PIX_GDP2>,
-					 <&clk_s_d2_flexgen CLK_PIX_GDP3>,
-					 <&clk_s_d2_flexgen CLK_PIX_GDP4>,
-					 <&clk_s_d2_quadfs 0>,
-					 <&clk_s_d2_quadfs 1>;
-
-				reset-names = "compo-main", "compo-aux";
-				resets = <&softreset STIH407_COMPO_SOFTRESET>,
-					 <&softreset STIH407_COMPO_SOFTRESET>;
-				st,vtg = <&vtg_main>, <&vtg_aux>;
-			};
-
-			sti-tvout@8d08000 {
-				compatible = "st,stih407-tvout";
-				reg = <0x8d08000 0x1000>;
-				reg-names = "tvout-reg";
-				reset-names = "tvout";
-				resets = <&softreset STIH407_HDTVOUT_SOFTRESET>;
-				#address-cells = <1>;
-				#size-cells = <1>;
-				assigned-clocks = <&clk_s_d2_flexgen CLK_PIX_HDMI>,
-						  <&clk_s_d2_flexgen CLK_TMDS_HDMI>,
-						  <&clk_s_d2_flexgen CLK_REF_HDMIPHY>,
-						  <&clk_s_d0_flexgen CLK_PCM_0>,
-						  <&clk_s_d2_flexgen CLK_PIX_HDDAC>,
-						  <&clk_s_d2_flexgen CLK_HDDAC>;
-
-				assigned-clock-parents = <&clk_s_d2_quadfs 0>,
-							 <&clk_tmdsout_hdmi>,
-							 <&clk_s_d2_quadfs 0>,
-							 <&clk_s_d0_quadfs 0>,
-							 <&clk_s_d2_quadfs 0>,
-							 <&clk_s_d2_quadfs 0>;
-			};
-
-			sti_hdmi: sti-hdmi@8d04000 {
-				compatible = "st,stih407-hdmi";
-				reg = <0x8d04000 0x1000>;
-				reg-names = "hdmi-reg";
-				#sound-dai-cells = <0>;
-				interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>;
-				interrupt-names = "irq";
-				clock-names = "pix",
-					      "tmds",
-					      "phy",
-					      "audio",
-					      "main_parent",
-					      "aux_parent";
-
-				clocks = <&clk_s_d2_flexgen CLK_PIX_HDMI>,
-					 <&clk_s_d2_flexgen CLK_TMDS_HDMI>,
-					 <&clk_s_d2_flexgen CLK_REF_HDMIPHY>,
-					 <&clk_s_d0_flexgen CLK_PCM_0>,
-					 <&clk_s_d2_quadfs 0>,
-					 <&clk_s_d2_quadfs 1>;
-
-				hdmi,hpd-gpio = <&pio5 3 GPIO_ACTIVE_LOW>;
-				reset-names = "hdmi";
-				resets = <&softreset STIH407_HDMI_TX_PHY_SOFTRESET>;
-				ddc = <&hdmiddc>;
-			};
-
-			sti-hda@8d02000 {
-				compatible = "st,stih407-hda";
-				reg = <0x8d02000 0x400>, <0x92b0120 0x4>;
-				reg-names = "hda-reg", "video-dacs-ctrl";
-				clock-names = "pix",
-					      "hddac",
-					      "main_parent",
-					      "aux_parent";
-				clocks = <&clk_s_d2_flexgen CLK_PIX_HDDAC>,
-					 <&clk_s_d2_flexgen CLK_HDDAC>,
-					 <&clk_s_d2_quadfs 0>,
-					 <&clk_s_d2_quadfs 1>;
-			};
-		};
-	};
-};
diff --git a/arch/arm/boot/dts/st/stih410-b2120.dts b/arch/arm/boot/dts/st/stih410-b2120.dts
deleted file mode 100644
index 538ff98ca1b1b6542f98a18f68968f030b8c55c0..0000000000000000000000000000000000000000
--- a/arch/arm/boot/dts/st/stih410-b2120.dts
+++ /dev/null
@@ -1,66 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-only
-/*
- * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
- * Author: Peter Griffin <peter.griffin@linaro.org>
- */
-/dts-v1/;
-#include "stih410.dtsi"
-#include "stihxxx-b2120.dtsi"
-/ {
-	model = "STiH410 B2120";
-	compatible = "st,stih410-b2120", "st,stih410";
-
-	chosen {
-		stdout-path = &sbc_serial0;
-	};
-
-	memory@40000000 {
-		device_type = "memory";
-		reg = <0x40000000 0x80000000>;
-	};
-
-	aliases {
-		serial0 = &sbc_serial0;
-		ethernet0 = &ethernet0;
-	};
-
-	usb2_picophy1: phy2 {
-		status = "okay";
-	};
-
-	usb2_picophy2: phy3 {
-		status = "okay";
-	};
-
-	soc {
-
-		mmc0: sdhci@9060000 {
-			max-frequency = <200000000>;
-			sd-uhs-sdr50;
-			sd-uhs-sdr104;
-			sd-uhs-ddr50;
-		};
-
-		ohci0: usb@9a03c00 {
-			status = "okay";
-		};
-
-		ehci0: usb@9a03e00 {
-			status = "okay";
-		};
-
-		ohci1: usb@9a83c00 {
-			status = "okay";
-		};
-
-		ehci1: usb@9a83e00 {
-			status = "okay";
-		};
-
-		sti-display-subsystem@0 {
-			sti-hda@8d02000 {
-				status = "okay";
-			};
-		};
-	};
-};
diff --git a/arch/arm/boot/dts/st/stihxxx-b2120.dtsi b/arch/arm/boot/dts/st/stihxxx-b2120.dtsi
deleted file mode 100644
index 8d9a2dfa76f1fe55b954803cab8d00003a8ebec0..0000000000000000000000000000000000000000
--- a/arch/arm/boot/dts/st/stihxxx-b2120.dtsi
+++ /dev/null
@@ -1,206 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-only
-/*
- * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
- * Author: Giuseppe Cavallaro <peppe.cavallaro@st.com>
- */
-#include <dt-bindings/clock/stih407-clks.h>
-#include <dt-bindings/gpio/gpio.h>
-#include <dt-bindings/media/c8sectpfe.h>
-/ {
-	leds {
-		compatible = "gpio-leds";
-		led-red {
-			label = "Front Panel LED";
-			gpios = <&pio4 1 GPIO_ACTIVE_HIGH>;
-			linux,default-trigger = "heartbeat";
-		};
-		led-green {
-			gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
-			default-state = "off";
-		};
-	};
-
-	sound: sound {
-		compatible = "simple-audio-card";
-		simple-audio-card,name = "STI-B2120";
-		status = "okay";
-		#address-cells = <1>;
-		#size-cells = <0>;
-
-		simple-audio-card,dai-link@0 {
-			reg = <0>;
-			/* HDMI */
-			format = "i2s";
-			mclk-fs = <128>;
-			cpu {
-				sound-dai = <&sti_uni_player0>;
-			};
-
-			codec {
-				sound-dai = <&sti_hdmi>;
-			};
-		};
-
-		simple-audio-card,dai-link@1 {
-			reg = <1>;
-			/* DAC */
-			format = "i2s";
-			mclk-fs = <256>;
-			frame-inversion;
-			cpu {
-				sound-dai = <&sti_uni_player2>;
-			};
-
-			codec {
-				sound-dai = <&sti_sasg_codec 1>;
-			};
-		};
-
-		simple-audio-card,dai-link@2 {
-			reg = <2>;
-			/* SPDIF */
-			format = "left_j";
-			mclk-fs = <128>;
-			cpu {
-				sound-dai = <&sti_uni_player3>;
-			};
-
-			codec {
-				sound-dai = <&sti_sasg_codec 0>;
-			};
-		};
-	};
-
-	miphy28lp_phy: miphy28lp {
-
-		phy_port0: port@9b22000 {
-			st,osc-rdy;
-		};
-
-		phy_port1: port@9b2a000 {
-			st,osc-force-ext;
-		};
-	};
-
-	soc {
-		sbc_serial0: serial@9530000 {
-			status = "okay";
-		};
-
-		pwm0: pwm@9810000 {
-			status = "okay";
-		};
-
-		pwm1: pwm@9510000 {
-			status = "okay";
-		};
-
-		ssc2: i2c@9842000 {
-			status = "okay";
-			clock-frequency = <100000>;
-			st,i2c-min-scl-pulse-width-us = <0>;
-			st,i2c-min-sda-pulse-width-us = <5>;
-		};
-
-		ssc3: i2c@9843000 {
-			status = "okay";
-			clock-frequency = <100000>;
-			st,i2c-min-scl-pulse-width-us = <0>;
-			st,i2c-min-sda-pulse-width-us = <5>;
-		};
-
-		i2c@9844000 {
-			status = "okay";
-		};
-
-		i2c@9845000 {
-			status = "okay";
-		};
-
-		i2c@9540000 {
-			status = "okay";
-		};
-
-		mmc0: sdhci@9060000 {
-			non-removable;
-			status = "okay";
-		};
-
-		mmc1: sdhci@9080000 {
-			status = "okay";
-		};
-
-		/* SSC11 to HDMI */
-		hdmiddc: i2c@9541000 {
-			status = "okay";
-			/* HDMI V1.3a supports Standard mode only */
-			clock-frequency = <100000>;
-			st,i2c-min-scl-pulse-width-us = <0>;
-			st,i2c-min-sda-pulse-width-us = <5>;
-		};
-
-		st_dwc3: dwc3@8f94000 {
-			status = "okay";
-		};
-
-		ethernet0: dwmac@9630000 {
-			st,tx-retime-src = "clkgen";
-			status = "okay";
-			phy-mode = "rgmii";
-			fixed-link = <0 1 1000 0 0>;
-		};
-
-		demux@8a20000 {
-			compatible	= "st,stih407-c8sectpfe";
-			status		= "okay";
-			reg		= <0x08a20000 0x10000>,
-					  <0x08a00000 0x4000>;
-			reg-names	= "c8sectpfe", "c8sectpfe-ram";
-			interrupts	= <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>,
-					  <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
-			interrupt-names	= "c8sectpfe-error-irq",
-					  "c8sectpfe-idle-irq";
-			pinctrl-0	= <&pinctrl_tsin0_serial>;
-			pinctrl-1	= <&pinctrl_tsin0_parallel>;
-			pinctrl-2	= <&pinctrl_tsin3_serial>;
-			pinctrl-3	= <&pinctrl_tsin4_serial_alt3>;
-			pinctrl-4	= <&pinctrl_tsin5_serial_alt1>;
-			pinctrl-names	= "tsin0-serial",
-					  "tsin0-parallel",
-					  "tsin3-serial",
-					  "tsin4-serial",
-					  "tsin5-serial";
-			clocks		= <&clk_s_c0_flexgen CLK_PROC_STFE>;
-			clock-names	= "c8sectpfe";
-
-			/* tsin0 is TSA on NIMA */
-			tsin0: port {
-				tsin-num = <0>;
-				serial-not-parallel;
-				i2c-bus = <&ssc2>;
-				reset-gpios = <&pio15 4 GPIO_ACTIVE_LOW>;
-				dvb-card = <STV0367_TDA18212_NIMA_1>;
-			};
-		};
-
-		sti_uni_player0: sti-uni-player@8d80000 {
-			status = "okay";
-		};
-
-		sti_uni_player2: sti-uni-player@8d82000 {
-			status = "okay";
-		};
-
-		sti_uni_player3: sti-uni-player@8d85000 {
-			status = "okay";
-		};
-
-		syscfg_core: core-syscfg@92b0000 {
-			sti_sasg_codec: sti-sasg-codec {
-				status = "okay";
-				pinctrl-names = "default";
-				pinctrl-0 = <&pinctrl_spdif_out>;
-			};
-		};
-	};
-};

-- 
2.50.1



^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH v2 2/3] ARM: sti: removal of stih415/stih416 related entries
  2025-07-14 13:49 [PATCH v2 0/3] STi device-tree cleanup Raphael Gallais-Pou
  2025-07-14 13:49 ` [PATCH v2 1/3] ARM: sti: drop B2120 board support Raphael Gallais-Pou
@ 2025-07-14 13:49 ` Raphael Gallais-Pou
  2025-07-28  9:25   ` Patrice CHOTARD
  2025-07-14 13:49 ` [PATCH v2 3/3] dt-bindings: arm: sti: drop B2120 board support Raphael Gallais-Pou
  2 siblings, 1 reply; 10+ messages in thread
From: Raphael Gallais-Pou @ 2025-07-14 13:49 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Patrice Chotard,
	Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel, Alain Volmat

From: Alain Volmat <avolmat@me.com>

ST's STiH415 and STiH416 platforms have already been removed since
a while.  Remove some remaining bits within the mach-sti.

Signed-off-by: Alain Volmat <avolmat@me.com>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
---
 arch/arm/mach-sti/Kconfig    | 20 +-------------------
 arch/arm/mach-sti/board-dt.c |  2 --
 2 files changed, 1 insertion(+), 21 deletions(-)

diff --git a/arch/arm/mach-sti/Kconfig b/arch/arm/mach-sti/Kconfig
index b3842c971d31b80edaf88ab907f4377bea0a2af5..e58699e13e1a55ce46e68908c7ef51e18b040dc9 100644
--- a/arch/arm/mach-sti/Kconfig
+++ b/arch/arm/mach-sti/Kconfig
@@ -19,31 +19,13 @@ menuconfig ARCH_STI
 	select PL310_ERRATA_769419 if CACHE_L2X0
 	select RESET_CONTROLLER
 	help
-	  Include support for STMicroelectronics' STiH415/416, STiH407/10 and
+	  Include support for STMicroelectronics' STiH407/10 and
 	  STiH418 family SoCs using the Device Tree for discovery.  More
 	  information can be found in Documentation/arch/arm/sti/ and
 	  Documentation/devicetree.
 
 if ARCH_STI
 
-config SOC_STIH415
-	bool "STiH415 STMicroelectronics Consumer Electronics family"
-	default y
-	help
-	  This enables support for STMicroelectronics Digital Consumer
-	  Electronics family StiH415 parts, primarily targeted at set-top-box
-	  and other digital audio/video applications using Flattned Device
-	  Trees.
-
-config SOC_STIH416
-	bool "STiH416 STMicroelectronics Consumer Electronics family"
-	default y
-	help
-	  This enables support for STMicroelectronics Digital Consumer
-	  Electronics family StiH416 parts, primarily targeted at set-top-box
-	  and other digital audio/video applications using Flattened Device
-	  Trees.
-
 config SOC_STIH407
 	bool "STiH407 STMicroelectronics Consumer Electronics family"
 	default y
diff --git a/arch/arm/mach-sti/board-dt.c b/arch/arm/mach-sti/board-dt.c
index 488084b61b4acafb569ee9c51f5769393d55a9ce..1aaf61184685d754de57b487aef9a6b45a759b23 100644
--- a/arch/arm/mach-sti/board-dt.c
+++ b/arch/arm/mach-sti/board-dt.c
@@ -10,8 +10,6 @@
 #include "smp.h"
 
 static const char *const stih41x_dt_match[] __initconst = {
-	"st,stih415",
-	"st,stih416",
 	"st,stih407",
 	"st,stih410",
 	"st,stih418",

-- 
2.50.1



^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH v2 3/3] dt-bindings: arm: sti: drop B2120 board support
  2025-07-14 13:49 [PATCH v2 0/3] STi device-tree cleanup Raphael Gallais-Pou
  2025-07-14 13:49 ` [PATCH v2 1/3] ARM: sti: drop B2120 board support Raphael Gallais-Pou
  2025-07-14 13:49 ` [PATCH v2 2/3] ARM: sti: removal of stih415/stih416 related entries Raphael Gallais-Pou
@ 2025-07-14 13:49 ` Raphael Gallais-Pou
  2025-07-28  9:26   ` Patrice CHOTARD
  2 siblings, 1 reply; 10+ messages in thread
From: Raphael Gallais-Pou @ 2025-07-14 13:49 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Patrice Chotard,
	Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel, Krzysztof Kozlowski

B2120 boards are internal boards which never were commercialised.

Remove them from bindings.

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>

---
Changes in v2:
- Added Krzysztof's acked-by
- Put this patch last to get a bisectable series
---
 Documentation/devicetree/bindings/arm/sti.yaml | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/sti.yaml b/Documentation/devicetree/bindings/arm/sti.yaml
index 842def3e3f2bce470763d3665c7603b9058b1e4e..177358895fe1c9f80f8c825142cf015d04286ce8 100644
--- a/Documentation/devicetree/bindings/arm/sti.yaml
+++ b/Documentation/devicetree/bindings/arm/sti.yaml
@@ -14,12 +14,8 @@ properties:
     const: '/'
   compatible:
     oneOf:
-      - items:
-          - const: st,stih407-b2120
-          - const: st,stih407
       - items:
           - enum:
-              - st,stih410-b2120
               - st,stih410-b2260
           - const: st,stih410
       - items:

-- 
2.50.1



^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 2/3] ARM: sti: removal of stih415/stih416 related entries
  2025-07-14 13:49 ` [PATCH v2 2/3] ARM: sti: removal of stih415/stih416 related entries Raphael Gallais-Pou
@ 2025-07-28  9:25   ` Patrice CHOTARD
  2025-08-18 14:02     ` Patrice CHOTARD
  0 siblings, 1 reply; 10+ messages in thread
From: Patrice CHOTARD @ 2025-07-28  9:25 UTC (permalink / raw)
  To: Raphael Gallais-Pou, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel, Alain Volmat



On 7/14/25 15:49, Raphael Gallais-Pou wrote:
> From: Alain Volmat <avolmat@me.com>
> 
> ST's STiH415 and STiH416 platforms have already been removed since
> a while.  Remove some remaining bits within the mach-sti.
> 
> Signed-off-by: Alain Volmat <avolmat@me.com>
> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
> ---
>  arch/arm/mach-sti/Kconfig    | 20 +-------------------
>  arch/arm/mach-sti/board-dt.c |  2 --
>  2 files changed, 1 insertion(+), 21 deletions(-)
> 
> diff --git a/arch/arm/mach-sti/Kconfig b/arch/arm/mach-sti/Kconfig
> index b3842c971d31b80edaf88ab907f4377bea0a2af5..e58699e13e1a55ce46e68908c7ef51e18b040dc9 100644
> --- a/arch/arm/mach-sti/Kconfig
> +++ b/arch/arm/mach-sti/Kconfig
> @@ -19,31 +19,13 @@ menuconfig ARCH_STI
>  	select PL310_ERRATA_769419 if CACHE_L2X0
>  	select RESET_CONTROLLER
>  	help
> -	  Include support for STMicroelectronics' STiH415/416, STiH407/10 and
> +	  Include support for STMicroelectronics' STiH407/10 and
>  	  STiH418 family SoCs using the Device Tree for discovery.  More
>  	  information can be found in Documentation/arch/arm/sti/ and
>  	  Documentation/devicetree.
>  
>  if ARCH_STI
>  
> -config SOC_STIH415
> -	bool "STiH415 STMicroelectronics Consumer Electronics family"
> -	default y
> -	help
> -	  This enables support for STMicroelectronics Digital Consumer
> -	  Electronics family StiH415 parts, primarily targeted at set-top-box
> -	  and other digital audio/video applications using Flattned Device
> -	  Trees.
> -
> -config SOC_STIH416
> -	bool "STiH416 STMicroelectronics Consumer Electronics family"
> -	default y
> -	help
> -	  This enables support for STMicroelectronics Digital Consumer
> -	  Electronics family StiH416 parts, primarily targeted at set-top-box
> -	  and other digital audio/video applications using Flattened Device
> -	  Trees.
> -
>  config SOC_STIH407
>  	bool "STiH407 STMicroelectronics Consumer Electronics family"
>  	default y
> diff --git a/arch/arm/mach-sti/board-dt.c b/arch/arm/mach-sti/board-dt.c
> index 488084b61b4acafb569ee9c51f5769393d55a9ce..1aaf61184685d754de57b487aef9a6b45a759b23 100644
> --- a/arch/arm/mach-sti/board-dt.c
> +++ b/arch/arm/mach-sti/board-dt.c
> @@ -10,8 +10,6 @@
>  #include "smp.h"
>  
>  static const char *const stih41x_dt_match[] __initconst = {
> -	"st,stih415",
> -	"st,stih416",
>  	"st,stih407",
>  	"st,stih410",
>  	"st,stih418",
> 


Hi Raphael

Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] ARM: sti: drop B2120 board support
  2025-07-14 13:49 ` [PATCH v2 1/3] ARM: sti: drop B2120 board support Raphael Gallais-Pou
@ 2025-07-28  9:25   ` Patrice CHOTARD
  2025-08-18 14:02     ` Patrice CHOTARD
  0 siblings, 1 reply; 10+ messages in thread
From: Patrice CHOTARD @ 2025-07-28  9:25 UTC (permalink / raw)
  To: Raphael Gallais-Pou, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel



On 7/14/25 15:49, Raphael Gallais-Pou wrote:
> B2120 boards are internal boards which never were commercialised.
> 
> Drop them.
> 
> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
> ---
>  arch/arm/boot/dts/st/Makefile           |   2 -
>  arch/arm/boot/dts/st/stih407-b2120.dts  |  27 -----
>  arch/arm/boot/dts/st/stih407.dtsi       | 145 ----------------------
>  arch/arm/boot/dts/st/stih410-b2120.dts  |  66 ----------
>  arch/arm/boot/dts/st/stihxxx-b2120.dtsi | 206 --------------------------------
>  5 files changed, 446 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/st/Makefile b/arch/arm/boot/dts/st/Makefile
> index cc9948b9870f7f73629573149bfd342af75b07da..f57fc6dc48a00c9a9323b4508e5e4e161b197c47 100644
> --- a/arch/arm/boot/dts/st/Makefile
> +++ b/arch/arm/boot/dts/st/Makefile
> @@ -13,8 +13,6 @@ dtb-$(CONFIG_ARCH_SPEAR3XX) += \
>  dtb-$(CONFIG_ARCH_SPEAR6XX) += \
>  	spear600-evb.dtb
>  dtb-$(CONFIG_ARCH_STI) += \
> -	stih407-b2120.dtb \
> -	stih410-b2120.dtb \
>  	stih410-b2260.dtb \
>  	stih418-b2199.dtb \
>  	stih418-b2264.dtb
> diff --git a/arch/arm/boot/dts/st/stih407-b2120.dts b/arch/arm/boot/dts/st/stih407-b2120.dts
> deleted file mode 100644
> index 9c79982ee7ba8fadb1a2a92e732bf7f652b74c38..0000000000000000000000000000000000000000
> --- a/arch/arm/boot/dts/st/stih407-b2120.dts
> +++ /dev/null
> @@ -1,27 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0-only
> -/*
> - * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
> - * Author: Giuseppe Cavallaro <peppe.cavallaro@st.com>
> - */
> -/dts-v1/;
> -#include "stih407.dtsi"
> -#include "stihxxx-b2120.dtsi"
> -/ {
> -	model = "STiH407 B2120";
> -	compatible = "st,stih407-b2120", "st,stih407";
> -
> -	chosen {
> -		stdout-path = &sbc_serial0;
> -	};
> -
> -	memory@40000000 {
> -		device_type = "memory";
> -		reg = <0x40000000 0x80000000>;
> -	};
> -
> -	aliases {
> -		serial0 = &sbc_serial0;
> -		ethernet0 = &ethernet0;
> -	};
> -
> -};
> diff --git a/arch/arm/boot/dts/st/stih407.dtsi b/arch/arm/boot/dts/st/stih407.dtsi
> deleted file mode 100644
> index aca43d2bdaad44ef2a0e8a120c679c217709af44..0000000000000000000000000000000000000000
> --- a/arch/arm/boot/dts/st/stih407.dtsi
> +++ /dev/null
> @@ -1,145 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0-only
> -/*
> - * Copyright (C) 2015 STMicroelectronics Limited.
> - * Author: Gabriel Fernandez <gabriel.fernandez@linaro.org>
> - */
> -#include "stih407-clock.dtsi"
> -#include "stih407-family.dtsi"
> -#include <dt-bindings/gpio/gpio.h>
> -/ {
> -	soc {
> -		sti-display-subsystem@0 {
> -			compatible = "st,sti-display-subsystem";
> -			#address-cells = <1>;
> -			#size-cells = <1>;
> -			reg = <0 0>;
> -			assigned-clocks = <&clk_s_d2_quadfs 0>,
> -					  <&clk_s_d2_quadfs 1>,
> -					  <&clk_s_c0_pll1 0>,
> -					  <&clk_s_c0_flexgen CLK_COMPO_DVP>,
> -					  <&clk_s_c0_flexgen CLK_MAIN_DISP>,
> -					  <&clk_s_d2_flexgen CLK_PIX_MAIN_DISP>,
> -					  <&clk_s_d2_flexgen CLK_PIX_AUX_DISP>,
> -					  <&clk_s_d2_flexgen CLK_PIX_GDP1>,
> -					  <&clk_s_d2_flexgen CLK_PIX_GDP2>,
> -					  <&clk_s_d2_flexgen CLK_PIX_GDP3>,
> -					  <&clk_s_d2_flexgen CLK_PIX_GDP4>;
> -
> -			assigned-clock-parents = <0>,
> -						 <0>,
> -						 <0>,
> -						 <&clk_s_c0_pll1 0>,
> -						 <&clk_s_c0_pll1 0>,
> -						 <&clk_s_d2_quadfs 0>,
> -						 <&clk_s_d2_quadfs 1>,
> -						 <&clk_s_d2_quadfs 0>,
> -						 <&clk_s_d2_quadfs 0>,
> -						 <&clk_s_d2_quadfs 0>,
> -						 <&clk_s_d2_quadfs 0>;
> -
> -			assigned-clock-rates = <297000000>,
> -					       <108000000>,
> -					       <0>,
> -					       <400000000>,
> -					       <400000000>;
> -
> -			ranges;
> -
> -			sti-compositor@9d11000 {
> -				compatible = "st,stih407-compositor";
> -				reg = <0x9d11000 0x1000>;
> -
> -				clock-names = "compo_main",
> -					      "compo_aux",
> -					      "pix_main",
> -					      "pix_aux",
> -					      "pix_gdp1",
> -					      "pix_gdp2",
> -					      "pix_gdp3",
> -					      "pix_gdp4",
> -					      "main_parent",
> -					      "aux_parent";
> -
> -				clocks = <&clk_s_c0_flexgen CLK_COMPO_DVP>,
> -					 <&clk_s_c0_flexgen CLK_COMPO_DVP>,
> -					 <&clk_s_d2_flexgen CLK_PIX_MAIN_DISP>,
> -					 <&clk_s_d2_flexgen CLK_PIX_AUX_DISP>,
> -					 <&clk_s_d2_flexgen CLK_PIX_GDP1>,
> -					 <&clk_s_d2_flexgen CLK_PIX_GDP2>,
> -					 <&clk_s_d2_flexgen CLK_PIX_GDP3>,
> -					 <&clk_s_d2_flexgen CLK_PIX_GDP4>,
> -					 <&clk_s_d2_quadfs 0>,
> -					 <&clk_s_d2_quadfs 1>;
> -
> -				reset-names = "compo-main", "compo-aux";
> -				resets = <&softreset STIH407_COMPO_SOFTRESET>,
> -					 <&softreset STIH407_COMPO_SOFTRESET>;
> -				st,vtg = <&vtg_main>, <&vtg_aux>;
> -			};
> -
> -			sti-tvout@8d08000 {
> -				compatible = "st,stih407-tvout";
> -				reg = <0x8d08000 0x1000>;
> -				reg-names = "tvout-reg";
> -				reset-names = "tvout";
> -				resets = <&softreset STIH407_HDTVOUT_SOFTRESET>;
> -				#address-cells = <1>;
> -				#size-cells = <1>;
> -				assigned-clocks = <&clk_s_d2_flexgen CLK_PIX_HDMI>,
> -						  <&clk_s_d2_flexgen CLK_TMDS_HDMI>,
> -						  <&clk_s_d2_flexgen CLK_REF_HDMIPHY>,
> -						  <&clk_s_d0_flexgen CLK_PCM_0>,
> -						  <&clk_s_d2_flexgen CLK_PIX_HDDAC>,
> -						  <&clk_s_d2_flexgen CLK_HDDAC>;
> -
> -				assigned-clock-parents = <&clk_s_d2_quadfs 0>,
> -							 <&clk_tmdsout_hdmi>,
> -							 <&clk_s_d2_quadfs 0>,
> -							 <&clk_s_d0_quadfs 0>,
> -							 <&clk_s_d2_quadfs 0>,
> -							 <&clk_s_d2_quadfs 0>;
> -			};
> -
> -			sti_hdmi: sti-hdmi@8d04000 {
> -				compatible = "st,stih407-hdmi";
> -				reg = <0x8d04000 0x1000>;
> -				reg-names = "hdmi-reg";
> -				#sound-dai-cells = <0>;
> -				interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>;
> -				interrupt-names = "irq";
> -				clock-names = "pix",
> -					      "tmds",
> -					      "phy",
> -					      "audio",
> -					      "main_parent",
> -					      "aux_parent";
> -
> -				clocks = <&clk_s_d2_flexgen CLK_PIX_HDMI>,
> -					 <&clk_s_d2_flexgen CLK_TMDS_HDMI>,
> -					 <&clk_s_d2_flexgen CLK_REF_HDMIPHY>,
> -					 <&clk_s_d0_flexgen CLK_PCM_0>,
> -					 <&clk_s_d2_quadfs 0>,
> -					 <&clk_s_d2_quadfs 1>;
> -
> -				hdmi,hpd-gpio = <&pio5 3 GPIO_ACTIVE_LOW>;
> -				reset-names = "hdmi";
> -				resets = <&softreset STIH407_HDMI_TX_PHY_SOFTRESET>;
> -				ddc = <&hdmiddc>;
> -			};
> -
> -			sti-hda@8d02000 {
> -				compatible = "st,stih407-hda";
> -				reg = <0x8d02000 0x400>, <0x92b0120 0x4>;
> -				reg-names = "hda-reg", "video-dacs-ctrl";
> -				clock-names = "pix",
> -					      "hddac",
> -					      "main_parent",
> -					      "aux_parent";
> -				clocks = <&clk_s_d2_flexgen CLK_PIX_HDDAC>,
> -					 <&clk_s_d2_flexgen CLK_HDDAC>,
> -					 <&clk_s_d2_quadfs 0>,
> -					 <&clk_s_d2_quadfs 1>;
> -			};
> -		};
> -	};
> -};
> diff --git a/arch/arm/boot/dts/st/stih410-b2120.dts b/arch/arm/boot/dts/st/stih410-b2120.dts
> deleted file mode 100644
> index 538ff98ca1b1b6542f98a18f68968f030b8c55c0..0000000000000000000000000000000000000000
> --- a/arch/arm/boot/dts/st/stih410-b2120.dts
> +++ /dev/null
> @@ -1,66 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0-only
> -/*
> - * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
> - * Author: Peter Griffin <peter.griffin@linaro.org>
> - */
> -/dts-v1/;
> -#include "stih410.dtsi"
> -#include "stihxxx-b2120.dtsi"
> -/ {
> -	model = "STiH410 B2120";
> -	compatible = "st,stih410-b2120", "st,stih410";
> -
> -	chosen {
> -		stdout-path = &sbc_serial0;
> -	};
> -
> -	memory@40000000 {
> -		device_type = "memory";
> -		reg = <0x40000000 0x80000000>;
> -	};
> -
> -	aliases {
> -		serial0 = &sbc_serial0;
> -		ethernet0 = &ethernet0;
> -	};
> -
> -	usb2_picophy1: phy2 {
> -		status = "okay";
> -	};
> -
> -	usb2_picophy2: phy3 {
> -		status = "okay";
> -	};
> -
> -	soc {
> -
> -		mmc0: sdhci@9060000 {
> -			max-frequency = <200000000>;
> -			sd-uhs-sdr50;
> -			sd-uhs-sdr104;
> -			sd-uhs-ddr50;
> -		};
> -
> -		ohci0: usb@9a03c00 {
> -			status = "okay";
> -		};
> -
> -		ehci0: usb@9a03e00 {
> -			status = "okay";
> -		};
> -
> -		ohci1: usb@9a83c00 {
> -			status = "okay";
> -		};
> -
> -		ehci1: usb@9a83e00 {
> -			status = "okay";
> -		};
> -
> -		sti-display-subsystem@0 {
> -			sti-hda@8d02000 {
> -				status = "okay";
> -			};
> -		};
> -	};
> -};
> diff --git a/arch/arm/boot/dts/st/stihxxx-b2120.dtsi b/arch/arm/boot/dts/st/stihxxx-b2120.dtsi
> deleted file mode 100644
> index 8d9a2dfa76f1fe55b954803cab8d00003a8ebec0..0000000000000000000000000000000000000000
> --- a/arch/arm/boot/dts/st/stihxxx-b2120.dtsi
> +++ /dev/null
> @@ -1,206 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0-only
> -/*
> - * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
> - * Author: Giuseppe Cavallaro <peppe.cavallaro@st.com>
> - */
> -#include <dt-bindings/clock/stih407-clks.h>
> -#include <dt-bindings/gpio/gpio.h>
> -#include <dt-bindings/media/c8sectpfe.h>
> -/ {
> -	leds {
> -		compatible = "gpio-leds";
> -		led-red {
> -			label = "Front Panel LED";
> -			gpios = <&pio4 1 GPIO_ACTIVE_HIGH>;
> -			linux,default-trigger = "heartbeat";
> -		};
> -		led-green {
> -			gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
> -			default-state = "off";
> -		};
> -	};
> -
> -	sound: sound {
> -		compatible = "simple-audio-card";
> -		simple-audio-card,name = "STI-B2120";
> -		status = "okay";
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -
> -		simple-audio-card,dai-link@0 {
> -			reg = <0>;
> -			/* HDMI */
> -			format = "i2s";
> -			mclk-fs = <128>;
> -			cpu {
> -				sound-dai = <&sti_uni_player0>;
> -			};
> -
> -			codec {
> -				sound-dai = <&sti_hdmi>;
> -			};
> -		};
> -
> -		simple-audio-card,dai-link@1 {
> -			reg = <1>;
> -			/* DAC */
> -			format = "i2s";
> -			mclk-fs = <256>;
> -			frame-inversion;
> -			cpu {
> -				sound-dai = <&sti_uni_player2>;
> -			};
> -
> -			codec {
> -				sound-dai = <&sti_sasg_codec 1>;
> -			};
> -		};
> -
> -		simple-audio-card,dai-link@2 {
> -			reg = <2>;
> -			/* SPDIF */
> -			format = "left_j";
> -			mclk-fs = <128>;
> -			cpu {
> -				sound-dai = <&sti_uni_player3>;
> -			};
> -
> -			codec {
> -				sound-dai = <&sti_sasg_codec 0>;
> -			};
> -		};
> -	};
> -
> -	miphy28lp_phy: miphy28lp {
> -
> -		phy_port0: port@9b22000 {
> -			st,osc-rdy;
> -		};
> -
> -		phy_port1: port@9b2a000 {
> -			st,osc-force-ext;
> -		};
> -	};
> -
> -	soc {
> -		sbc_serial0: serial@9530000 {
> -			status = "okay";
> -		};
> -
> -		pwm0: pwm@9810000 {
> -			status = "okay";
> -		};
> -
> -		pwm1: pwm@9510000 {
> -			status = "okay";
> -		};
> -
> -		ssc2: i2c@9842000 {
> -			status = "okay";
> -			clock-frequency = <100000>;
> -			st,i2c-min-scl-pulse-width-us = <0>;
> -			st,i2c-min-sda-pulse-width-us = <5>;
> -		};
> -
> -		ssc3: i2c@9843000 {
> -			status = "okay";
> -			clock-frequency = <100000>;
> -			st,i2c-min-scl-pulse-width-us = <0>;
> -			st,i2c-min-sda-pulse-width-us = <5>;
> -		};
> -
> -		i2c@9844000 {
> -			status = "okay";
> -		};
> -
> -		i2c@9845000 {
> -			status = "okay";
> -		};
> -
> -		i2c@9540000 {
> -			status = "okay";
> -		};
> -
> -		mmc0: sdhci@9060000 {
> -			non-removable;
> -			status = "okay";
> -		};
> -
> -		mmc1: sdhci@9080000 {
> -			status = "okay";
> -		};
> -
> -		/* SSC11 to HDMI */
> -		hdmiddc: i2c@9541000 {
> -			status = "okay";
> -			/* HDMI V1.3a supports Standard mode only */
> -			clock-frequency = <100000>;
> -			st,i2c-min-scl-pulse-width-us = <0>;
> -			st,i2c-min-sda-pulse-width-us = <5>;
> -		};
> -
> -		st_dwc3: dwc3@8f94000 {
> -			status = "okay";
> -		};
> -
> -		ethernet0: dwmac@9630000 {
> -			st,tx-retime-src = "clkgen";
> -			status = "okay";
> -			phy-mode = "rgmii";
> -			fixed-link = <0 1 1000 0 0>;
> -		};
> -
> -		demux@8a20000 {
> -			compatible	= "st,stih407-c8sectpfe";
> -			status		= "okay";
> -			reg		= <0x08a20000 0x10000>,
> -					  <0x08a00000 0x4000>;
> -			reg-names	= "c8sectpfe", "c8sectpfe-ram";
> -			interrupts	= <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>,
> -					  <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
> -			interrupt-names	= "c8sectpfe-error-irq",
> -					  "c8sectpfe-idle-irq";
> -			pinctrl-0	= <&pinctrl_tsin0_serial>;
> -			pinctrl-1	= <&pinctrl_tsin0_parallel>;
> -			pinctrl-2	= <&pinctrl_tsin3_serial>;
> -			pinctrl-3	= <&pinctrl_tsin4_serial_alt3>;
> -			pinctrl-4	= <&pinctrl_tsin5_serial_alt1>;
> -			pinctrl-names	= "tsin0-serial",
> -					  "tsin0-parallel",
> -					  "tsin3-serial",
> -					  "tsin4-serial",
> -					  "tsin5-serial";
> -			clocks		= <&clk_s_c0_flexgen CLK_PROC_STFE>;
> -			clock-names	= "c8sectpfe";
> -
> -			/* tsin0 is TSA on NIMA */
> -			tsin0: port {
> -				tsin-num = <0>;
> -				serial-not-parallel;
> -				i2c-bus = <&ssc2>;
> -				reset-gpios = <&pio15 4 GPIO_ACTIVE_LOW>;
> -				dvb-card = <STV0367_TDA18212_NIMA_1>;
> -			};
> -		};
> -
> -		sti_uni_player0: sti-uni-player@8d80000 {
> -			status = "okay";
> -		};
> -
> -		sti_uni_player2: sti-uni-player@8d82000 {
> -			status = "okay";
> -		};
> -
> -		sti_uni_player3: sti-uni-player@8d85000 {
> -			status = "okay";
> -		};
> -
> -		syscfg_core: core-syscfg@92b0000 {
> -			sti_sasg_codec: sti-sasg-codec {
> -				status = "okay";
> -				pinctrl-names = "default";
> -				pinctrl-0 = <&pinctrl_spdif_out>;
> -			};
> -		};
> -	};
> -};
> 


Hi Rahael

Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 3/3] dt-bindings: arm: sti: drop B2120 board support
  2025-07-14 13:49 ` [PATCH v2 3/3] dt-bindings: arm: sti: drop B2120 board support Raphael Gallais-Pou
@ 2025-07-28  9:26   ` Patrice CHOTARD
  2025-08-18 14:02     ` Patrice CHOTARD
  0 siblings, 1 reply; 10+ messages in thread
From: Patrice CHOTARD @ 2025-07-28  9:26 UTC (permalink / raw)
  To: Raphael Gallais-Pou, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel, Krzysztof Kozlowski



On 7/14/25 15:49, Raphael Gallais-Pou wrote:
> B2120 boards are internal boards which never were commercialised.
> 
> Remove them from bindings.
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
> 
> ---
> Changes in v2:
> - Added Krzysztof's acked-by
> - Put this patch last to get a bisectable series
> ---
>  Documentation/devicetree/bindings/arm/sti.yaml | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/sti.yaml b/Documentation/devicetree/bindings/arm/sti.yaml
> index 842def3e3f2bce470763d3665c7603b9058b1e4e..177358895fe1c9f80f8c825142cf015d04286ce8 100644
> --- a/Documentation/devicetree/bindings/arm/sti.yaml
> +++ b/Documentation/devicetree/bindings/arm/sti.yaml
> @@ -14,12 +14,8 @@ properties:
>      const: '/'
>    compatible:
>      oneOf:
> -      - items:
> -          - const: st,stih407-b2120
> -          - const: st,stih407
>        - items:
>            - enum:
> -              - st,stih410-b2120
>                - st,stih410-b2260
>            - const: st,stih410
>        - items:
> 


Hi Raphael

Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 1/3] ARM: sti: drop B2120 board support
  2025-07-28  9:25   ` Patrice CHOTARD
@ 2025-08-18 14:02     ` Patrice CHOTARD
  0 siblings, 0 replies; 10+ messages in thread
From: Patrice CHOTARD @ 2025-08-18 14:02 UTC (permalink / raw)
  To: Raphael Gallais-Pou, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel



On 7/28/25 11:25, Patrice CHOTARD wrote:
> 
> 
> On 7/14/25 15:49, Raphael Gallais-Pou wrote:
>> B2120 boards are internal boards which never were commercialised.
>>
>> Drop them.
>>
>> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
>> ---
>>  arch/arm/boot/dts/st/Makefile           |   2 -
>>  arch/arm/boot/dts/st/stih407-b2120.dts  |  27 -----
>>  arch/arm/boot/dts/st/stih407.dtsi       | 145 ----------------------
>>  arch/arm/boot/dts/st/stih410-b2120.dts  |  66 ----------
>>  arch/arm/boot/dts/st/stihxxx-b2120.dtsi | 206 --------------------------------
>>  5 files changed, 446 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/st/Makefile b/arch/arm/boot/dts/st/Makefile
>> index cc9948b9870f7f73629573149bfd342af75b07da..f57fc6dc48a00c9a9323b4508e5e4e161b197c47 100644
>> --- a/arch/arm/boot/dts/st/Makefile
>> +++ b/arch/arm/boot/dts/st/Makefile
>> @@ -13,8 +13,6 @@ dtb-$(CONFIG_ARCH_SPEAR3XX) += \
>>  dtb-$(CONFIG_ARCH_SPEAR6XX) += \
>>  	spear600-evb.dtb
>>  dtb-$(CONFIG_ARCH_STI) += \
>> -	stih407-b2120.dtb \
>> -	stih410-b2120.dtb \
>>  	stih410-b2260.dtb \
>>  	stih418-b2199.dtb \
>>  	stih418-b2264.dtb
>> diff --git a/arch/arm/boot/dts/st/stih407-b2120.dts b/arch/arm/boot/dts/st/stih407-b2120.dts
>> deleted file mode 100644
>> index 9c79982ee7ba8fadb1a2a92e732bf7f652b74c38..0000000000000000000000000000000000000000
>> --- a/arch/arm/boot/dts/st/stih407-b2120.dts
>> +++ /dev/null
>> @@ -1,27 +0,0 @@
>> -// SPDX-License-Identifier: GPL-2.0-only
>> -/*
>> - * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
>> - * Author: Giuseppe Cavallaro <peppe.cavallaro@st.com>
>> - */
>> -/dts-v1/;
>> -#include "stih407.dtsi"
>> -#include "stihxxx-b2120.dtsi"
>> -/ {
>> -	model = "STiH407 B2120";
>> -	compatible = "st,stih407-b2120", "st,stih407";
>> -
>> -	chosen {
>> -		stdout-path = &sbc_serial0;
>> -	};
>> -
>> -	memory@40000000 {
>> -		device_type = "memory";
>> -		reg = <0x40000000 0x80000000>;
>> -	};
>> -
>> -	aliases {
>> -		serial0 = &sbc_serial0;
>> -		ethernet0 = &ethernet0;
>> -	};
>> -
>> -};
>> diff --git a/arch/arm/boot/dts/st/stih407.dtsi b/arch/arm/boot/dts/st/stih407.dtsi
>> deleted file mode 100644
>> index aca43d2bdaad44ef2a0e8a120c679c217709af44..0000000000000000000000000000000000000000
>> --- a/arch/arm/boot/dts/st/stih407.dtsi
>> +++ /dev/null
>> @@ -1,145 +0,0 @@
>> -// SPDX-License-Identifier: GPL-2.0-only
>> -/*
>> - * Copyright (C) 2015 STMicroelectronics Limited.
>> - * Author: Gabriel Fernandez <gabriel.fernandez@linaro.org>
>> - */
>> -#include "stih407-clock.dtsi"
>> -#include "stih407-family.dtsi"
>> -#include <dt-bindings/gpio/gpio.h>
>> -/ {
>> -	soc {
>> -		sti-display-subsystem@0 {
>> -			compatible = "st,sti-display-subsystem";
>> -			#address-cells = <1>;
>> -			#size-cells = <1>;
>> -			reg = <0 0>;
>> -			assigned-clocks = <&clk_s_d2_quadfs 0>,
>> -					  <&clk_s_d2_quadfs 1>,
>> -					  <&clk_s_c0_pll1 0>,
>> -					  <&clk_s_c0_flexgen CLK_COMPO_DVP>,
>> -					  <&clk_s_c0_flexgen CLK_MAIN_DISP>,
>> -					  <&clk_s_d2_flexgen CLK_PIX_MAIN_DISP>,
>> -					  <&clk_s_d2_flexgen CLK_PIX_AUX_DISP>,
>> -					  <&clk_s_d2_flexgen CLK_PIX_GDP1>,
>> -					  <&clk_s_d2_flexgen CLK_PIX_GDP2>,
>> -					  <&clk_s_d2_flexgen CLK_PIX_GDP3>,
>> -					  <&clk_s_d2_flexgen CLK_PIX_GDP4>;
>> -
>> -			assigned-clock-parents = <0>,
>> -						 <0>,
>> -						 <0>,
>> -						 <&clk_s_c0_pll1 0>,
>> -						 <&clk_s_c0_pll1 0>,
>> -						 <&clk_s_d2_quadfs 0>,
>> -						 <&clk_s_d2_quadfs 1>,
>> -						 <&clk_s_d2_quadfs 0>,
>> -						 <&clk_s_d2_quadfs 0>,
>> -						 <&clk_s_d2_quadfs 0>,
>> -						 <&clk_s_d2_quadfs 0>;
>> -
>> -			assigned-clock-rates = <297000000>,
>> -					       <108000000>,
>> -					       <0>,
>> -					       <400000000>,
>> -					       <400000000>;
>> -
>> -			ranges;
>> -
>> -			sti-compositor@9d11000 {
>> -				compatible = "st,stih407-compositor";
>> -				reg = <0x9d11000 0x1000>;
>> -
>> -				clock-names = "compo_main",
>> -					      "compo_aux",
>> -					      "pix_main",
>> -					      "pix_aux",
>> -					      "pix_gdp1",
>> -					      "pix_gdp2",
>> -					      "pix_gdp3",
>> -					      "pix_gdp4",
>> -					      "main_parent",
>> -					      "aux_parent";
>> -
>> -				clocks = <&clk_s_c0_flexgen CLK_COMPO_DVP>,
>> -					 <&clk_s_c0_flexgen CLK_COMPO_DVP>,
>> -					 <&clk_s_d2_flexgen CLK_PIX_MAIN_DISP>,
>> -					 <&clk_s_d2_flexgen CLK_PIX_AUX_DISP>,
>> -					 <&clk_s_d2_flexgen CLK_PIX_GDP1>,
>> -					 <&clk_s_d2_flexgen CLK_PIX_GDP2>,
>> -					 <&clk_s_d2_flexgen CLK_PIX_GDP3>,
>> -					 <&clk_s_d2_flexgen CLK_PIX_GDP4>,
>> -					 <&clk_s_d2_quadfs 0>,
>> -					 <&clk_s_d2_quadfs 1>;
>> -
>> -				reset-names = "compo-main", "compo-aux";
>> -				resets = <&softreset STIH407_COMPO_SOFTRESET>,
>> -					 <&softreset STIH407_COMPO_SOFTRESET>;
>> -				st,vtg = <&vtg_main>, <&vtg_aux>;
>> -			};
>> -
>> -			sti-tvout@8d08000 {
>> -				compatible = "st,stih407-tvout";
>> -				reg = <0x8d08000 0x1000>;
>> -				reg-names = "tvout-reg";
>> -				reset-names = "tvout";
>> -				resets = <&softreset STIH407_HDTVOUT_SOFTRESET>;
>> -				#address-cells = <1>;
>> -				#size-cells = <1>;
>> -				assigned-clocks = <&clk_s_d2_flexgen CLK_PIX_HDMI>,
>> -						  <&clk_s_d2_flexgen CLK_TMDS_HDMI>,
>> -						  <&clk_s_d2_flexgen CLK_REF_HDMIPHY>,
>> -						  <&clk_s_d0_flexgen CLK_PCM_0>,
>> -						  <&clk_s_d2_flexgen CLK_PIX_HDDAC>,
>> -						  <&clk_s_d2_flexgen CLK_HDDAC>;
>> -
>> -				assigned-clock-parents = <&clk_s_d2_quadfs 0>,
>> -							 <&clk_tmdsout_hdmi>,
>> -							 <&clk_s_d2_quadfs 0>,
>> -							 <&clk_s_d0_quadfs 0>,
>> -							 <&clk_s_d2_quadfs 0>,
>> -							 <&clk_s_d2_quadfs 0>;
>> -			};
>> -
>> -			sti_hdmi: sti-hdmi@8d04000 {
>> -				compatible = "st,stih407-hdmi";
>> -				reg = <0x8d04000 0x1000>;
>> -				reg-names = "hdmi-reg";
>> -				#sound-dai-cells = <0>;
>> -				interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>;
>> -				interrupt-names = "irq";
>> -				clock-names = "pix",
>> -					      "tmds",
>> -					      "phy",
>> -					      "audio",
>> -					      "main_parent",
>> -					      "aux_parent";
>> -
>> -				clocks = <&clk_s_d2_flexgen CLK_PIX_HDMI>,
>> -					 <&clk_s_d2_flexgen CLK_TMDS_HDMI>,
>> -					 <&clk_s_d2_flexgen CLK_REF_HDMIPHY>,
>> -					 <&clk_s_d0_flexgen CLK_PCM_0>,
>> -					 <&clk_s_d2_quadfs 0>,
>> -					 <&clk_s_d2_quadfs 1>;
>> -
>> -				hdmi,hpd-gpio = <&pio5 3 GPIO_ACTIVE_LOW>;
>> -				reset-names = "hdmi";
>> -				resets = <&softreset STIH407_HDMI_TX_PHY_SOFTRESET>;
>> -				ddc = <&hdmiddc>;
>> -			};
>> -
>> -			sti-hda@8d02000 {
>> -				compatible = "st,stih407-hda";
>> -				reg = <0x8d02000 0x400>, <0x92b0120 0x4>;
>> -				reg-names = "hda-reg", "video-dacs-ctrl";
>> -				clock-names = "pix",
>> -					      "hddac",
>> -					      "main_parent",
>> -					      "aux_parent";
>> -				clocks = <&clk_s_d2_flexgen CLK_PIX_HDDAC>,
>> -					 <&clk_s_d2_flexgen CLK_HDDAC>,
>> -					 <&clk_s_d2_quadfs 0>,
>> -					 <&clk_s_d2_quadfs 1>;
>> -			};
>> -		};
>> -	};
>> -};
>> diff --git a/arch/arm/boot/dts/st/stih410-b2120.dts b/arch/arm/boot/dts/st/stih410-b2120.dts
>> deleted file mode 100644
>> index 538ff98ca1b1b6542f98a18f68968f030b8c55c0..0000000000000000000000000000000000000000
>> --- a/arch/arm/boot/dts/st/stih410-b2120.dts
>> +++ /dev/null
>> @@ -1,66 +0,0 @@
>> -// SPDX-License-Identifier: GPL-2.0-only
>> -/*
>> - * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
>> - * Author: Peter Griffin <peter.griffin@linaro.org>
>> - */
>> -/dts-v1/;
>> -#include "stih410.dtsi"
>> -#include "stihxxx-b2120.dtsi"
>> -/ {
>> -	model = "STiH410 B2120";
>> -	compatible = "st,stih410-b2120", "st,stih410";
>> -
>> -	chosen {
>> -		stdout-path = &sbc_serial0;
>> -	};
>> -
>> -	memory@40000000 {
>> -		device_type = "memory";
>> -		reg = <0x40000000 0x80000000>;
>> -	};
>> -
>> -	aliases {
>> -		serial0 = &sbc_serial0;
>> -		ethernet0 = &ethernet0;
>> -	};
>> -
>> -	usb2_picophy1: phy2 {
>> -		status = "okay";
>> -	};
>> -
>> -	usb2_picophy2: phy3 {
>> -		status = "okay";
>> -	};
>> -
>> -	soc {
>> -
>> -		mmc0: sdhci@9060000 {
>> -			max-frequency = <200000000>;
>> -			sd-uhs-sdr50;
>> -			sd-uhs-sdr104;
>> -			sd-uhs-ddr50;
>> -		};
>> -
>> -		ohci0: usb@9a03c00 {
>> -			status = "okay";
>> -		};
>> -
>> -		ehci0: usb@9a03e00 {
>> -			status = "okay";
>> -		};
>> -
>> -		ohci1: usb@9a83c00 {
>> -			status = "okay";
>> -		};
>> -
>> -		ehci1: usb@9a83e00 {
>> -			status = "okay";
>> -		};
>> -
>> -		sti-display-subsystem@0 {
>> -			sti-hda@8d02000 {
>> -				status = "okay";
>> -			};
>> -		};
>> -	};
>> -};
>> diff --git a/arch/arm/boot/dts/st/stihxxx-b2120.dtsi b/arch/arm/boot/dts/st/stihxxx-b2120.dtsi
>> deleted file mode 100644
>> index 8d9a2dfa76f1fe55b954803cab8d00003a8ebec0..0000000000000000000000000000000000000000
>> --- a/arch/arm/boot/dts/st/stihxxx-b2120.dtsi
>> +++ /dev/null
>> @@ -1,206 +0,0 @@
>> -// SPDX-License-Identifier: GPL-2.0-only
>> -/*
>> - * Copyright (C) 2014 STMicroelectronics (R&D) Limited.
>> - * Author: Giuseppe Cavallaro <peppe.cavallaro@st.com>
>> - */
>> -#include <dt-bindings/clock/stih407-clks.h>
>> -#include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/media/c8sectpfe.h>
>> -/ {
>> -	leds {
>> -		compatible = "gpio-leds";
>> -		led-red {
>> -			label = "Front Panel LED";
>> -			gpios = <&pio4 1 GPIO_ACTIVE_HIGH>;
>> -			linux,default-trigger = "heartbeat";
>> -		};
>> -		led-green {
>> -			gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
>> -			default-state = "off";
>> -		};
>> -	};
>> -
>> -	sound: sound {
>> -		compatible = "simple-audio-card";
>> -		simple-audio-card,name = "STI-B2120";
>> -		status = "okay";
>> -		#address-cells = <1>;
>> -		#size-cells = <0>;
>> -
>> -		simple-audio-card,dai-link@0 {
>> -			reg = <0>;
>> -			/* HDMI */
>> -			format = "i2s";
>> -			mclk-fs = <128>;
>> -			cpu {
>> -				sound-dai = <&sti_uni_player0>;
>> -			};
>> -
>> -			codec {
>> -				sound-dai = <&sti_hdmi>;
>> -			};
>> -		};
>> -
>> -		simple-audio-card,dai-link@1 {
>> -			reg = <1>;
>> -			/* DAC */
>> -			format = "i2s";
>> -			mclk-fs = <256>;
>> -			frame-inversion;
>> -			cpu {
>> -				sound-dai = <&sti_uni_player2>;
>> -			};
>> -
>> -			codec {
>> -				sound-dai = <&sti_sasg_codec 1>;
>> -			};
>> -		};
>> -
>> -		simple-audio-card,dai-link@2 {
>> -			reg = <2>;
>> -			/* SPDIF */
>> -			format = "left_j";
>> -			mclk-fs = <128>;
>> -			cpu {
>> -				sound-dai = <&sti_uni_player3>;
>> -			};
>> -
>> -			codec {
>> -				sound-dai = <&sti_sasg_codec 0>;
>> -			};
>> -		};
>> -	};
>> -
>> -	miphy28lp_phy: miphy28lp {
>> -
>> -		phy_port0: port@9b22000 {
>> -			st,osc-rdy;
>> -		};
>> -
>> -		phy_port1: port@9b2a000 {
>> -			st,osc-force-ext;
>> -		};
>> -	};
>> -
>> -	soc {
>> -		sbc_serial0: serial@9530000 {
>> -			status = "okay";
>> -		};
>> -
>> -		pwm0: pwm@9810000 {
>> -			status = "okay";
>> -		};
>> -
>> -		pwm1: pwm@9510000 {
>> -			status = "okay";
>> -		};
>> -
>> -		ssc2: i2c@9842000 {
>> -			status = "okay";
>> -			clock-frequency = <100000>;
>> -			st,i2c-min-scl-pulse-width-us = <0>;
>> -			st,i2c-min-sda-pulse-width-us = <5>;
>> -		};
>> -
>> -		ssc3: i2c@9843000 {
>> -			status = "okay";
>> -			clock-frequency = <100000>;
>> -			st,i2c-min-scl-pulse-width-us = <0>;
>> -			st,i2c-min-sda-pulse-width-us = <5>;
>> -		};
>> -
>> -		i2c@9844000 {
>> -			status = "okay";
>> -		};
>> -
>> -		i2c@9845000 {
>> -			status = "okay";
>> -		};
>> -
>> -		i2c@9540000 {
>> -			status = "okay";
>> -		};
>> -
>> -		mmc0: sdhci@9060000 {
>> -			non-removable;
>> -			status = "okay";
>> -		};
>> -
>> -		mmc1: sdhci@9080000 {
>> -			status = "okay";
>> -		};
>> -
>> -		/* SSC11 to HDMI */
>> -		hdmiddc: i2c@9541000 {
>> -			status = "okay";
>> -			/* HDMI V1.3a supports Standard mode only */
>> -			clock-frequency = <100000>;
>> -			st,i2c-min-scl-pulse-width-us = <0>;
>> -			st,i2c-min-sda-pulse-width-us = <5>;
>> -		};
>> -
>> -		st_dwc3: dwc3@8f94000 {
>> -			status = "okay";
>> -		};
>> -
>> -		ethernet0: dwmac@9630000 {
>> -			st,tx-retime-src = "clkgen";
>> -			status = "okay";
>> -			phy-mode = "rgmii";
>> -			fixed-link = <0 1 1000 0 0>;
>> -		};
>> -
>> -		demux@8a20000 {
>> -			compatible	= "st,stih407-c8sectpfe";
>> -			status		= "okay";
>> -			reg		= <0x08a20000 0x10000>,
>> -					  <0x08a00000 0x4000>;
>> -			reg-names	= "c8sectpfe", "c8sectpfe-ram";
>> -			interrupts	= <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>,
>> -					  <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
>> -			interrupt-names	= "c8sectpfe-error-irq",
>> -					  "c8sectpfe-idle-irq";
>> -			pinctrl-0	= <&pinctrl_tsin0_serial>;
>> -			pinctrl-1	= <&pinctrl_tsin0_parallel>;
>> -			pinctrl-2	= <&pinctrl_tsin3_serial>;
>> -			pinctrl-3	= <&pinctrl_tsin4_serial_alt3>;
>> -			pinctrl-4	= <&pinctrl_tsin5_serial_alt1>;
>> -			pinctrl-names	= "tsin0-serial",
>> -					  "tsin0-parallel",
>> -					  "tsin3-serial",
>> -					  "tsin4-serial",
>> -					  "tsin5-serial";
>> -			clocks		= <&clk_s_c0_flexgen CLK_PROC_STFE>;
>> -			clock-names	= "c8sectpfe";
>> -
>> -			/* tsin0 is TSA on NIMA */
>> -			tsin0: port {
>> -				tsin-num = <0>;
>> -				serial-not-parallel;
>> -				i2c-bus = <&ssc2>;
>> -				reset-gpios = <&pio15 4 GPIO_ACTIVE_LOW>;
>> -				dvb-card = <STV0367_TDA18212_NIMA_1>;
>> -			};
>> -		};
>> -
>> -		sti_uni_player0: sti-uni-player@8d80000 {
>> -			status = "okay";
>> -		};
>> -
>> -		sti_uni_player2: sti-uni-player@8d82000 {
>> -			status = "okay";
>> -		};
>> -
>> -		sti_uni_player3: sti-uni-player@8d85000 {
>> -			status = "okay";
>> -		};
>> -
>> -		syscfg_core: core-syscfg@92b0000 {
>> -			sti_sasg_codec: sti-sasg-codec {
>> -				status = "okay";
>> -				pinctrl-names = "default";
>> -				pinctrl-0 = <&pinctrl_spdif_out>;
>> -			};
>> -		};
>> -	};
>> -};
>>
> 
> 
> Hi Rahael
> 
> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
> 
> Thanks


Applied to sti-next

Thanks
Patrice


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 2/3] ARM: sti: removal of stih415/stih416 related entries
  2025-07-28  9:25   ` Patrice CHOTARD
@ 2025-08-18 14:02     ` Patrice CHOTARD
  0 siblings, 0 replies; 10+ messages in thread
From: Patrice CHOTARD @ 2025-08-18 14:02 UTC (permalink / raw)
  To: Raphael Gallais-Pou, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel, Alain Volmat



On 7/28/25 11:25, Patrice CHOTARD wrote:
> 
> 
> On 7/14/25 15:49, Raphael Gallais-Pou wrote:
>> From: Alain Volmat <avolmat@me.com>
>>
>> ST's STiH415 and STiH416 platforms have already been removed since
>> a while.  Remove some remaining bits within the mach-sti.
>>
>> Signed-off-by: Alain Volmat <avolmat@me.com>
>> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
>> ---
>>  arch/arm/mach-sti/Kconfig    | 20 +-------------------
>>  arch/arm/mach-sti/board-dt.c |  2 --
>>  2 files changed, 1 insertion(+), 21 deletions(-)
>>
>> diff --git a/arch/arm/mach-sti/Kconfig b/arch/arm/mach-sti/Kconfig
>> index b3842c971d31b80edaf88ab907f4377bea0a2af5..e58699e13e1a55ce46e68908c7ef51e18b040dc9 100644
>> --- a/arch/arm/mach-sti/Kconfig
>> +++ b/arch/arm/mach-sti/Kconfig
>> @@ -19,31 +19,13 @@ menuconfig ARCH_STI
>>  	select PL310_ERRATA_769419 if CACHE_L2X0
>>  	select RESET_CONTROLLER
>>  	help
>> -	  Include support for STMicroelectronics' STiH415/416, STiH407/10 and
>> +	  Include support for STMicroelectronics' STiH407/10 and
>>  	  STiH418 family SoCs using the Device Tree for discovery.  More
>>  	  information can be found in Documentation/arch/arm/sti/ and
>>  	  Documentation/devicetree.
>>  
>>  if ARCH_STI
>>  
>> -config SOC_STIH415
>> -	bool "STiH415 STMicroelectronics Consumer Electronics family"
>> -	default y
>> -	help
>> -	  This enables support for STMicroelectronics Digital Consumer
>> -	  Electronics family StiH415 parts, primarily targeted at set-top-box
>> -	  and other digital audio/video applications using Flattned Device
>> -	  Trees.
>> -
>> -config SOC_STIH416
>> -	bool "STiH416 STMicroelectronics Consumer Electronics family"
>> -	default y
>> -	help
>> -	  This enables support for STMicroelectronics Digital Consumer
>> -	  Electronics family StiH416 parts, primarily targeted at set-top-box
>> -	  and other digital audio/video applications using Flattened Device
>> -	  Trees.
>> -
>>  config SOC_STIH407
>>  	bool "STiH407 STMicroelectronics Consumer Electronics family"
>>  	default y
>> diff --git a/arch/arm/mach-sti/board-dt.c b/arch/arm/mach-sti/board-dt.c
>> index 488084b61b4acafb569ee9c51f5769393d55a9ce..1aaf61184685d754de57b487aef9a6b45a759b23 100644
>> --- a/arch/arm/mach-sti/board-dt.c
>> +++ b/arch/arm/mach-sti/board-dt.c
>> @@ -10,8 +10,6 @@
>>  #include "smp.h"
>>  
>>  static const char *const stih41x_dt_match[] __initconst = {
>> -	"st,stih415",
>> -	"st,stih416",
>>  	"st,stih407",
>>  	"st,stih410",
>>  	"st,stih418",
>>
> 
> 
> Hi Raphael
> 
> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
> 
> Thanks
Applied to sti-next

Thanks
Patrice


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v2 3/3] dt-bindings: arm: sti: drop B2120 board support
  2025-07-28  9:26   ` Patrice CHOTARD
@ 2025-08-18 14:02     ` Patrice CHOTARD
  0 siblings, 0 replies; 10+ messages in thread
From: Patrice CHOTARD @ 2025-08-18 14:02 UTC (permalink / raw)
  To: Raphael Gallais-Pou, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Russell King
  Cc: devicetree, linux-kernel, linux-arm-kernel, Krzysztof Kozlowski



On 7/28/25 11:26, Patrice CHOTARD wrote:
> 
> 
> On 7/14/25 15:49, Raphael Gallais-Pou wrote:
>> B2120 boards are internal boards which never were commercialised.
>>
>> Remove them from bindings.
>>
>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
>>
>> ---
>> Changes in v2:
>> - Added Krzysztof's acked-by
>> - Put this patch last to get a bisectable series
>> ---
>>  Documentation/devicetree/bindings/arm/sti.yaml | 4 ----
>>  1 file changed, 4 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/sti.yaml b/Documentation/devicetree/bindings/arm/sti.yaml
>> index 842def3e3f2bce470763d3665c7603b9058b1e4e..177358895fe1c9f80f8c825142cf015d04286ce8 100644
>> --- a/Documentation/devicetree/bindings/arm/sti.yaml
>> +++ b/Documentation/devicetree/bindings/arm/sti.yaml
>> @@ -14,12 +14,8 @@ properties:
>>      const: '/'
>>    compatible:
>>      oneOf:
>> -      - items:
>> -          - const: st,stih407-b2120
>> -          - const: st,stih407
>>        - items:
>>            - enum:
>> -              - st,stih410-b2120
>>                - st,stih410-b2260
>>            - const: st,stih410
>>        - items:
>>
> 
> 
> Hi Raphael
> 
> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
> 
> Thanks
Applied to sti-next

Thanks
Patrice


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2025-08-18 14:15 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-14 13:49 [PATCH v2 0/3] STi device-tree cleanup Raphael Gallais-Pou
2025-07-14 13:49 ` [PATCH v2 1/3] ARM: sti: drop B2120 board support Raphael Gallais-Pou
2025-07-28  9:25   ` Patrice CHOTARD
2025-08-18 14:02     ` Patrice CHOTARD
2025-07-14 13:49 ` [PATCH v2 2/3] ARM: sti: removal of stih415/stih416 related entries Raphael Gallais-Pou
2025-07-28  9:25   ` Patrice CHOTARD
2025-08-18 14:02     ` Patrice CHOTARD
2025-07-14 13:49 ` [PATCH v2 3/3] dt-bindings: arm: sti: drop B2120 board support Raphael Gallais-Pou
2025-07-28  9:26   ` Patrice CHOTARD
2025-08-18 14:02     ` Patrice CHOTARD

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).