On Tue, Aug 28, 2007 at 10:50:15AM +0200, Johannes Berg wrote: > On Tue, 2007-08-28 at 15:25 +0800, Zhu Yi wrote: > > > I know the method is not common used in the Linux drivers, but this is > > decided by the hardware layout. For 3945 and 4965, 90% of the > > hardware/firmware layout are the same and only 10% are different. This > > makes it possible to support two slightly differs hardwares in one > > driver. If the hardwares differ a lot (i.e 2100 and 2200), I won't even > > think about to do it in this way. > > To be fair, that's not what you do, you're building two drivers from one > source rather than have a single driver that supports both. I'm fairly > certain Christoph wouldn't object to the latter. Absolutely not. The right way to do it is to have a least a common driver core, and if the device-specific parts get too big they can but don't have to be separate modules. With s"light amount of duplicated code" I meant possibly duplicated bits if the split between common and device-specific bits doesn't work out perfectly because e.g. both device types do the same kind of operation on differently layed out data structures, which AFAIK is the case for some operations in this driver. > > johannes ---end quoted text--- - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
