lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <512c4b4c-8db6-45e5-ad11-82f341d40962@foss.st.com>
Date: Mon, 18 Aug 2025 16:02:35 +0200
From: Patrice CHOTARD <patrice.chotard@...s.st.com>
To: Raphael Gallais-Pou <rgallaispou@...il.com>,
        Rob Herring
	<robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley
	<conor+dt@...nel.org>,
        Russell King <linux@...linux.org.uk>
CC: <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v2 1/3] ARM: sti: drop B2120 board support



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@...il.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@...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@...00000 {
>> -		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@...aro.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@...1000 {
>> -				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@...8000 {
>> -				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@...4000 {
>> -				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@...2000 {
>> -				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@...aro.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@...00000 {
>> -		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@...0000 {
>> -			max-frequency = <200000000>;
>> -			sd-uhs-sdr50;
>> -			sd-uhs-sdr104;
>> -			sd-uhs-ddr50;
>> -		};
>> -
>> -		ohci0: usb@...3c00 {
>> -			status = "okay";
>> -		};
>> -
>> -		ehci0: usb@...3e00 {
>> -			status = "okay";
>> -		};
>> -
>> -		ohci1: usb@...3c00 {
>> -			status = "okay";
>> -		};
>> -
>> -		ehci1: usb@...3e00 {
>> -			status = "okay";
>> -		};
>> -
>> -		sti-display-subsystem@0 {
>> -			sti-hda@...2000 {
>> -				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@...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@...2000 {
>> -			st,osc-rdy;
>> -		};
>> -
>> -		phy_port1: port@...a000 {
>> -			st,osc-force-ext;
>> -		};
>> -	};
>> -
>> -	soc {
>> -		sbc_serial0: serial@...0000 {
>> -			status = "okay";
>> -		};
>> -
>> -		pwm0: pwm@...0000 {
>> -			status = "okay";
>> -		};
>> -
>> -		pwm1: pwm@...0000 {
>> -			status = "okay";
>> -		};
>> -
>> -		ssc2: i2c@...2000 {
>> -			status = "okay";
>> -			clock-frequency = <100000>;
>> -			st,i2c-min-scl-pulse-width-us = <0>;
>> -			st,i2c-min-sda-pulse-width-us = <5>;
>> -		};
>> -
>> -		ssc3: i2c@...3000 {
>> -			status = "okay";
>> -			clock-frequency = <100000>;
>> -			st,i2c-min-scl-pulse-width-us = <0>;
>> -			st,i2c-min-sda-pulse-width-us = <5>;
>> -		};
>> -
>> -		i2c@...4000 {
>> -			status = "okay";
>> -		};
>> -
>> -		i2c@...5000 {
>> -			status = "okay";
>> -		};
>> -
>> -		i2c@...0000 {
>> -			status = "okay";
>> -		};
>> -
>> -		mmc0: sdhci@...0000 {
>> -			non-removable;
>> -			status = "okay";
>> -		};
>> -
>> -		mmc1: sdhci@...0000 {
>> -			status = "okay";
>> -		};
>> -
>> -		/* SSC11 to HDMI */
>> -		hdmiddc: i2c@...1000 {
>> -			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@...4000 {
>> -			status = "okay";
>> -		};
>> -
>> -		ethernet0: dwmac@...0000 {
>> -			st,tx-retime-src = "clkgen";
>> -			status = "okay";
>> -			phy-mode = "rgmii";
>> -			fixed-link = <0 1 1000 0 0>;
>> -		};
>> -
>> -		demux@...0000 {
>> -			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@...0000 {
>> -			status = "okay";
>> -		};
>> -
>> -		sti_uni_player2: sti-uni-player@...2000 {
>> -			status = "okay";
>> -		};
>> -
>> -		sti_uni_player3: sti-uni-player@...5000 {
>> -			status = "okay";
>> -		};
>> -
>> -		syscfg_core: core-syscfg@...0000 {
>> -			sti_sasg_codec: sti-sasg-codec {
>> -				status = "okay";
>> -				pinctrl-names = "default";
>> -				pinctrl-0 = <&pinctrl_spdif_out>;
>> -			};
>> -		};
>> -	};
>> -};
>>
> 
> 
> Hi Rahael
> 
> Reviewed-by: Patrice Chotard <patrice.chotard@...s.st.com>
> 
> Thanks


Applied to sti-next

Thanks
Patrice

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ