From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Sune Subject: Re: [PATCH v6 1/5] ethdev: Added ETH_SPEED_CAP bitmap for ports Date: Thu, 19 Nov 2015 00:08:05 +0100 Message-ID: References: <1443993003-1059-1-git-send-email-marcdevel@gmail.com> <1445810400-8978-1-git-send-email-marcdevel@gmail.com> <1445810400-8978-2-git-send-email-marcdevel@gmail.com> <18320386.mavPrt1dXu@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: dev@dpdk.org To: Thomas Monjalon Return-path: Received: from mail-io0-f172.google.com (mail-io0-f172.google.com [209.85.223.172]) by dpdk.org (Postfix) with ESMTP id 10CB3374E for ; Thu, 19 Nov 2015 00:08:06 +0100 (CET) Received: by iofh3 with SMTP id h3so71093603iof.3 for ; Wed, 18 Nov 2015 15:08:05 -0800 (PST) In-Reply-To: <18320386.mavPrt1dXu@xps13> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Thomas, 2015-11-01 23:11 GMT+01:00 Thomas Monjalon : > 2015-10-25 22:59, Marc Sune: > > +#define ETH_SPEED_CAP_NOT_PHY (0) /*< No phy media > */ > > +#define ETH_SPEED_CAP_10M_HD (1 << 0) /*< 10 Mbps half-duplex> */ > > +#define ETH_SPEED_CAP_10M_FD (1 << 1) /*< 10 Mbps full-duplex> */ > > +#define ETH_SPEED_CAP_100M_HD (1 << 2) /*< 100 Mbps > half-duplex> */ > > +#define ETH_SPEED_CAP_100M_FD (1 << 3) /*< 100 Mbps > full-duplex> */ > > +#define ETH_SPEED_CAP_1G (1 << 4) /*< 1 Gbps > */ > > +#define ETH_SPEED_CAP_2_5G (1 << 5) /*< 2.5 Gbps > */ > > +#define ETH_SPEED_CAP_5G (1 << 6) /*< 5 Gbps > */ > > +#define ETH_SPEED_CAP_10G (1 << 7) /*< 10 Mbps > */ > > +#define ETH_SPEED_CAP_20G (1 << 8) /*< 20 Gbps > */ > > +#define ETH_SPEED_CAP_25G (1 << 9) /*< 25 Gbps > */ > > +#define ETH_SPEED_CAP_40G (1 << 10) /*< 40 Gbps > */ > > +#define ETH_SPEED_CAP_50G (1 << 11) /*< 50 Gbps > */ > > +#define ETH_SPEED_CAP_56G (1 << 12) /*< 56 Gbps > */ > > +#define ETH_SPEED_CAP_100G (1 << 13) /*< 100 Gbps > */ > > In the patch 3, you rename this flags. It would be easier to understand if > the right names were used in the first patch. > > > @@ -837,6 +860,7 @@ struct rte_eth_dev_info { > > uint16_t vmdq_queue_base; /**< First queue ID for VMDQ pools. */ > > uint16_t vmdq_queue_num; /**< Queue number for VMDQ pools. */ > > uint16_t vmdq_pool_base; /**< First ID of VMDQ pools. */ > > + uint32_t speed_capa; /**< Supported speeds bitmap > (ETH_SPEED_CAP_). */ > > When renaming ETH_SPEED_CAP, this line is not changed later. > This is made in purpose. In patch 3/5 the bitmap speeds are renamed to ETH_LINK_SPEED_XXX and numeric values are moved ETH_SPEED_NUM_XXX, to make clear the difference. I cannot, in this commit, rename them directly to ETH_LINK_SPEED_XXX since they would collide with the current numeric speeds. Thanks Marc