Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

staging: r8188eu: clean up comparsions to NULL in hal directory

Clean up comparsions to NULL in the hal directory reported by
checkpatch.

x == NULL -> !x
x != NULL -> x

Signed-off-by: Michael Straube <straube.linux@gmail.com>
Link: https://lore.kernel.org/r/20210801084614.4328-1-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

authored by

Michael Straube and committed by
Greg Kroah-Hartman
725a3f1c 11e14fc3

+29 -29
+1 -1
drivers/staging/r8188eu/hal/Hal8188ERateAdaptive.c
··· 486 486 struct odm_ra_info *pRaInfo = &dm_odm->RAInfo[macid]; 487 487 u8 WirelessMode = 0xFF; /* invalid value */ 488 488 u8 max_rate_idx = 0x13; /* MCS7 */ 489 - if (dm_odm->pWirelessMode != NULL) 489 + if (dm_odm->pWirelessMode) 490 490 WirelessMode = *(dm_odm->pWirelessMode); 491 491 492 492 if (WirelessMode != 0xFF) {
+2 -2
drivers/staging/r8188eu/hal/HalHWImg8188E_BB.c
··· 195 195 196 196 if (biol) { 197 197 pxmit_frame = rtw_IOL_accquire_xmit_frame(adapter); 198 - if (pxmit_frame == NULL) { 198 + if (!pxmit_frame) { 199 199 pr_info("rtw_IOL_accquire_xmit_frame failed\n"); 200 200 return HAL_STATUS_FAILURE; 201 201 } ··· 474 474 475 475 if (biol) { 476 476 pxmit_frame = rtw_IOL_accquire_xmit_frame(adapter); 477 - if (pxmit_frame == NULL) { 477 + if (!pxmit_frame) { 478 478 pr_info("rtw_IOL_accquire_xmit_frame failed\n"); 479 479 return HAL_STATUS_FAILURE; 480 480 }
+1 -1
drivers/staging/r8188eu/hal/HalHWImg8188E_MAC.c
··· 153 153 154 154 if (biol) { 155 155 pxmit_frame = rtw_IOL_accquire_xmit_frame(adapt); 156 - if (pxmit_frame == NULL) { 156 + if (!pxmit_frame) { 157 157 pr_info("rtw_IOL_accquire_xmit_frame failed\n"); 158 158 return HAL_STATUS_FAILURE; 159 159 }
+1 -1
drivers/staging/r8188eu/hal/HalHWImg8188E_RF.c
··· 163 163 164 164 if (biol) { 165 165 pxmit_frame = rtw_IOL_accquire_xmit_frame(Adapter); 166 - if (pxmit_frame == NULL) { 166 + if (!pxmit_frame) { 167 167 pr_info("rtw_IOL_accquire_xmit_frame failed\n"); 168 168 return HAL_STATUS_FAILURE; 169 169 }
+1 -1
drivers/staging/r8188eu/hal/hal_com.c
··· 322 322 int i; 323 323 u8 trigger; 324 324 325 - if (buf == NULL) 325 + if (!buf) 326 326 goto exit; 327 327 328 328 trigger = rtw_read8(adapter, REG_C2HEVT_CLEAR);
+2 -2
drivers/staging/r8188eu/hal/hal_intf.c
··· 224 224 225 225 s32 rtw_hal_init_xmit_priv(struct adapter *adapt) 226 226 { 227 - if (adapt->HalFunc.init_xmit_priv != NULL) 227 + if (adapt->HalFunc.init_xmit_priv) 228 228 return adapt->HalFunc.init_xmit_priv(adapt); 229 229 return _FAIL; 230 230 } 231 231 232 232 void rtw_hal_free_xmit_priv(struct adapter *adapt) 233 233 { 234 - if (adapt->HalFunc.free_xmit_priv != NULL) 234 + if (adapt->HalFunc.free_xmit_priv) 235 235 adapt->HalFunc.free_xmit_priv(adapt); 236 236 } 237 237
+2 -2
drivers/staging/r8188eu/hal/rtl8188e_cmd.c
··· 490 490 491 491 DBG_88E("%s\n", __func__); 492 492 ReservedPagePacket = (u8 *)rtw_zmalloc(1000); 493 - if (ReservedPagePacket == NULL) { 493 + if (!ReservedPagePacket) { 494 494 DBG_88E("%s: alloc ReservedPagePacket fail!\n", __func__); 495 495 return; 496 496 } ··· 559 559 560 560 TotalPacketLen = BufIndex + QosNullLength; 561 561 pmgntframe = alloc_mgtxmitframe(pxmitpriv); 562 - if (pmgntframe == NULL) 562 + if (!pmgntframe) 563 563 goto exit; 564 564 565 565 /* update attribute */
+5 -5
drivers/staging/r8188eu/hal/rtl8188e_hal_init.c
··· 77 77 u8 u1temp = 0; 78 78 79 79 efuseTbl = (u8 *)rtw_zmalloc(EFUSE_MAP_LEN_88E); 80 - if (efuseTbl == NULL) { 80 + if (!efuseTbl) { 81 81 DBG_88E("%s: alloc efuseTbl fail!\n", __func__); 82 82 goto exit; 83 83 } 84 84 85 85 eFuseWord = (u16 **)rtw_malloc2d(EFUSE_MAX_SECTION_88E, EFUSE_MAX_WORD_UNIT, sizeof(u16)); 86 - if (eFuseWord == NULL) { 86 + if (!eFuseWord) { 87 87 DBG_88E("%s: alloc eFuseWord fail!\n", __func__); 88 88 goto exit; 89 89 } ··· 811 811 } 812 812 813 813 efuseTbl = (u8 *)rtw_zmalloc(EFUSE_MAP_LEN_88E); 814 - if (efuseTbl == NULL) { 814 + if (!efuseTbl) { 815 815 DBG_88E("%s: alloc efuseTbl fail!\n", __func__); 816 816 goto exit; 817 817 } 818 818 819 819 eFuseWord = (u16 **)rtw_malloc2d(EFUSE_MAX_SECTION_88E, EFUSE_MAX_WORD_UNIT, sizeof(u16)); 820 - if (eFuseWord == NULL) { 820 + if (!eFuseWord) { 821 821 DBG_88E("%s: alloc eFuseWord fail!\n", __func__); 822 822 goto exit; 823 823 } ··· 1240 1240 1241 1241 EFUSE_GetEfuseDefinition(pAdapter, EFUSE_WIFI, TYPE_EFUSE_MAX_SECTION, (void *)&max_section, bPseudoTest); 1242 1242 1243 - if (data == NULL) 1243 + if (!data) 1244 1244 return false; 1245 1245 if (offset > max_section) 1246 1246 return false;
+1 -1
drivers/staging/r8188eu/hal/rtl8188e_rxdesc.c
··· 28 28 struct rx_pkt_attrib *pattrib; 29 29 struct signal_stat *signal_stat; 30 30 31 - if (prframe == NULL || padapter == NULL) 31 + if (!prframe || !padapter) 32 32 return; 33 33 34 34 pattrib = &prframe->attrib;
+1 -1
drivers/staging/r8188eu/hal/rtl8188eu_recv.c
··· 45 45 _rtw_init_queue(&precvpriv->free_recv_buf_queue); 46 46 47 47 precvpriv->pallocated_recv_buf = rtw_zmalloc(NR_RECVBUFF * sizeof(struct recv_buf) + 4); 48 - if (precvpriv->pallocated_recv_buf == NULL) { 48 + if (!precvpriv->pallocated_recv_buf) { 49 49 res = _FAIL; 50 50 RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("alloc recv_buf fail!\n")); 51 51 goto exit;
+4 -4
drivers/staging/r8188eu/hal/rtl8188eu_xmit.c
··· 444 444 RT_TRACE(_module_rtl8192c_xmit_c_, _drv_info_, ("+xmitframe_complete\n")); 445 445 446 446 /* check xmitbuffer is ok */ 447 - if (pxmitbuf == NULL) { 447 + if (!pxmitbuf) { 448 448 pxmitbuf = rtw_alloc_xmitbuf(pxmitpriv); 449 - if (pxmitbuf == NULL) 449 + if (!pxmitbuf) 450 450 return false; 451 451 } 452 452 ··· 455 455 rtw_free_xmitframe(pxmitpriv, pxmitframe); 456 456 457 457 pxmitframe = rtw_dequeue_xframe(pxmitpriv, pxmitpriv->hwxmits, pxmitpriv->hwxmit_entry); 458 - if (pxmitframe == NULL) { 458 + if (!pxmitframe) { 459 459 /* no more xmit frame, release xmit buffer */ 460 460 rtw_free_xmitbuf(pxmitpriv, pxmitbuf); 461 461 return false; ··· 638 638 goto enqueue; 639 639 640 640 pxmitbuf = rtw_alloc_xmitbuf(pxmitpriv); 641 - if (pxmitbuf == NULL) 641 + if (!pxmitbuf) 642 642 goto enqueue; 643 643 644 644 spin_unlock_bh(&pxmitpriv->lock);
+2 -2
drivers/staging/r8188eu/hal/usb_halinit.c
··· 2117 2117 if (mac_id >= NUM_STA) /* CAM_SIZE */ 2118 2118 return; 2119 2119 psta = pmlmeinfo->FW_sta_info[mac_id].psta; 2120 - if (psta == NULL) 2120 + if (!psta) 2121 2121 return; 2122 2122 switch (mac_id) { 2123 2123 case 0:/* for infra mode */ ··· 2253 2253 struct hal_ops *halfunc = &adapt->HalFunc; 2254 2254 2255 2255 adapt->HalData = rtw_zmalloc(sizeof(struct hal_data_8188e)); 2256 - if (adapt->HalData == NULL) 2256 + if (!adapt->HalData) 2257 2257 DBG_88E("cant not alloc memory for HAL DATA\n"); 2258 2258 adapt->hal_data_sz = sizeof(struct hal_data_8188e); 2259 2259
+6 -6
drivers/staging/r8188eu/hal/usb_ops_linux.c
··· 38 38 /* Acquire IO memory for vendorreq */ 39 39 pIo_buf = dvobjpriv->usb_vendor_req_buf; 40 40 41 - if (pIo_buf == NULL) { 41 + if (!pIo_buf) { 42 42 DBG_88E("[%s] pIo_buf == NULL\n", __func__); 43 43 status = -ENOMEM; 44 44 goto release_mutex; ··· 318 318 prxstat = (struct recv_stat *)pbuf; 319 319 320 320 precvframe = rtw_alloc_recvframe(pfree_recv_queue); 321 - if (precvframe == NULL) { 321 + if (!precvframe) { 322 322 RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("recvbuf2recvframe: precvframe==NULL\n")); 323 323 DBG_88E("%s()-%d: rtw_alloc_recvframe() failed! RX Drop!\n", __func__, __LINE__); 324 324 goto _exit_recvbuf2recvframe; ··· 587 587 return _FAIL; 588 588 } 589 589 590 - if ((!precvbuf->reuse) || (precvbuf->pskb == NULL)) { 590 + if (!precvbuf->reuse || !precvbuf->pskb) { 591 591 precvbuf->pskb = skb_dequeue(&precvpriv->free_recv_skb_queue); 592 - if (NULL != precvbuf->pskb) 592 + if (precvbuf->pskb) 593 593 precvbuf->reuse = true; 594 594 } 595 595 596 596 rtl8188eu_init_recvbuf(adapter, precvbuf); 597 597 598 598 /* re-assign for linux based on skb */ 599 - if ((!precvbuf->reuse) || (precvbuf->pskb == NULL)) { 599 + if (!precvbuf->reuse || !precvbuf->pskb) { 600 600 precvbuf->pskb = netdev_alloc_skb(adapter->pnetdev, MAX_RECVBUF_SZ + RECVBUFF_ALIGN_SZ); 601 - if (precvbuf->pskb == NULL) { 601 + if (!precvbuf->pskb) { 602 602 RT_TRACE(_module_hci_ops_os_c_, _drv_err_, ("init_recvbuf(): alloc_skb fail!\n")); 603 603 DBG_88E("#### usb_read_port() alloc_skb fail!#####\n"); 604 604 return _FAIL;