Re: [PATCH v8 00/45] powerpc/powernv: PCI hotplug support

On Wed, Apr 13, 2016 at 8:30 PM, Gavin Shan [off-list ref] wrote:

On Thu, Apr 14, 2016 at 09:57:32AM +1000, Alistair Popple wrote:

quoted

Hi Gavin,

<snip>

quoted

quoted

Why exactly cannot EEH reset changes go to a smaller separate patchset
(before hotplug)?

As I explained before, the patchset's order is: PCI generic part,
PowerNV PCI related, EEH related, device-tree part and hotplug driver.

The EEH reset change is included in PATCH[37/45]. There is no point
to reorder the patches.

I don't understand all of the dependencies but if possible splitting the
series up into a set of smaller self-contained patch series makes things
easier to review and may make it easier for you to get this functionality
reviewed and accepted into upstream.

Thanks, Alistair. I will move those cleanup/refactor related patches
to form a separate series which is expected to be merged first. That
will helps the reviewers to focus on the patches with complicated
changes as you suggested. Alexey, please let me know if that way is
you like to see or not.

As I said last cycle, I'll happily take the DT refactoring patches
separately, but you have to tell me if you want me to apply them and
it has to be well before the merge window.

Rob