On Wed, Apr 24, 2013 at 7:08 PM, Liujiang (Gerry, Euler) <jiang.liu@xxxxxxxxxx> wrote: > Hi Yinghai, > I have worked out a similar patch last night. Please refer to attachment. > I think following code may not work because acpiphp_handle_to_bridge() > will return NULL. for root bus we have pcibios_add_bus. ==> acpi_pci_add_bus==>call acpiphp_enumerate_slots. so if there is hotplug slot on root bus directly, it will allocate the acpiphp_bridge and register the slots. then acpiphp_handle_to_bridge will return the right acpiphp_bridge with slots list. > Could you please help to review my version? > (Due to work environment limitations, sorry for sending out without > attachment and outlook). looks the same to my patch, except you missed the acpiphp_check_bridge calling. Thanks Yinghai -- 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