Hi Ferruh,
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Friday, December 14, 2018 5:30 AM
> To: Lu, Wenzhuo <wenzhuo...@intel.com>; dev@dpdk.org
> Cc: Yang, Qiming <qiming.y...@intel.com>; Li, Xiaoyun
> <xiaoyun...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; Olivier MATZ
> <olivier.m...@6wind.com>
> Subject: Re: [dpdk-dev] [PATCH v3 32/34] net/ice: support descriptor ops
>
> On 12/12/2018 7:00 AM, Wenzhuo Lu wrote:
> > Add below ops,
> > rx_descriptor_done
> > rx_descriptor_status
> > tx_descriptor_status
>
> I guess this is our mistake to not clarify this, sorry about it, but
> "rx_descriptor_status" replaces "rx_descriptor_done", it is and extended
> version, cc'ed Olivier to correct me in case I am wrong.
>
> So when "rx_descriptor_status" implemented, "rx_descriptor_done" can be
> dropped.
>
> Please see commit log of:
> Commit b1b700ce7d6f ("ethdev: add descriptor status API")
>
> copy-paste related part:
> "
> The descriptor_done() API, and probably the rx_queue_count() API could
> be replaced by this new API as soon as it is implemented on all PMDs.
> "
Thanks for the reminder. Will remove it.
