On Fri, Sep 03, 2010 at 03:25:21PM +0100, Berg, Johannes wrote: > > > set_bit(STATUS_SCAN_HW, &priv->status); > > > > > > - if (priv->cfg->ops->hcmd->set_pan_params && > > > - priv->cfg->ops->hcmd->set_pan_params(priv)) > > > - goto done; > > > + if (priv->cfg->ops->hcmd->set_pan_params) { > > > + ret = priv->cfg->ops->hcmd->set_pan_params(priv); > > > + if (ret) > > > + return ret; > > > + } > > > STATUS_SCAN_HW bit still set here > > Good catch. +1 >Stanislaw, would you mind fixing that? Seems easier than for me to re-send a part of your patch series, but I can do if you prefer. I will fix. Stanislaw -- 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