Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-05-02 Thread Alexey Kardashevskiy
On 05/03/2016 09:41 AM, Gavin Shan wrote: On Wed, Apr 20, 2016 at 11:55:56AM +1000, Alistair Popple wrote: On Tue, 19 Apr 2016 20:36:48 Alexey Kardashevskiy wrote: On 02/17/2016 02:44 PM, Gavin Shan wrote: This adds standalone driver to support PCI hotplug for PowerPC PowerNV platform that run

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-05-02 Thread Gavin Shan
On Wed, Apr 20, 2016 at 11:55:56AM +1000, Alistair Popple wrote: >On Tue, 19 Apr 2016 20:36:48 Alexey Kardashevskiy wrote: >> On 02/17/2016 02:44 PM, Gavin Shan wrote: >> > This adds standalone driver to support PCI hotplug for PowerPC PowerNV >> > platform that runs on top of skiboot firmware. The

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-05-02 Thread Gavin Shan
On Mon, May 02, 2016 at 04:11:53PM +1000, Alexey Kardashevskiy wrote: >On 05/02/2016 01:44 PM, Gavin Shan wrote: >>On Tue, Apr 19, 2016 at 08:36:48PM +1000, Alexey Kardashevskiy wrote: >>>On 02/17/2016 02:44 PM, Gavin Shan wrote: This adds standalone driver to support PCI hotplug for PowerPC Po

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-05-01 Thread Alexey Kardashevskiy
On 05/02/2016 01:44 PM, Gavin Shan wrote: On Tue, Apr 19, 2016 at 08:36:48PM +1000, Alexey Kardashevskiy wrote: On 02/17/2016 02:44 PM, Gavin Shan wrote: This adds standalone driver to support PCI hotplug for PowerPC PowerNV platform that runs on top of skiboot firmware. The firmware identifies

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-05-01 Thread Gavin Shan
On Tue, Apr 19, 2016 at 08:36:48PM +1000, Alexey Kardashevskiy wrote: >On 02/17/2016 02:44 PM, Gavin Shan wrote: >>This adds standalone driver to support PCI hotplug for PowerPC PowerNV >>platform that runs on top of skiboot firmware. The firmware identifies >>hotpluggable slots and marked their de

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-04-19 Thread Alistair Popple
On Tue, 19 Apr 2016 20:36:48 Alexey Kardashevskiy wrote: > On 02/17/2016 02:44 PM, Gavin Shan wrote: > > This adds standalone driver to support PCI hotplug for PowerPC PowerNV > > platform that runs on top of skiboot firmware. The firmware identifies > > hotpluggable slots and marked their device t

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-04-19 Thread Alexey Kardashevskiy
On 02/17/2016 02:44 PM, Gavin Shan wrote: This adds standalone driver to support PCI hotplug for PowerPC PowerNV platform that runs on top of skiboot firmware. The firmware identifies hotpluggable slots and marked their device tree node with proper "ibm,slot-pluggable" and "ibm,reset-by-firmware"

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-04-14 Thread Gavin Shan
On Fri, Apr 15, 2016 at 10:47:52AM +1000, Alistair Popple wrote: >Hi Gavin, > >I was reading through this to understand how it all works and noticed a couple >of things, comments below. > Alistair, thanks for your time on review. > >On Wed, 17 Feb 2016 14:44:28 Gavin Shan wrote: > > > >> + >> +st

Re: [PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-04-14 Thread Alistair Popple
Hi Gavin, I was reading through this to understand how it all works and noticed a couple of things, comments below. - Alistair On Wed, 17 Feb 2016 14:44:28 Gavin Shan wrote: > + > +static void pnv_php_handle_poweron(struct pnv_php_slot *php_slot) > +{ > + void *fdt, *fdt1, *dt; > + in

[PATCH v8 45/45] PCI/hotplug: PowerPC PowerNV PCI hotplug driver

2016-02-16 Thread Gavin Shan
This adds standalone driver to support PCI hotplug for PowerPC PowerNV platform that runs on top of skiboot firmware. The firmware identifies hotpluggable slots and marked their device tree node with proper "ibm,slot-pluggable" and "ibm,reset-by-firmware". The driver scans device tree nodes to crea