Hi Bjorn, On 11/14/2016 1:52 PM, Bjorn Helgaas wrote: > On Mon, Oct 31, 2016 at 05:38:33PM -0700, Ray Jui wrote: [...] >> drivers/pci/host/pcie-iproc.c | 58 +++++++++++++++++++++++++++++++++++++++++-- >> drivers/pci/host/pcie-iproc.h | 3 +++ >> 2 files changed, 59 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/pci/host/pcie-iproc.c b/drivers/pci/host/pcie-iproc.c >> index f3b3340..07ec478 100644 >> --- a/drivers/pci/host/pcie-iproc.c >> +++ b/drivers/pci/host/pcie-iproc.c [...] >> +static int iproc_pcie_config_read32(struct pci_bus *bus, unsigned int devfn, >> + int where, int size, u32 *val) >> +{ >> + int ret; >> + >> + iproc_pcie_apb_err_disable(bus, true); >> + ret = pci_generic_config_read32(bus, devfn, where, size, val); >> + iproc_pcie_apb_err_disable(bus, false); > > I guess this means that any other APB errors (unrelated to this config > access, e.g., a driver doing MMIo to a device at the same time the user is > doing "lspci") that happen to occur during this window will also be > ignored. > I believe this only disables APB errors caused by unsupported request generated from PCIe. Thanks, Ray -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html