staging: ks7010: remove non-zero comparison
authorTobin C. Harding <me@tobin.cc>
Tue, 21 Mar 2017 02:37:10 +0000 (13:37 +1100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 21 Mar 2017 07:56:07 +0000 (08:56 +0100)
Comparison, does not equal zero, is redundant

'if (foo != 0)'  is equal to  'if (foo)'

Typical kernel coding style is to use the shorter form.

Remove unnecessary non-zero comparison.

Signed-off-by: Tobin C. Harding <me@tobin.cc>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ks7010/ks7010_sdio.c
drivers/staging/ks7010/ks_hostif.c

index 40ec028de9acec86bb4816a64bce1744b73b2e81..8823e934aff7f62a9dc7c18a845516ea9e296704 100644 (file)
@@ -61,7 +61,7 @@ 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)
+       if (rc)
                DPRINTK(1, "sdio error=%d size=%d\n", rc, length);
 
        return rc;
@@ -80,7 +80,7 @@ static int ks7010_sdio_write(struct ks_wlan_private *priv, unsigned int address,
        else    /* CMD53 */
                rc = sdio_memcpy_toio(card->func, address, buffer, length);
 
-       if (rc != 0)
+       if (rc)
                DPRINTK(1, "sdio error=%d size=%d\n", rc, length);
 
        return rc;
index 3eb0f2e5189efe97f7bb37151d398220167a2f81..83cda1faf7ce38c3ec759f20a0166c068307294a 100644 (file)
@@ -528,7 +528,7 @@ void hostif_mib_get_confirm(struct ks_wlan_private *priv)
        mib_val_size = get_WORD(priv);  /* MIB value size */
        mib_val_type = get_WORD(priv);  /* MIB value type */
 
-       if (mib_status != 0) {
+       if (mib_status) {
                /* in case of error */
                DPRINTK(1, "attribute=%08X, status=%08X\n", mib_attribute,
                        mib_status);
@@ -604,7 +604,7 @@ void hostif_mib_set_confirm(struct ks_wlan_private *priv)
        mib_status = get_DWORD(priv);   /* MIB Status */
        mib_attribute = get_DWORD(priv);        /* MIB attribute */
 
-       if (mib_status != 0) {
+       if (mib_status) {
                /* in case of error */
                DPRINTK(1, "error :: attribute=%08X, status=%08X\n",
                        mib_attribute, mib_status);
@@ -834,7 +834,7 @@ void hostif_scan_indication(struct ks_wlan_private *priv)
        DPRINTK(3, "scan_ind_count = %d\n", priv->scan_ind_count);
        ap_info = (struct ap_info_t *)(priv->rxp);
 
-       if (priv->scan_ind_count != 0) {
+       if (priv->scan_ind_count) {
                for (i = 0; i < priv->aplist.size; i++) {       /* bssid check */
                        if (!memcmp
                            (ap_info->bssid,