* [PATCH v4 0/2] Add support for Xilinx Versal CPM5 Root Port @ 2022-06-08 16:40 Bharat Kumar Gogada 2022-06-08 16:40 ` [PATCH v4 1/2] dt-bindings: PCI: xilinx-cpm: Add " Bharat Kumar Gogada 2022-06-08 16:40 ` [PATCH v4 2/2] PCI: xilinx-cpm: Add support for " Bharat Kumar Gogada 0 siblings, 2 replies; 10+ messages in thread From: Bharat Kumar Gogada @ 2022-06-08 16:40 UTC (permalink / raw) To: linux-pci, linux-kernel, devicetree Cc: lorenzo.pieralisi, bhelgaas, michals, robh, Bharat Kumar Gogada Xilinx Versal Premium series has CPM5 block which supports Root Port functioning at Gen5 speed. Xilinx Versal CPM5 has few changes with existing CPM block. - CPM5 has dedicated register space for control and status registers. - CPM5 legacy interrupt handling needs additonal register bit to enable and handle legacy interrupts. Changes in v4: - Removed versioning in compatible string - reg property definitions are added Bharat Kumar Gogada (2): dt-bindings: PCI: xilinx-cpm: Add Versal CPM5 Root Port PCI: xilinx-cpm: Add support for Versal CPM5 Root Port .../bindings/pci/xilinx-versal-cpm.yaml | 48 +++++++++++++++++-- drivers/pci/controller/pcie-xilinx-cpm.c | 33 ++++++++++++- 2 files changed, 76 insertions(+), 5 deletions(-) -- 2.17.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v4 1/2] dt-bindings: PCI: xilinx-cpm: Add Versal CPM5 Root Port 2022-06-08 16:40 [PATCH v4 0/2] Add support for Xilinx Versal CPM5 Root Port Bharat Kumar Gogada @ 2022-06-08 16:40 ` Bharat Kumar Gogada 2022-06-09 21:10 ` Rob Herring 2022-06-08 16:40 ` [PATCH v4 2/2] PCI: xilinx-cpm: Add support for " Bharat Kumar Gogada 1 sibling, 1 reply; 10+ messages in thread From: Bharat Kumar Gogada @ 2022-06-08 16:40 UTC (permalink / raw) To: linux-pci, linux-kernel, devicetree Cc: lorenzo.pieralisi, bhelgaas, michals, robh, Bharat Kumar Gogada Xilinx Versal Premium series has CPM5 block which supports Root Port functionality at Gen5 speed. Add support for YAML schemas documentation for Versal CPM5 Root Port driver. Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> --- .../bindings/pci/xilinx-versal-cpm.yaml | 48 +++++++++++++++++-- 1 file changed, 44 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml index cca395317a4c..80597f2974e5 100644 --- a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml +++ b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml @@ -14,17 +14,27 @@ allOf: properties: compatible: - const: xlnx,versal-cpm-host-1.00 + contains: + enum: + - xlnx,versal-cpm-host-1.00 + - xlnx,versal-cpm5-host reg: items: - description: CPM system level control and status registers. - description: Configuration space region and bridge registers. + - description: CPM5 control and status registers. + minItems: 2 reg-names: - items: - - const: cpm_slcr - - const: cfg + oneOf: + - items: + - const: cpm_slcr + - const: cfg + - items: + - const: cpm_slcr + - const: cfg + - const: cpm_csr interrupts: maxItems: 1 @@ -95,4 +105,34 @@ examples: interrupt-controller; }; }; + + cpm5_pcie: pcie@fcdd0000 { + compatible = "xlnx,versal-cpm5-host"; + device_type = "pci"; + #address-cells = <3>; + #interrupt-cells = <1>; + #size-cells = <2>; + interrupts = <0 72 4>; + interrupt-parent = <&gic>; + interrupt-map-mask = <0 0 0 7>; + interrupt-map = <0 0 0 1 &pcie_intc_1 0>, + <0 0 0 2 &pcie_intc_1 1>, + <0 0 0 3 &pcie_intc_1 2>, + <0 0 0 4 &pcie_intc_1 3>; + bus-range = <0x00 0xff>; + ranges = <0x02000000 0x0 0xe0000000 0x0 0xe0000000 0x0 0x10000000>, + <0x43000000 0x80 0x00000000 0x80 0x00000000 0x0 0x80000000>; + msi-map = <0x0 &its_gic 0x0 0x10000>; + reg = <0x00 0xfcdd0000 0x00 0x1000>, + <0x06 0x00000000 0x00 0x1000000>, + <0x00 0xfce20000 0x00 0x1000000>; + reg-names = "cpm_slcr", "cfg", "cpm_csr"; + + pcie_intc_1: interrupt-controller { + #address-cells = <0>; + #interrupt-cells = <1>; + interrupt-controller; + }; + }; + }; -- 2.17.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4 1/2] dt-bindings: PCI: xilinx-cpm: Add Versal CPM5 Root Port 2022-06-08 16:40 ` [PATCH v4 1/2] dt-bindings: PCI: xilinx-cpm: Add " Bharat Kumar Gogada @ 2022-06-09 21:10 ` Rob Herring 2022-06-10 8:50 ` Bharat Kumar Gogada 0 siblings, 1 reply; 10+ messages in thread From: Rob Herring @ 2022-06-09 21:10 UTC (permalink / raw) To: Bharat Kumar Gogada Cc: linux-pci, linux-kernel, devicetree, lorenzo.pieralisi, bhelgaas, michals On Wed, Jun 08, 2022 at 10:10:45PM +0530, Bharat Kumar Gogada wrote: > Xilinx Versal Premium series has CPM5 block which supports Root Port > functionality at Gen5 speed. > > Add support for YAML schemas documentation for Versal CPM5 Root Port driver. > > Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> > --- > .../bindings/pci/xilinx-versal-cpm.yaml | 48 +++++++++++++++++-- > 1 file changed, 44 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml > index cca395317a4c..80597f2974e5 100644 > --- a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml > +++ b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml > @@ -14,17 +14,27 @@ allOf: > > properties: > compatible: > - const: xlnx,versal-cpm-host-1.00 > + contains: Drop 'contains'. > + enum: > + - xlnx,versal-cpm-host-1.00 > + - xlnx,versal-cpm5-host > > reg: > items: > - description: CPM system level control and status registers. > - description: Configuration space region and bridge registers. > + - description: CPM5 control and status registers. > + minItems: 2 > > reg-names: > - items: > - - const: cpm_slcr > - - const: cfg > + oneOf: You don't need oneOf. > + - items: > + - const: cpm_slcr > + - const: cfg > + - items: > + - const: cpm_slcr > + - const: cfg > + - const: cpm_csr Just add 'minItems: 2' > > interrupts: > maxItems: 1 > @@ -95,4 +105,34 @@ examples: > interrupt-controller; > }; > }; > + > + cpm5_pcie: pcie@fcdd0000 { > + compatible = "xlnx,versal-cpm5-host"; > + device_type = "pci"; > + #address-cells = <3>; > + #interrupt-cells = <1>; > + #size-cells = <2>; > + interrupts = <0 72 4>; > + interrupt-parent = <&gic>; > + interrupt-map-mask = <0 0 0 7>; > + interrupt-map = <0 0 0 1 &pcie_intc_1 0>, > + <0 0 0 2 &pcie_intc_1 1>, > + <0 0 0 3 &pcie_intc_1 2>, > + <0 0 0 4 &pcie_intc_1 3>; > + bus-range = <0x00 0xff>; > + ranges = <0x02000000 0x0 0xe0000000 0x0 0xe0000000 0x0 0x10000000>, > + <0x43000000 0x80 0x00000000 0x80 0x00000000 0x0 0x80000000>; > + msi-map = <0x0 &its_gic 0x0 0x10000>; > + reg = <0x00 0xfcdd0000 0x00 0x1000>, > + <0x06 0x00000000 0x00 0x1000000>, > + <0x00 0xfce20000 0x00 0x1000000>; > + reg-names = "cpm_slcr", "cfg", "cpm_csr"; > + > + pcie_intc_1: interrupt-controller { > + #address-cells = <0>; > + #interrupt-cells = <1>; > + interrupt-controller; > + }; > + }; > + > }; > -- > 2.17.1 > > ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v4 1/2] dt-bindings: PCI: xilinx-cpm: Add Versal CPM5 Root Port 2022-06-09 21:10 ` Rob Herring @ 2022-06-10 8:50 ` Bharat Kumar Gogada 0 siblings, 0 replies; 10+ messages in thread From: Bharat Kumar Gogada @ 2022-06-10 8:50 UTC (permalink / raw) To: Rob Herring Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, lorenzo.pieralisi@arm.com, bhelgaas@google.com, Michal Simek > On Wed, Jun 08, 2022 at 10:10:45PM +0530, Bharat Kumar Gogada wrote: > > Xilinx Versal Premium series has CPM5 block which supports Root Port > > functionality at Gen5 speed. > > > > Add support for YAML schemas documentation for Versal CPM5 Root Port > driver. > > > > Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> > > --- > > .../bindings/pci/xilinx-versal-cpm.yaml | 48 +++++++++++++++++-- > > 1 file changed, 44 insertions(+), 4 deletions(-) > > > > diff --git > > a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml > > b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml > > index cca395317a4c..80597f2974e5 100644 > > --- a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml > > +++ b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml > > @@ -14,17 +14,27 @@ allOf: > > > > properties: > > compatible: > > - const: xlnx,versal-cpm-host-1.00 > > + contains: > > Drop 'contains'. > > > + enum: > > + - xlnx,versal-cpm-host-1.00 > > + - xlnx,versal-cpm5-host > > > > reg: > > items: > > - description: CPM system level control and status registers. > > - description: Configuration space region and bridge registers. > > + - description: CPM5 control and status registers. > > + minItems: 2 > > > > reg-names: > > - items: > > - - const: cpm_slcr > > - - const: cfg > > + oneOf: > > You don't need oneOf. > > > + - items: > > + - const: cpm_slcr > > + - const: cfg > > > + - items: > > + - const: cpm_slcr > > + - const: cfg > > + - const: cpm_csr > > Just add 'minItems: 2' > Thanks Rob, will change this in next patch. Regards, Bharat ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v4 2/2] PCI: xilinx-cpm: Add support for Versal CPM5 Root Port 2022-06-08 16:40 [PATCH v4 0/2] Add support for Xilinx Versal CPM5 Root Port Bharat Kumar Gogada 2022-06-08 16:40 ` [PATCH v4 1/2] dt-bindings: PCI: xilinx-cpm: Add " Bharat Kumar Gogada @ 2022-06-08 16:40 ` Bharat Kumar Gogada 2022-06-08 19:14 ` Bjorn Helgaas 1 sibling, 1 reply; 10+ messages in thread From: Bharat Kumar Gogada @ 2022-06-08 16:40 UTC (permalink / raw) To: linux-pci, linux-kernel, devicetree Cc: lorenzo.pieralisi, bhelgaas, michals, robh, Bharat Kumar Gogada Xilinx Versal Premium series has CPM5 block which supports Root Port functioning at Gen5 speed. Xilinx Versal CPM5 has few changes with existing CPM block. - CPM5 has dedicated register space for control and status registers. - CPM5 legacy interrupt handling needs additional register bit to enable and handle legacy interrupts. Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> --- drivers/pci/controller/pcie-xilinx-cpm.c | 33 +++++++++++++++++++++++- 1 file changed, 32 insertions(+), 1 deletion(-) diff --git a/drivers/pci/controller/pcie-xilinx-cpm.c b/drivers/pci/controller/pcie-xilinx-cpm.c index c7cd44ed4dfc..a3b04083b6b3 100644 --- a/drivers/pci/controller/pcie-xilinx-cpm.c +++ b/drivers/pci/controller/pcie-xilinx-cpm.c @@ -35,6 +35,10 @@ #define XILINX_CPM_PCIE_MISC_IR_ENABLE 0x00000348 #define XILINX_CPM_PCIE_MISC_IR_LOCAL BIT(1) +#define XILINX_CPM_PCIE_IR_STATUS 0x000002A0 +#define XILINX_CPM_PCIE_IR_ENABLE 0x000002A8 +#define XILINX_CPM_PCIE_IR_LOCAL BIT(0) + /* Interrupt registers definitions */ #define XILINX_CPM_PCIE_INTR_LINK_DOWN 0 #define XILINX_CPM_PCIE_INTR_HOT_RESET 3 @@ -109,6 +113,7 @@ * @intx_irq: legacy interrupt number * @irq: Error interrupt number * @lock: lock protecting shared register access + * @is_cpm5: value to check cpm version */ struct xilinx_cpm_pcie { struct device *dev; @@ -120,6 +125,7 @@ struct xilinx_cpm_pcie { int intx_irq; int irq; raw_spinlock_t lock; + bool is_cpm5; }; static u32 pcie_read(struct xilinx_cpm_pcie *port, u32 reg) @@ -285,6 +291,14 @@ static void xilinx_cpm_pcie_event_flow(struct irq_desc *desc) generic_handle_domain_irq(port->cpm_domain, i); pcie_write(port, val, XILINX_CPM_PCIE_REG_IDR); + if (port->is_cpm5) { + val = readl_relaxed(port->cpm_base + XILINX_CPM_PCIE_IR_STATUS); + if (val) + writel_relaxed(val, + port->cpm_base + + XILINX_CPM_PCIE_IR_STATUS); + } + /* * XILINX_CPM_PCIE_MISC_IR_STATUS register is mapped to * CPM SLCR block. @@ -484,6 +498,12 @@ static void xilinx_cpm_pcie_init_port(struct xilinx_cpm_pcie *port) */ writel(XILINX_CPM_PCIE_MISC_IR_LOCAL, port->cpm_base + XILINX_CPM_PCIE_MISC_IR_ENABLE); + + if (port->is_cpm5) { + writel(XILINX_CPM_PCIE_IR_LOCAL, + port->cpm_base + XILINX_CPM_PCIE_IR_ENABLE); + } + /* Enable the Bridge enable bit */ pcie_write(port, pcie_read(port, XILINX_CPM_PCIE_REG_RPSC) | XILINX_CPM_PCIE_REG_RPSC_BEN, @@ -504,6 +524,9 @@ static int xilinx_cpm_pcie_parse_dt(struct xilinx_cpm_pcie *port, struct platform_device *pdev = to_platform_device(dev); struct resource *res; + port->is_cpm5 = of_device_is_compatible(dev->of_node, + "xlnx,versal-cpm5-host"); + port->cpm_base = devm_platform_ioremap_resource_byname(pdev, "cpm_slcr"); if (IS_ERR(port->cpm_base)) @@ -518,7 +541,14 @@ static int xilinx_cpm_pcie_parse_dt(struct xilinx_cpm_pcie *port, if (IS_ERR(port->cfg)) return PTR_ERR(port->cfg); - port->reg_base = port->cfg->win; + if (port->is_cpm5) { + port->reg_base = devm_platform_ioremap_resource_byname(pdev, + "cpm_csr"); + if (IS_ERR(port->reg_base)) + return PTR_ERR(port->reg_base); + } else { + port->reg_base = port->cfg->win; + } return 0; } @@ -593,6 +623,7 @@ static int xilinx_cpm_pcie_probe(struct platform_device *pdev) static const struct of_device_id xilinx_cpm_pcie_of_match[] = { { .compatible = "xlnx,versal-cpm-host-1.00", }, + { .compatible = "xlnx,versal-cpm5-host", }, {} }; -- 2.17.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4 2/2] PCI: xilinx-cpm: Add support for Versal CPM5 Root Port 2022-06-08 16:40 ` [PATCH v4 2/2] PCI: xilinx-cpm: Add support for " Bharat Kumar Gogada @ 2022-06-08 19:14 ` Bjorn Helgaas 2022-06-09 7:59 ` Michal Simek 2022-06-10 8:47 ` Bharat Kumar Gogada 0 siblings, 2 replies; 10+ messages in thread From: Bjorn Helgaas @ 2022-06-08 19:14 UTC (permalink / raw) To: Bharat Kumar Gogada Cc: linux-pci, linux-kernel, devicetree, lorenzo.pieralisi, bhelgaas, michals, robh On Wed, Jun 08, 2022 at 10:10:46PM +0530, Bharat Kumar Gogada wrote: > Xilinx Versal Premium series has CPM5 block which supports Root Port > functioning at Gen5 speed. > > Xilinx Versal CPM5 has few changes with existing CPM block. > - CPM5 has dedicated register space for control and status registers. > - CPM5 legacy interrupt handling needs additional register bit > to enable and handle legacy interrupts. > > Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> > --- > drivers/pci/controller/pcie-xilinx-cpm.c | 33 +++++++++++++++++++++++- > 1 file changed, 32 insertions(+), 1 deletion(-) Per MAINTAINERS, xilinx-cpm lacks a maintainer. Can we get one? > diff --git a/drivers/pci/controller/pcie-xilinx-cpm.c b/drivers/pci/controller/pcie-xilinx-cpm.c > index c7cd44ed4dfc..a3b04083b6b3 100644 > --- a/drivers/pci/controller/pcie-xilinx-cpm.c > +++ b/drivers/pci/controller/pcie-xilinx-cpm.c > @@ -35,6 +35,10 @@ > #define XILINX_CPM_PCIE_MISC_IR_ENABLE 0x00000348 > #define XILINX_CPM_PCIE_MISC_IR_LOCAL BIT(1) > > +#define XILINX_CPM_PCIE_IR_STATUS 0x000002A0 > +#define XILINX_CPM_PCIE_IR_ENABLE 0x000002A8 > +#define XILINX_CPM_PCIE_IR_LOCAL BIT(0) > + > /* Interrupt registers definitions */ > #define XILINX_CPM_PCIE_INTR_LINK_DOWN 0 > #define XILINX_CPM_PCIE_INTR_HOT_RESET 3 > @@ -109,6 +113,7 @@ > * @intx_irq: legacy interrupt number > * @irq: Error interrupt number > * @lock: lock protecting shared register access > + * @is_cpm5: value to check cpm version s/cpm version/CPM version/ to match commit log usage. > + port->is_cpm5 = of_device_is_compatible(dev->of_node, > + "xlnx,versal-cpm5-host"); One use of of_device_is_compatible() is OK, I guess, but of_device_get_match_data() is a better pattern if we ever need more. I would lean toward of_device_get_match_data() even here, just to reduce the number of ways to identify device-specific things across drivers. Bjorn ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 2/2] PCI: xilinx-cpm: Add support for Versal CPM5 Root Port 2022-06-08 19:14 ` Bjorn Helgaas @ 2022-06-09 7:59 ` Michal Simek 2022-06-09 17:32 ` Bjorn Helgaas 2022-06-10 8:47 ` Bharat Kumar Gogada 1 sibling, 1 reply; 10+ messages in thread From: Michal Simek @ 2022-06-09 7:59 UTC (permalink / raw) To: Bjorn Helgaas, Bharat Kumar Gogada Cc: linux-pci, linux-kernel, devicetree, lorenzo.pieralisi, bhelgaas, robh On 6/8/22 21:14, Bjorn Helgaas wrote: > On Wed, Jun 08, 2022 at 10:10:46PM +0530, Bharat Kumar Gogada wrote: >> Xilinx Versal Premium series has CPM5 block which supports Root Port >> functioning at Gen5 speed. >> >> Xilinx Versal CPM5 has few changes with existing CPM block. >> - CPM5 has dedicated register space for control and status registers. >> - CPM5 legacy interrupt handling needs additional register bit >> to enable and handle legacy interrupts. >> >> Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> >> --- >> drivers/pci/controller/pcie-xilinx-cpm.c | 33 +++++++++++++++++++++++- >> 1 file changed, 32 insertions(+), 1 deletion(-) > > Per MAINTAINERS, xilinx-cpm lacks a maintainer. Can we get one? Bharat should become maintainer for this driver. My fragment should cover xilinx things in general in case Bharat is not available. Thanks, Michal ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 2/2] PCI: xilinx-cpm: Add support for Versal CPM5 Root Port 2022-06-09 7:59 ` Michal Simek @ 2022-06-09 17:32 ` Bjorn Helgaas 2022-06-10 8:49 ` Bharat Kumar Gogada 0 siblings, 1 reply; 10+ messages in thread From: Bjorn Helgaas @ 2022-06-09 17:32 UTC (permalink / raw) To: Michal Simek Cc: Bharat Kumar Gogada, linux-pci, linux-kernel, devicetree, lorenzo.pieralisi, bhelgaas, robh On Thu, Jun 09, 2022 at 09:59:08AM +0200, Michal Simek wrote: > On 6/8/22 21:14, Bjorn Helgaas wrote: > > On Wed, Jun 08, 2022 at 10:10:46PM +0530, Bharat Kumar Gogada wrote: > > > Xilinx Versal Premium series has CPM5 block which supports Root Port > > > functioning at Gen5 speed. > > > > > > Xilinx Versal CPM5 has few changes with existing CPM block. > > > - CPM5 has dedicated register space for control and status registers. > > > - CPM5 legacy interrupt handling needs additional register bit > > > to enable and handle legacy interrupts. > > > > > > Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> > > > --- > > > drivers/pci/controller/pcie-xilinx-cpm.c | 33 +++++++++++++++++++++++- > > > 1 file changed, 32 insertions(+), 1 deletion(-) > > > > Per MAINTAINERS, xilinx-cpm lacks a maintainer. Can we get one? > > Bharat should become maintainer for this driver. > > My fragment should cover xilinx things in general in case Bharat is > not available. Great! Can one of you post a patch to show exactly what you have in mind? Thanks, Bjorn ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v4 2/2] PCI: xilinx-cpm: Add support for Versal CPM5 Root Port 2022-06-09 17:32 ` Bjorn Helgaas @ 2022-06-10 8:49 ` Bharat Kumar Gogada 0 siblings, 0 replies; 10+ messages in thread From: Bharat Kumar Gogada @ 2022-06-10 8:49 UTC (permalink / raw) To: Bjorn Helgaas, Michal Simek Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, lorenzo.pieralisi@arm.com, bhelgaas@google.com, robh@kernel.org > > On Thu, Jun 09, 2022 at 09:59:08AM +0200, Michal Simek wrote: > > On 6/8/22 21:14, Bjorn Helgaas wrote: > > > On Wed, Jun 08, 2022 at 10:10:46PM +0530, Bharat Kumar Gogada wrote: > > > > Xilinx Versal Premium series has CPM5 block which supports Root > > > > Port functioning at Gen5 speed. > > > > > > > > Xilinx Versal CPM5 has few changes with existing CPM block. > > > > - CPM5 has dedicated register space for control and status registers. > > > > - CPM5 legacy interrupt handling needs additional register bit > > > > to enable and handle legacy interrupts. > > > > > > > > Signed-off-by: Bharat Kumar Gogada > > > > <bharat.kumar.gogada@xilinx.com> > > > > --- > > > > drivers/pci/controller/pcie-xilinx-cpm.c | 33 > +++++++++++++++++++++++- > > > > 1 file changed, 32 insertions(+), 1 deletion(-) > > > > > > Per MAINTAINERS, xilinx-cpm lacks a maintainer. Can we get one? > > > > Bharat should become maintainer for this driver. > > > > My fragment should cover xilinx things in general in case Bharat is > > not available. > > Great! Can one of you post a patch to show exactly what you have in mind? > Thanks, will send patch. Regards, Bharat ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v4 2/2] PCI: xilinx-cpm: Add support for Versal CPM5 Root Port 2022-06-08 19:14 ` Bjorn Helgaas 2022-06-09 7:59 ` Michal Simek @ 2022-06-10 8:47 ` Bharat Kumar Gogada 1 sibling, 0 replies; 10+ messages in thread From: Bharat Kumar Gogada @ 2022-06-10 8:47 UTC (permalink / raw) To: Bjorn Helgaas Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, lorenzo.pieralisi@arm.com, bhelgaas@google.com, Michal Simek, robh@kernel.org > > On Wed, Jun 08, 2022 at 10:10:46PM +0530, Bharat Kumar Gogada wrote: > > Xilinx Versal Premium series has CPM5 block which supports Root Port > > functioning at Gen5 speed. > > > > Xilinx Versal CPM5 has few changes with existing CPM block. > > - CPM5 has dedicated register space for control and status registers. > > - CPM5 legacy interrupt handling needs additional register bit > > to enable and handle legacy interrupts. > > > > Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com> > > --- > > drivers/pci/controller/pcie-xilinx-cpm.c | 33 > > +++++++++++++++++++++++- > > 1 file changed, 32 insertions(+), 1 deletion(-) > > Per MAINTAINERS, xilinx-cpm lacks a maintainer. Can we get one? > > > diff --git a/drivers/pci/controller/pcie-xilinx-cpm.c > > b/drivers/pci/controller/pcie-xilinx-cpm.c > > index c7cd44ed4dfc..a3b04083b6b3 100644 > > --- a/drivers/pci/controller/pcie-xilinx-cpm.c > > +++ b/drivers/pci/controller/pcie-xilinx-cpm.c > > @@ -35,6 +35,10 @@ > > #define XILINX_CPM_PCIE_MISC_IR_ENABLE 0x00000348 > > #define XILINX_CPM_PCIE_MISC_IR_LOCAL BIT(1) > > > > +#define XILINX_CPM_PCIE_IR_STATUS 0x000002A0 > > +#define XILINX_CPM_PCIE_IR_ENABLE 0x000002A8 > > +#define XILINX_CPM_PCIE_IR_LOCAL BIT(0) > > + > > /* Interrupt registers definitions */ > > #define XILINX_CPM_PCIE_INTR_LINK_DOWN 0 > > #define XILINX_CPM_PCIE_INTR_HOT_RESET 3 > > @@ -109,6 +113,7 @@ > > * @intx_irq: legacy interrupt number > > * @irq: Error interrupt number > > * @lock: lock protecting shared register access > > + * @is_cpm5: value to check cpm version > > s/cpm version/CPM version/ to match commit log usage. > > > + port->is_cpm5 = of_device_is_compatible(dev->of_node, > > + "xlnx,versal-cpm5-host"); > > One use of of_device_is_compatible() is OK, I guess, but > of_device_get_match_data() is a better pattern if we ever need more. > > I would lean toward of_device_get_match_data() even here, just to reduce > the number of ways to identify device-specific things across drivers. > Thanks Bjorn, will add this change in next patch. Regards, bharat ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-06-10 8:50 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-06-08 16:40 [PATCH v4 0/2] Add support for Xilinx Versal CPM5 Root Port Bharat Kumar Gogada 2022-06-08 16:40 ` [PATCH v4 1/2] dt-bindings: PCI: xilinx-cpm: Add " Bharat Kumar Gogada 2022-06-09 21:10 ` Rob Herring 2022-06-10 8:50 ` Bharat Kumar Gogada 2022-06-08 16:40 ` [PATCH v4 2/2] PCI: xilinx-cpm: Add support for " Bharat Kumar Gogada 2022-06-08 19:14 ` Bjorn Helgaas 2022-06-09 7:59 ` Michal Simek 2022-06-09 17:32 ` Bjorn Helgaas 2022-06-10 8:49 ` Bharat Kumar Gogada 2022-06-10 8:47 ` Bharat Kumar Gogada
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).