On Wed, 2013-05-29 at 21:52 +0200, Johannes Berg wrote: > > atomic_inc(&sdata->u.mesh.estab_plinks); > > - return mesh_accept_plinks_update(sdata); > > + mesh_accept_plinks_update(sdata); > > + return BSS_CHANGED_BEACON; > > These changes seem odd to me, why doesn't mesh_accept_plinks_update() > just return the change value then? Oh wait, n/m, but shouldn't you have something like return mesh_accept_...(...) | BSS_CHANGED_BEACON; ? 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