Michal Kazior <michal.kazior@xxxxxxxxx> writes: > All configuration sequences should be protected > with conf_mutex to avoid concurrent/conflicting > requests. > > This should make sure that wep tx key setup is not > performed while hw is restarted (at least). > > Signed-off-by: Michal Kazior <michal.kazior@xxxxxxxxx> [...] > @@ -1902,10 +1904,12 @@ static void ath10k_tx_wep_key_work(struct work_struct *work) > ath10k_warn("failed to update wep key index for vdev %d: %d\n", > arvif->vdev_id, > ret); > - return; > + goto unlock; > } > > arvif->def_wep_key_idx = keyidx; > +unlock: > + mutex_unlock(&arvif->ar->conf_mutex); > } Empty line before the label, please. -- Kalle Valo -- 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