[Skiboot] [PATCH 05/10] phb4: Fix endian of TLP headers print
    Michael Neuling 
    mikey at neuling.org
       
    Thu Jul 20 16:22:15 AEST 2017
    
    
  
Byte swap TLP headers so they are the same as the PCIe spec.
Also remove redundant print.
Suggested-by: Rob Lippert <rlippert at google.com>
Signed-off-by: Michael Neuling <mikey at neuling.org>
---
 hw/phb4.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/hw/phb4.c b/hw/phb4.c
index cf9ee61871..6c3ca60ae2 100644
--- a/hw/phb4.c
+++ b/hw/phb4.c
@@ -203,12 +203,12 @@ static void phb4_eeh_dump_regs(struct phb4 *p)
 	PHBERR(p, "       uncorrErrorStatus = %08x\n", s->uncorrErrorStatus);
 	PHBERR(p, "         corrErrorStatus = %08x\n", s->corrErrorStatus);
 	PHBERR(p, "       uncorrErrorStatus = %08x\n", s->uncorrErrorStatus);
-	PHBERR(p, "                 tlpHdr1 = %08x\n", s->tlpHdr1);
-	PHBERR(p, "                 tlpHdr2 = %08x\n", s->tlpHdr2);
-	PHBERR(p, "                 tlpHdr3 = %08x\n", s->tlpHdr3);
-	PHBERR(p, "                 tlpHdr4 = %08x\n", s->tlpHdr4);
+	/* Byte swap TLP headers so they are the same as the PCIe spec */
+	PHBERR(p, "                 tlpHdr1 = %08x\n", bswap_32(s->tlpHdr1));
+	PHBERR(p, "                 tlpHdr2 = %08x\n", bswap_32(s->tlpHdr2));
+	PHBERR(p, "                 tlpHdr3 = %08x\n", bswap_32(s->tlpHdr3));
+	PHBERR(p, "                 tlpHdr4 = %08x\n", bswap_32(s->tlpHdr4));
 	PHBERR(p, "                sourceId = %08x\n", s->sourceId);
-	PHBERR(p, "                 tlpHdr1 = %08x\n", s->tlpHdr1);
 	PHBERR(p, "                    nFir = %016llx\n", s->nFir);
 	PHBERR(p, "                nFirMask = %016llx\n", s->nFirMask);
 	PHBERR(p, "                 nFirWOF = %016llx\n", s->nFirWOF);
-- 
2.11.0
    
    
More information about the Skiboot
mailing list