From: Rogério Brito <rbr...@ime.usp.br> Date: Mon, 27 Apr 2009 09:16:33 -0300
> Is this anything close to what needs to be done? It's not without > failures, because the function mace_set_timeout receives a pointer to a > struct net_device, but is marked inline and is used by mace_tx_timeout, > which receives an unsigned long (which calls mace_set_timeout). > > Perhaps it would be a case of removing the inline hint to the compiler? > I guess that BenH or Paul could comment here better... > > Signed-off-by: Rogério Brito <rbr...@ime.usp.br> You can fix the mace_set_timeout() function arguments by having a helper function that simply wraps around it and provides the second expection of argument types. Your patch is also wrong, it's missing a lot of netdev_ops entries that are implicitly obtained via alloc_etherdev(), namely: .ndo_change_mtu = eth_change_mtu, .ndo_set_mac_address = eth_mac_addr, .ndo_validate_addr = eth_validate_addr, _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev