greearb@xxxxxxxxxxxxxxx writes: > From: Ben Greear <greearb@xxxxxxxxxxxxxxx> > > Properly clean up driver state in case firmware fails > to start scan for some reason. > > Signed-off-by: Ben Greear <greearb@xxxxxxxxxxxxxxx> Why just RFC? And "ath10k:" prefix missing. > --- a/drivers/net/wireless/ath/ath10k/wmi.c > +++ b/drivers/net/wireless/ath/ath10k/wmi.c > @@ -770,7 +770,25 @@ static int ath10k_wmi_event_scan(struct ath10k *ar, struct sk_buff *skb) > ath10k_dbg(ATH10K_DBG_WMI, "WMI_SCAN_EVENT_PREEMPTED\n"); > break; > case WMI_SCAN_EVENT_START_FAILED: > - ath10k_dbg(ATH10K_DBG_WMI, "WMI_SCAN_EVENT_START_FAILED\n"); > + ath10k_warn("WMI_SCAN_EVENT_START_FAILED, reason: %i\n", reason); "scan failed to start: %i\n" > + ar->scan_channel = NULL; > + if (!ar->scan.in_progress) { > + ath10k_warn("scan-start-failed: no scan requested, ignoring\n"); "scan failed to start but no scan requested, ignoring\n" -- 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