Hello, I tried to delve into the iwlwifi code and I have 3 short questions about iwlwifi; I hope that I can get some advice here. 1) I see that there seems to be common code in 3945 and agn. Are there intentions to make use of agn in 3945 in the future to avoid code duplication, or is it a design decision to make a full separation between 3945 and agn (so even when it is possible to use common code, to avoid it in order not to cause possible problems in the future). 2) In some places in iwlwifi we have: #ifdef IEEE80211_CONF_CHANNEL_SWITCH. For example, iwl-agn.c or iwl3945-base.c. What is this define ? is it not (an uneeded) legacy from when the drivers were not part of the tree? 3) Little question about implementation: In iwl3945-base.c: iwl3945_commit_rxon() calls iwl3945_clear_stations_table(priv); so is it necessary in iwl3945_set_mode() to call also iwl3945_clear_stations_table() , since we anyhow call iwl3945_commit_rxon() afterwards ? the only thing iwl3945_clear_stations_table() does is zeros priv->num_stations and zeros priv->stations. David S -- 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
