staging: ks7010: remove max_event_count field which is only being used in debug
authorSergio Paracuellos <sergio.paracuellos@gmail.com>
Tue, 20 Mar 2018 14:36:44 +0000 (15:36 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 22 Mar 2018 17:20:54 +0000 (18:20 +0100)
This patch removes max_event_count field of sme_info structure which is a
write only variable just being used for debug purposes.

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

index 81391df9e6d2473584715e21b34c3630068184a3..ce1ac119a03162a834f55084cf5779628cd3497c 100644 (file)
@@ -2409,10 +2409,6 @@ void hostif_sme_enqueue(struct ks_wlan_private *priv, unsigned short event)
        if (cnt_smeqbody(priv) < (SME_EVENT_BUFF_SIZE - 1)) {
                priv->sme_i.event_buff[priv->sme_i.qtail] = event;
                inc_smeqtail(priv);
-#ifdef DEBUG
-               if (priv->sme_i.max_event_count < cnt_smeqbody(priv))
-                       priv->sme_i.max_event_count = cnt_smeqbody(priv);
-#endif
        } else {
                /* in case of buffer overflow */
                netdev_err(priv->net_dev, "sme queue buffer overflow\n");
@@ -2461,9 +2457,6 @@ int hostif_init(struct ks_wlan_private *priv)
        priv->sme_i.sme_status = SME_IDLE;
        priv->sme_i.qhead = 0;
        priv->sme_i.qtail = 0;
-#ifdef DEBUG
-       priv->sme_i.max_event_count = 0;
-#endif
        spin_lock_init(&priv->sme_i.sme_spin);
        priv->sme_i.sme_flag = 0;
 
index 6bad88d287fde44e78dfa5bbb55a656afa7ed218..aeabbe37619b440c0cac7d728664f0f1cc43ad1a 100644 (file)
@@ -192,10 +192,6 @@ struct sme_info {
        int event_buff[SME_EVENT_BUFF_SIZE];
        unsigned int qhead;
        unsigned int qtail;
-#ifdef DEBUG
-       /* for debug */
-       unsigned int max_event_count;
-#endif
        spinlock_t sme_spin;
        unsigned long sme_flag;
 };