Re: [dpdk-dev] [PATCH v2 0/4] NFP PF support based on new CPP interface

[Alejandro Lucero <alejandro.lucero@netronome.com>]

On Fri, Apr 6, 2018 at 4:47 PM, Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 4/6/2018 3:44 PM, Ferruh Yigit wrote:
> > On 4/5/2018 3:42 PM, Alejandro Lucero wrote:
> >> NFP PMD PF support requires to access the NFP chip for initialization.
> >> Current NFP PMD PF support was added based on the NSPU interface. This
> >> implies to do initialization through the NSP, a embedded ARM processor
> >> which does initialization tasks on demand. The main problem with this
> >> approach is it requires to add support for new NSP commands each time
> >> a new functionality is required, which does not scale well and it is
> >> not really flexible.
> >>
> >> Using the new CPP user space interface, the PMD can do whatever could
> >> be done by the NSP, this is current commands and any new functionality
> >> required. This CPP interface allows to access any single chip component
> >> facilitating initialization, firmware uploading, firmware debugging or
> >> extended stats.
> >>
> >> The changes just change the PMD PF initialization and do not touch the
> >> datapath at all. No performance changes nor PMD functionalities are
> affected.
> >>
> >> The initial impact using the new CPP interface is the way firmware
> upload
> >> is handled, which helps the PMD detecting the card type and the
> firmware file
> >> to upload. Future commits will include extended stats and some sort of
> debug
> >> channel.
> >>
> >> The specific CPP code is contained in the first patch, which has not
> been
> >> splitted up because is completely internal to the NFP functionality. The
> >> second patch makes the PMD changes required for using the new interface.
> >>
> >> v2:
> >>  - removing unused reference to zlib.h
> >>  - fix build errors
> >>  - add SPDX tags in new files
> >>  - rebase changes nfp.rst
> >>
> >> Alejandro Lucero (4):
> >>   net/nfp: add NFP CPP support
> >>   net/nfp: update PMD for using new CPP interface
> >>   doc: update NFP guide
> >>   net/nfp: remove files
> >
> > Series applied to dpdk-next-net/master, thanks.
>
> btw, some nfp files seems still missing spdx tags, can you please check
> them?
>

Yes, those are not new files. I will send a patch soon.

Thanks!