DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] ethdev: rename FEC API parameters
@ 2021-01-15 15:52 Ferruh Yigit
  2021-01-15 16:48 ` Ajit Khaparde
  2021-01-16  1:48 ` Min Hu (Connor)
  0 siblings, 2 replies; 4+ messages in thread
From: Ferruh Yigit @ 2021-01-15 15:52 UTC (permalink / raw)
  To: Thomas Monjalon, Andrew Rybchenko
  Cc: Ferruh Yigit, dev, Min Hu, Wei Hu, Chengwen Feng,
	Chengchang Tang, Andrew Rybchenko, Ajit Khaparde

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 <ferruh.yigit@intel.com>
---
Cc: Min Hu (Connor) <humin29@huawei.com>
Cc: Wei Hu (Xavier) <xavier.huwei@huawei.com>
Cc: Chengwen Feng <fengchengwen@huawei.com>
Cc: Chengchang Tang <tangchengchang@huawei.com>
Cc: Andrew Rybchenko <arybchenko@solarflare.com>
Cc: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 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
-- 
2.29.2


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] ethdev: rename FEC API parameters
  2021-01-15 15:52 [dpdk-dev] [PATCH] ethdev: rename FEC API parameters Ferruh Yigit
@ 2021-01-15 16:48 ` Ajit Khaparde
  2021-01-16  1:48 ` Min Hu (Connor)
  1 sibling, 0 replies; 4+ messages in thread
From: Ajit Khaparde @ 2021-01-15 16:48 UTC (permalink / raw)
  To: Ferruh Yigit
  Cc: Thomas Monjalon, Andrew Rybchenko, dpdk-dev, Min Hu, Wei Hu,
	Chengwen Feng, Chengchang Tang, Andrew Rybchenko

[-- Attachment #1: Type: text/plain, Size: 5474 bytes --]

On Fri, Jan 15, 2021 at 7:52 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> 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 <ferruh.yigit@intel.com>

Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>

> ---
> Cc: Min Hu (Connor) <humin29@huawei.com>
> Cc: Wei Hu (Xavier) <xavier.huwei@huawei.com>
> Cc: Chengwen Feng <fengchengwen@huawei.com>
> Cc: Chengchang Tang <tangchengchang@huawei.com>
> Cc: Andrew Rybchenko <arybchenko@solarflare.com>
> Cc: Ajit Khaparde <ajit.khaparde@broadcom.com>
> ---
>  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
> --
> 2.29.2
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] ethdev: rename FEC API parameters
  2021-01-15 15:52 [dpdk-dev] [PATCH] ethdev: rename FEC API parameters Ferruh Yigit
  2021-01-15 16:48 ` Ajit Khaparde
@ 2021-01-16  1:48 ` Min Hu (Connor)
  2021-01-29 16:54   ` Ferruh Yigit
  1 sibling, 1 reply; 4+ messages in thread
From: Min Hu (Connor) @ 2021-01-16  1:48 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, Andrew Rybchenko
  Cc: dev, Wei Hu, Chengwen Feng, Chengchang Tang, Andrew Rybchenko,
	Ajit Khaparde

Hi, Ferruh,
	this change origin from the opinions by Andrew Rybchenko
<arybchenko@solarflare.com>. 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 <ferruh.yigit@intel.com>
> ---
> Cc: Min Hu (Connor) <humin29@huawei.com>
> Cc: Wei Hu (Xavier) <xavier.huwei@huawei.com>
> Cc: Chengwen Feng <fengchengwen@huawei.com>
> Cc: Chengchang Tang <tangchengchang@huawei.com>
> Cc: Andrew Rybchenko <arybchenko@solarflare.com>
> Cc: Ajit Khaparde <ajit.khaparde@broadcom.com>
> ---
>   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
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] ethdev: rename FEC API parameters
  2021-01-16  1:48 ` Min Hu (Connor)
@ 2021-01-29 16:54   ` Ferruh Yigit
  0 siblings, 0 replies; 4+ messages in thread
From: Ferruh Yigit @ 2021-01-29 16:54 UTC (permalink / raw)
  To: Min Hu (Connor), Thomas Monjalon, Andrew Rybchenko
  Cc: dev, Wei Hu, Chengwen Feng, Chengchang Tang, Andrew Rybchenko,
	Ajit Khaparde

On 1/16/2021 1:48 AM, Min Hu (Connor) wrote:
> Hi, Ferruh,
>      this change origin from the opinions by Andrew Rybchenko
> <arybchenko@solarflare.com>. 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.
> 

Got it, if the parameter name explicitly selected this way, dropping this patch.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-01-29 16:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-15 15:52 [dpdk-dev] [PATCH] ethdev: rename FEC API parameters Ferruh Yigit
2021-01-15 16:48 ` Ajit Khaparde
2021-01-16  1:48 ` Min Hu (Connor)
2021-01-29 16:54   ` Ferruh Yigit

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git