[PATCH] powerpc: convert mace to netdev_ops

2009-05-03 Thread Rogério Brito
Hi, Dave. On Apr 27 2009, David Miller wrote: 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. I hope that this version is a slightly better fix to convert mace to netdev_ops.

Re: [PATCH] powerpc: convert mace to netdev_ops

2009-05-03 Thread David Miller
From: Rogério Brito rbr...@ime.usp.br Date: Sun, 3 May 2009 08:48:20 -0300 I hope that this version is a slightly better fix to convert mace to netdev_ops. This is against this morning's net-2.6 tree. Signed-off-by: Rogério Brito rbr...@ime.usp.br I'll apply this, thanks! I made one

Re: [PATCH] powerpc: convert mace to netdev_ops

2009-05-03 Thread David Miller
From: David Miller da...@davemloft.net Date: Sun, 03 May 2009 14:15:24 -0700 (PDT) From: Rogério Brito rbr...@ime.usp.br Date: Sun, 3 May 2009 08:48:20 -0300 I hope that this version is a slightly better fix to convert mace to netdev_ops. This is against this morning's net-2.6 tree.

[PATCH] powerpc: convert mace to netdev_ops (was: Re: [2.6.30-rc3] powerpc: compilation error of mace module)

2009-04-27 Thread Rogério Brito
Hi, Dave. On Apr 26 2009, David Miller wrote: Or, if you're bored, feel free to convert the mace driver over to netdev_ops :-) 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

Re: [PATCH] powerpc: convert mace to netdev_ops

2009-04-27 Thread David Miller
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

Re: [PATCH] powerpc: convert mace to netdev_ops

2009-04-27 Thread Rogério Brito
Hi, Dave. On Apr 27 2009, David Miller wrote: 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. Hummm, this means that I'm not that bad... The wrapper function was the first thing