2016-12-08 10:34, Shreyansh Jain: > On Wednesday 07 December 2016 06:54 PM, Thomas Monjalon wrote: > > 2016-12-07 18:40, Shreyansh Jain: > >> Is there anything specific that you are looking for in patchset v2? > >> I was thinking of: > >> 0. fixing BSD compilation issue reported by CI > >> 1. improving the test_pci.c > >> 2. hotplugging > >> 3. trying to move PCI to drives/bus/pci/linux/* and resolving how > >> drivers link to it, and how EAL resources like devargs are consumed. > > > > I am concerned about the time needed for all these changes. > > Please let's make sure that the basic parts are well done and pushed, first. > > That's why I suggest to postpone 1, 2 and 3 to next release if possible. > > The priority is to have a clean bus model, > > and if time permits, integrate the NXP driver. > > > > I am absolutely OK with this. > > Just a confirmation, though. Is it OK to have a broken test_pci.c (1) in > a series? It is not getting compiled by default, but it would definitely > not work if compiled (symbols are missing).
No, that's why I said "if possible" :). I thought 1. was about improvements. But if it does not compile, it is a must.
