From: kernel test robot drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_p2p.c:1259:13-24: Unneeded variable: "rWlanStatus". Return "WLAN_STATUS_SUCCESS" on line 1282 Remove unneeded variable used to store return value. Generated by: scripts/coccinelle/misc/returnvar.cocci Fixes: 9a72d145abe4 ("mt6625l: include wifi-folder") Signed-off-by: kernel test robot --- tree: https://github.com/frank-w/BPI-R2-4.14 5.8-rc head: ca0d6f7786586ca7ba424d5967d780dd3c75748b commit: 9a72d145abe49d9cb1e16d479b0b18caa20cdd71 [5/38] mt6625l: include wifi-folder Please take the patch only if it's a positive warning. Thanks! wlan_p2p.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) --- a/drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_p2p.c +++ b/drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_p2p.c @@ -1256,7 +1256,6 @@ WLAN_STATUS wlanoidGetSecCheckResponse(IN P_ADAPTER_T prAdapter, IN PVOID pvQueryBuffer, IN UINT_32 u4QueryBufferLen, OUT PUINT_32 pu4QueryInfoLen) { - WLAN_STATUS rWlanStatus = WLAN_STATUS_SUCCESS; /* P_WLAN_MAC_HEADER_T prWlanHdr = (P_WLAN_MAC_HEADER_T)NULL; */ P_GLUE_INFO_T prGlueInfo; @@ -1279,7 +1278,7 @@ wlanoidGetSecCheckResponse(IN P_ADAPTER_ kalMemCopy((PUINT_8) (pvQueryBuffer + OFFSET_OF(IW_P2P_TRANSPORT_STRUCT, aucBuffer)), prGlueInfo->prP2PInfo->aucSecCheckRsp, u4QueryBufferLen); - return rWlanStatus; + return WLAN_STATUS_SUCCESS; } /* end of wlanoidGetSecCheckResponse() */ #endif