Re: [PATCH 5/5] dt-bindings: PCI: add support for new generation controller

[Honghui Zhang]

On Mon, 2017-07-24 at 15:26 -0500, Rob Herring wrote:
> On Fri, Jul 21, 2017 at 10:34:49AM +0800, honghui.zh...@mediatek.com wrote:
> > From: Ryder Lee 
> > 
> > Add support for MediaTek new generation controller and update related
> > properities.
> > 
> > Signed-off-by: Ryder Lee 
> > Signed-off-by: Honghui Zhang 
> > ---
> >  .../devicetree/bindings/pci/mediatek-pcie.txt  | 84 
> > --
> >  1 file changed, 79 insertions(+), 5 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt 
> > b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> > index 294f4a3..a1f3767 100644
> > --- a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> > +++ b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> > @@ -4,17 +4,27 @@ Required properties:
> >  - compatible: Should contain one of the following string:
> > "mediatek,mt7623-pcie"
> > "mediatek,mt2701-pcie"
> > +   "mediatek,generic-pcie-v2"
> 
> No, please use SoC specific compatible strings.
> 
Thanks, I will rewrite it as below:
"mediatek,mt2701-pcie"
"mediatek,mt7623-pcie"
"mediatek,mt2712-pcie"
"mediatek,mt7622-pcie"

> >  - device_type: Must be "pci"
> > -- reg: Base addresses and lengths of the PCIe controller.
> > +- reg: Base addresses and lengths of the PICe subsys and root ports.
> > +- reg-names: Names of the above areas to use during resource look-up.
> >  - #address-cells: Address representation for root ports (must be 3)
> >  - #size-cells: Size representation for root ports (must be 2)
> >  - clocks: Must contain an entry for each entry in clock-names.
> >See ../clocks/clock-bindings.txt for details.
> >  - clock-names: Must include the following entries:
> >- free_ck :for reference clock of PCIe subsys
> > -  - sys_ck0 :for clock of Port0
> > -  - sys_ck1 :for clock of Port1
> > -  - sys_ck2 :for clock of Port2
> > +  - sys_ckN :transaction layer and data link layer clock
> > +  The "sys_ck" might be divided into the following parts in some v2 chips:
> 
> Please be specific about what clocks apply to which compatible. Each 
> valid combination of clocks should correspond to at least one 
> compatible.
> 

Thanks, I will change it in the next version.

> > +- ahb_ckN :AHB slave interface operating clock for CSR access and RC
> > +  initiated MMIO access
> > +- axi_ckN :application layer MMIO channel operating clock
> > +- aux_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> > +  pcie_mac_ck/pcie_pipe_ck is turned off
> > +- obff_ckN :OBFF functional block operating clock
> > +- pipe_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> > +   pcie_mac_ck/pcie_pipe_ck is turned off
> > +  where N starting from 0 to the maximum number of root ports.
> >  - phys: List of PHY specifiers (used by generic PHY framework).
> >  - phy-names : Must be "pcie-phy0", "pcie-phy1", "pcie-phyN".. based on the
> >number of PHYs as specified in *phys* property.
> > @@ -33,6 +43,9 @@ Required properties for MT7623/MT2701:
> >  - reset-names: Must be "pcie-rst0", "pcie-rst1", "pcie-rstN".. based on the
> >number of root ports.
> >  
> > +Required properties for mediatek generic-pcie-v2:
> > +-interrupts: A list of interrupt outputs of the controller.
> > +
> 
> How mnay interrupts? Not sure why you moved this in the previous patch 
> if all versions have some interrupts.
> 
There would be 1 interrupt per port for the new SoCs.
The legacy controller(7623, 2701) does not have a interrupt status
register to identify the interrupt source, so we use interrupt-map to
bind the interrupt to each port, thus we do not need the "interrupt"
property.
But the new SoCs doest not have this problem, and support msi. So we use
the standard "interrupt" property.

How about:
Requirted properties for "medaitek,mt2712-pcie" and
"mediatek,mt7622-pcie"
-interrupts: A list for interrupt outputs of the controller, there must
be one interrupt for each port.
 
> >  In addition, the device tree node must have sub-nodes describing each
> >  PCIe port interface, having the following mandatory properties:
> >  
> > @@ -50,7 +63,7 @@ Required properties:
> >property is sufficient.
> >  - num-lanes: Number of lanes to use for this port.
> >  
> > -Examples:
> > +Examples for mt7623:
> >  
> > hifsys: syscon@1a00 {
> > compatible = "mediatek,mt7623-hifsys",
> > @@ -68,6 +81,7 @@ Examples:
> >   <0 0x1a142000 0 0x1000>, /* Port0 registers */
> >   <0 0x1a143000 0 0x1000>, /* Port1 registers */
> >   <0 0x1a144000 0 0x1000>; /* Port2 registers */
> > +   reg-names = "subsys", "port0", "port1", "port2";
> > #address-cells = <3>;
> > #size-cells = <2>;
> > #interrupt-cells = <1>;
> > @@ -127,3 +141,63 @@ Examples:
> > num-lanes = <1>;
> >

Re: [PATCH 5/5] dt-bindings: PCI: add support for new generation controller

[Honghui Zhang]

On Mon, 2017-07-24 at 15:26 -0500, Rob Herring wrote:
> On Fri, Jul 21, 2017 at 10:34:49AM +0800, honghui.zh...@mediatek.com wrote:
> > From: Ryder Lee 
> > 
> > Add support for MediaTek new generation controller and update related
> > properities.
> > 
> > Signed-off-by: Ryder Lee 
> > Signed-off-by: Honghui Zhang 
> > ---
> >  .../devicetree/bindings/pci/mediatek-pcie.txt  | 84 
> > --
> >  1 file changed, 79 insertions(+), 5 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt 
> > b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> > index 294f4a3..a1f3767 100644
> > --- a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> > +++ b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> > @@ -4,17 +4,27 @@ Required properties:
> >  - compatible: Should contain one of the following string:
> > "mediatek,mt7623-pcie"
> > "mediatek,mt2701-pcie"
> > +   "mediatek,generic-pcie-v2"
> 
> No, please use SoC specific compatible strings.
> 
Thanks, I will rewrite it as below:
"mediatek,mt2701-pcie"
"mediatek,mt7623-pcie"
"mediatek,mt2712-pcie"
"mediatek,mt7622-pcie"

> >  - device_type: Must be "pci"
> > -- reg: Base addresses and lengths of the PCIe controller.
> > +- reg: Base addresses and lengths of the PICe subsys and root ports.
> > +- reg-names: Names of the above areas to use during resource look-up.
> >  - #address-cells: Address representation for root ports (must be 3)
> >  - #size-cells: Size representation for root ports (must be 2)
> >  - clocks: Must contain an entry for each entry in clock-names.
> >See ../clocks/clock-bindings.txt for details.
> >  - clock-names: Must include the following entries:
> >- free_ck :for reference clock of PCIe subsys
> > -  - sys_ck0 :for clock of Port0
> > -  - sys_ck1 :for clock of Port1
> > -  - sys_ck2 :for clock of Port2
> > +  - sys_ckN :transaction layer and data link layer clock
> > +  The "sys_ck" might be divided into the following parts in some v2 chips:
> 
> Please be specific about what clocks apply to which compatible. Each 
> valid combination of clocks should correspond to at least one 
> compatible.
> 

Thanks, I will change it in the next version.

> > +- ahb_ckN :AHB slave interface operating clock for CSR access and RC
> > +  initiated MMIO access
> > +- axi_ckN :application layer MMIO channel operating clock
> > +- aux_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> > +  pcie_mac_ck/pcie_pipe_ck is turned off
> > +- obff_ckN :OBFF functional block operating clock
> > +- pipe_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> > +   pcie_mac_ck/pcie_pipe_ck is turned off
> > +  where N starting from 0 to the maximum number of root ports.
> >  - phys: List of PHY specifiers (used by generic PHY framework).
> >  - phy-names : Must be "pcie-phy0", "pcie-phy1", "pcie-phyN".. based on the
> >number of PHYs as specified in *phys* property.
> > @@ -33,6 +43,9 @@ Required properties for MT7623/MT2701:
> >  - reset-names: Must be "pcie-rst0", "pcie-rst1", "pcie-rstN".. based on the
> >number of root ports.
> >  
> > +Required properties for mediatek generic-pcie-v2:
> > +-interrupts: A list of interrupt outputs of the controller.
> > +
> 
> How mnay interrupts? Not sure why you moved this in the previous patch 
> if all versions have some interrupts.
> 
There would be 1 interrupt per port for the new SoCs.
The legacy controller(7623, 2701) does not have a interrupt status
register to identify the interrupt source, so we use interrupt-map to
bind the interrupt to each port, thus we do not need the "interrupt"
property.
But the new SoCs doest not have this problem, and support msi. So we use
the standard "interrupt" property.

How about:
Requirted properties for "medaitek,mt2712-pcie" and
"mediatek,mt7622-pcie"
-interrupts: A list for interrupt outputs of the controller, there must
be one interrupt for each port.
 
> >  In addition, the device tree node must have sub-nodes describing each
> >  PCIe port interface, having the following mandatory properties:
> >  
> > @@ -50,7 +63,7 @@ Required properties:
> >property is sufficient.
> >  - num-lanes: Number of lanes to use for this port.
> >  
> > -Examples:
> > +Examples for mt7623:
> >  
> > hifsys: syscon@1a00 {
> > compatible = "mediatek,mt7623-hifsys",
> > @@ -68,6 +81,7 @@ Examples:
> >   <0 0x1a142000 0 0x1000>, /* Port0 registers */
> >   <0 0x1a143000 0 0x1000>, /* Port1 registers */
> >   <0 0x1a144000 0 0x1000>; /* Port2 registers */
> > +   reg-names = "subsys", "port0", "port1", "port2";
> > #address-cells = <3>;
> > #size-cells = <2>;
> > #interrupt-cells = <1>;
> > @@ -127,3 +141,63 @@ Examples:
> > num-lanes = <1>;
> > };
> > };
> > +
> > +Examples for mt2712:
> > +   pcie: 

Re: [PATCH 5/5] dt-bindings: PCI: add support for new generation controller

[Rob Herring]

On Fri, Jul 21, 2017 at 10:34:49AM +0800, honghui.zh...@mediatek.com wrote:
> From: Ryder Lee 
> 
> Add support for MediaTek new generation controller and update related
> properities.
> 
> Signed-off-by: Ryder Lee 
> Signed-off-by: Honghui Zhang 
> ---
>  .../devicetree/bindings/pci/mediatek-pcie.txt  | 84 
> --
>  1 file changed, 79 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt 
> b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> index 294f4a3..a1f3767 100644
> --- a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> +++ b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> @@ -4,17 +4,27 @@ Required properties:
>  - compatible: Should contain one of the following string:
>   "mediatek,mt7623-pcie"
>   "mediatek,mt2701-pcie"
> + "mediatek,generic-pcie-v2"

No, please use SoC specific compatible strings.

>  - device_type: Must be "pci"
> -- reg: Base addresses and lengths of the PCIe controller.
> +- reg: Base addresses and lengths of the PICe subsys and root ports.
> +- reg-names: Names of the above areas to use during resource look-up.
>  - #address-cells: Address representation for root ports (must be 3)
>  - #size-cells: Size representation for root ports (must be 2)
>  - clocks: Must contain an entry for each entry in clock-names.
>See ../clocks/clock-bindings.txt for details.
>  - clock-names: Must include the following entries:
>- free_ck :for reference clock of PCIe subsys
> -  - sys_ck0 :for clock of Port0
> -  - sys_ck1 :for clock of Port1
> -  - sys_ck2 :for clock of Port2
> +  - sys_ckN :transaction layer and data link layer clock
> +  The "sys_ck" might be divided into the following parts in some v2 chips:

Please be specific about what clocks apply to which compatible. Each 
valid combination of clocks should correspond to at least one 
compatible.

> +- ahb_ckN :AHB slave interface operating clock for CSR access and RC
> +initiated MMIO access
> +- axi_ckN :application layer MMIO channel operating clock
> +- aux_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> +pcie_mac_ck/pcie_pipe_ck is turned off
> +- obff_ckN :OBFF functional block operating clock
> +- pipe_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> + pcie_mac_ck/pcie_pipe_ck is turned off
> +  where N starting from 0 to the maximum number of root ports.
>  - phys: List of PHY specifiers (used by generic PHY framework).
>  - phy-names : Must be "pcie-phy0", "pcie-phy1", "pcie-phyN".. based on the
>number of PHYs as specified in *phys* property.
> @@ -33,6 +43,9 @@ Required properties for MT7623/MT2701:
>  - reset-names: Must be "pcie-rst0", "pcie-rst1", "pcie-rstN".. based on the
>number of root ports.
>  
> +Required properties for mediatek generic-pcie-v2:
> +-interrupts: A list of interrupt outputs of the controller.
> +

How mnay interrupts? Not sure why you moved this in the previous patch 
if all versions have some interrupts.

>  In addition, the device tree node must have sub-nodes describing each
>  PCIe port interface, having the following mandatory properties:
>  
> @@ -50,7 +63,7 @@ Required properties:
>property is sufficient.
>  - num-lanes: Number of lanes to use for this port.
>  
> -Examples:
> +Examples for mt7623:
>  
>   hifsys: syscon@1a00 {
>   compatible = "mediatek,mt7623-hifsys",
> @@ -68,6 +81,7 @@ Examples:
> <0 0x1a142000 0 0x1000>, /* Port0 registers */
> <0 0x1a143000 0 0x1000>, /* Port1 registers */
> <0 0x1a144000 0 0x1000>; /* Port2 registers */
> + reg-names = "subsys", "port0", "port1", "port2";
>   #address-cells = <3>;
>   #size-cells = <2>;
>   #interrupt-cells = <1>;
> @@ -127,3 +141,63 @@ Examples:
>   num-lanes = <1>;
>   };
>   };
> +
> +Examples for mt2712:
> + pcie: pcie@0x1170 {

Drop the '0x'.

> + compatible = "mediatek,generic-pcie-v2";
> + device_type = "pci";
> + reg = <0 0x1170 0 0x1000>,
> +   <0 0x112FF000 0 0x1000>;
> + reg-names = "port0", "port1";
> + #address-cells = <3>;
> + #size-cells = <2>;
> + interrupts = ,
> +  ;
> + clocks = < CLK_PERI_PCIE0>,
> +  < CLK_PERI_PCIE1>,
> +  < CLK_TOP_PE2_MAC_P0_SEL>,
> +  < CLK_TOP_PE2_MAC_P1_SEL>;
> + clock-names = "sys_ck0", "sys_ck1", "ahb0", "ahb1";
> + bus-range = <0x00 0xff>;
> + ranges = <0x8200 0 0x2000  0x0 0x2000  0 
> 0x1000>;
> +
> + pcie0: pcie@0,0 {
> + device_type = "pci";
> +   

Re: [PATCH 5/5] dt-bindings: PCI: add support for new generation controller

[Rob Herring]

On Fri, Jul 21, 2017 at 10:34:49AM +0800, honghui.zh...@mediatek.com wrote:
> From: Ryder Lee 
> 
> Add support for MediaTek new generation controller and update related
> properities.
> 
> Signed-off-by: Ryder Lee 
> Signed-off-by: Honghui Zhang 
> ---
>  .../devicetree/bindings/pci/mediatek-pcie.txt  | 84 
> --
>  1 file changed, 79 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt 
> b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> index 294f4a3..a1f3767 100644
> --- a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> +++ b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
> @@ -4,17 +4,27 @@ Required properties:
>  - compatible: Should contain one of the following string:
>   "mediatek,mt7623-pcie"
>   "mediatek,mt2701-pcie"
> + "mediatek,generic-pcie-v2"

No, please use SoC specific compatible strings.

>  - device_type: Must be "pci"
> -- reg: Base addresses and lengths of the PCIe controller.
> +- reg: Base addresses and lengths of the PICe subsys and root ports.
> +- reg-names: Names of the above areas to use during resource look-up.
>  - #address-cells: Address representation for root ports (must be 3)
>  - #size-cells: Size representation for root ports (must be 2)
>  - clocks: Must contain an entry for each entry in clock-names.
>See ../clocks/clock-bindings.txt for details.
>  - clock-names: Must include the following entries:
>- free_ck :for reference clock of PCIe subsys
> -  - sys_ck0 :for clock of Port0
> -  - sys_ck1 :for clock of Port1
> -  - sys_ck2 :for clock of Port2
> +  - sys_ckN :transaction layer and data link layer clock
> +  The "sys_ck" might be divided into the following parts in some v2 chips:

Please be specific about what clocks apply to which compatible. Each 
valid combination of clocks should correspond to at least one 
compatible.

> +- ahb_ckN :AHB slave interface operating clock for CSR access and RC
> +initiated MMIO access
> +- axi_ckN :application layer MMIO channel operating clock
> +- aux_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> +pcie_mac_ck/pcie_pipe_ck is turned off
> +- obff_ckN :OBFF functional block operating clock
> +- pipe_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
> + pcie_mac_ck/pcie_pipe_ck is turned off
> +  where N starting from 0 to the maximum number of root ports.
>  - phys: List of PHY specifiers (used by generic PHY framework).
>  - phy-names : Must be "pcie-phy0", "pcie-phy1", "pcie-phyN".. based on the
>number of PHYs as specified in *phys* property.
> @@ -33,6 +43,9 @@ Required properties for MT7623/MT2701:
>  - reset-names: Must be "pcie-rst0", "pcie-rst1", "pcie-rstN".. based on the
>number of root ports.
>  
> +Required properties for mediatek generic-pcie-v2:
> +-interrupts: A list of interrupt outputs of the controller.
> +

How mnay interrupts? Not sure why you moved this in the previous patch 
if all versions have some interrupts.

>  In addition, the device tree node must have sub-nodes describing each
>  PCIe port interface, having the following mandatory properties:
>  
> @@ -50,7 +63,7 @@ Required properties:
>property is sufficient.
>  - num-lanes: Number of lanes to use for this port.
>  
> -Examples:
> +Examples for mt7623:
>  
>   hifsys: syscon@1a00 {
>   compatible = "mediatek,mt7623-hifsys",
> @@ -68,6 +81,7 @@ Examples:
> <0 0x1a142000 0 0x1000>, /* Port0 registers */
> <0 0x1a143000 0 0x1000>, /* Port1 registers */
> <0 0x1a144000 0 0x1000>; /* Port2 registers */
> + reg-names = "subsys", "port0", "port1", "port2";
>   #address-cells = <3>;
>   #size-cells = <2>;
>   #interrupt-cells = <1>;
> @@ -127,3 +141,63 @@ Examples:
>   num-lanes = <1>;
>   };
>   };
> +
> +Examples for mt2712:
> + pcie: pcie@0x1170 {

Drop the '0x'.

> + compatible = "mediatek,generic-pcie-v2";
> + device_type = "pci";
> + reg = <0 0x1170 0 0x1000>,
> +   <0 0x112FF000 0 0x1000>;
> + reg-names = "port0", "port1";
> + #address-cells = <3>;
> + #size-cells = <2>;
> + interrupts = ,
> +  ;
> + clocks = < CLK_PERI_PCIE0>,
> +  < CLK_PERI_PCIE1>,
> +  < CLK_TOP_PE2_MAC_P0_SEL>,
> +  < CLK_TOP_PE2_MAC_P1_SEL>;
> + clock-names = "sys_ck0", "sys_ck1", "ahb0", "ahb1";
> + bus-range = <0x00 0xff>;
> + ranges = <0x8200 0 0x2000  0x0 0x2000  0 
> 0x1000>;
> +
> + pcie0: pcie@0,0 {
> + device_type = "pci";
> + reg = <0x 0 0 0 0>;
> + #address-cells = 

[PATCH 5/5] dt-bindings: PCI: add support for new generation controller

[honghui.zhang]

From: Ryder Lee 

Add support for MediaTek new generation controller and update related
properities.

Signed-off-by: Ryder Lee 
Signed-off-by: Honghui Zhang 
---
 .../devicetree/bindings/pci/mediatek-pcie.txt  | 84 --
 1 file changed, 79 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt 
b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
index 294f4a3..a1f3767 100644
--- a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
+++ b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
@@ -4,17 +4,27 @@ Required properties:
 - compatible: Should contain one of the following string:
"mediatek,mt7623-pcie"
"mediatek,mt2701-pcie"
+   "mediatek,generic-pcie-v2"
 - device_type: Must be "pci"
-- reg: Base addresses and lengths of the PCIe controller.
+- reg: Base addresses and lengths of the PICe subsys and root ports.
+- reg-names: Names of the above areas to use during resource look-up.
 - #address-cells: Address representation for root ports (must be 3)
 - #size-cells: Size representation for root ports (must be 2)
 - clocks: Must contain an entry for each entry in clock-names.
   See ../clocks/clock-bindings.txt for details.
 - clock-names: Must include the following entries:
   - free_ck :for reference clock of PCIe subsys
-  - sys_ck0 :for clock of Port0
-  - sys_ck1 :for clock of Port1
-  - sys_ck2 :for clock of Port2
+  - sys_ckN :transaction layer and data link layer clock
+  The "sys_ck" might be divided into the following parts in some v2 chips:
+- ahb_ckN :AHB slave interface operating clock for CSR access and RC
+  initiated MMIO access
+- axi_ckN :application layer MMIO channel operating clock
+- aux_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
+  pcie_mac_ck/pcie_pipe_ck is turned off
+- obff_ckN :OBFF functional block operating clock
+- pipe_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
+   pcie_mac_ck/pcie_pipe_ck is turned off
+  where N starting from 0 to the maximum number of root ports.
 - phys: List of PHY specifiers (used by generic PHY framework).
 - phy-names : Must be "pcie-phy0", "pcie-phy1", "pcie-phyN".. based on the
   number of PHYs as specified in *phys* property.
@@ -33,6 +43,9 @@ Required properties for MT7623/MT2701:
 - reset-names: Must be "pcie-rst0", "pcie-rst1", "pcie-rstN".. based on the
   number of root ports.
 
+Required properties for mediatek generic-pcie-v2:
+-interrupts: A list of interrupt outputs of the controller.
+
 In addition, the device tree node must have sub-nodes describing each
 PCIe port interface, having the following mandatory properties:
 
@@ -50,7 +63,7 @@ Required properties:
   property is sufficient.
 - num-lanes: Number of lanes to use for this port.
 
-Examples:
+Examples for mt7623:
 
hifsys: syscon@1a00 {
compatible = "mediatek,mt7623-hifsys",
@@ -68,6 +81,7 @@ Examples:
  <0 0x1a142000 0 0x1000>, /* Port0 registers */
  <0 0x1a143000 0 0x1000>, /* Port1 registers */
  <0 0x1a144000 0 0x1000>; /* Port2 registers */
+   reg-names = "subsys", "port0", "port1", "port2";
#address-cells = <3>;
#size-cells = <2>;
#interrupt-cells = <1>;
@@ -127,3 +141,63 @@ Examples:
num-lanes = <1>;
};
};
+
+Examples for mt2712:
+   pcie: pcie@0x1170 {
+   compatible = "mediatek,generic-pcie-v2";
+   device_type = "pci";
+   reg = <0 0x1170 0 0x1000>,
+ <0 0x112FF000 0 0x1000>;
+   reg-names = "port0", "port1";
+   #address-cells = <3>;
+   #size-cells = <2>;
+   interrupts = ,
+;
+   clocks = < CLK_PERI_PCIE0>,
+< CLK_PERI_PCIE1>,
+< CLK_TOP_PE2_MAC_P0_SEL>,
+< CLK_TOP_PE2_MAC_P1_SEL>;
+   clock-names = "sys_ck0", "sys_ck1", "ahb0", "ahb1";
+   bus-range = <0x00 0xff>;
+   ranges = <0x8200 0 0x2000  0x0 0x2000  0 
0x1000>;
+
+   pcie0: pcie@0,0 {
+   device_type = "pci";
+   reg = <0x 0 0 0 0>;
+   #address-cells = <3>;
+   #size-cells = <2>;
+   #interrupt-cells = <1>;
+   ranges;
+   num-lanes = <1>;
+   interrupt-map-mask = <0 0 0 7>;
+   interrupt-map = <0 0 0 1 _intc0 0>,
+   <0 0 0 2 _intc0 1>,
+   <0 0 0 3 _intc0 2>,
+   <0 0 0 4 

[PATCH 5/5] dt-bindings: PCI: add support for new generation controller

[honghui.zhang]

From: Ryder Lee 

Add support for MediaTek new generation controller and update related
properities.

Signed-off-by: Ryder Lee 
Signed-off-by: Honghui Zhang 
---
 .../devicetree/bindings/pci/mediatek-pcie.txt  | 84 --
 1 file changed, 79 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt 
b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
index 294f4a3..a1f3767 100644
--- a/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
+++ b/Documentation/devicetree/bindings/pci/mediatek-pcie.txt
@@ -4,17 +4,27 @@ Required properties:
 - compatible: Should contain one of the following string:
"mediatek,mt7623-pcie"
"mediatek,mt2701-pcie"
+   "mediatek,generic-pcie-v2"
 - device_type: Must be "pci"
-- reg: Base addresses and lengths of the PCIe controller.
+- reg: Base addresses and lengths of the PICe subsys and root ports.
+- reg-names: Names of the above areas to use during resource look-up.
 - #address-cells: Address representation for root ports (must be 3)
 - #size-cells: Size representation for root ports (must be 2)
 - clocks: Must contain an entry for each entry in clock-names.
   See ../clocks/clock-bindings.txt for details.
 - clock-names: Must include the following entries:
   - free_ck :for reference clock of PCIe subsys
-  - sys_ck0 :for clock of Port0
-  - sys_ck1 :for clock of Port1
-  - sys_ck2 :for clock of Port2
+  - sys_ckN :transaction layer and data link layer clock
+  The "sys_ck" might be divided into the following parts in some v2 chips:
+- ahb_ckN :AHB slave interface operating clock for CSR access and RC
+  initiated MMIO access
+- axi_ckN :application layer MMIO channel operating clock
+- aux_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
+  pcie_mac_ck/pcie_pipe_ck is turned off
+- obff_ckN :OBFF functional block operating clock
+- pipe_ckN :pe2_mac_bridge and pe2_mac_core operating clock when
+   pcie_mac_ck/pcie_pipe_ck is turned off
+  where N starting from 0 to the maximum number of root ports.
 - phys: List of PHY specifiers (used by generic PHY framework).
 - phy-names : Must be "pcie-phy0", "pcie-phy1", "pcie-phyN".. based on the
   number of PHYs as specified in *phys* property.
@@ -33,6 +43,9 @@ Required properties for MT7623/MT2701:
 - reset-names: Must be "pcie-rst0", "pcie-rst1", "pcie-rstN".. based on the
   number of root ports.
 
+Required properties for mediatek generic-pcie-v2:
+-interrupts: A list of interrupt outputs of the controller.
+
 In addition, the device tree node must have sub-nodes describing each
 PCIe port interface, having the following mandatory properties:
 
@@ -50,7 +63,7 @@ Required properties:
   property is sufficient.
 - num-lanes: Number of lanes to use for this port.
 
-Examples:
+Examples for mt7623:
 
hifsys: syscon@1a00 {
compatible = "mediatek,mt7623-hifsys",
@@ -68,6 +81,7 @@ Examples:
  <0 0x1a142000 0 0x1000>, /* Port0 registers */
  <0 0x1a143000 0 0x1000>, /* Port1 registers */
  <0 0x1a144000 0 0x1000>; /* Port2 registers */
+   reg-names = "subsys", "port0", "port1", "port2";
#address-cells = <3>;
#size-cells = <2>;
#interrupt-cells = <1>;
@@ -127,3 +141,63 @@ Examples:
num-lanes = <1>;
};
};
+
+Examples for mt2712:
+   pcie: pcie@0x1170 {
+   compatible = "mediatek,generic-pcie-v2";
+   device_type = "pci";
+   reg = <0 0x1170 0 0x1000>,
+ <0 0x112FF000 0 0x1000>;
+   reg-names = "port0", "port1";
+   #address-cells = <3>;
+   #size-cells = <2>;
+   interrupts = ,
+;
+   clocks = < CLK_PERI_PCIE0>,
+< CLK_PERI_PCIE1>,
+< CLK_TOP_PE2_MAC_P0_SEL>,
+< CLK_TOP_PE2_MAC_P1_SEL>;
+   clock-names = "sys_ck0", "sys_ck1", "ahb0", "ahb1";
+   bus-range = <0x00 0xff>;
+   ranges = <0x8200 0 0x2000  0x0 0x2000  0 
0x1000>;
+
+   pcie0: pcie@0,0 {
+   device_type = "pci";
+   reg = <0x 0 0 0 0>;
+   #address-cells = <3>;
+   #size-cells = <2>;
+   #interrupt-cells = <1>;
+   ranges;
+   num-lanes = <1>;
+   interrupt-map-mask = <0 0 0 7>;
+   interrupt-map = <0 0 0 1 _intc0 0>,
+   <0 0 0 2 _intc0 1>,
+   <0 0 0 3 _intc0 2>,
+   <0 0 0 4 _intc0 3>;
+   pcie_intc0: interrupt-controller {
+