Commit c61583b1 authored by Michael Straube's avatar Michael Straube Committed by Greg Kroah-Hartman
Browse files

staging: rtl8188eu: refactor cckratesonly_included() 



Refactor cckratesonly_included() to improve readability and slightly
reduce object file size. Also change the return type to bool.

Signed-off-by: default avatarMichael Straube <straube.linux@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 3101957f

drivers/staging/rtl8188eu/core/rtw_wlan_util.c

+5 −5
Original line number Diff line number Diff line
@@ -69,16 +69,16 @@ bool cckrates_included(unsigned char *rate, int ratelen) 
	return false;

}



int cckratesonly_included(unsigned char *rate, int ratelen)

bool cckratesonly_included(unsigned char *rate, int ratelen)

{

	int i;



	for (i = 0; i < ratelen; i++) {

		if  ((((rate[i]) & 0x7f) != 2) && (((rate[i]) & 0x7f) != 4) &&

			   (((rate[i]) & 0x7f) != 11)  && (((rate[i]) & 0x7f) != 22))

		u8 r = rate[i] & 0x7f;



		if (r != 2 && r != 4 && r != 11 && r != 22)

			return false;

	}



	return true;

}

drivers/staging/rtl8188eu/include/rtw_mlme_ext.h

+1 −1
Original line number Diff line number Diff line
@@ -581,7 +581,7 @@ void addba_timer_hdl(struct timer_list *t); 
		  msecs_to_jiffies(ms))



bool cckrates_included(unsigned char *rate, int ratelen);

int cckratesonly_included(unsigned char *rate, int ratelen);

bool cckratesonly_included(unsigned char *rate, int ratelen);



void process_addba_req(struct adapter *padapter, u8 *paddba_req, u8 *addr);

CRA Git | Maintained and supported by SUSTech CRA and CCSE