]> git.ipfire.org Git - thirdparty/pciutils.git/blobdiff - ls-caps.c
lspci: Clarify unknown capability IDs
[thirdparty/pciutils.git] / ls-caps.c
index 7ff6c67884dd31e8b1b8b853b629465b98e4b85c..bc7829c120e59844b104107920cda10e430d5d7e 100644 (file)
--- a/ls-caps.c
+++ b/ls-caps.c
@@ -1,7 +1,7 @@
 /*
  *     The PCI Utilities -- Show Capabilities
  *
- *     Copyright (c) 1997--2010 Martin Mares <mj@ucw.cz>
+ *     Copyright (c) 1997--2018 Martin Mares <mj@ucw.cz>
  *
  *     Can be freely distributed and used under the terms of the GNU GPL.
  */
@@ -25,7 +25,7 @@ cap_pm(struct device *d, int where, int cap)
         FLAG(cap, PCI_PM_CAP_DSI),
         FLAG(cap, PCI_PM_CAP_D1),
         FLAG(cap, PCI_PM_CAP_D2),
-        pm_aux_current[(cap >> 6) & 7],
+        pm_aux_current[(cap & PCI_PM_CAP_AUX_C_MASK) >> 6],
         FLAG(cap, PCI_PM_CAP_PME_D0),
         FLAG(cap, PCI_PM_CAP_PME_D1),
         FLAG(cap, PCI_PM_CAP_PME_D2),
@@ -136,17 +136,17 @@ cap_pcix_nobridge(struct device *d, int where)
         1 << (9 + ((command & PCI_PCIX_COMMAND_MAX_MEM_READ_BYTE_COUNT) >> 2U)),
         max_outstanding[(command & PCI_PCIX_COMMAND_MAX_OUTSTANDING_SPLIT_TRANS) >> 4U]);
   printf("\t\tStatus: Dev=%02x:%02x.%d 64bit%c 133MHz%c SCD%c USC%c DC=%s DMMRBC=%u DMOST=%u DMCRS=%u RSCEM%c 266MHz%c 533MHz%c\n",
-        ((status >> 8) & 0xff),
-        ((status >> 3) & 0x1f),
+        (status & PCI_PCIX_STATUS_BUS) >> 8,
+        (status & PCI_PCIX_STATUS_DEVICE) >> 3,
         (status & PCI_PCIX_STATUS_FUNCTION),
         FLAG(status, PCI_PCIX_STATUS_64BIT),
         FLAG(status, PCI_PCIX_STATUS_133MHZ),
         FLAG(status, PCI_PCIX_STATUS_SC_DISCARDED),
         FLAG(status, PCI_PCIX_STATUS_UNEXPECTED_SC),
         ((status & PCI_PCIX_STATUS_DEVICE_COMPLEXITY) ? "bridge" : "simple"),
-        1 << (9 + ((status >> 21) & 3U)),
-        max_outstanding[(status >> 23) & 7U],
-        1 << (3 + ((status >> 26) & 7U)),
+        1 << (9 + ((status & PCI_PCIX_STATUS_DESIGNED_MAX_MEM_READ_BYTE_COUNT) >> 21)),
+        max_outstanding[(status & PCI_PCIX_STATUS_DESIGNED_MAX_OUTSTANDING_SPLIT_TRANS) >> 23],
+        1 << (3 + ((status & PCI_PCIX_STATUS_DESIGNED_MAX_CUMULATIVE_READ_SIZE) >> 26)),
         FLAG(status, PCI_PCIX_STATUS_RCVD_SC_ERR_MESS),
         FLAG(status, PCI_PCIX_STATUS_266MHZ),
         FLAG(status, PCI_PCIX_STATUS_533MHZ));
@@ -175,11 +175,11 @@ cap_pcix_bridge(struct device *d, int where)
         FLAG(secstatus, PCI_PCIX_BRIDGE_SEC_STATUS_UNEXPECTED_SC),
         FLAG(secstatus, PCI_PCIX_BRIDGE_SEC_STATUS_SC_OVERRUN),
         FLAG(secstatus, PCI_PCIX_BRIDGE_SEC_STATUS_SPLIT_REQUEST_DELAYED),
-        sec_clock_freq[(secstatus >> 6) & 7]);
+        sec_clock_freq[(secstatus & PCI_PCIX_BRIDGE_SEC_STATUS_CLOCK_FREQ) >> 6]);
   status = get_conf_long(d, where + PCI_PCIX_BRIDGE_STATUS);
   printf("\t\tStatus: Dev=%02x:%02x.%d 64bit%c 133MHz%c SCD%c USC%c SCO%c SRD%c\n",
-        ((status >> 8) & 0xff),
-        ((status >> 3) & 0x1f),
+        (status & PCI_PCIX_BRIDGE_STATUS_BUS) >> 8,
+        (status & PCI_PCIX_BRIDGE_STATUS_DEVICE) >> 3,
         (status & PCI_PCIX_BRIDGE_STATUS_FUNCTION),
         FLAG(status, PCI_PCIX_BRIDGE_STATUS_64BIT),
         FLAG(status, PCI_PCIX_BRIDGE_STATUS_133MHZ),
@@ -231,7 +231,6 @@ cap_ht_pri(struct device *d, int where, int cmd)
 {
   u16 lctr0, lcnf0, lctr1, lcnf1, eh;
   u8 rid, lfrer0, lfcap0, ftr, lfrer1, lfcap1, mbu, mlu, bn;
-  char *fmt;
 
   printf("HyperTransport: Slave or Primary Interface\n");
   if (verbose < 2)
@@ -243,22 +242,17 @@ cap_ht_pri(struct device *d, int where, int cmd)
   if (rid < 0x22 && rid > 0x11)
     printf("\t\t!!! Possibly incomplete decoding\n");
 
-  if (rid >= 0x22)
-    fmt = "\t\tCommand: BaseUnitID=%u UnitCnt=%u MastHost%c DefDir%c DUL%c\n";
-  else
-    fmt = "\t\tCommand: BaseUnitID=%u UnitCnt=%u MastHost%c DefDir%c\n";
-  printf(fmt,
+  printf("\t\tCommand: BaseUnitID=%u UnitCnt=%u MastHost%c DefDir%c",
         (cmd & PCI_HT_PRI_CMD_BUID),
         (cmd & PCI_HT_PRI_CMD_UC) >> 5,
         FLAG(cmd, PCI_HT_PRI_CMD_MH),
-        FLAG(cmd, PCI_HT_PRI_CMD_DD),
-        FLAG(cmd, PCI_HT_PRI_CMD_DUL));
-  lctr0 = get_conf_word(d, where + PCI_HT_PRI_LCTR0);
+        FLAG(cmd, PCI_HT_PRI_CMD_DD));
   if (rid >= 0x22)
-    fmt = "\t\tLink Control 0: CFlE%c CST%c CFE%c <LkFail%c Init%c EOC%c TXO%c <CRCErr=%x IsocEn%c LSEn%c ExtCTL%c 64b%c\n";
-  else
-    fmt = "\t\tLink Control 0: CFlE%c CST%c CFE%c <LkFail%c Init%c EOC%c TXO%c <CRCErr=%x\n";
-  printf(fmt,
+    printf(" DUL%c", FLAG(cmd, PCI_HT_PRI_CMD_DUL));
+  printf("\n");
+
+  lctr0 = get_conf_word(d, where + PCI_HT_PRI_LCTR0);
+  printf("\t\tLink Control 0: CFlE%c CST%c CFE%c <LkFail%c Init%c EOC%c TXO%c <CRCErr=%x",
         FLAG(lctr0, PCI_HT_LCTR_CFLE),
         FLAG(lctr0, PCI_HT_LCTR_CST),
         FLAG(lctr0, PCI_HT_LCTR_CFE),
@@ -266,31 +260,35 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(lctr0, PCI_HT_LCTR_INIT),
         FLAG(lctr0, PCI_HT_LCTR_EOC),
         FLAG(lctr0, PCI_HT_LCTR_TXO),
-        (lctr0 & PCI_HT_LCTR_CRCERR) >> 8,
-        FLAG(lctr0, PCI_HT_LCTR_ISOCEN),
-        FLAG(lctr0, PCI_HT_LCTR_LSEN),
-        FLAG(lctr0, PCI_HT_LCTR_EXTCTL),
-        FLAG(lctr0, PCI_HT_LCTR_64B));
-  lcnf0 = get_conf_word(d, where + PCI_HT_PRI_LCNF0);
+        (lctr0 & PCI_HT_LCTR_CRCERR) >> 8);
   if (rid >= 0x22)
-    fmt = "\t\tLink Config 0: MLWI=%1$s DwFcIn%5$c MLWO=%2$s DwFcOut%6$c LWI=%3$s DwFcInEn%7$c LWO=%4$s DwFcOutEn%8$c\n";
+    printf(" IsocEn%c LSEn%c ExtCTL%c 64b%c",
+          FLAG(lctr0, PCI_HT_LCTR_ISOCEN),
+          FLAG(lctr0, PCI_HT_LCTR_LSEN),
+          FLAG(lctr0, PCI_HT_LCTR_EXTCTL),
+          FLAG(lctr0, PCI_HT_LCTR_64B));
+  printf("\n");
+
+  lcnf0 = get_conf_word(d, where + PCI_HT_PRI_LCNF0);
+  if (rid < 0x22)
+    printf("\t\tLink Config 0: MLWI=%s MLWO=%s LWI=%s LWO=%s\n",
+          ht_link_width(lcnf0 & PCI_HT_LCNF_MLWI),
+          ht_link_width((lcnf0 & PCI_HT_LCNF_MLWO) >> 4),
+          ht_link_width((lcnf0 & PCI_HT_LCNF_LWI) >> 8),
+          ht_link_width((lcnf0 & PCI_HT_LCNF_LWO) >> 12));
   else
-    fmt = "\t\tLink Config 0: MLWI=%s MLWO=%s LWI=%s LWO=%s\n";
-  printf(fmt,
-        ht_link_width(lcnf0 & PCI_HT_LCNF_MLWI),
-        ht_link_width((lcnf0 & PCI_HT_LCNF_MLWO) >> 4),
-        ht_link_width((lcnf0 & PCI_HT_LCNF_LWI) >> 8),
-        ht_link_width((lcnf0 & PCI_HT_LCNF_LWO) >> 12),
-        FLAG(lcnf0, PCI_HT_LCNF_DFI),
-        FLAG(lcnf0, PCI_HT_LCNF_DFO),
-        FLAG(lcnf0, PCI_HT_LCNF_DFIE),
-        FLAG(lcnf0, PCI_HT_LCNF_DFOE));
+    printf("\t\tLink Config 0: MLWI=%s DwFcIn%c MLWO=%s DwFcOut%c LWI=%s DwFcInEn%c LWO=%s DwFcOutEn%c\n",
+           ht_link_width(lcnf0 & PCI_HT_LCNF_MLWI),
+          FLAG(lcnf0, PCI_HT_LCNF_DFI),
+          ht_link_width((lcnf0 & PCI_HT_LCNF_MLWO) >> 4),
+          FLAG(lcnf0, PCI_HT_LCNF_DFO),
+          ht_link_width((lcnf0 & PCI_HT_LCNF_LWI) >> 8),
+          FLAG(lcnf0, PCI_HT_LCNF_DFIE),
+          ht_link_width((lcnf0 & PCI_HT_LCNF_LWO) >> 12),
+          FLAG(lcnf0, PCI_HT_LCNF_DFOE));
+
   lctr1 = get_conf_word(d, where + PCI_HT_PRI_LCTR1);
-  if (rid >= 0x22)
-    fmt = "\t\tLink Control 1: CFlE%c CST%c CFE%c <LkFail%c Init%c EOC%c TXO%c <CRCErr=%x IsocEn%c LSEn%c ExtCTL%c 64b%c\n";
-  else
-    fmt = "\t\tLink Control 1: CFlE%c CST%c CFE%c <LkFail%c Init%c EOC%c TXO%c <CRCErr=%x\n";
-  printf(fmt,
+  printf("\t\tLink Control 1: CFlE%c CST%c CFE%c <LkFail%c Init%c EOC%c TXO%c <CRCErr=%x",
         FLAG(lctr1, PCI_HT_LCTR_CFLE),
         FLAG(lctr1, PCI_HT_LCTR_CST),
         FLAG(lctr1, PCI_HT_LCTR_CFE),
@@ -298,29 +296,38 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(lctr1, PCI_HT_LCTR_INIT),
         FLAG(lctr1, PCI_HT_LCTR_EOC),
         FLAG(lctr1, PCI_HT_LCTR_TXO),
-        (lctr1 & PCI_HT_LCTR_CRCERR) >> 8,
+        (lctr1 & PCI_HT_LCTR_CRCERR) >> 8);
+  if (rid >= 0x22)
+    printf(" IsocEn%c LSEn%c ExtCTL%c 64b%c",
         FLAG(lctr1, PCI_HT_LCTR_ISOCEN),
         FLAG(lctr1, PCI_HT_LCTR_LSEN),
         FLAG(lctr1, PCI_HT_LCTR_EXTCTL),
         FLAG(lctr1, PCI_HT_LCTR_64B));
+  printf("\n");
+
   lcnf1 = get_conf_word(d, where + PCI_HT_PRI_LCNF1);
-  if (rid >= 0x22)
-    fmt = "\t\tLink Config 1: MLWI=%1$s DwFcIn%5$c MLWO=%2$s DwFcOut%6$c LWI=%3$s DwFcInEn%7$c LWO=%4$s DwFcOutEn%8$c\n";
+  if (rid < 0x22)
+    printf("\t\tLink Config 1: MLWI=%s MLWO=%s LWI=%s LWO=%s\n",
+          ht_link_width(lcnf1 & PCI_HT_LCNF_MLWI),
+          ht_link_width((lcnf1 & PCI_HT_LCNF_MLWO) >> 4),
+          ht_link_width((lcnf1 & PCI_HT_LCNF_LWI) >> 8),
+          ht_link_width((lcnf1 & PCI_HT_LCNF_LWO) >> 12));
   else
-    fmt = "\t\tLink Config 1: MLWI=%s MLWO=%s LWI=%s LWO=%s\n";
-  printf(fmt,
-        ht_link_width(lcnf1 & PCI_HT_LCNF_MLWI),
-        ht_link_width((lcnf1 & PCI_HT_LCNF_MLWO) >> 4),
-        ht_link_width((lcnf1 & PCI_HT_LCNF_LWI) >> 8),
-        ht_link_width((lcnf1 & PCI_HT_LCNF_LWO) >> 12),
-        FLAG(lcnf1, PCI_HT_LCNF_DFI),
-        FLAG(lcnf1, PCI_HT_LCNF_DFO),
-        FLAG(lcnf1, PCI_HT_LCNF_DFIE),
-        FLAG(lcnf1, PCI_HT_LCNF_DFOE));
+    printf("\t\tLink Config 1: MLWI=%s DwFcIn%c MLWO=%s DwFcOut%c LWI=%s DwFcInEn%c LWO=%s DwFcOutEn%c\n",
+          ht_link_width(lcnf1 & PCI_HT_LCNF_MLWI),
+          FLAG(lcnf1, PCI_HT_LCNF_DFI),
+          ht_link_width((lcnf1 & PCI_HT_LCNF_MLWO) >> 4),
+          FLAG(lcnf1, PCI_HT_LCNF_DFO),
+          ht_link_width((lcnf1 & PCI_HT_LCNF_LWI) >> 8),
+          FLAG(lcnf1, PCI_HT_LCNF_DFIE),
+          ht_link_width((lcnf1 & PCI_HT_LCNF_LWO) >> 12),
+          FLAG(lcnf1, PCI_HT_LCNF_DFOE));
+
   printf("\t\tRevision ID: %u.%02u\n",
         (rid & PCI_HT_RID_MAJ) >> 5, (rid & PCI_HT_RID_MIN));
   if (rid < 0x22)
     return;
+
   lfrer0 = get_conf_byte(d, where + PCI_HT_PRI_LFRER0);
   printf("\t\tLink Frequency 0: %s\n", ht_link_freq(lfrer0 & PCI_HT_LFRER_FREQ));
   printf("\t\tLink Error 0: <Prot%c <Ovfl%c <EOC%c CTLTm%c\n",
@@ -328,6 +335,7 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(lfrer0, PCI_HT_LFRER_OV),
         FLAG(lfrer0, PCI_HT_LFRER_EOC),
         FLAG(lfrer0, PCI_HT_LFRER_CTLT));
+
   lfcap0 = get_conf_byte(d, where + PCI_HT_PRI_LFCAP0);
   printf("\t\tLink Frequency Capability 0: 200MHz%c 300MHz%c 400MHz%c 500MHz%c 600MHz%c 800MHz%c 1.0GHz%c 1.2GHz%c 1.4GHz%c 1.6GHz%c Vend%c\n",
         FLAG(lfcap0, PCI_HT_LFCAP_200),
@@ -341,6 +349,7 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(lfcap0, PCI_HT_LFCAP_1400),
         FLAG(lfcap0, PCI_HT_LFCAP_1600),
         FLAG(lfcap0, PCI_HT_LFCAP_VEND));
+
   ftr = get_conf_byte(d, where + PCI_HT_PRI_FTR);
   printf("\t\tFeature Capability: IsocFC%c LDTSTOP%c CRCTM%c ECTLT%c 64bA%c UIDRD%c\n",
         FLAG(ftr, PCI_HT_FTR_ISOCFC),
@@ -349,6 +358,7 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(ftr, PCI_HT_FTR_ECTLT),
         FLAG(ftr, PCI_HT_FTR_64BA),
         FLAG(ftr, PCI_HT_FTR_UIDRD));
+
   lfrer1 = get_conf_byte(d, where + PCI_HT_PRI_LFRER1);
   printf("\t\tLink Frequency 1: %s\n", ht_link_freq(lfrer1 & PCI_HT_LFRER_FREQ));
   printf("\t\tLink Error 1: <Prot%c <Ovfl%c <EOC%c CTLTm%c\n",
@@ -356,6 +366,7 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(lfrer1, PCI_HT_LFRER_OV),
         FLAG(lfrer1, PCI_HT_LFRER_EOC),
         FLAG(lfrer1, PCI_HT_LFRER_CTLT));
+
   lfcap1 = get_conf_byte(d, where + PCI_HT_PRI_LFCAP1);
   printf("\t\tLink Frequency Capability 1: 200MHz%c 300MHz%c 400MHz%c 500MHz%c 600MHz%c 800MHz%c 1.0GHz%c 1.2GHz%c 1.4GHz%c 1.6GHz%c Vend%c\n",
         FLAG(lfcap1, PCI_HT_LFCAP_200),
@@ -369,6 +380,7 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(lfcap1, PCI_HT_LFCAP_1400),
         FLAG(lfcap1, PCI_HT_LFCAP_1600),
         FLAG(lfcap1, PCI_HT_LFCAP_VEND));
+
   eh = get_conf_word(d, where + PCI_HT_PRI_EH);
   printf("\t\tError Handling: PFlE%c OFlE%c PFE%c OFE%c EOCFE%c RFE%c CRCFE%c SERRFE%c CF%c RE%c PNFE%c ONFE%c EOCNFE%c RNFE%c CRCNFE%c SERRNFE%c\n",
         FLAG(eh, PCI_HT_EH_PFLE),
@@ -387,9 +399,11 @@ cap_ht_pri(struct device *d, int where, int cmd)
         FLAG(eh, PCI_HT_EH_RNFE),
         FLAG(eh, PCI_HT_EH_CRCNFE),
         FLAG(eh, PCI_HT_EH_SERRNFE));
+
   mbu = get_conf_byte(d, where + PCI_HT_PRI_MBU);
   mlu = get_conf_byte(d, where + PCI_HT_PRI_MLU);
   printf("\t\tPrefetchable memory behind bridge Upper: %02x-%02x\n", mbu, mlu);
+
   bn = get_conf_byte(d, where + PCI_HT_PRI_BN);
   printf("\t\tBus Number: %02x\n", bn);
 }
@@ -686,7 +700,7 @@ static void cap_express_dev(struct device *d, int where, int type)
   printf("\n");
 
   w = get_conf_word(d, where + PCI_EXP_DEVCTL);
-  printf("\t\tDevCtl:\tReport errors: Correctable%c Non-Fatal%c Fatal%c Unsupported%c\n",
+  printf("\t\tDevCtl:\tCorrErr%c NonFatalErr%c FatalErr%c UnsupReq%c\n",
        FLAG(w, PCI_EXP_DEVCTL_CERE),
        FLAG(w, PCI_EXP_DEVCTL_NFERE),
        FLAG(w, PCI_EXP_DEVCTL_FERE),
@@ -707,7 +721,7 @@ static void cap_express_dev(struct device *d, int where, int type)
        128 << ((w & PCI_EXP_DEVCTL_READRQ) >> 12));
 
   w = get_conf_word(d, where + PCI_EXP_DEVSTA);
-  printf("\t\tDevSta:\tCorrErr%c UncorrErr%c FatalErr%c UnsuppReq%c AuxPwr%c TransPend%c\n",
+  printf("\t\tDevSta:\tCorrErr%c NonFatalErr%c FatalErr%c UnsupReq%c AuxPwr%c TransPend%c\n",
        FLAG(w, PCI_EXP_DEVSTA_CED),
        FLAG(w, PCI_EXP_DEVSTA_NFED),
        FLAG(w, PCI_EXP_DEVSTA_FED),
@@ -726,11 +740,22 @@ static char *link_speed(int speed)
        return "5GT/s";
       case 3:
        return "8GT/s";
+      case 4:
+        return "16GT/s";
       default:
        return "unknown";
     }
 }
 
+static char *link_compare(int sta, int cap)
+{
+  if (sta < cap)
+    return "downgraded";
+  if (sta > cap)
+    return "strange";
+  return "ok";
+}
+
 static char *aspm_support(int code)
 {
   switch (code)
@@ -756,16 +781,27 @@ static const char *aspm_enabled(int code)
 
 static void cap_express_link(struct device *d, int where, int type)
 {
-  u32 t;
+  u32 t, aspm, cap_speed, cap_width, sta_speed, sta_width;
   u16 w;
 
   t = get_conf_long(d, where + PCI_EXP_LNKCAP);
-  printf("\t\tLnkCap:\tPort #%d, Speed %s, Width x%d, ASPM %s, Exit Latency L0s %s, L1 %s\n",
+  aspm = (t & PCI_EXP_LNKCAP_ASPM) >> 10;
+  cap_speed = t & PCI_EXP_LNKCAP_SPEED;
+  cap_width = (t & PCI_EXP_LNKCAP_WIDTH) >> 4;
+  printf("\t\tLnkCap:\tPort #%d, Speed %s, Width x%d, ASPM %s",
        t >> 24,
-       link_speed(t & PCI_EXP_LNKCAP_SPEED), (t & PCI_EXP_LNKCAP_WIDTH) >> 4,
-       aspm_support((t & PCI_EXP_LNKCAP_ASPM) >> 10),
-       latency_l0s((t & PCI_EXP_LNKCAP_L0S) >> 12),
-       latency_l1((t & PCI_EXP_LNKCAP_L1) >> 15));
+       link_speed(cap_speed), cap_width,
+       aspm_support(aspm));
+  if (aspm)
+    {
+      printf(", Exit Latency ");
+      if (aspm & 1)
+       printf("L0s %s", latency_l0s((t & PCI_EXP_LNKCAP_L0S) >> 12));
+      if (aspm & 2)
+       printf("%sL1 %s", (aspm & 1) ? ", " : "",
+           latency_l1((t & PCI_EXP_LNKCAP_L1) >> 15));
+    }
+  printf("\n");
   printf("\t\t\tClockPM%c Surprise%c LLActRep%c BwNot%c ASPMOptComp%c\n",
        FLAG(t, PCI_EXP_LNKCAP_CLOCKPM),
        FLAG(t, PCI_EXP_LNKCAP_SURPRISE),
@@ -788,9 +824,14 @@ static void cap_express_link(struct device *d, int where, int type)
        FLAG(w, PCI_EXP_LNKCTL_AUTBWIE));
 
   w = get_conf_word(d, where + PCI_EXP_LNKSTA);
-  printf("\t\tLnkSta:\tSpeed %s, Width x%d, TrErr%c Train%c SlotClk%c DLActive%c BWMgmt%c ABWMgmt%c\n",
-       link_speed(w & PCI_EXP_LNKSTA_SPEED),
-       (w & PCI_EXP_LNKSTA_WIDTH) >> 4,
+  sta_speed = w & PCI_EXP_LNKSTA_SPEED;
+  sta_width = (w & PCI_EXP_LNKSTA_WIDTH) >> 4;
+  printf("\t\tLnkSta:\tSpeed %s (%s), Width x%d (%s)\n",
+       link_speed(sta_speed),
+       link_compare(sta_speed, cap_speed),
+       sta_width,
+       link_compare(sta_width, cap_width));
+  printf("\t\t\tTrErr%c Train%c SlotClk%c DLActive%c BWMgmt%c ABWMgmt%c\n",
        FLAG(w, PCI_EXP_LNKSTA_TR_ERR),
        FLAG(w, PCI_EXP_LNKSTA_TRAIN),
        FLAG(w, PCI_EXP_LNKSTA_SL_CLK),
@@ -820,7 +861,7 @@ static void cap_express_slot(struct device *d, int where)
        FLAG(t, PCI_EXP_SLTCAP_HPC),
        FLAG(t, PCI_EXP_SLTCAP_HPS));
   printf("\t\t\tSlot #%d, PowerLimit %.3fW; Interlock%c NoCompl%c\n",
-       t >> 19,
+       (t & PCI_EXP_SLTCAP_PSN) >> 19,
        power_limit((t & PCI_EXP_SLTCAP_PWR_VAL) >> 7, (t & PCI_EXP_SLTCAP_PWR_SCL) >> 15),
        FLAG(t, PCI_EXP_SLTCAP_INTERLOCK),
        FLAG(t, PCI_EXP_SLTCAP_NOCMDCOMP));
@@ -868,7 +909,7 @@ static void cap_express_root(struct device *d, int where)
   printf("\t\tRootCap: CRSVisible%c\n",
        FLAG(w, PCI_EXP_RTCAP_CRSVIS));
 
-  w = get_conf_word(d, where + PCI_EXP_RTSTA);
+  w = get_conf_long(d, where + PCI_EXP_RTSTA);
   printf("\t\tRootSta: PME ReqID %04x, PMEStatus%c PMEPending%c\n",
        w & PCI_EXP_RTSTA_PME_REQID,
        FLAG(w, PCI_EXP_RTSTA_PME_STATUS),
@@ -961,10 +1002,29 @@ static const char *cap_express_devctl2_obff(int obff)
     }
 }
 
+static int
+device_has_memory_space_bar(struct device *d)
+{
+  struct pci_dev *p = d->dev;
+  int i, found = 0;
+
+  for (i=0; i<6; i++)
+    if (p->base_addr[i] && p->size[i])
+      {
+        if (!(p->base_addr[i] & PCI_BASE_ADDRESS_SPACE_IO))
+          {
+            found = 1;
+            break;
+          }
+      }
+  return found;
+}
+
 static void cap_express_dev2(struct device *d, int where, int type)
 {
   u32 l;
   u16 w;
+  int has_mem_bar = device_has_memory_space_bar(d);
 
   l = get_conf_long(d, where + PCI_EXP_DEVCAP2);
   printf("\t\tDevCap2: Completion Timeout: %s, TimeoutDis%c, LTR%c, OBFF %s",
@@ -976,6 +1036,20 @@ static void cap_express_dev2(struct device *d, int where, int type)
     printf(" ARIFwd%c\n", FLAG(l, PCI_EXP_DEV2_ARI));
   else
     printf("\n");
+  if (type == PCI_EXP_TYPE_ROOT_PORT || type == PCI_EXP_TYPE_UPSTREAM ||
+      type == PCI_EXP_TYPE_DOWNSTREAM || has_mem_bar)
+    {
+       printf("\t\t\t AtomicOpsCap:");
+       if (type == PCI_EXP_TYPE_ROOT_PORT || type == PCI_EXP_TYPE_UPSTREAM ||
+           type == PCI_EXP_TYPE_DOWNSTREAM)
+         printf(" Routing%c", FLAG(l, PCI_EXP_DEVCAP2_ATOMICOP_ROUTING));
+       if (type == PCI_EXP_TYPE_ROOT_PORT || has_mem_bar)
+         printf(" 32bit%c 64bit%c 128bitCAS%c",
+               FLAG(l, PCI_EXP_DEVCAP2_32BIT_ATOMICOP_COMP),
+               FLAG(l, PCI_EXP_DEVCAP2_64BIT_ATOMICOP_COMP),
+               FLAG(l, PCI_EXP_DEVCAP2_128BIT_CAS_COMP));
+       printf("\n");
+    }
 
   w = get_conf_word(d, where + PCI_EXP_DEVCTL2);
   printf("\t\tDevCtl2: Completion Timeout: %s, TimeoutDis%c, LTR%c, OBFF %s",
@@ -987,6 +1061,19 @@ static void cap_express_dev2(struct device *d, int where, int type)
     printf(" ARIFwd%c\n", FLAG(w, PCI_EXP_DEV2_ARI));
   else
     printf("\n");
+  if (type == PCI_EXP_TYPE_ROOT_PORT || type == PCI_EXP_TYPE_UPSTREAM ||
+      type == PCI_EXP_TYPE_DOWNSTREAM || type == PCI_EXP_TYPE_ENDPOINT ||
+      type == PCI_EXP_TYPE_ROOT_INT_EP || type == PCI_EXP_TYPE_LEG_END)
+    {
+      printf("\t\t\t AtomicOpsCtl:");
+      if (type == PCI_EXP_TYPE_ROOT_PORT || type == PCI_EXP_TYPE_ENDPOINT ||
+          type == PCI_EXP_TYPE_ROOT_INT_EP || type == PCI_EXP_TYPE_LEG_END)
+        printf(" ReqEn%c", FLAG(w, PCI_EXP_DEV2_ATOMICOP_REQUESTER_EN));
+      if (type == PCI_EXP_TYPE_ROOT_PORT || type == PCI_EXP_TYPE_UPSTREAM ||
+          type == PCI_EXP_TYPE_DOWNSTREAM)
+        printf(" EgressBlck%c", FLAG(w, PCI_EXP_DEV2_ATOMICOP_EGRESS_BLOCK));
+      printf("\n");
+    }
 }
 
 static const char *cap_express_link2_speed(int type)
@@ -1000,6 +1087,8 @@ static const char *cap_express_link2_speed(int type)
        return "5GT/s";
       case 3:
        return "8GT/s";
+      case 4:
+        return "16GT/s";
       default:
        return "Unknown";
     }
@@ -1075,7 +1164,7 @@ static void cap_express_slot2(struct device *d UNUSED, int where UNUSED)
   /* No capabilities that require this field in PCIe rev2.0 spec. */
 }
 
-static void
+static int
 cap_express(struct device *d, int where, int cap)
 {
   int type = (cap & PCI_EXP_FLAGS_TYPE) >> 4;
@@ -1109,7 +1198,9 @@ cap_express(struct device *d, int where, int cap)
       printf("PCI-Express to PCI/PCI-X Bridge");
       break;
     case PCI_EXP_TYPE_PCIE_BRIDGE:
-      printf("PCI/PCI-X to PCI-Express Bridge");
+      slot = cap & PCI_EXP_FLAGS_SLOT;
+      printf("PCI/PCI-X to PCI-Express Bridge (Slot%c)",
+            FLAG(cap, PCI_EXP_FLAGS_SLOT));
       break;
     case PCI_EXP_TYPE_ROOT_INT_EP:
       link = 0;
@@ -1124,7 +1215,7 @@ cap_express(struct device *d, int where, int cap)
   }
   printf(", MSI %02x\n", (cap & PCI_EXP_FLAGS_IRQ) >> 9);
   if (verbose < 2)
-    return;
+    return type;
 
   size = 16;
   if (slot)
@@ -1132,7 +1223,7 @@ cap_express(struct device *d, int where, int cap)
   if (type == PCI_EXP_TYPE_ROOT_PORT)
     size = 32;
   if (!config_fetch(d, where + PCI_EXP_DEVCAP, size))
-    return;
+    return type;
 
   cap_express_dev(d, where, type);
   if (link)
@@ -1143,19 +1234,20 @@ cap_express(struct device *d, int where, int cap)
     cap_express_root(d, where);
 
   if ((cap & PCI_EXP_FLAGS_VERS) < 2)
-    return;
+    return type;
 
   size = 16;
   if (slot)
     size = 24;
   if (!config_fetch(d, where + PCI_EXP_DEVCAP2, size))
-    return;
+    return type;
 
   cap_express_dev2(d, where, type);
   if (link)
     cap_express_link2(d, where, type);
   if (slot)
     cap_express_slot2(d, where);
+  return type;
 }
 
 static void
@@ -1412,6 +1504,7 @@ void
 show_caps(struct device *d, int where)
 {
   int can_have_ext_caps = 0;
+  int type = -1;
 
   if (get_conf_word(d, PCI_STATUS) & PCI_STATUS_CAP_LIST)
     {
@@ -1490,7 +1583,7 @@ show_caps(struct device *d, int where)
              printf("Secure device <?>\n");
              break;
            case PCI_CAP_ID_EXP:
-             cap_express(d, where, cap);
+             type = cap_express(d, where, cap);
              can_have_ext_caps = 1;
              break;
            case PCI_CAP_ID_MSIX:
@@ -1506,11 +1599,11 @@ show_caps(struct device *d, int where)
              cap_ea(d, where, cap);
              break;
            default:
-             printf("#%02x [%04x]\n", id, cap);
+             printf("Capability ID %#02x [%04x]\n", id, cap);
            }
          where = next;
        }
     }
   if (can_have_ext_caps)
-    show_ext_caps(d);
+    show_ext_caps(d, type);
 }