diff mbox series

dt-bindings: PCI: hisi: Delete the useless HiSilicon PCIe file

Message ID 1616842062-21823-1-git-send-email-liudongdong3@huawei.com (mailing list archive)
State Accepted
Delegated to: Bjorn Helgaas
Headers show
Series dt-bindings: PCI: hisi: Delete the useless HiSilicon PCIe file | expand

Commit Message

Dongdong Liu March 27, 2021, 10:47 a.m. UTC
The hisilicon-pcie.txt file is no longer useful since commit
c2fa6cf76d20 (PCI: dwc: hisi: Remove non-ECAM HiSilicon
hip05/hip06 driver), so delete it.

Signed-off-by: Dongdong Liu <liudongdong3@huawei.com>
---
 .../devicetree/bindings/pci/hisilicon-pcie.txt     | 43 ----------------------
 1 file changed, 43 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/pci/hisilicon-pcie.txt

Comments

Krzysztof Wilczyński March 27, 2021, 11:32 a.m. UTC | #1
Hi,

Such a strong word "useless" in the subject.  I would think that the
file became "obsolete". :-) (N.B. nothing to fix, though).

> The hisilicon-pcie.txt file is no longer useful since commit
> c2fa6cf76d20 (PCI: dwc: hisi: Remove non-ECAM HiSilicon
> hip05/hip06 driver), so delete it.

Thank you for cleaning things up!

Reviewed-by: Krzysztof Wilczyński <kw@linux.com>

Krzysztof
Zhou Wang March 29, 2021, 1:23 a.m. UTC | #2
On 2021/3/27 18:47, Dongdong Liu wrote:
> The hisilicon-pcie.txt file is no longer useful since commit
> c2fa6cf76d20 (PCI: dwc: hisi: Remove non-ECAM HiSilicon
> hip05/hip06 driver), so delete it.

No, it is no needed now. Thanks for removing this, and please
also remove related code in MAINTAINERS file :)

Best,
Zhou

> 
> Signed-off-by: Dongdong Liu <liudongdong3@huawei.com>
> ---
>  .../devicetree/bindings/pci/hisilicon-pcie.txt     | 43 ----------------------
>  1 file changed, 43 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
> 
> diff --git a/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt b/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
> deleted file mode 100644
> index d6796ef..0000000
> --- a/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
> +++ /dev/null
> @@ -1,43 +0,0 @@
> -HiSilicon Hip05 and Hip06 PCIe host bridge DT description
> -
> -HiSilicon PCIe host controller is based on the Synopsys DesignWare PCI core.
> -It shares common functions with the PCIe DesignWare core driver and inherits
> -common properties defined in
> -Documentation/devicetree/bindings/pci/designware-pcie.txt.
> -
> -Additional properties are described here:
> -
> -Required properties
> -- compatible: Should contain "hisilicon,hip05-pcie" or "hisilicon,hip06-pcie".
> -- reg: Should contain rc_dbi, config registers location and length.
> -- reg-names: Must include the following entries:
> -  "rc_dbi": controller configuration registers;
> -  "config": PCIe configuration space registers.
> -- msi-parent: Should be its_pcie which is an ITS receiving MSI interrupts.
> -- port-id: Should be 0, 1, 2 or 3.
> -
> -Optional properties:
> -- status: Either "ok" or "disabled".
> -- dma-coherent: Present if DMA operations are coherent.
> -
> -Hip05 Example (note that Hip06 is the same except compatible):
> -	pcie@b0080000 {
> -		compatible = "hisilicon,hip05-pcie", "snps,dw-pcie";
> -		reg = <0 0xb0080000 0 0x10000>, <0x220 0x00000000 0 0x2000>;
> -		reg-names = "rc_dbi", "config";
> -		bus-range = <0  15>;
> -		msi-parent = <&its_pcie>;
> -		#address-cells = <3>;
> -		#size-cells = <2>;
> -		device_type = "pci";
> -		dma-coherent;
> -		ranges = <0x82000000 0 0x00000000 0x220 0x00000000 0 0x10000000>;
> -		num-lanes = <8>;
> -		port-id = <1>;
> -		#interrupt-cells = <1>;
> -		interrupt-map-mask = <0xf800 0 0 7>;
> -		interrupt-map = <0x0 0 0 1 &mbigen_pcie 1 10
> -				 0x0 0 0 2 &mbigen_pcie 2 11
> -				 0x0 0 0 3 &mbigen_pcie 3 12
> -				 0x0 0 0 4 &mbigen_pcie 4 13>;
> -	};
>
Dongdong Liu March 29, 2021, 6:07 a.m. UTC | #3
Hi Zhou

Mant thanks for your review
On 2021/3/29 9:23, Zhou Wang wrote:
> On 2021/3/27 18:47, Dongdong Liu wrote:
>> The hisilicon-pcie.txt file is no longer useful since commit
>> c2fa6cf76d20 (PCI: dwc: hisi: Remove non-ECAM HiSilicon
>> hip05/hip06 driver), so delete it.
>
> No, it is no needed now. Thanks for removing this, and please
> also remove related code in MAINTAINERS file :)
>
OK, Will do.

Thanks,
Dongdong
> Best,
> Zhou
>
>>
>> Signed-off-by: Dongdong Liu <liudongdong3@huawei.com>
>> ---
>>  .../devicetree/bindings/pci/hisilicon-pcie.txt     | 43 ----------------------
>>  1 file changed, 43 deletions(-)
>>  delete mode 100644 Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
>>
>> diff --git a/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt b/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
>> deleted file mode 100644
>> index d6796ef..0000000
>> --- a/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
>> +++ /dev/null
>> @@ -1,43 +0,0 @@
>> -HiSilicon Hip05 and Hip06 PCIe host bridge DT description
>> -
>> -HiSilicon PCIe host controller is based on the Synopsys DesignWare PCI core.
>> -It shares common functions with the PCIe DesignWare core driver and inherits
>> -common properties defined in
>> -Documentation/devicetree/bindings/pci/designware-pcie.txt.
>> -
>> -Additional properties are described here:
>> -
>> -Required properties
>> -- compatible: Should contain "hisilicon,hip05-pcie" or "hisilicon,hip06-pcie".
>> -- reg: Should contain rc_dbi, config registers location and length.
>> -- reg-names: Must include the following entries:
>> -  "rc_dbi": controller configuration registers;
>> -  "config": PCIe configuration space registers.
>> -- msi-parent: Should be its_pcie which is an ITS receiving MSI interrupts.
>> -- port-id: Should be 0, 1, 2 or 3.
>> -
>> -Optional properties:
>> -- status: Either "ok" or "disabled".
>> -- dma-coherent: Present if DMA operations are coherent.
>> -
>> -Hip05 Example (note that Hip06 is the same except compatible):
>> -	pcie@b0080000 {
>> -		compatible = "hisilicon,hip05-pcie", "snps,dw-pcie";
>> -		reg = <0 0xb0080000 0 0x10000>, <0x220 0x00000000 0 0x2000>;
>> -		reg-names = "rc_dbi", "config";
>> -		bus-range = <0  15>;
>> -		msi-parent = <&its_pcie>;
>> -		#address-cells = <3>;
>> -		#size-cells = <2>;
>> -		device_type = "pci";
>> -		dma-coherent;
>> -		ranges = <0x82000000 0 0x00000000 0x220 0x00000000 0 0x10000000>;
>> -		num-lanes = <8>;
>> -		port-id = <1>;
>> -		#interrupt-cells = <1>;
>> -		interrupt-map-mask = <0xf800 0 0 7>;
>> -		interrupt-map = <0x0 0 0 1 &mbigen_pcie 1 10
>> -				 0x0 0 0 2 &mbigen_pcie 2 11
>> -				 0x0 0 0 3 &mbigen_pcie 3 12
>> -				 0x0 0 0 4 &mbigen_pcie 4 13>;
>> -	};
>>
>
> .
>
Lorenzo Pieralisi March 30, 2021, 4:01 p.m. UTC | #4
On Sat, 27 Mar 2021 18:47:42 +0800, Dongdong Liu wrote:
> The hisilicon-pcie.txt file is no longer useful since commit
> c2fa6cf76d20 (PCI: dwc: hisi: Remove non-ECAM HiSilicon
> hip05/hip06 driver), so delete it.

Applied to pci/misc, thanks!

[1/1] dt-bindings: PCI: hisi: Delete the obsolete HiSilicon PCIe file
      https://git.kernel.org/lpieralisi/pci/c/756d4c369c

Thanks,
Lorenzo
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt b/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
deleted file mode 100644
index d6796ef..0000000
--- a/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt
+++ /dev/null
@@ -1,43 +0,0 @@ 
-HiSilicon Hip05 and Hip06 PCIe host bridge DT description
-
-HiSilicon PCIe host controller is based on the Synopsys DesignWare PCI core.
-It shares common functions with the PCIe DesignWare core driver and inherits
-common properties defined in
-Documentation/devicetree/bindings/pci/designware-pcie.txt.
-
-Additional properties are described here:
-
-Required properties
-- compatible: Should contain "hisilicon,hip05-pcie" or "hisilicon,hip06-pcie".
-- reg: Should contain rc_dbi, config registers location and length.
-- reg-names: Must include the following entries:
-  "rc_dbi": controller configuration registers;
-  "config": PCIe configuration space registers.
-- msi-parent: Should be its_pcie which is an ITS receiving MSI interrupts.
-- port-id: Should be 0, 1, 2 or 3.
-
-Optional properties:
-- status: Either "ok" or "disabled".
-- dma-coherent: Present if DMA operations are coherent.
-
-Hip05 Example (note that Hip06 is the same except compatible):
-	pcie@b0080000 {
-		compatible = "hisilicon,hip05-pcie", "snps,dw-pcie";
-		reg = <0 0xb0080000 0 0x10000>, <0x220 0x00000000 0 0x2000>;
-		reg-names = "rc_dbi", "config";
-		bus-range = <0  15>;
-		msi-parent = <&its_pcie>;
-		#address-cells = <3>;
-		#size-cells = <2>;
-		device_type = "pci";
-		dma-coherent;
-		ranges = <0x82000000 0 0x00000000 0x220 0x00000000 0 0x10000000>;
-		num-lanes = <8>;
-		port-id = <1>;
-		#interrupt-cells = <1>;
-		interrupt-map-mask = <0xf800 0 0 7>;
-		interrupt-map = <0x0 0 0 1 &mbigen_pcie 1 10
-				 0x0 0 0 2 &mbigen_pcie 2 11
-				 0x0 0 0 3 &mbigen_pcie 3 12
-				 0x0 0 0 4 &mbigen_pcie 4 13>;
-	};