Hi Ferruh, It seems I have missed some comments, so just reply this again.
> -----Original Message----- > From: Yigit, Ferruh [mailto:[email protected]] > Sent: Wednesday, March 06, 2019 20:45 > To: Xu, Rosen <[email protected]>; [email protected] > Cc: Yigit, Ferruh <[email protected]>; Zhang, Tianfei > <[email protected]>; Wei, Dan <[email protected]>; Pei, Andy > <[email protected]>; Yang, Qiming <[email protected]>; Wang, > Haiyue <[email protected]>; Chen, Santos <[email protected]>; > Zhang, Zhang <[email protected]> > Subject: Re: [dpdk-dev] [PATCH v1 06/11] config: add build enablement for > IPN3KE > > On 2/28/2019 7:13 AM, Rosen Xu wrote: > > Add build enablement for Intel FPGA Acceleration NIC IPN3KE. > > No need to make this a separate patch, can squash into previous one. I will merge it in patch v2. > > > > Signed-off-by: Rosen Xu <[email protected]> > > --- > > config/common_base | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/config/common_base b/config/common_base index > > 7c6da51..4fac8ba 100644 > > --- a/config/common_base > > +++ b/config/common_base > > @@ -316,6 +316,12 @@ CONFIG_RTE_LIBRTE_AVF_DEBUG_RX=n > > CONFIG_RTE_LIBRTE_AVF_16BYTE_RX_DESC=n > > > > # > > +# Compile burst-oriented IPN3KE PMD driver # > > +CONFIG_RTE_LIBRTE_IPN3KE_PMD=n > > Why PMD is disabled by default? It should not be disabled by default. I will change it to y in patch v2. > > +CONFIG_RTE_LIBRTE_IPN3KE_DEBUG=n > > _DEBUG seems not used at all, if so can you please remove it? Currently it's not used, I will remove it in patch v2. > > + > > +# > > # Compile burst-oriented Mellanox ConnectX-3 (MLX4) PMD # > > CONFIG_RTE_LIBRTE_MLX4_PMD=n > >
