diff options
author | Jason Cooper <jason@lakedaemon.net> | 2010-09-30 15:15:37 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-05 12:22:24 -0400 |
commit | a1b04b0b33d2d5d2ecbc7c67845619bee493461e (patch) | |
tree | af21cf5da6d33f5051d4eb019e3564199f65297e | |
parent | 57c096b897fe59e106452500afbbe0e0f1f8e149 (diff) |
staging: brcm80211: fix checkpatch error 'assignment in if condition'
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Acked-by: Henry Ptasinski <henryp@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/staging/brcm80211/brcmfmac/bcmsdh_sdmmc.c | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/drivers/staging/brcm80211/brcmfmac/bcmsdh_sdmmc.c b/drivers/staging/brcm80211/brcmfmac/bcmsdh_sdmmc.c index a6616010d61..ba990a54d9b 100644 --- a/drivers/staging/brcm80211/brcmfmac/bcmsdh_sdmmc.c +++ b/drivers/staging/brcm80211/brcmfmac/bcmsdh_sdmmc.c | |||
@@ -123,7 +123,8 @@ extern sdioh_info_t *sdioh_attach(osl_t *osh, void *bar0, uint irq) | |||
123 | return NULL; | 123 | return NULL; |
124 | } | 124 | } |
125 | 125 | ||
126 | if ((sd = (sdioh_info_t *) MALLOC(osh, sizeof(sdioh_info_t))) == NULL) { | 126 | sd = (sdioh_info_t *) MALLOC(osh, sizeof(sdioh_info_t)); |
127 | if (sd == NULL) { | ||
127 | sd_err(("sdioh_attach: out of memory, malloced %d bytes\n", | 128 | sd_err(("sdioh_attach: out of memory, malloced %d bytes\n", |
128 | MALLOCED(osh))); | 129 | MALLOCED(osh))); |
129 | return NULL; | 130 | return NULL; |
@@ -417,12 +418,14 @@ sdioh_iovar_op(sdioh_info_t *si, const char *name, | |||
417 | sd_trace(("%s: Enter (%s %s)\n", __func__, (set ? "set" : "get"), | 418 | sd_trace(("%s: Enter (%s %s)\n", __func__, (set ? "set" : "get"), |
418 | name)); | 419 | name)); |
419 | 420 | ||
420 | if ((vi = bcm_iovar_lookup(sdioh_iovars, name)) == NULL) { | 421 | vi = bcm_iovar_lookup(sdioh_iovars, name); |
422 | if (vi == NULL) { | ||
421 | bcmerror = BCME_UNSUPPORTED; | 423 | bcmerror = BCME_UNSUPPORTED; |
422 | goto exit; | 424 | goto exit; |
423 | } | 425 | } |
424 | 426 | ||
425 | if ((bcmerror = bcm_iovar_lencheck(vi, arg, len, set)) != 0) | 427 | bcmerror = bcm_iovar_lencheck(vi, arg, len, set); |
428 | if (bcmerror != 0) | ||
426 | goto exit; | 429 | goto exit; |
427 | 430 | ||
428 | /* Set up params so get and set can share the convenience variables */ | 431 | /* Set up params so get and set can share the convenience variables */ |
@@ -1037,12 +1040,11 @@ sdioh_request_buffer(sdioh_info_t *sd, uint pio_dma, uint fix_inc, uint write, | |||
1037 | sd_data(("%s: Creating new %s Packet, len=%d\n", | 1040 | sd_data(("%s: Creating new %s Packet, len=%d\n", |
1038 | __func__, write ? "TX" : "RX", buflen_u)); | 1041 | __func__, write ? "TX" : "RX", buflen_u)); |
1039 | #ifdef DHD_USE_STATIC_BUF | 1042 | #ifdef DHD_USE_STATIC_BUF |
1040 | if (!(mypkt = | 1043 | mypkt = PKTGET_STATIC(sd->osh, buflen_u, write ? TRUE : FALSE); |
1041 | PKTGET_STATIC(sd->osh, buflen_u, write ? TRUE : FALSE))) { | ||
1042 | #else | 1044 | #else |
1043 | if (!(mypkt = PKTGET(sd->osh, buflen_u, | 1045 | mypkt = PKTGET(sd->osh, buflen_u, write ? TRUE : FALSE); |
1044 | write ? TRUE : FALSE))) { | ||
1045 | #endif /* DHD_USE_STATIC_BUF */ | 1046 | #endif /* DHD_USE_STATIC_BUF */ |
1047 | if (!mypkt) { | ||
1046 | sd_err(("%s: PKTGET failed: len %d\n", | 1048 | sd_err(("%s: PKTGET failed: len %d\n", |
1047 | __func__, buflen_u)); | 1049 | __func__, buflen_u)); |
1048 | return SDIOH_API_RC_FAIL; | 1050 | return SDIOH_API_RC_FAIL; |
@@ -1073,13 +1075,12 @@ sdioh_request_buffer(sdioh_info_t *sd, uint pio_dma, uint fix_inc, uint write, | |||
1073 | sd_data(("%s: Creating aligned %s Packet, len=%d\n", | 1075 | sd_data(("%s: Creating aligned %s Packet, len=%d\n", |
1074 | __func__, write ? "TX" : "RX", PKTLEN(pkt))); | 1076 | __func__, write ? "TX" : "RX", PKTLEN(pkt))); |
1075 | #ifdef DHD_USE_STATIC_BUF | 1077 | #ifdef DHD_USE_STATIC_BUF |
1076 | if (!(mypkt = | 1078 | mypkt = PKTGET_STATIC(sd->osh, PKTLEN(pkt), |
1077 | PKTGET_STATIC(sd->osh, PKTLEN(pkt), | 1079 | write ? TRUE : FALSE); |
1078 | write ? TRUE : FALSE))) { | ||
1079 | #else | 1080 | #else |
1080 | if (!(mypkt = | 1081 | mypkt = PKTGET(sd->osh, PKTLEN(pkt), write ? TRUE : FALSE); |
1081 | PKTGET(sd->osh, PKTLEN(pkt), write ? TRUE : FALSE))) { | ||
1082 | #endif /* DHD_USE_STATIC_BUF */ | 1082 | #endif /* DHD_USE_STATIC_BUF */ |
1083 | if (!mypkt) { | ||
1083 | sd_err(("%s: PKTGET failed: len %d\n", | 1084 | sd_err(("%s: PKTGET failed: len %d\n", |
1084 | __func__, PKTLEN(pkt))); | 1085 | __func__, PKTLEN(pkt))); |
1085 | return SDIOH_API_RC_FAIL; | 1086 | return SDIOH_API_RC_FAIL; |