Re: [PATCH v9 09/11] PCI, acpiphp: Don't bailout even no slots found yet.

2013-01-20 Thread Yinghai Lu
> > If that's the case: > > Acked-by: Rafael J. Wysocki > > but please say something like this in the changelog: > > "The result returned by acpiphp_get_num_slots() is meaningless, because > the bridge the slots are under may be added after this function has been > called, so drop acpiphp_get_nu

Re: [PATCH v9 09/11] PCI, acpiphp: Don't bailout even no slots found yet.

2013-01-20 Thread Rafael J. Wysocki
On Thursday, January 17, 2013 11:53:20 PM Yinghai Lu wrote: > Could have root bus hot added later and there may be slots that need acpiphp. The patch implies that acpiphp_get_num_slots() is not necessary any more, but the changelog above doesn't really explain why that is so. Do I guess correctly

[PATCH v9 09/11] PCI, acpiphp: Don't bailout even no slots found yet.

2013-01-17 Thread Yinghai Lu
Could have root bus hot added later and there may be slots that need acpiphp. Signed-off-by: Yinghai Lu --- drivers/pci/hotplug/acpiphp.h |1 - drivers/pci/hotplug/acpiphp_core.c | 23 ++- drivers/pci/hotplug/acpiphp_glue.c | 22 -- 3 files ch