Commit c133b9bd authored by Tobin C. Harding's avatar Tobin C. Harding Committed by Greg Kroah-Hartman

staging: ks7010: remove non-zero comparison

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: default avatarTobin C. Harding <me@tobin.cc>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f283dd69
...@@ -61,7 +61,7 @@ static int ks7010_sdio_read(struct ks_wlan_private *priv, unsigned int address, ...@@ -61,7 +61,7 @@ static int ks7010_sdio_read(struct ks_wlan_private *priv, unsigned int address,
else /* CMD53 multi-block transfer */ else /* CMD53 multi-block transfer */
rc = sdio_memcpy_fromio(card->func, buffer, address, length); rc = sdio_memcpy_fromio(card->func, buffer, address, length);
if (rc != 0) if (rc)
DPRINTK(1, "sdio error=%d size=%d\n", rc, length); DPRINTK(1, "sdio error=%d size=%d\n", rc, length);
return rc; return rc;
...@@ -80,7 +80,7 @@ static int ks7010_sdio_write(struct ks_wlan_private *priv, unsigned int address, ...@@ -80,7 +80,7 @@ static int ks7010_sdio_write(struct ks_wlan_private *priv, unsigned int address,
else /* CMD53 */ else /* CMD53 */
rc = sdio_memcpy_toio(card->func, address, buffer, length); rc = sdio_memcpy_toio(card->func, address, buffer, length);
if (rc != 0) if (rc)
DPRINTK(1, "sdio error=%d size=%d\n", rc, length); DPRINTK(1, "sdio error=%d size=%d\n", rc, length);
return rc; return rc;
......
...@@ -528,7 +528,7 @@ void hostif_mib_get_confirm(struct ks_wlan_private *priv) ...@@ -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_size = get_WORD(priv); /* MIB value size */
mib_val_type = get_WORD(priv); /* MIB value type */ mib_val_type = get_WORD(priv); /* MIB value type */
if (mib_status != 0) { if (mib_status) {
/* in case of error */ /* in case of error */
DPRINTK(1, "attribute=%08X, status=%08X\n", mib_attribute, DPRINTK(1, "attribute=%08X, status=%08X\n", mib_attribute,
mib_status); mib_status);
...@@ -604,7 +604,7 @@ void hostif_mib_set_confirm(struct ks_wlan_private *priv) ...@@ -604,7 +604,7 @@ void hostif_mib_set_confirm(struct ks_wlan_private *priv)
mib_status = get_DWORD(priv); /* MIB Status */ mib_status = get_DWORD(priv); /* MIB Status */
mib_attribute = get_DWORD(priv); /* MIB attribute */ mib_attribute = get_DWORD(priv); /* MIB attribute */
if (mib_status != 0) { if (mib_status) {
/* in case of error */ /* in case of error */
DPRINTK(1, "error :: attribute=%08X, status=%08X\n", DPRINTK(1, "error :: attribute=%08X, status=%08X\n",
mib_attribute, mib_status); mib_attribute, mib_status);
...@@ -834,7 +834,7 @@ void hostif_scan_indication(struct ks_wlan_private *priv) ...@@ -834,7 +834,7 @@ void hostif_scan_indication(struct ks_wlan_private *priv)
DPRINTK(3, "scan_ind_count = %d\n", priv->scan_ind_count); DPRINTK(3, "scan_ind_count = %d\n", priv->scan_ind_count);
ap_info = (struct ap_info_t *)(priv->rxp); 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 */ for (i = 0; i < priv->aplist.size; i++) { /* bssid check */
if (!memcmp if (!memcmp
(ap_info->bssid, (ap_info->bssid,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment