Commit 5a436cb1 authored by Colin Ian King's avatar Colin Ian King Committed by Greg Kroah-Hartman

staging: rtl8723bs: remove temporary variable CrystalCap

Currently variable CrystalCap is being initialized with the value
0x20 that is never read so that is redundant and can be removed.
Clean up the code by removing the need for variable CrystalCap
since the calculation of the return value is relatively simple.

Addresses-Coverity: ("Unused Value")
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20200223151438.415542-1-colin.king@canonical.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent df08620e
...@@ -40,16 +40,11 @@ static void odm_SetCrystalCap(void *pDM_VOID, u8 CrystalCap) ...@@ -40,16 +40,11 @@ static void odm_SetCrystalCap(void *pDM_VOID, u8 CrystalCap)
static u8 odm_GetDefaultCrytaltalCap(void *pDM_VOID) static u8 odm_GetDefaultCrytaltalCap(void *pDM_VOID)
{ {
PDM_ODM_T pDM_Odm = (PDM_ODM_T)pDM_VOID; PDM_ODM_T pDM_Odm = (PDM_ODM_T)pDM_VOID;
u8 CrystalCap = 0x20;
struct adapter *Adapter = pDM_Odm->Adapter; struct adapter *Adapter = pDM_Odm->Adapter;
struct hal_com_data *pHalData = GET_HAL_DATA(Adapter); struct hal_com_data *pHalData = GET_HAL_DATA(Adapter);
CrystalCap = pHalData->CrystalCap; return pHalData->CrystalCap & 0x3f;
CrystalCap = CrystalCap & 0x3f;
return CrystalCap;
} }
static void odm_SetATCStatus(void *pDM_VOID, bool ATCStatus) static void odm_SetATCStatus(void *pDM_VOID, bool ATCStatus)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment