Am Mittwoch, den 10.08.2011, 18:05 +0200 schrieb Johannes Berg: > On Wed, 2011-08-10 at 17:48 +0200, Alexander Simon wrote: > > Am Mittwoch, den 10.08.2011, 15:53 +0200 schrieb Johannes Berg: > > > On Mon, 2011-08-08 at 14:04 +0200, Alexander Simon wrote: > > > > > > > +struct cfg80211_bss *cfg80211_get_bss_ht(struct wiphy *wiphy, > > > > + struct ieee80211_channel *channel, > > > > + const u8 *bssid, > > > > + const u8 *ssid, size_t ssid_len, > > > > + u16 capa_mask, u16 capa_val, > > > > + enum nl80211_channel_type channel_type); > > > > > > indentation? Also, you really need kernel-doc for this. > > Pardon me, but I don't know what you mean... I just browsed > > Documentation/networking. Could you point me the exact file, please? > > Documentation/kernel-doc-nano-HOWTO.txt maybe? Uh, okay... I thought as there is no documentation for cfg80211_get_bss, there was no need to document the _ht version... The problem I'm facing here is that I really *need* to check the HT mode _inside_ the loop. Hence i need a _ht version. OR I could change cfg80211_get_bss only, causing to add the new parameters in *all* calls of cfg80211_get_bss... Whatever you prefer. To conclude this: * I'll write a _ht version that has a check_ht bool * if (check_ht == 1), match channel_type parameter * _ht version calls non-ht with check_ht = 0 * documentation for _ht (for non-ht also???) Indentation will be fixed, of course. Alex -- 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
