Re: [PATCH v7 3/8] net/hns3: remove some basic address dump

[fengchengwen]

Acked-by: Chengwen Feng <fengcheng...@huawei.com>

On 2024/9/14 15:13, Jie Hai wrote:
> For security reasons, some address registers are not suitable
> to be exposed, remove them.
> 
> Cc: sta...@dpdk.org
> 
> Signed-off-by: Jie Hai <haij...@huawei.com>
> Acked-by: Huisong Li<lihuis...@huawei.com>
> ---
>  drivers/net/hns3/hns3_regs.c | 12 ++----------
>  1 file changed, 2 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/net/hns3/hns3_regs.c b/drivers/net/hns3/hns3_regs.c
> index be1be6a89c94..53d829a4fc68 100644
> --- a/drivers/net/hns3/hns3_regs.c
> +++ b/drivers/net/hns3/hns3_regs.c
> @@ -17,13 +17,9 @@
>  
>  static int hns3_get_dfx_reg_line(struct hns3_hw *hw, uint32_t *lines);
>  
> -static const uint32_t cmdq_reg_addrs[] = {HNS3_CMDQ_TX_ADDR_L_REG,
> -                                       HNS3_CMDQ_TX_ADDR_H_REG,
> -                                       HNS3_CMDQ_TX_DEPTH_REG,
> +static const uint32_t cmdq_reg_addrs[] = {HNS3_CMDQ_TX_DEPTH_REG,
>                                         HNS3_CMDQ_TX_TAIL_REG,
>                                         HNS3_CMDQ_TX_HEAD_REG,
> -                                       HNS3_CMDQ_RX_ADDR_L_REG,
> -                                       HNS3_CMDQ_RX_ADDR_H_REG,
>                                         HNS3_CMDQ_RX_DEPTH_REG,
>                                         HNS3_CMDQ_RX_TAIL_REG,
>                                         HNS3_CMDQ_RX_HEAD_REG,
> @@ -44,9 +40,7 @@ static const uint32_t common_vf_reg_addrs[] = 
> {HNS3_MISC_VECTOR_REG_BASE,
>                                              HNS3_FUN_RST_ING,
>                                              HNS3_GRO_EN_REG};
>  
> -static const uint32_t ring_reg_addrs[] = {HNS3_RING_RX_BASEADDR_L_REG,
> -                                       HNS3_RING_RX_BASEADDR_H_REG,
> -                                       HNS3_RING_RX_BD_NUM_REG,
> +static const uint32_t ring_reg_addrs[] = {HNS3_RING_RX_BD_NUM_REG,
>                                         HNS3_RING_RX_BD_LEN_REG,
>                                         HNS3_RING_RX_EN_REG,
>                                         HNS3_RING_RX_MERGE_EN_REG,
> @@ -57,8 +51,6 @@ static const uint32_t ring_reg_addrs[] = 
> {HNS3_RING_RX_BASEADDR_L_REG,
>                                         HNS3_RING_RX_FBD_OFFSET_REG,
>                                         HNS3_RING_RX_STASH_REG,
>                                         HNS3_RING_RX_BD_ERR_REG,
> -                                       HNS3_RING_TX_BASEADDR_L_REG,
> -                                       HNS3_RING_TX_BASEADDR_H_REG,
>                                         HNS3_RING_TX_BD_NUM_REG,
>                                         HNS3_RING_TX_EN_REG,
>                                         HNS3_RING_TX_PRIORITY_REG,