ath10k: update comment regarding warm reset
authorMichal Kazior <michal.kazior@tieto.com>
Fri, 22 Aug 2014 12:33:15 +0000 (14:33 +0200)
committerKalle Valo <kvalo@qca.qualcomm.com>
Mon, 25 Aug 2014 08:31:58 +0000 (11:31 +0300)
The old comment was a little out of date. HTT Rx
ring is a more relevant problem when stopping
transport layer.

Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/ath10k/pci.c

index 21f7dc3..daa38ce 100644 (file)
@@ -1243,11 +1243,10 @@ static void ath10k_pci_hif_stop(struct ath10k *ar)
        ath10k_pci_irq_disable(ar);
        ath10k_pci_flush(ar);
 
-       /* Make the sure the device won't access any structures on the host by
-        * resetting it. The device was fed with PCI CE ringbuffer
-        * configuration during init. If ringbuffers are freed and the device
-        * were to access them this could lead to memory corruption on the
-        * host. */
+       /* Most likely the device has HTT Rx ring configured. The only way to
+        * prevent the device from accessing (and possible corrupting) host
+        * memory is to reset the chip now.
+        */
        ath10k_pci_warm_reset(ar);
 
        ar_pci->started = 0;