]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
staging: ks7010: avoid one level indentation in devio_rec_ind function
authorSergio Paracuellos <sergio.paracuellos@gmail.com>
Mon, 23 Apr 2018 13:44:49 +0000 (15:44 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 25 Apr 2018 13:47:23 +0000 (15:47 +0200)
This commit changes logic to handle with the status of the device
at first checking for close state to return directly instead
of just do the stuff when device is open. This improves readability
avoiding one level indentation.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ks7010/ks_hostif.c

index 3b10f9ac0fa000910589b5013f19b2cc12724015..156e732999f25a9aecbaaea56718bbf27cbf0da1 100644 (file)
@@ -1562,24 +1562,25 @@ void hostif_mic_failure_request(struct ks_wlan_private *priv,
 static void devio_rec_ind(struct ks_wlan_private *priv, unsigned char *p,
                          unsigned int size)
 {
-       if (priv->is_device_open) {
-               spin_lock(&priv->dev_read_lock);        /* request spin lock */
-               priv->dev_data[atomic_read(&priv->rec_count)] = p;
-               priv->dev_size[atomic_read(&priv->rec_count)] = size;
-
-               if (atomic_read(&priv->event_count) != DEVICE_STOCK_COUNT) {
-                       /* rx event count inc */
-                       atomic_inc(&priv->event_count);
-               }
-               atomic_inc(&priv->rec_count);
-               if (atomic_read(&priv->rec_count) == DEVICE_STOCK_COUNT)
-                       atomic_set(&priv->rec_count, 0);
+       if (!priv->is_device_open)
+               return;
 
-               wake_up_interruptible_all(&priv->devread_wait);
+       spin_lock(&priv->dev_read_lock);        /* request spin lock */
+       priv->dev_data[atomic_read(&priv->rec_count)] = p;
+       priv->dev_size[atomic_read(&priv->rec_count)] = size;
 
-               /* release spin lock */
-               spin_unlock(&priv->dev_read_lock);
+       if (atomic_read(&priv->event_count) != DEVICE_STOCK_COUNT) {
+               /* rx event count inc */
+               atomic_inc(&priv->event_count);
        }
+       atomic_inc(&priv->rec_count);
+       if (atomic_read(&priv->rec_count) == DEVICE_STOCK_COUNT)
+               atomic_set(&priv->rec_count, 0);
+
+       wake_up_interruptible_all(&priv->devread_wait);
+
+       /* release spin lock */
+       spin_unlock(&priv->dev_read_lock);
 }
 
 void hostif_receive(struct ks_wlan_private *priv, unsigned char *p,