> -----Original Message----- > From: Ferruh Yigit [mailto:ferruh.yi...@intel.com] > Sent: Wednesday, January 11, 2017 8:34 PM > To: Shreyansh Jain <shreyansh.j...@nxp.com>; dev@dpdk.org > Cc: Stephen Hemminger <sthem...@microsoft.com>; Jan Blunck > <jblu...@infradead.org> > Subject: Re: [PATCH 1/2] add rte_bus->probe > > On 1/11/2017 4:53 AM, Shreyansh Jain wrote: > > On Tuesday 10 January 2017 11:32 PM, Ferruh Yigit wrote: > >> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> > >> --- > >> lib/librte_eal/common/eal_common_bus.c | 7 ++++--- > >> lib/librte_eal/common/include/rte_bus.h | 3 +++ > >> lib/librte_eal/linuxapp/eal/eal_pci.c | 1 + > >> 3 files changed, 8 insertions(+), 3 deletions(-) > >> > >> diff --git a/lib/librte_eal/common/eal_common_bus.c > b/lib/librte_eal/common/eal_common_bus.c > >> index f8c2e03..e8d1143 100644 > >> --- a/lib/librte_eal/common/eal_common_bus.c > >> +++ b/lib/librte_eal/common/eal_common_bus.c > >> @@ -145,6 +145,7 @@ rte_eal_bus_register(struct rte_bus *bus) > >> /* A bus should mandatorily have the scan and match implemented */ > >> RTE_VERIFY(bus->scan); > >> RTE_VERIFY(bus->match); > >> + RTE_VERIFY(bus->probe); > > > > v6 of my patches would include the above. > > Since I am aware of you are working on something similar, I added this > (in a dirty way) just to able to test next patch.
:) I understood this after sending my mail. I understand your point. > > Thanks, > ferruh > > <...>