ath10k: reduce indentation by moving powersave check within function
authorRajkumar Manoharan <rmanohar@qti.qualcomm.com>
Wed, 16 Dec 2015 14:52:19 +0000 (16:52 +0200)
committerKalle Valo <kvalo@qca.qualcomm.com>
Thu, 31 Dec 2015 13:23:15 +0000 (15:23 +0200)
For devices that does not support PCI power save, force wake up
API is used. So move powersave check inside force wakeup to reduce
one level indentation.

Signed-off-by: Rajkumar Manoharan <rmanohar@qti.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/ath10k/pci.c

index d77ba4c09e7807b3f2318833dcbcfadf3a82ae66..ee925c618535f464a213b59d35a0a4c8688da59f 100644 (file)
@@ -487,6 +487,9 @@ static int ath10k_pci_force_wake(struct ath10k *ar)
        unsigned long flags;
        int ret = 0;
 
+       if (ar_pci->pci_ps)
+               return ret;
+
        spin_lock_irqsave(&ar_pci->ps_lock, flags);
 
        if (!ar_pci->ps_awake) {
@@ -2480,12 +2483,10 @@ static int ath10k_pci_hif_resume(struct ath10k *ar)
        u32 val;
        int ret = 0;
 
-       if (ar_pci->pci_ps == 0) {
-               ret = ath10k_pci_force_wake(ar);
-               if (ret) {
-                       ath10k_err(ar, "failed to wake up target: %d\n", ret);
-                       return ret;
-               }
+       ret = ath10k_pci_force_wake(ar);
+       if (ret) {
+               ath10k_err(ar, "failed to wake up target: %d\n", ret);
+               return ret;
        }
 
        /* Suspend/Resume resets the PCI configuration space, so we have to
@@ -2592,13 +2593,10 @@ static irqreturn_t ath10k_pci_interrupt_handler(int irq, void *arg)
        struct ath10k_pci *ar_pci = ath10k_pci_priv(ar);
        int ret;
 
-       if (ar_pci->pci_ps == 0) {
-               ret = ath10k_pci_force_wake(ar);
-               if (ret) {
-                       ath10k_warn(ar, "failed to wake device up on irq: %d\n",
-                                   ret);
-                       return IRQ_NONE;
-               }
+       ret = ath10k_pci_force_wake(ar);
+       if (ret) {
+               ath10k_warn(ar, "failed to wake device up on irq: %d\n", ret);
+               return IRQ_NONE;
        }
 
        if (ar_pci->num_msi_intrs == 0) {
@@ -3071,12 +3069,10 @@ static int ath10k_pci_probe(struct pci_dev *pdev,
                goto err_sleep;
        }
 
-       if (ar_pci->pci_ps == 0) {
-               ret = ath10k_pci_force_wake(ar);
-               if (ret) {
-                       ath10k_warn(ar, "failed to wake up device : %d\n", ret);
-                       goto err_free_pipes;
-               }
+       ret = ath10k_pci_force_wake(ar);
+       if (ret) {
+               ath10k_warn(ar, "failed to wake up device : %d\n", ret);
+               goto err_free_pipes;
        }
 
        ath10k_pci_ce_deinit(ar);