On Thu, May 12, 2016 at 12:50:07PM +0200, Tomasz Nowicki wrote: > On 12.05.2016 00:43, Bjorn Helgaas wrote: > >@@ -70,9 +71,11 @@ struct pci_config_window *pci_ecam_create(struct device *dev, > > cfg->res.flags = IORESOURCE_MEM | IORESOURCE_BUSY; > > cfg->res.name = "PCI ECAM"; > > > >- err = request_resource(&iomem_resource, &cfg->res); > >- if (err) { > >- dev_err(dev, "request ECAM res %pR failed\n", &cfg->res); > >+ conflict = request_resource(&iomem_resource, &cfg->res); > > We need request_resource_conflict here then: > - conflict = request_resource(&iomem_resource, &cfg->res); > + conflict = request_resource_conflict(&iomem_resource, &cfg->res); Whoops, fixed, thanks! -- 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