Re: [PATCH] net/mlx5: DR, remove redundant object_range assignment

[Yevgeny Kliteynik]

On 24-Mar-25 21:41, Qasim Ijaz wrote:
> The initial assignment of object_range from
> pool->dmn->info.caps.log_header_modify_argument_granularity is
> redundant because it is immediately overwritten by the max_t() call.
>
> Remove the unnecessary assignment.
>
> Signed-off-by: Qasim Ijaz <qasdev00@xxxxxxxxx>
> ---
>   drivers/net/ethernet/mellanox/mlx5/core/steering/sws/dr_arg.c | 3 ---
>   1 file changed, 3 deletions(-)
>
> diff --git a/drivers/net/ethernet/mellanox/mlx5/core/steering/sws/dr_arg.c b/drivers/net/ethernet/mellanox/mlx5/core/steering/sws/dr_arg.c
> index 01ed6442095d..c2218dc556c7 100644
> --- a/drivers/net/ethernet/mellanox/mlx5/core/steering/sws/dr_arg.c
> +++ b/drivers/net/ethernet/mellanox/mlx5/core/steering/sws/dr_arg.c
> @@ -39,9 +39,6 @@ static int dr_arg_pool_alloc_objs(struct dr_arg_pool *pool)
>   
>   	INIT_LIST_HEAD(&cur_list);
>   
> -	object_range =
> -		pool->dmn->info.caps.log_header_modify_argument_granularity;
> -
>   	object_range =
>   		max_t(u32, pool->dmn->info.caps.log_header_modify_argument_granularity,
>   		      DR_ICM_MODIFY_HDR_GRANULARITY_4K);

Reviewed-by: Yevgeny Kliteynik <kliteyn@xxxxxxxxxx>