ath11k: Fix error code in ath11k_core_suspend()
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 16 Dec 2020 08:31:19 +0000 (11:31 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 17 Dec 2020 06:48:07 +0000 (08:48 +0200)
The "if (!ret)" condition is inverted and it should be "if (ret)".  It means
that we return success when we had intended to return an error code. This also
caused a spurious warning even when the suspend was successful:

[  297.186612] ath11k_pci 0000:06:00.0: failed to suspend hif: 0

Fixes: d1b0c33850d2 ("ath11k: implement suspend for QCA6390 PCI devices")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/X9nF17L2/EKOSbn/@mwanda
drivers/net/wireless/ath/ath11k/core.c

index b97c38b..350b791 100644 (file)
@@ -185,7 +185,7 @@ int ath11k_core_suspend(struct ath11k_base *ab)
        ath11k_hif_ce_irq_disable(ab);
 
        ret = ath11k_hif_suspend(ab);
-       if (!ret) {
+       if (ret) {
                ath11k_warn(ab, "failed to suspend hif: %d\n", ret);
                return ret;
        }