RE: [PATCH v4 40/50] IB/hfi1: add sysfs routines and documentation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > > > > +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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux