> > @@ -204,7 +205,7 @@ static int txgbe_request_irq(struct wx *wx) > > > > static void txgbe_up_complete(struct wx *wx) > > { > > - u32 reg; > > + struct txgbe *txgbe = (struct txgbe *)wx->priv; > > Personal choice I guess, but normally we tend to rely on compilers > accepting the implicit cast from void * to whatever struct pointer > in the kernel. Davem used to strongly push back against a cast when the pointer was a void *. Please remove the cast. Andrew