From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D1607A0A02; Sat, 16 Jan 2021 02:48:22 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 534A6140ED2; Sat, 16 Jan 2021 02:48:22 +0100 (CET) Received: from szxga06-in.huawei.com (szxga06-in.huawei.com [45.249.212.32]) by mails.dpdk.org (Postfix) with ESMTP id 38974140ED1 for ; Sat, 16 Jan 2021 02:48:19 +0100 (CET) Received: from DGGEMS402-HUB.china.huawei.com (unknown [172.30.72.58]) by szxga06-in.huawei.com (SkyGuard) with ESMTP id 4DHgsW2KF6zj1rs; Sat, 16 Jan 2021 09:47:27 +0800 (CST) Received: from [10.67.103.128] (10.67.103.128) by DGGEMS402-HUB.china.huawei.com (10.3.19.202) with Microsoft SMTP Server id 14.3.498.0; Sat, 16 Jan 2021 09:48:09 +0800 To: Ferruh Yigit , Thomas Monjalon , Andrew Rybchenko CC: , Wei Hu , Chengwen Feng , Chengchang Tang , Andrew Rybchenko , Ajit Khaparde References: <20210115155232.1406688-1-ferruh.yigit@intel.com> From: "Min Hu (Connor)" Message-ID: <6b95ed0f-a6ec-a259-37fc-3ce6b74dd43b@huawei.com> Date: Sat, 16 Jan 2021 09:48:09 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.3.1 MIME-Version: 1.0 In-Reply-To: <20210115155232.1406688-1-ferruh.yigit@intel.com> Content-Type: text/plain; charset="gbk"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.103.128] X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH] ethdev: rename FEC API parameters X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, Ferruh, this change origin from the opinions by Andrew Rybchenko . snippets like this: > +__rte_experimental > +int rte_eth_fec_get(uint16_t port_id, uint32_t *mode); > + > +/** > + * @warning > + * @b EXPERIMENTAL: this API may change, or be removed, without prior notice > + * > + * Set Forward Error Correction(FEC) mode. > + * > + * @param port_id > + * The port identifier of the Ethernet device. > + * @param mode > + * The FEC mode. mode -> fec_capa A bitmask of enabled FEC modes. If AUTO bit is set, other bits specify FEC modes which may be negotiated. If AUTO bit is clear, specify FEC modes to be used (only one valid mode per speed may be set). > + * @return > + * - (0) if successful. > + * - (-EINVAL) if the FEC mode is not valid. Much more info could be referred to the patch: Re: [dpdk-dev] [PATCH V13 1/3] ethdev: introduce FEC API from arybchenko@solarflare.com thanks. ÔÚ 2021/1/15 23:52, Ferruh Yigit дµÀ: > The FEC APIs to get/set FEC mode has parameter name as 'fec_capa', > renamed them as 'fec_mode' to be more accurate with API. > > No functional change. > > Signed-off-by: Ferruh Yigit > --- > Cc: Min Hu (Connor) > Cc: Wei Hu (Xavier) > Cc: Chengwen Feng > Cc: Chengchang Tang > Cc: Andrew Rybchenko > Cc: Ajit Khaparde > --- > lib/librte_ethdev/rte_ethdev.c | 10 +++++----- > lib/librte_ethdev/rte_ethdev.h | 8 ++++---- > lib/librte_ethdev/rte_ethdev_driver.h | 8 ++++---- > 3 files changed, 13 insertions(+), 13 deletions(-) > > diff --git a/lib/librte_ethdev/rte_ethdev.c b/lib/librte_ethdev/rte_ethdev.c > index 17ddacc78ded..332d7a789380 100644 > --- a/lib/librte_ethdev/rte_ethdev.c > +++ b/lib/librte_ethdev/rte_ethdev.c > @@ -3948,28 +3948,28 @@ rte_eth_fec_get_capability(uint16_t port_id, > } > > int > -rte_eth_fec_get(uint16_t port_id, uint32_t *fec_capa) > +rte_eth_fec_get(uint16_t port_id, uint32_t *fec_mode) > { > struct rte_eth_dev *dev; > > - if (fec_capa == NULL) > + if (fec_mode == NULL) > return -EINVAL; > > RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV); > dev = &rte_eth_devices[port_id]; > RTE_FUNC_PTR_OR_ERR_RET(*dev->dev_ops->fec_get, -ENOTSUP); > - return eth_err(port_id, (*dev->dev_ops->fec_get)(dev, fec_capa)); > + return eth_err(port_id, (*dev->dev_ops->fec_get)(dev, fec_mode)); > } > > int > -rte_eth_fec_set(uint16_t port_id, uint32_t fec_capa) > +rte_eth_fec_set(uint16_t port_id, uint32_t fec_mode) > { > struct rte_eth_dev *dev; > > RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV); > dev = &rte_eth_devices[port_id]; > RTE_FUNC_PTR_OR_ERR_RET(*dev->dev_ops->fec_set, -ENOTSUP); > - return eth_err(port_id, (*dev->dev_ops->fec_set)(dev, fec_capa)); > + return eth_err(port_id, (*dev->dev_ops->fec_set)(dev, fec_mode)); > } > > /* > diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h > index 2cbce958cf6c..0aefb7cb2f4a 100644 > --- a/lib/librte_ethdev/rte_ethdev.h > +++ b/lib/librte_ethdev/rte_ethdev.h > @@ -3699,7 +3699,7 @@ int rte_eth_fec_get_capability(uint16_t port_id, > * > * @param port_id > * The port identifier of the Ethernet device. > - * @param fec_capa > + * @param fec_mode > * A bitmask of enabled FEC modes. If AUTO bit is set, other > * bits specify FEC modes which may be negotiated. If AUTO > * bit is clear, specify FEC modes to be used (only one valid > @@ -3712,7 +3712,7 @@ int rte_eth_fec_get_capability(uint16_t port_id, > * - (-ENODEV) if *port_id* invalid. > */ > __rte_experimental > -int rte_eth_fec_get(uint16_t port_id, uint32_t *fec_capa); > +int rte_eth_fec_get(uint16_t port_id, uint32_t *fec_mode); > > /** > * @warning > @@ -3722,7 +3722,7 @@ int rte_eth_fec_get(uint16_t port_id, uint32_t *fec_capa); > * > * @param port_id > * The port identifier of the Ethernet device. > - * @param fec_capa > + * @param fec_mode > * A bitmask of allowed FEC modes. If AUTO bit is set, other > * bits specify FEC modes which may be negotiated. If AUTO > * bit is clear, specify FEC modes to be used (only one valid > @@ -3735,7 +3735,7 @@ int rte_eth_fec_get(uint16_t port_id, uint32_t *fec_capa); > * - (-ENODEV) if *port_id* invalid. > */ > __rte_experimental > -int rte_eth_fec_set(uint16_t port_id, uint32_t fec_capa); > +int rte_eth_fec_set(uint16_t port_id, uint32_t fec_mode); > > /** > * Get current status of the Ethernet link flow control for Ethernet device > diff --git a/lib/librte_ethdev/rte_ethdev_driver.h b/lib/librte_ethdev/rte_ethdev_driver.h > index 0eacfd842581..616a14d0361e 100644 > --- a/lib/librte_ethdev/rte_ethdev_driver.h > +++ b/lib/librte_ethdev/rte_ethdev_driver.h > @@ -625,7 +625,7 @@ typedef int (*eth_fec_get_capability_t)(struct rte_eth_dev *dev, > * > * @param dev > * ethdev handle of port. > - * @param fec_capa > + * @param fec_mode > * a bitmask of enabled FEC modes. If AUTO bit is set, other > * bits specify FEC modes which may be negotiated. If AUTO > * bit is clear, specify FEC modes to be used (only one valid > @@ -642,7 +642,7 @@ typedef int (*eth_fec_get_capability_t)(struct rte_eth_dev *dev, > * Device is removed. > */ > typedef int (*eth_fec_get_t)(struct rte_eth_dev *dev, > - uint32_t *fec_capa); > + uint32_t *fec_mode); > > /** > * @internal > @@ -650,7 +650,7 @@ typedef int (*eth_fec_get_t)(struct rte_eth_dev *dev, > * > * @param dev > * ethdev handle of port. > - * @param fec_capa > + * @param fec_mode > * bitmask of allowed FEC modes. It must be only one > * if AUTO is disabled. If AUTO is enabled, other > * bits specify FEC modes which may be negotiated. > @@ -667,7 +667,7 @@ typedef int (*eth_fec_get_t)(struct rte_eth_dev *dev, > * @retval -EIO > * Device is removed. > */ > -typedef int (*eth_fec_set_t)(struct rte_eth_dev *dev, uint32_t fec_capa); > +typedef int (*eth_fec_set_t)(struct rte_eth_dev *dev, uint32_t fec_mode); > > /** > * @internal >