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: Shreeya Patel <shreeya.patel23498@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20200313104920.19974-1-shreeya.patel23498@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/rtl8723bs/hal/sdio_halinit.c b/drivers/staging/rtl8723bs/hal/sdio_halinit.c
index e813382..4894f7d 100644 (file)
--- a/drivers/staging/rtl8723bs/hal/sdio_halinit.c
+++ b/drivers/staging/rtl8723bs/hal/sdio_halinit.c
@@ -551,18 +551,8 @@ static void HalRxAggr8723BSdio(struct adapter *padapter)
 
        pregistrypriv = &padapter->registrypriv;
 
-       if (pregistrypriv->wifi_spec) {
-               /*  2010.04.27 hpfan */
-               /*  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;
-       }
+       valueDMATimeout = 0x06;
+       valueDMAPageCount = 0x06;
 
        rtw_write8(padapter, REG_RXDMA_AGG_PG_TH + 1, valueDMATimeout);
        rtw_write8(padapter, REG_RXDMA_AGG_PG_TH, valueDMAPageCount);