PCI: Use function 0 VPD for identical functions, regular VPD for others
authorAlex Williamson <alex.williamson@redhat.com>
Wed, 16 Sep 2015 04:24:46 +0000 (22:24 -0600)
committerBen Hutchings <ben@decadent.org.uk>
Fri, 27 Nov 2015 12:48:19 +0000 (12:48 +0000)
commitb2af40d1de4d8d3e22c8a6aeb785ab1d1c41801a
treed3c764691220b978374dcebe362995fc6482c4ea
parent991e923fb23b1fff62283078fd4734f42e1bf398
PCI: Use function 0 VPD for identical functions, regular VPD for others

commit da2d03ea27f6ed9d2005a67b20dd021ddacf1e4d upstream.

932c435caba8 ("PCI: Add dev_flags bit to access VPD through function 0")
added PCI_DEV_FLAGS_VPD_REF_F0.  Previously, we set the flag on every
non-zero function of quirked devices.  If a function turned out to be
different from function 0, i.e., it had a different class, vendor ID, or
device ID, the flag remained set but we didn't make VPD accessible at all.

Flip this around so we only set PCI_DEV_FLAGS_VPD_REF_F0 for functions that
are identical to function 0, and allow regular VPD access for any other
functions.

[bhelgaas: changelog, stable tag]
Fixes: 932c435caba8 ("PCI: Add dev_flags bit to access VPD through function 0")
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Bjorn Helgaas <helgaas@kernel.org>
Acked-by: Myron Stowe <myron.stowe@redhat.com>
Acked-by: Mark Rustad <mark.d.rustad@intel.com>
[bwh: Backported to 3.2: adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/pci/access.c
drivers/pci/quirks.c