On 5/29/2019 8:14 PM, Stephen Hemminger wrote:
> The device private pointer (dev_private) is of type void *
> therefore no cast is necessary in C.
> 
> Signed-off-by: Stephen Hemminger <step...@networkplumber.org>

<...>

> @@ -768,9 +765,7 @@ static int ena_link_update(struct rte_eth_dev *dev,
>                          __rte_unused int wait_to_complete)
>  {
>       struct rte_eth_link *link = &dev->data->dev_link;
> -     struct ena_adapter *adapter;
> -
> -     adapter = (struct ena_adapter *)(dev->data->dev_private);
> +     struct ena_adapter *adapter =dev->data->dev_private;

Needs space around "=", fixing while merging.

<...>

> @@ -2506,8 +2490,7 @@ static int ena_xstats_get(struct rte_eth_dev *dev,
>                         struct rte_eth_xstat *xstats,
>                         unsigned int n)
>  {
> -     struct ena_adapter *adapter =
> -                     (struct ena_adapter *)(dev->data->dev_private);
> +     struct ena_adapter *adapter =dev->data->dev_private;

Needs space around "=", fixing while merging.

Reply via email to