On 4/26/2021 4:34 AM, Min Hu (Connor) wrote:
> From: Chengwen Feng <fengcheng...@huawei.com>
> 
> This patch deletes some unused capabilities, include:
> 1. Delete some unused firmware capabilities definition.
> 2. Delete some unused driver capabilities definition.
> 
> Signed-off-by: Chengwen Feng <fengcheng...@huawei.com>
> Signed-off-by: Min Hu (Connor) <humi...@huawei.com>

<...>

> diff --git a/drivers/net/hns3/hns3_ethdev.h b/drivers/net/hns3/hns3_ethdev.h
> index d27c725..de17878 100644
> --- a/drivers/net/hns3/hns3_ethdev.h
> +++ b/drivers/net/hns3/hns3_ethdev.h
> @@ -855,17 +855,20 @@ enum {
>  
>  #define HNS3_DEVARG_DEV_CAPS_MASK    "dev_caps_mask"
>  
> -#define HNS3_DEV_SUPPORT_DCB_B                       0x0
> -#define HNS3_DEV_SUPPORT_COPPER_B            0x1
> -#define HNS3_DEV_SUPPORT_UDP_GSO_B           0x2
> -#define HNS3_DEV_SUPPORT_FD_QUEUE_REGION_B   0x3
> -#define HNS3_DEV_SUPPORT_PTP_B                       0x4
> -#define HNS3_DEV_SUPPORT_TX_PUSH_B           0x5
> -#define HNS3_DEV_SUPPORT_INDEP_TXRX_B                0x6
> -#define HNS3_DEV_SUPPORT_STASH_B             0x7
> -#define HNS3_DEV_SUPPORT_RXD_ADV_LAYOUT_B    0x9
> -#define HNS3_DEV_SUPPORT_OUTER_UDP_CKSUM_B   0xA
> -#define HNS3_DEV_SUPPORT_RAS_IMP_B           0xB
> +enum {
> +     HNS3_DEV_SUPPORT_DCB_B,
> +     HNS3_DEV_SUPPORT_COPPER_B,
> +     HNS3_DEV_SUPPORT_FD_QUEUE_REGION_B,
> +     HNS3_DEV_SUPPORT_PTP_B,
> +     HNS3_DEV_SUPPORT_TX_PUSH_B,
> +     HNS3_DEV_SUPPORT_INDEP_TXRX_B,
> +     HNS3_DEV_SUPPORT_STASH_B,
> +     HNS3_DEV_SUPPORT_SIMPLE_BD_B,
> +     HNS3_DEV_SUPPORT_RXD_ADV_LAYOUT_B,
> +     HNS3_DEV_SUPPORT_OUTER_UDP_CKSUM_B,
> +     HNS3_DEV_SUPPORT_RAS_IMP_B,
> +     HNS3_DEV_SUPPORT_TM_B,
> +};

This cause some values to be changed, because of added/removed items, is this
expected?
If so can you please mention from this value change in the commit log, to 
clarify.

Reply via email to