On Wed, 2014-03-12 at 14:26 +0100, Michal Kazior wrote:
> On 12 March 2014 14:06, Luciano Coelho <luciano.coelho@xxxxxxxxx> wrote:
>
> [...]
>
> > + if (sdata->vif.bss_conf.chandef.width != sdata->reserved_chandef.width)
> > + tmp_changed |= BSS_CHANGED_BANDWIDTH;
> > +
> > + sdata->vif.bss_conf.chandef = sdata->reserved_chandef;
>
> > +
> > + /* unref our reservation before assigning */
> > + ctx->refcount--;
> > + sdata->reserved_chanctx = NULL;
> > +
> > + if (old_ctx == ctx) {
> > + /* This is our own context, just change it */
> > + ret = __ieee80211_vif_change_channel(sdata, old_ctx,
> > + &tmp_changed);
> > + if (ret)
> > + goto out;
> > + } else {
>
> > + if (sdata->vif.bss_conf.chandef.width !=
> > + sdata->reserved_chandef.width)
> > + tmp_changed |= BSS_CHANGED_BANDWIDTH;
> > +
> > + sdata->vif.bss_conf.chandef = sdata->reserved_chandef;
>
> You're already doing this a few lines earlier, before branching.
Yeah... too many rebases and I'm not concentrating enough... *sigh*.
--
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
