Commit 9a27592d authored by Shreeya Patel's avatar Shreeya Patel Committed by Greg Kroah-Hartman

Staging: rtl8723bs: sdio_halinit: Remove unnecessary conditions

Remove if and else conditions since both are leading to the
initialization of "valueDMATimeout" and "valueDMAPageCount" with
the same value.

Found using coccinelle script.
Signed-off-by: default avatarShreeya Patel <shreeya.patel23498@gmail.com>
Reviewed-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20200313104920.19974-1-shreeya.patel23498@gmail.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent af68516e
...@@ -551,18 +551,8 @@ static void HalRxAggr8723BSdio(struct adapter *padapter) ...@@ -551,18 +551,8 @@ static void HalRxAggr8723BSdio(struct adapter *padapter)
pregistrypriv = &padapter->registrypriv; pregistrypriv = &padapter->registrypriv;
if (pregistrypriv->wifi_spec) { valueDMATimeout = 0x06;
/* 2010.04.27 hpfan */ valueDMAPageCount = 0x06;
/* Adjust RxAggrTimeout to close to zero disable RxAggr, suggested by designer */
/* Timeout value is calculated by 34 / (2^n) */
valueDMATimeout = 0x06;
valueDMAPageCount = 0x06;
} else {
/* 20130530, Isaac@SD1 suggest 3 kinds of parameter */
/* TX/RX Balance */
valueDMATimeout = 0x06;
valueDMAPageCount = 0x06;
}
rtw_write8(padapter, REG_RXDMA_AGG_PG_TH + 1, valueDMATimeout); rtw_write8(padapter, REG_RXDMA_AGG_PG_TH + 1, valueDMATimeout);
rtw_write8(padapter, REG_RXDMA_AGG_PG_TH, valueDMAPageCount); rtw_write8(padapter, REG_RXDMA_AGG_PG_TH, valueDMAPageCount);
......
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