On Mon, 2008-12-29 at 20:29 +0530, Vivek Natarajan wrote: > u16 ah_currentRD5G; > u16 ah_currentRD2G; > char ah_iso[4]; > + enum ath9k_power_mode ah_powerMode; > + enum ath9k_power_mode ah_restoreMode; I think camelCase is discouraged in Linux sources. Sure, there are exceptions, but they are normally caused by names used in standards or other documentation. I cannot find "powerMode" in any drivers other than ath9k. > - enum ath9k_power_mode ah_powerMode; That's good. Let's follow Linux conventions for the new variable names. -- Regards, Pavel Roskin -- 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