On Fri, Jul 27, 2012 at 2:16 PM, Johannes Berg
[off-list ref] wrote:

From: Michal Kazior <redacted>

Split functionality for further reuse.

Will prevent code duplication when channel context
channel_type merging is introduced.

Signed-off-by: Michal Kazior <redacted>
Signed-off-by: Johannes Berg <redacted>
---

+
+bool ieee80211_set_channel_type(struct ieee80211_local *local,
+                               struct ieee80211_sub_if_data *sdata,
+                               enum nl80211_channel_type chantype)
+{
+       enum nl80211_channel_type superchan;
+       enum nl80211_channel_type compatchan = NL80211_CHAN_NO_HT;
+

no need to initialize compatchan.

+       superchan = ieee80211_get_superchan(local, sdata);
+       if (!ieee80211_channel_types_are_compatible(superchan, chantype,
+                                                   &compatchan))
+               return false;
+

as it should be assigned here.

Eliad.