On Tue, Sep 21, 2021 at 09:46:56PM +0200, Michael Straube wrote: > In this driver DynamicTxHighPowerLvl is set to TxHighPwrLevel_Normal > and never changed. In the file rtl8188e_rf6052.c there are some if > statements that check DynamicTxHighPowerLvl for other values than > TxHighPwrLevel_Normal. The code in the if blocks is never executed. > Remove the dead code. > > Signed-off-by: Michael Straube <straube.linux@xxxxxxxxx> > --- > drivers/staging/r8188eu/hal/rtl8188e_rf6052.c | 57 +++++-------------- > 1 file changed, 14 insertions(+), 43 deletions(-) > > diff --git a/drivers/staging/r8188eu/hal/rtl8188e_rf6052.c b/drivers/staging/r8188eu/hal/rtl8188e_rf6052.c > index edaa9a6dfdb1..946a1b97d96f 100644 > --- a/drivers/staging/r8188eu/hal/rtl8188e_rf6052.c > +++ b/drivers/staging/r8188eu/hal/rtl8188e_rf6052.c > @@ -85,7 +85,6 @@ rtl8188e_PHY_RF6052SetCckTxPower( > u8 *pPowerlevel) > { > struct hal_data_8188e *pHalData = GET_HAL_DATA(Adapter); > - struct dm_priv *pdmpriv = &pHalData->dmpriv; > struct mlme_ext_priv *pmlmeext = &Adapter->mlmeextpriv; > u32 TxAGC[2] = {0, 0}, tmpval = 0, pwrtrac_value; > bool TurboScanOff = false; > @@ -112,34 +111,19 @@ rtl8188e_PHY_RF6052SetCckTxPower( > } > } > } else { > - /* Driver dynamic Tx power shall not affect Tx power. > - * It shall be determined by power training mechanism. > -i * Currently, we cannot fully disable driver dynamic ^ You're a vimster. :) regards, dan carpenter