On Tue, 2017-06-20 at 22:55 +0300, Sergey Matyukevich wrote:
> + if (sme->channel) {
> + /* FIXME: need to set proper nl80211_channel_type
> value */
> + cfg80211_chandef_create(&chandef, sme->channel,
> + NL80211_CHAN_HT20);
> + /* fall-back to minimal safe chandef description */
> + if (!cfg80211_chandef_valid(&chandef))
> + cfg80211_chandef_create(&chandef, sme-
> >channel,
> + NL80211_CHAN_HT20);
>
This seems odd since you just do the same thing over again? Not that I
could see how it would be invalid anyway.
johannes
