On Wed, Apr 13, 2011 at 04:35:19PM +0530, Kalle Valo wrote: > Vasanthakumar Thiagarajan <vasanth@xxxxxxxxxxx> writes: > > > -#define AR_WA 0x4004 > > +#define AR_WA (AR_SREV_9340(ah) ? 0x40c4 : 0x4004) > > So now allthe macros require this ah variable? That's confusing as > that's not visible when using the macro. All register read/write functions require ath9k hw structure (ah) to be available anyway, so it is reasonable to assume that ah will be available to determine that register offset as well. Vasanth -- 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