[PATCH] PCI: xilinx-cpm: Add cpm5_csr register mapping for CPM5_HOST1 variant

[PATCH] PCI: xilinx-cpm: Add cpm5_csr register mapping for CPM5_HOST1 variant

[Thippeswamy Havalige]

This commit updates the CPM5 variant check to include CPM5_HOST1.
Previously, only CPM5 was considered when mapping the "cpm_csr" register.

With this change, CPM5_HOST1 is also supported, ensuring proper resource
mapping for this variant.

Signed-off-by: Thippeswamy Havalige <thippeswamy.havalige@amd.com>
---
 drivers/pci/controller/pcie-xilinx-cpm.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/pci/controller/pcie-xilinx-cpm.c b/drivers/pci/controller/pcie-xilinx-cpm.c
index d0ab187..13ca493 100644
--- a/drivers/pci/controller/pcie-xilinx-cpm.c
+++ b/drivers/pci/controller/pcie-xilinx-cpm.c
@@ -542,7 +542,8 @@ static int xilinx_cpm_pcie_parse_dt(struct xilinx_cpm_pcie *port,
 	if (IS_ERR(port->cfg))
 		return PTR_ERR(port->cfg);
 
-	if (port->variant->version == CPM5) {
+	if (port->variant->version == CPM5 ||
+	    port->variant->version == CPM5_HOST1) {
 		port->reg_base = devm_platform_ioremap_resource_byname(pdev,
 								    "cpm_csr");
 		if (IS_ERR(port->reg_base))
-- 
1.8.3.1

Re: [PATCH] PCI: xilinx-cpm: Add cpm5_csr register mapping for CPM5_HOST1 variant

[Krzysztof Wilczyński]

Hello,

> This commit updates the CPM5 variant check to include CPM5_HOST1.
> Previously, only CPM5 was considered when mapping the "cpm_csr" register.
> 
> With this change, CPM5_HOST1 is also supported, ensuring proper resource
> mapping for this variant.

Applied to controller/xilinx-cpm, thank you!

	Krzysztof

Re: [PATCH] PCI: xilinx-cpm: Add cpm5_csr register mapping for CPM5_HOST1 variant

[Krzysztof Wilczyński]

Hello,

> This commit updates the CPM5 variant check to include CPM5_HOST1.
> Previously, only CPM5 was considered when mapping the "cpm_csr" register.

The subject says "cpm5_csr" but here we say "cpm_csr", I think it's only
the latter?  Let me know, so I can update the branch correctly.  Maybe both
things are correct.  I am just double-checking.

Thank you!

	Krzysztof

RE: [PATCH] PCI: xilinx-cpm: Add cpm5_csr register mapping for CPM5_HOST1 variant

[Havalige, Thippeswamy]

[AMD Official Use Only - AMD Internal Distribution Only]

HI Krzysztof

> -----Original Message-----
> From: Krzysztof Wilczyński <kw@linux.com>
> Sent: Sunday, March 23, 2025 6:21 PM
> To: Havalige, Thippeswamy <thippeswamy.havalige@amd.com>
> Cc: bhelgaas@google.com; lpieralisi@kernel.org;
> manivannan.sadhasivam@linaro.org; robh@kernel.org; krzk+dt@kernel.org;
> conor+dt@kernel.org; linux-pci@vger.kernel.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org; Simek, Michal <michal.simek@amd.com>;
> Gogada, Bharat Kumar <bharat.kumar.gogada@amd.com>
> Subject: Re: [PATCH] PCI: xilinx-cpm: Add cpm5_csr register mapping for
> CPM5_HOST1 variant
>
> Hello,
>
> > This commit updates the CPM5 variant check to include CPM5_HOST1.
> > Previously, only CPM5 was considered when mapping the "cpm_csr"
> register.
>
> The subject says "cpm5_csr" but here we say "cpm_csr", I think it's only
> the latter?  Let me know, so I can update the branch correctly.  Maybe both
> things are correct.  I am just double-checking.
Yes, You're correct, cpm_csr is correct one and not cpm5_csr.

Thank you for correcting.

Regards,
Thippeswamy H
>
> Thank you!
>
>       Krzysztof