On Mon, 2012-07-02 at 18:01 +0300, Vladimir Kondratiev wrote: > So, now I'll focus on the last patch - bitrate calculation. > > My plan is: > > - add new element like NL80211_RATE_INFO_BITRATE_HT with u32 data > - change nl80211_put_sta_rate() to fill both > NL80211_RATE_INFO_BITRATE (with u16 value) > and NL80211_RATE_INFO_BITRATE_HT (with u32 value) > - when bitrate not fit into u16, put 0 for u16 value and fill only u32 I think you should leave out the BITRATE value completely, not fill it with 0. > This opens migration path for the drivers and user space tools to the > new u32 representation without breaking existing code. Other than that, yes, good plan. johannes -- 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
