On Thu July 29 2010 02:24:46 Luis R. Rodriguez wrote: > So if you want to define an API I do believe its best to treat these > separately, for legacy I think your API can be fine tuned to consider > FIXED_A, FIXED_B, or DIVERSITY. hi luis, i already tried to explain several times why FIXED_A, FIXED_B, or DIVERSITY is not enough even for "legacy". please re-read the mails and the description of the first patch - i really don't want to re-iterate it *again*. thanks :) as felix pointed out, it's best to think of this as the antenna mask, which can be used to mask out antennas which are not attached or should not be used. it does not matter if the device is "legacy" or 802.11n or "802.11xyz" - we can always use this API to inform mac80211 / the driver about the available antennas if they are in fact different from the default HW settings. based on this information 802.11n drivers can derive which chainmasks to use, and in some cases wether diversity should be used; "legacy" drivers can derive wether to use diversity or not. actually thinking about it like this, i would like to remove the special casing of "0" for tx diversity (which means go back to my original proposal): if an antenna can be used for TX it should be set to one in the mask, and the driver can enable diversity or mulitple chainmasks as suitable. > Would we keep two values for some of these settings, an "actual hw" > capablity and then also a "configured" values? Would this be exposed > and visible through iw? the actual HW capability could be exported via the wiphy info as well, but i think this could be dealt with in a seperate patch, as well as other improvements. for now let's just define an API for an antenna mask, which can be used to mask out antennas seperately for RX and TX. i'll try to make that more clear in the descriptions and re-send my proposal. bruno -- 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
