Re: [PATCH 4/4] ppc/pnv/pci: Fix PHB xscom registers memory region name

[Cédric Le Goater]

On 1/27/23 13:28, Frederic Barrat wrote:

The name is for the region mapping the PHB xscom registers. It was
apparently a bad cut-and-paste from the per-stack pci xscom area just
above, so we had two regions with the same name.

Signed-off-by: Frederic Barrat <fbar...@linux.ibm.com>

Reviewed-by: Cédric Le Goater <c...@kaod.org>

Thanks,

C.

---
  hw/pci-host/pnv_phb4.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/pci-host/pnv_phb4.c b/hw/pci-host/pnv_phb4.c
index ccbde841fc..542f9e2932 100644
--- a/hw/pci-host/pnv_phb4.c
+++ b/hw/pci-host/pnv_phb4.c
@@ -1497,7 +1497,7 @@ static void pnv_phb4_xscom_realize(PnvPHB4 *phb)
                            PHB4_PEC_PCI_STK_REGS_COUNT);

/* PHB pass-through */

-    snprintf(name, sizeof(name), "xscom-pec-%d.%d-pci-phb-%d",
+    snprintf(name, sizeof(name), "xscom-pec-%d.%d-phb-%d",
               pec->chip_id, pec->index, stack_no);
      pnv_xscom_region_init(&phb->phb_regs_mr, OBJECT(phb),
                            &pnv_phb4_xscom_ops, phb, name, 0x40);