From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============7377061340536787917==" MIME-Version: 1.0 From: kernel test robot To: kbuild-all@lists.01.org Subject: [PATCH] mt6625l: fix returnvar.cocci warnings Date: Sun, 12 Jul 2020 20:19:20 +0800 Message-ID: <20200712121920.GA73132@f669aaa75a90> In-Reply-To: <202007122043.encHIg4V%lkp@intel.com> List-Id: --===============7377061340536787917== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: kernel test robot drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_oid.c:5224:15-22: = Unneeded variable: "rStatus". Return "WLAN_STATUS_SUCCESS" on line 5227 drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_oid.c:5259:15-22: = Unneeded variable: "rStatus". Return "WLAN_STATUS_SUCCESS" on line 5262 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 wi= fi-folder Please take the patch only if it's a positive warning. Thanks! wlan_oid.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) --- a/drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_oid.c +++ b/drivers/misc/mediatek/connectivity/wlan/gen2/common/wlan_oid.c @@ -5221,10 +5221,9 @@ wlanoidSetMcrWrite(IN P_ADAPTER_T prAdap #if 1 /* low power test special command */ if (prMcrWrInfo->u4McrOffset =3D=3D 0x11111110) { - WLAN_STATUS rStatus =3D WLAN_STATUS_SUCCESS; /* DbgPrint("Enter test mode\n"); */ prAdapter->fgTestMode =3D TRUE; - return rStatus; + return WLAN_STATUS_SUCCESS; } if (prMcrWrInfo->u4McrOffset =3D=3D 0x11111111) { /* DbgPrint("nicpmSetAcpiPowerD3\n"); */ @@ -5256,10 +5255,9 @@ wlanoidSetMcrWrite(IN P_ADAPTER_T prAdap #if 1 /* low power test special command */ if (prMcrWrInfo->u4McrOffset =3D=3D 0x11111110) { - WLAN_STATUS rStatus =3D WLAN_STATUS_SUCCESS; /* DbgPrint("Enter test mode\n"); */ prAdapter->fgTestMode =3D TRUE; - return rStatus; + return WLAN_STATUS_SUCCESS; } if (prMcrWrInfo->u4McrOffset =3D=3D 0x11111111) { /* DbgPrint("nicpmSetAcpiPowerD3\n"); */ --===============7377061340536787917==--