> > > > > +HFI1 > > > > > + > > > > > + The hfi1 driver also creates these additional files: > > > > > + > > > > > + hw_rev - hardware revision > > >=20 > > > I'm checking on this to see if it is indeed a duplicate. > > >=20 > > > > Our hardware architect has indicated there is not PCIe equivalent for > > this = case. > > Please get someone knowledgeable at Intel to look at this. There is a (rev 04) > when using lspci on my nic here. This seems to be the hardware revision. >
I'm not sure what else to say other that the source for the sysfs file display is not from the PCI_REVISION_ID. > 00:19.0 Ethernet controller: Intel Corporation Ethernet Connection I217-LM > (rev 04) > > lspci.c does the following and to get that info. See where PCI_REVISION_ID is > used: > > > > static void > show_terse(struct device *d) > { > int c; > struct pci_dev *p = d->dev; > char classbuf[128], devbuf[128]; > > show_slot_name(d); > printf(" %s: %s", > pci_lookup_name(pacc, classbuf, sizeof(classbuf), > PCI_LOOKUP_CLASS, > p->device_class), > pci_lookup_name(pacc, devbuf, sizeof(devbuf), > PCI_LOOKUP_VENDOR | PCI_LOOKUP_DEVICE, > p->vendor_id, p->device_id)); > if (c = get_conf_byte(d, PCI_REVISION_ID)) > printf(" (rev %02x)", c); Note, that a zero value suppresses the print, which is the case right now. Mike -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html