On Wed, 2012-06-20 at 11:11 +0200, Michal Kazior wrote: > >> +void > >> +cfg80211_get_chan_state(struct cfg80211_registered_device *rdev, > >> + struct wireless_dev *wdev, > >> + struct ieee80211_channel **chan, > >> + enum cfg80211_chan_mode *chanmode) > > > > Why not return the mode rather than void& pointer? > > I think returning the mode would be ambiguous - the function name isn't > "get_chan_mode". chan_mode isn't a result on its own. But then again, > it's just a matter of preference. No, ok, you're right -- I missed that it was returning both the channel & the mode. 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
