On Wed, Apr 24, 2013 at 8:16 PM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > 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. even there is not slot directly on root bus, but have slot in following bridge, acpiphp_bridge for root bus still get created, just no direct slots under it. > > >> 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
