* [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 = ðernet0;
- };
-
-};
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 = ðernet0;
- };
-
- 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 = ðernet0;
> - };
> -
> -};
> 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 = ðernet0;
> - };
> -
> - 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 = ðernet0;
>> - };
>> -
>> -};
>> 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 = ðernet0;
>> - };
>> -
>> - 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).