[PATCH v2 01/14] ethdev: remove unnecessary deref on function pointers

[Andrew Rybchenko]

On 2/20/25 05:32, Stephen Hemminger wrote:
> The expression *dev->dev_ops->rx_queue_start and
> dev->dev_ops->rx_queue_start are equivalent.
> Remove the unnecessary asterisk and parenthesis.
> 
> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>

Reviewed-by: Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>

one nit below

> @@ -6995,11 +6966,11 @@ rte_eth_ip_reassembly_capability_get(uint16_t port_id,
>   		return -EINVAL;
>   	}
>   
> -	if (*dev->dev_ops->ip_reassembly_capability_get == NULL)
> +	if (dev->dev_ops->ip_reassembly_capability_get == NULL)
>   		return -ENOTSUP;
>   	memset(reassembly_capa, 0, sizeof(struct rte_eth_ip_reassembly_params));
>   
> -	ret = eth_err(port_id, (*dev->dev_ops->ip_reassembly_capability_get)
> +	ret = eth_err(port_id, (dev->dev_ops->ip_reassembly_capability_get)

Why are parenthesis kept above instead of moving to the next line and
remove parenthesis as it is done in the rest of cases?

>   					(dev, reassembly_capa));
>   
>   	rte_eth_trace_ip_reassembly_capability_get(port_id, reassembly_capa,