On 5/29/2019 8:15 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> > --- > drivers/net/ixgbe/ixgbe_ethdev.c | 33 +++++++++++--------------------- > drivers/net/ixgbe/ixgbe_rxtx.c | 14 +++++--------- > 2 files changed, 16 insertions(+), 31 deletions(-) > > diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c > b/drivers/net/ixgbe/ixgbe_ethdev.c > index fae93bb29ffd..1d78085a9159 100644 > --- a/drivers/net/ixgbe/ixgbe_ethdev.c > +++ b/drivers/net/ixgbe/ixgbe_ethdev.c > @@ -2408,8 +2408,7 @@ ixgbe_dev_configure(struct rte_eth_dev *dev) > { > struct ixgbe_interrupt *intr = > IXGBE_DEV_PRIVATE_TO_INTR(dev->data->dev_private); > - struct ixgbe_adapter *adapter = > - (struct ixgbe_adapter *)dev->data->dev_private; > + struct ixgbe_adapter *adapter =dev->data->dev_private;
Needs space around "=", fixing while merging, same for all occurrences below.