On Sat, Jun 02, 2018 at 12:46:59AM -0500, Bjorn Helgaas wrote: > > I think here hotplug_is_native() would work as well. The idea is > > that we need to avoid scanning bridges that are handled by pciehp > > (or shpchp). > > I made that change on my branch. That way we won't have to worry > about the cases where we set is_hotplug_bridge based on some ACPI > methods. I think those would be bugs because I think we *do* want > acpiphp to scan them, and if we tested "!dev->is_hotplug_bridge", we > would not scan them. I agree.