Re: [PATCH v7 38/50] powerpc/powernv: Exclude root bus in pnv_pci_reset_secondary_bus()

2015-11-12 Thread Gavin Shan
On Fri, Nov 13, 2015 at 09:59:27AM +1100, Daniel Axtens wrote: >Gavin Shan writes: > >> When pnv_pci_reset_secondary_bus() is called to issue reset on >> the indicated secondary bus, the bus can't be root bus. So we >> needn't consider root bus in the function. > >It took me a while to convince my

Re: [PATCH v7 38/50] powerpc/powernv: Exclude root bus in pnv_pci_reset_secondary_bus()

2015-11-12 Thread Daniel Axtens
Gavin Shan writes: > When pnv_pci_reset_secondary_bus() is called to issue reset on > the indicated secondary bus, the bus can't be root bus. So we > needn't consider root bus in the function. It took me a while to convince myself that this is correct. For the record, this is why it's correct:

[PATCH v7 38/50] powerpc/powernv: Exclude root bus in pnv_pci_reset_secondary_bus()

2015-11-04 Thread Gavin Shan
When pnv_pci_reset_secondary_bus() is called to issue reset on the indicated secondary bus, the bus can't be root bus. So we needn't consider root bus in the function. Signed-off-by: Gavin Shan --- arch/powerpc/platforms/powernv/eeh-powernv.c | 12 ++-- 1 file changed, 2 insertions(+), 1