On Sun, Sep 7, 2008 at 11:44 PM, Holger Schurig <hs4233@xxxxxxxxxxxxxxxxxxxx> wrote: > On Friday 05 September 2008 21:11:49 Luis R. Rodriguez wrote: >> >> +struct ieee80211_freq_range { >> >> + u32 start_freq; >> >> + u32 end_freq; >> >> + u32 max_bandwidth; >> > >> > u32 seems excessive for the bandwidth, no? >> >> I think its fine, remember this is in KHz. > > max_bandwitth_khz ??? Seems reasonable here. >> >> u16 center_freq; >> >> + u8 max_bandwidth; >> > >> > and here you're using u8 anyway :) >> >> and this is in MHz > > max_bandwidth_mhz ??? I'm in favor but then center_freq_mhz should also be used, but lets not go there and simply just add _khz for the freq_range. Luis -- 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