On Fri, Apr 15, 2016 at 11:10:21AM -0500, Rob Herring wrote: >On Wed, Apr 13, 2016 at 8:30 PM, Gavin Shan <gwshan@xxxxxxxxxxxxxxxxxx> wrote: >> On Thu, Apr 14, 2016 at 09:57:32AM +1000, Alistair Popple wrote: >>>Hi Gavin, >>> >>><snip> >>> >>>> >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. > Thanks, Rob. I hope to post next revision (v9) soon and the device-tree related cleanup patches should be ready for next merge window in it. >Rob > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html