staging: ks7010: drop counting sd errors
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Tue, 31 May 2016 10:56:42 +0000 (12:56 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Jun 2016 05:42:53 +0000 (22:42 -0700)
They were counted but never really used anywhere. Also change the printk
to a debug print, since it mostly shows on the expected -ENOMEDIUM on
card removal.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ks7010/ks7010_sdio.c
drivers/staging/ks7010/ks_wlan.h

index a8a14f9..c166022 100644 (file)
@@ -234,8 +234,8 @@ int ks_wlan_hw_power_save(struct ks_wlan_private *priv)
 static int ks7010_sdio_read(struct ks_wlan_private *priv, unsigned int address,
                            unsigned char *buffer, int length)
 {
-       int rc = -1;
        struct ks_sdio_card *card;
+       int rc;
 
        card = priv->ks_wlan_hw.sdio_card;
 
@@ -244,12 +244,8 @@ static int ks7010_sdio_read(struct ks_wlan_private *priv, unsigned int address,
        else    /* CMD53 multi-block transfer */
                rc = sdio_memcpy_fromio(card->func, buffer, address, length);
 
-       if (rc != 0) {
-               printk("sdio error erorr=%d size=%d\n", rc, length);
-               ++priv->sdio_error_count;
-       } else {
-               priv->sdio_error_count = 0;
-       }
+       if (rc != 0)
+               DPRINTK(1, "sdio error=%d size=%d\n", rc, length);
 
        return rc;
 }
@@ -257,8 +253,8 @@ static int ks7010_sdio_read(struct ks_wlan_private *priv, unsigned int address,
 static int ks7010_sdio_write(struct ks_wlan_private *priv, unsigned int address,
                             unsigned char *buffer, int length)
 {
-       int rc = -1;
        struct ks_sdio_card *card;
+       int rc;
 
        card = priv->ks_wlan_hw.sdio_card;
 
@@ -268,12 +264,8 @@ static int ks7010_sdio_write(struct ks_wlan_private *priv, unsigned int address,
                rc = sdio_memcpy_toio(card->func, (unsigned int)address, buffer,
                                      length);
 
-       if (rc != 0) {
-               printk("sdio error erorr=%d size=%d\n", rc, length);
-               ++priv->sdio_error_count;
-       } else {
-               priv->sdio_error_count = 0;
-       }
+       if (rc != 0)
+               DPRINTK(1, "sdio error=%d size=%d\n", rc, length);
 
        return rc;
 }
index f0f9f8e..a2126c2 100644 (file)
@@ -500,9 +500,7 @@ struct ks_wlan_private {
        unsigned long last_doze;
        unsigned long last_wakeup;
 
-       uint sdio_error_count;  /* SDIO error */
        uint wakeup_count;      /* for detect wakeup loop */
-
 };
 
 #endif /* _KS_WLAN_H */