Re: [dpdk-dev] [PATCH v7 1/7] net/ark: stub PMD for Atomic Rules Arkville

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 3/29/2017 10:32 PM, Ed Czeck wrote:
> Enable Arkville on supported configurations
> Add overview documentation
> Minimum driver support for valid compile
> Arkville PMD is not supported on ARM or PowerPC at this time
> 
> v6:
> * Address review comments
> * Unify messaging, logging and debug macros to ark_logs.h
> 
> v5:
> * Address comments from Ferruh Yigit <ferruh.yigit@intel.com>
> * Added documentation on driver args
> * Makefile fixes
> * Safe argument processing
> * vdev args to dev args
> 
> v4:
> * Address issues report from review
> * Add internal comments on driver arg
> * provide a bare-bones dev init to avoid compiler warnings
> 
> v3:
> * Split large patch into several smaller ones
> 
> Signed-off-by: Ed Czeck <ed.czeck@atomicrules.com>
> Signed-off-by: John Miller <john.miller@atomicrules.com>

Hi Ed,

Can you please update release notes [1] to announce the ARK PMD?


Apart from that, overall PMD looks good to me.

Still I suggest pushing the PMD integration to RC2 phase, to give more
chance for reviews.

If PMD gets acked before that time, we can integrate earlier. If there
is no objection till RC2, it gets merged in RC2.

This means less time for testing after integration, is merging on RC2
will leave you enough time for testing?

Thanks,
ferruh


[1]
doc/guides/rel_notes/release_17_05.rst