On Mon, Dec 31, 2007 at 08:22:27PM -0600, Jay Cliburn wrote: > > [Neglected to cc netdev in the first wave.] > > Hello Jeff, > > Happy New Year to you and all. > > In preparation for a future atl2 driver for the Atheros L2 10/100 chip, > we propose to move the existing atl1 driver to a new directory > (drivers/net/atlx), then split out functions and definitions that both > atl1 and atl2 can share. The final structure will look like this: > > directory or file status > ======================= ========================== > drivers/net/atl1/ deleted > drivers/net/atlx/ new > drivers/net/atlx/atl1.c atl1-specific functions > drivers/net/atlx/atl1.h atl1-specific definitions > drivers/net/atlx/atlx.c atl1-atl2 shared functions > drivers/net/atlx/atlx.h atl1-atl2 shared definitions > > The first two patches submitted in this patchset accomplish the relocation > by movng the atl1 driver -- lock, stock, and barrel -- over to > drivers/net/atlx, then splitting out shareable functions and definitions. > Some transitory hackery will be present until the atl2 merge. Please > overlook it for now. > > The remaining 24 patches bring the atl1 driver up to par with the current > vendor driver version 1.2.40.2. NAPI support is included and it seems > to work, but it needs to be scrutinized by an experienced eye. I had a > hard time finding much current NAPI documentation, so I just hacked at > it by looking at the e1000 driver.
How much shared code is going to be there in the end? Unless it's a lot I fear that the maintaince burden far outweights the benefit of having shared code. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
