Re: [PATCH net-next 2/3] net: hisilicon: hns: Remove unused hns_rcb_start

[Jijie Shao <shaojijie@huawei.com>]

on 2024/12/18 8:57, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
>
> hns_rcb_start() has been unused since 2016's
> commit 454784d85de3 ("net: hns: delete redundancy ring enable operations")
>
> Remove it.

Thanks,
Reviewed-by: Jijie Shao<shaojijie@huawei.com>

>
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> ---
>   drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c | 5 -----
>   drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.h | 1 -
>   2 files changed, 6 deletions(-)
>
> diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c b/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c
> index 46af467aa596..635b3a95dd82 100644
> --- a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c
> +++ b/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c
> @@ -195,11 +195,6 @@ void hns_rcb_ring_enable_hw(struct hnae_queue *q, u32 val)
>   	dsaf_write_dev(q, RCB_RING_PREFETCH_EN_REG, !!val);
>   }
>   
> -void hns_rcb_start(struct hnae_queue *q, u32 val)
> -{
> -	hns_rcb_ring_enable_hw(q, val);
> -}
> -
>   /**
>    *hns_rcb_common_init_commit_hw - make rcb common init completed
>    *@rcb_common: rcb common device
> diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.h b/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.h
> index 0f4cc184ef39..68f81547dfb4 100644
> --- a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.h
> +++ b/drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.h
> @@ -116,7 +116,6 @@ int hns_rcb_buf_size2type(u32 buf_size);
>   int hns_rcb_common_get_cfg(struct dsaf_device *dsaf_dev, int comm_index);
>   void hns_rcb_common_free_cfg(struct dsaf_device *dsaf_dev, u32 comm_index);
>   int hns_rcb_common_init_hw(struct rcb_common_cb *rcb_common);
> -void hns_rcb_start(struct hnae_queue *q, u32 val);
>   int hns_rcb_get_cfg(struct rcb_common_cb *rcb_common);
>   void hns_rcb_get_queue_mode(enum dsaf_mode dsaf_mode,
>   			    u16 *max_vfn, u16 *max_q_per_vf);