Re: [PATCH v2 3/5] dt-bindings: media: remove support of stih407-c8sectpfe

Patrice CHOTARD
2025-09-12T14:31:33Z

On 9/12/25 13:36, Raphael Gallais-Pou wrote:
> Remove files documentation from stih407-c8sectpfe driver.
>
> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>
> ---
> .../admin-guide/media/platform-cardlist.rst | 2 -
> .../bindings/media/stih407-c8sectpfe.txt | 88 ----------------------
> 2 files changed, 90 deletions(-)
>
> diff --git a/Documentation/admin-guide/media/platform-cardlist.rst b/Documentation/admin-guide/media/platform-cardlist.rst
> index 1230ae4037ad551087d4cddc8a02eab5eac2be71..63f4b19c3628f3488fd2ccd1a6dab385d46b2503 100644
> --- a/Documentation/admin-guide/media/platform-cardlist.rst
> +++ b/Documentation/admin-guide/media/platform-cardlist.rst
> @@ -18,8 +18,6 @@ am437x-vpfe TI AM437x VPFE
> aspeed-video Aspeed AST2400 and AST2500
> atmel-isc ATMEL Image Sensor Controller (ISC)
> atmel-isi ATMEL Image Sensor Interface (ISI)
> -c8sectpfe SDR platform devices
> -c8sectpfe SDR platform devices
> cafe_ccic Marvell 88ALP01 (Cafe) CMOS Camera Controller
> cdns-csi2rx Cadence MIPI-CSI2 RX Controller
> cdns-csi2tx Cadence MIPI-CSI2 TX Controller
> diff --git a/Documentation/devicetree/bindings/media/stih407-c8sectpfe.txt b/Documentation/devicetree/bindings/media/stih407-c8sectpfe.txt
> deleted file mode 100644
> index 880d4d70c9fd741ac13101721ced18f04336c373..0000000000000000000000000000000000000000
> --- a/Documentation/devicetree/bindings/media/stih407-c8sectpfe.txt
> +++ /dev/null
> @@ -1,88 +0,0 @@
> -STMicroelectronics STi c8sectpfe binding
> -============================================
> -
> -This document describes the c8sectpfe device bindings that is used to get transport
> -stream data into the SoC on the TS pins, and into DDR for further processing.
> -
> -It is typically used in conjunction with one or more demodulator and tuner devices
> -which converts from the RF to digital domain. Demodulators and tuners are usually
> -located on an external DVB frontend card connected to SoC TS input pins.
> -
> -Currently 7 TS input (tsin) channels are supported on the stih407 family SoC.
> -
> -Required properties (controller (parent) node):
> -- compatible : Should be "stih407-c8sectpfe"
> -
> -- reg : Address and length of register sets for each device in
> - "reg-names"
> -
> -- reg-names : The names of the register addresses corresponding to the
> - registers filled in "reg":
> - - c8sectpfe: c8sectpfe registers
> - - c8sectpfe-ram: c8sectpfe internal sram
> -
> -- clocks : phandle list of c8sectpfe clocks
> -- clock-names : should be "c8sectpfe"
> -See: Documentation/devicetree/bindings/clock/clock-bindings.txt
> -
> -- pinctrl-names : a pinctrl state named tsin%d-serial or tsin%d-parallel (where %d is tsin-num)
> - must be defined for each tsin child node.
> -- pinctrl-0 : phandle referencing pin configuration for this tsin configuration
> -See: Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt
> -
> -
> -Required properties (tsin (child) node):
> -
> -- tsin-num : tsin id of the InputBlock (must be between 0 to 6)
> -- i2c-bus : phandle to the I2C bus DT node which the demodulators & tuners on this tsin channel are connected.
> -- reset-gpios : reset gpio for this tsin channel.
> -
> -Optional properties (tsin (child) node):
> -
> -- invert-ts-clk : Bool property to control sense of ts input clock (data stored on falling edge of clk).
> -- serial-not-parallel : Bool property to configure input bus width (serial on ts_data<7>).
> -- async-not-sync : Bool property to control if data is received in asynchronous mode
> - (all bits/bytes with ts_valid or ts_packet asserted are valid).
> -
> -- dvb-card : Describes the NIM card connected to this tsin channel.
> -
> -Example:
> -
> -/* stih410 SoC b2120 + b2004a + stv0367-pll(NIMB) + stv0367-tda18212 (NIMA) DT example) */
> -
> - c8sectpfe@8a20000 {
> - compatible = "st,stih407-c8sectpfe";
> - reg = <0x08a20000 0x10000>, <0x08a00000 0x4000>;
> - reg-names = "stfe", "stfe-ram";
> - interrupts = <GIC_SPI 34 IRQ_TYPE_NONE>, <GIC_SPI 35 IRQ_TYPE_NONE>;
> - interrupt-names = "stfe-error-irq", "stfe-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@0 {
> - tsin-num = <0>;
> - serial-not-parallel;
> - i2c-bus = <&ssc2>;
> - reset-gpios = <&pio15 4 GPIO_ACTIVE_HIGH>;
> - dvb-card = <STV0367_TDA18212_NIMA_1>;
> - };
> -
> - tsin3: port@3 {
> - tsin-num = <3>;
> - serial-not-parallel;
> - i2c-bus = <&ssc3>;
> - reset-gpios = <&pio15 7 GPIO_ACTIVE_HIGH>;
> - dvb-card = <STV0367_TDA18212_NIMB_1>;
> - };
> - };
>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks
Patrice