From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3 2/4] ethdev: add siblings iterators Date: Wed, 03 Apr 2019 01:48:41 +0200 Message-ID: <30477655.WWphSIAI62@xps> References: <20181130002716.27325-1-thomas@monjalon.net> <20190401022700.1570-3-thomas@monjalon.net> <09b6795e-9247-6599-1c5e-1d398739b1e6@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: gaetan.rivet@6wind.com, Andrew Rybchenko , dev@dpdk.org To: Ferruh Yigit Return-path: Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id 52E3C1B0F9 for ; Wed, 3 Apr 2019 01:48:46 +0200 (CEST) In-Reply-To: <09b6795e-9247-6599-1c5e-1d398739b1e6@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 03/04/2019 01:42, Ferruh Yigit: > On 4/1/2019 3:26 AM, Thomas Monjalon wrote: > > +__rte_experimental > > +uint16_t rte_eth_find_next_of(uint16_t port_id_start, > > + const struct rte_device *parent); > > Minor nit, but other instances using the tag as: > > uint16_t __rte_experimental > rte_eth_find_next_of(uint16_t port_id_start, > const struct rte_device *parent); > > What do you think updating it for consistency? Same for two APIs. I think I did it this way to minimize the patch removing it later. I'm OK to change it for consistency.