Skip to contentSkip to conversation

PCI: dwc: Remove not-going-to-be-supported code for Baikal SoC

PATCH Applied Andy Shevchenko 4 participants 8 messages Feb 20 – Mar 2

Subsystems (3)

v2 of 4 Applied Latest is v4 →

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 PCI: dwc: Remove not-going-to-be-supported code for Baikal SoC

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

---

v2: removed YAML file

 .../bindings/pci/baikal,bt1-pcie.yaml         | 168 -----
 drivers/pci/controller/dwc/Kconfig            |   9 -
 drivers/pci/controller/dwc/Makefile           |   1 -
 drivers/pci/controller/dwc/pcie-bt1.c         | 645 ------------------
 4 files changed, 823 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/pci/baikal,bt1-pcie.yaml
 delete mode 100644 drivers/pci/controller/dwc/pcie-bt1.c
Documentation/devicetree/bindings/pci/baikal,bt1-pcie.yaml deleted +0 -168
deleted file mode 100644
index 8eaa07ae9774..000000000000
@@ -1,168 +0,0 @@
-# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
-%YAML 1.2
----
-$id: http://devicetree.org/schemas/pci/baikal,bt1-pcie.yaml#
-$schema: http://devicetree.org/meta-schemas/core.yaml#
-
-title: Baikal-T1 PCIe Root Port Controller
-
-maintainers:
-  - Serge Semin <[email protected]>
-
-description:
-  Embedded into Baikal-T1 SoC Root Complex controller with a single port
-  activated. It's based on the DWC RC PCIe v4.60a IP-core, which is configured
-  to have just a single Root Port function and is capable of establishing the
-  link up to Gen.3 speed on x4 lanes. It doesn't have embedded clock and reset
-  control module, so the proper interface initialization is supposed to be
-  performed by software. There four in- and four outbound iATU regions
-  which can be used to emit all required TLP types on the PCIe bus.
-
-allOf:
-  - $ref: /schemas/pci/snps,dw-pcie.yaml#
-
-properties:
-  compatible:
-    const: baikal,bt1-pcie
-
-  reg:
-    description:
-      DBI, DBI2 and at least 4KB outbound iATU-capable region for the
-      peripheral devices CFG-space access.
-    maxItems: 3
-
-  reg-names:
-    items:
-      - const: dbi
-      - const: dbi2
-      - const: config
-
-  interrupts:
-    description:
-      MSI, AER, PME, Hot-plug, Link Bandwidth Management, Link Equalization
-      request and eight Read/Write eDMA IRQ lines are available.
-    maxItems: 14
-
-  interrupt-names:
-    items:
-      - const: dma0
-      - const: dma1
-      - const: dma2
-      - const: dma3
-      - const: dma4
-      - const: dma5
-      - const: dma6
-      - const: dma7
-      - const: msi
-      - const: aer
-      - const: pme
-      - const: hp
-      - const: bw_mg
-      - const: l_eq
-
-  clocks:
-    description:
-      DBI (attached to the APB bus), AXI-bus master and slave interfaces
-      are fed up by the dedicated application clocks. A common reference
-      clock signal is supposed to be attached to the corresponding Ref-pad
-      of the SoC. It will be redistributed amongst the controller core
-      sub-modules (pipe, core, aux, etc).
-    maxItems: 4
-
-  clock-names:
-    items:
-      - const: dbi
-      - const: mstr
-      - const: slv
-      - const: ref
-
-  resets:
-    description:
-      A comprehensive controller reset logic is supposed to be implemented
-      by software, so almost all the possible application and core reset
-      signals are exposed via the system CCU module.
-    maxItems: 9
-
-  reset-names:
-    items:
-      - const: mstr
-      - const: slv
-      - const: pwr
-      - const: hot
-      - const: phy
-      - const: core
-      - const: pipe
-      - const: sticky
-      - const: non-sticky
-
-  baikal,bt1-syscon:
-    $ref: /schemas/types.yaml#/definitions/phandle
-    description:
-      Phandle to the Baikal-T1 System Controller DT node. It's required to
-      access some additional PM, Reset-related and LTSSM signals.
-
-  num-lanes:
-    maximum: 4
-
-  max-link-speed:
-    maximum: 3
-
-required:
-  - compatible
-  - reg
-  - reg-names
-  - interrupts
-  - interrupt-names
-
-unevaluatedProperties: false
-
-examples:
-  - |
-    #include <dt-bindings/interrupt-controller/mips-gic.h>
-    #include <dt-bindings/gpio/gpio.h>
-
-    pcie@1f052000 {
-      compatible = "baikal,bt1-pcie";
-      device_type = "pci";
-      reg = <0x1f052000 0x1000>, <0x1f053000 0x1000>, <0x1bdbf000 0x1000>;
-      reg-names = "dbi", "dbi2", "config";
-      #address-cells = <3>;
-      #size-cells = <2>;
-      ranges = <0x81000000 0 0x00000000 0x1bdb0000 0 0x00008000>,
-               <0x82000000 0 0x20000000 0x08000000 0 0x13db0000>;
-      bus-range = <0x0 0xff>;
-
-      interrupts = <GIC_SHARED 80 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 81 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 82 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 83 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 84 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 85 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 86 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 87 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 88 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 89 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 90 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 91 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 92 IRQ_TYPE_LEVEL_HIGH>,
-                   <GIC_SHARED 93 IRQ_TYPE_LEVEL_HIGH>;
-      interrupt-names = "dma0", "dma1", "dma2", "dma3",
-                        "dma4", "dma5", "dma6", "dma7",
-                        "msi", "aer", "pme", "hp", "bw_mg",
-                        "l_eq";
-
-      clocks = <&ccu_sys 1>, <&ccu_axi 6>, <&ccu_axi 7>, <&clk_pcie>;
-      clock-names = "dbi", "mstr", "slv", "ref";
-
-      resets = <&ccu_axi 6>, <&ccu_axi 7>, <&ccu_sys 7>, <&ccu_sys 10>,
-               <&ccu_sys 4>, <&ccu_sys 6>, <&ccu_sys 5>, <&ccu_sys 8>,
-               <&ccu_sys 9>;
-      reset-names = "mstr", "slv", "pwr", "hot", "phy", "core", "pipe",
-                    "sticky", "non-sticky";
-
-      reset-gpios = <&port0 0 GPIO_ACTIVE_LOW>;
-
-      num-lanes = <4>;
-      max-link-speed = <3>;
-    };
-...
drivers/pci/controller/dwc/Kconfig +0 -9
index 6c7e726d1909..3c1136a92e88 100644
@@ -84,15 +84,6 @@ config PCIE_ARTPEC6_EP
 	  Enables support for the PCIe controller in the ARTPEC-6 SoC to work in
 	  endpoint mode. This uses the DesignWare core.
 
-config PCIE_BT1
-	tristate "Baikal-T1 PCIe controller"
-	depends on MIPS_BAIKAL_T1 || COMPILE_TEST
-	depends on PCI_MSI
-	select PCIE_DW_HOST
-	help
-	  Enables support for the PCIe controller in the Baikal-T1 SoC to work
-	  in host mode. It's based on the Synopsys DWC PCIe v4.60a IP-core.
-
 config PCI_IMX6
 	bool
 
drivers/pci/controller/dwc/Makefile +0 -1
index 67ba59c02038..4867f30ab7c3 100644
@@ -5,7 +5,6 @@ obj-$(CONFIG_PCIE_DW_HOST) += pcie-designware-host.o
 obj-$(CONFIG_PCIE_DW_EP) += pcie-designware-ep.o
 obj-$(CONFIG_PCIE_DW_PLAT) += pcie-designware-plat.o
 obj-$(CONFIG_PCIE_AMD_MDB) += pcie-amd-mdb.o
-obj-$(CONFIG_PCIE_BT1) += pcie-bt1.o
 obj-$(CONFIG_PCI_DRA7XX) += pci-dra7xx.o
 obj-$(CONFIG_PCI_EXYNOS) += pci-exynos.o
 obj-$(CONFIG_PCIE_FU740) += pcie-fu740.o
drivers/pci/controller/dwc/pcie-bt1.c deleted +0 -645

645 lines not on this page · raw message

Signature · 1 line
2.50.1
maintainer Acked-by on v2 1/1
#1 lore raw
Quoted 9 lines · Andy Shevchenko

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

---

v2: removed YAML file

Quoted 8 lines
 .../bindings/pci/baikal,bt1-pcie.yaml         | 168 -----
 drivers/pci/controller/dwc/Kconfig            |   9 -
 drivers/pci/controller/dwc/Makefile           |   1 -
 drivers/pci/controller/dwc/pcie-bt1.c         | 645 ------------------
 4 files changed, 823 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/pci/baikal,bt1-pcie.yaml
 delete mode 100644 drivers/pci/controller/dwc/pcie-bt1.c
#2 lore raw
Quoted 5 lines · Rob Herring (Arm)
Quoted 2 lines · Andy Shevchenko

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

Thanks, Rob!

Any chances to see this in PCI tree?

Signature · 2 lines
With Best Regards,
Andy Shevchenko
#3 lore raw
In reply to Andy Shevchenko

+ Arnd

Quoted 5 lines · Andy Shevchenko

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

I see Arnd's comment in the above thread:

"I don't think we'll merge the rest of Baikal, we should probably remove the entire platform."

But is there more to the story?

- Mani

Signature · 1 line
மணிவண்ணன் சதாசிவம்
#4 lore raw
Quoted 7 lines · Manivannan Sadhasivam
Quoted 5 lines · Andy Shevchenko

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

Quoted 6 lines

I see Arnd's comment in the above thread:

"I don't think we'll merge the rest of Baikal, we should probably remove the entire platform."

But is there more to the story?

The Baikal enterprise is Russian entity under OFAC sanctions.

Signature · 2 lines
With Best Regards,
Andy Shevchenko
#5 lore raw
Quoted 8 lines · Andy Shevchenko
Quoted 7 lines · Manivannan Sadhasivam
Quoted 5 lines · Andy Shevchenko

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

Quoted 9 lines
Quoted 6 lines

I see Arnd's comment in the above thread:

"I don't think we'll merge the rest of Baikal, we should probably remove the entire platform."

But is there more to the story?

The Baikal enterprise is Russian entity under OFAC sanctions.

Yeah, I knew that part, but didn't know that we decided to remove the platform altogether. But I can see patches floating around for other subsystems and getting applied, so I will do that same for these PCI host controller patches. Thanks!

- Mani

Signature · 1 line
மணிவண்ணன் சதாசிவம்
#6 lore raw
Quoted 11 lines · Manivannan Sadhasivam
Quoted 10 lines · Andy Shevchenko
Quoted 9 lines · Manivannan Sadhasivam
Quoted 5 lines · Andy Shevchenko

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

I see Arnd's comment in the above thread:

Quoted 8 lines
Quoted 6 lines
Quoted 4 lines

"I don't think we'll merge the rest of Baikal, we should probably remove the entire platform."

But is there more to the story?

The Baikal enterprise is Russian entity under OFAC sanctions.

Yeah, I knew that part,

Then I can't tell you anything new :-)

Quoted 1 line

but didn't know that we decided to remove the platform altogether.

It was never fully added to begin with. That's why Arnd's reply is the above. It's half-baked and hence a dead (stale) code altogether. Bulk which would only waste our resources for supporting unused code.

Quoted 2 lines

But I can see patches floating around for other subsystems and getting applied, so I will do that same for these PCI host controller patches.

Thank you!

Signature · 2 lines
With Best Regards,
Andy Shevchenko
#7 lore raw
In reply to Andy Shevchenko
Quoted 4 lines · Andy Shevchenko

As noticed in the discussion [1] the Baikal SoC and platforms are not going to be finalized, hence remove stale code.

Applied, thanks!

[1/1] PCI: dwc: Remove not-going-to-be-supported code for Baikal SoC

      commit: 5e5ea39ff55297fc9d6338f26346c2a7738a78ea

Best regards,

Signature · 1 line
Manivannan Sadhasivam <[email protected]>