[Skiboot] [PATCH 6/6] hw/phb3: Enable ECRC on endpoints

Gavin Shan gwshan at linux.vnet.ibm.com
Thu Mar 30 10:05:31 AEDT 2017


As the ECRC is updated properly on root port, we needn't care the
setting on endpoints. This reverts commit 60ce59ccd0e9 ("hw/phb3:
Disable ECRC on Broadcom adapter behind PMC switch"), meaning ECRC
is always enabled on endpoints.

Reported-by: Mark E Schreiter <markes at us.ibm.com>
Signed-off-by: Gavin Shan <gwshan at linux.vnet.ibm.com>
Tested-by: Mark E Schreiter <markes at us.ibm.com>
---
 hw/phb3.c | 26 ++------------------------
 1 file changed, 2 insertions(+), 24 deletions(-)

diff --git a/hw/phb3.c b/hw/phb3.c
index 3463ca1..bde6060 100644
--- a/hw/phb3.c
+++ b/hw/phb3.c
@@ -544,24 +544,6 @@ static void phb3_switch_port_init(struct phb *phb,
 	pci_cfg_write32(phb, bdfn, aercap + PCIECAP_AER_CAPCTL, val32);
 }
 
-static inline bool phb3_endpoint_report_ecrc(struct pci_device *pd)
-{
-	if (!pd || !pd->parent)
-		return true;
-
-	/* No ECRC generation and check on Broadcom ethernet adapter
-	 * when it seats behind a PMC's PCIe switch downstream port.
-	 * Otherwise, the Broadcom ethernet adapter's config space
-	 * can't be accessed because of frozen PE error even after
-	 * the frozen PE error is cleared.
-	 */
-	if (pd->vdid == 0x168a14e4 || // Broadcom bnx2x CHIP_NUM_57800
-	    pd->parent->vdid == 0x854611f8)
-		return false;
-
-	return true;
-}
-
 static void phb3_endpoint_init(struct phb *phb,
 			       struct pci_device *dev,
 			       int ecap, int aercap)
@@ -599,12 +581,8 @@ static void phb3_endpoint_init(struct phb *phb,
 
 	/* Enable ECRC generation and check */
 	pci_cfg_read32(phb, bdfn, aercap + PCIECAP_AER_CAPCTL, &val32);
-	if (phb3_endpoint_report_ecrc(dev))
-		val32 |= (PCIECAP_AER_CAPCTL_ECRCG_EN |
-			  PCIECAP_AER_CAPCTL_ECRCC_EN);
-	else
-		val32 &= ~(PCIECAP_AER_CAPCTL_ECRCG_EN |
-			   PCIECAP_AER_CAPCTL_ECRCC_EN);
+	val32 |= (PCIECAP_AER_CAPCTL_ECRCG_EN |
+		  PCIECAP_AER_CAPCTL_ECRCC_EN);
 	pci_cfg_write32(phb, bdfn, aercap + PCIECAP_AER_CAPCTL, val32);
 }
 
-- 
2.7.4



More information about the Skiboot mailing list