Re: [dpdk-dev] [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver for fsl-mc bus

2016-12-07 Thread Hemant Agrawal
> -Original Message- > From: David Marchand [mailto:david.march...@6wind.com] > Sent: Wednesday, December 07, 2016 5:52 PM > To: Thomas Monjalon > Cc: Shreyansh Jain ; Hemant Agrawal > ; dev@dpdk.org; Richardson, Bruce > > Subject: Re: [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver

Re: [dpdk-dev] [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver for fsl-mc bus

2016-12-07 Thread David Marchand
On Wed, Dec 7, 2016 at 11:40 AM, Thomas Monjalon wrote: > 2016-12-07 15:43, Shreyansh Jain: >> IMO, the way Bus is kept is debatable. >> - should it be in EAL (lib/librte_eal/linuxapp/eal_pci.c like Bus >> patches) [1]? >> - Should it a 'handler/driver' parallel to device drivers? >> >> I pers

Re: [dpdk-dev] [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver for fsl-mc bus

2016-12-07 Thread Thomas Monjalon
2016-12-07 15:43, Shreyansh Jain: > IMO, the way Bus is kept is debatable. > - should it be in EAL (lib/librte_eal/linuxapp/eal_pci.c like Bus > patches) [1]? > - Should it a 'handler/driver' parallel to device drivers? > > I personally prefer a clean layer for buses with: > > - RTE_SDK/dr

Re: [dpdk-dev] [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver for fsl-mc bus

2016-12-07 Thread Shreyansh Jain
+ CC: David Marchand On Sunday 04 December 2016 11:47 PM, Hemant Agrawal wrote: The DPAA2 bus driver is a rte_bus driver which scans the fsl-mc bus. Signed-off-by: Hemant Agrawal --- drivers/net/Makefile| 2 +- drivers/net/dpaa2/Makefile | 60 ++

Re: [dpdk-dev] [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver for fsl-mc bus

2016-12-06 Thread Hemant Agrawal
On 12/7/2016 1:19 AM, Ferruh Yigit wrote: On 12/4/2016 6:17 PM, Hemant Agrawal wrote: The DPAA2 bus driver is a rte_bus driver which scans the fsl-mc bus. Signed-off-by: Hemant Agrawal --- drivers/net/Makefile| 2 +- drivers/net/dpaa2/Makefile | 60

Re: [dpdk-dev] [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver for fsl-mc bus

2016-12-06 Thread Ferruh Yigit
On 12/4/2016 6:17 PM, Hemant Agrawal wrote: > The DPAA2 bus driver is a rte_bus driver which scans the fsl-mc bus. > > Signed-off-by: Hemant Agrawal > --- > drivers/net/Makefile| 2 +- > drivers/net/dpaa2/Makefile | 60 ++ > drivers/net/dpa

[dpdk-dev] [PATCH 10/32] net/dpaa2: introducing dpaa2 bus driver for fsl-mc bus

2016-12-04 Thread Hemant Agrawal
The DPAA2 bus driver is a rte_bus driver which scans the fsl-mc bus. Signed-off-by: Hemant Agrawal --- drivers/net/Makefile| 2 +- drivers/net/dpaa2/Makefile | 60 ++ drivers/net/dpaa2/dpaa2_bus.c | 99 +++