Hi Souptick,
On Thu, Jun 22, 2017 at 9:16 PM, Souptick Joarder <jrdr.linux@xxxxxxxxx> wrote:
> As wmm_enable is initialized to false, hence the else condition never
> execute and boundary is assigned with TX_PAGE_BOUNDARY. So the if-else
> condition can be removed and boundary will be assigned with TX_PAGE_BOUNDARY.
>
> Signed-off-by: Souptick Joarder <jrdr.linux@xxxxxxxxx>
> ---
> drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> index f95a645..e9c5f6a 100644
> --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/hw.c
> @@ -845,13 +845,7 @@ static int _rtl92cu_init_mac(struct ieee80211_hw *hw)
> pr_err("Failed to init power on!\n");
> return err;
> }
> - if (!wmm_enable) {
> boundary = TX_PAGE_BOUNDARY;
Firstly this isn't indented properly.
Secondly, can this be moved up to where boundary is defined?
Thanks,
--
Julian Calaby
Email: julian.calaby@xxxxxxxxx
Profile: http://www.google.com/profiles/julian.calaby/
