From: kernel test robot drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_lib.c:1689:13-21: Unneeded variable: "u4Status". Return "WLAN_STATUS_SUCCESS" on line 1775 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_lib.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) --- a/drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_lib.c +++ b/drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_lib.c @@ -1686,7 +1686,6 @@ wlanAdapterStart(IN P_ADAPTER_T prAdapte WLAN_STATUS wlanAdapterStop(IN P_ADAPTER_T prAdapter) { UINT_32 i, u4Value = 0; - WLAN_STATUS u4Status = WLAN_STATUS_SUCCESS; ASSERT(prAdapter); @@ -1772,7 +1771,7 @@ WLAN_STATUS wlanAdapterStop(IN P_ADAPTER nicRestoreSpiDefMode(prAdapter); #endif - return u4Status; + return WLAN_STATUS_SUCCESS; } /* wlanAdapterStop */ #if 0