Eduardo Abinader <eduardoabinader@xxxxxxxxx> writes: > Signed-off-by: Eduardo Abinader <eduardoabinader@xxxxxxxxx> No empty commit logs, please. > --- a/drivers/net/wireless/ath/ath10k/mac.c > +++ b/drivers/net/wireless/ath/ath10k/mac.c > @@ -6786,13 +6786,13 @@ static int ath10k_sta_state(struct ieee80211_hw *hw, > /* > * Disassociation. > */ > - ath10k_dbg(ar, ATH10K_DBG_MAC, "mac sta %pM disassociated\n", > - sta->addr); > - > ret = ath10k_station_disassoc(ar, vif, sta); > if (ret) > ath10k_warn(ar, "failed to disassociate station: %pM vdev %i: %i\n", > sta->addr, arvif->vdev_id, ret); > + else > + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac sta %pM disassociated\n", > + sta->addr); I don't understand, how is this better? -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
