linux-staging.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Fabio Aiuto <fabioaiuto83@gmail.com>, gregkh@linuxfoundation.org
Cc: Larry.Finger@lwfinger.net, linux-staging@lists.linux.dev,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 01/18] staging: rtl8723bs: remove all 5Ghz network types
Date: Mon, 21 Jun 2021 11:40:47 +0200	[thread overview]
Message-ID: <7d973fa3-5f2f-ca23-5fd3-5f7e11ce6e98@redhat.com> (raw)
In-Reply-To: <96b241e97e425a3115305b3d6658037e1e4c862c.1624099125.git.fabioaiuto83@gmail.com>

Hi,

Thank you for your work in this.

On 6/19/21 12:47 PM, Fabio Aiuto wrote:
> remove all 5Ghz network types. rtl8723bs works on
> 802.11bgn standards and on 2.4Ghz band.
> 
> So remove all code related to 802.11a and 802.11ac
> standards, which the device doesn't support.
> 
> Signed-off-by: Fabio Aiuto <fabioaiuto83@gmail.com>
> ---
>  drivers/staging/rtl8723bs/core/rtw_ap.c       | 10 +---
>  .../staging/rtl8723bs/core/rtw_ieee80211.c    | 22 ++------
>  drivers/staging/rtl8723bs/core/rtw_mlme.c     | 10 ----
>  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 55 -------------------
>  .../staging/rtl8723bs/core/rtw_wlan_util.c    | 14 +----
>  drivers/staging/rtl8723bs/hal/odm.c           | 30 ----------
>  drivers/staging/rtl8723bs/hal/odm.h           |  3 -
>  .../rtl8723bs/hal/odm_EdcaTurboCheck.c        |  4 +-
>  drivers/staging/rtl8723bs/hal/sdio_halinit.c  |  5 --
>  drivers/staging/rtl8723bs/include/hal_phy.h   |  3 -
>  drivers/staging/rtl8723bs/include/ieee80211.h | 22 +-------
>  drivers/staging/rtl8723bs/include/wifi.h      |  1 -
>  drivers/staging/rtl8723bs/os_dep/os_intfs.c   |  2 +-
>  13 files changed, 14 insertions(+), 167 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c b/drivers/staging/rtl8723bs/core/rtw_ap.c
> index 23bbdf084631..b0abadd4b4dd 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_ap.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
> @@ -343,10 +343,7 @@ void add_RATid(struct adapter *padapter, struct sta_info *psta, u8 rssi_level)
>  
>  	if (pcur_network->Configuration.DSConfig > 14) {

This if sounds like it is checking for a channel-nr greater
then 14, so maybe this entire block should go away ?

>  		if (tx_ra_bitmap & 0xffff000)
> -			sta_band |= WIRELESS_11_5N;
> -
> -		if (tx_ra_bitmap & 0xff0)
> -			sta_band |= WIRELESS_11A;
> +			sta_band |= WIRELESS_INVALID;

You are replacing 2 if-s with 1 here, so you should also or
the 2 masks being checked for togehter, iow there now should
be one if like this:

		if (tx_ra_bitmap & (0xffff000 | 0xff0))
			sta_band |= WIRELESS_INVALID;

Which can be simplified to:

		if (tx_ra_bitmap & 0xffffff0)
			sta_band |= WIRELESS_INVALID;

The rest of the patch looks ok (from a quick scan).

Regards,

Hans



>  	} else {
>  		if (tx_ra_bitmap & 0xffff000)
>  			sta_band |= WIRELESS_11_24N;
> @@ -412,7 +409,7 @@ void update_bmc_sta(struct adapter *padapter)
>  		} else if (network_type == WIRELESS_INVALID) { /*  error handling */
>  
>  			if (pcur_network->Configuration.DSConfig > 14)
> -				network_type = WIRELESS_11A;
> +				network_type = WIRELESS_INVALID;
>  			else
>  				network_type = WIRELESS_11B;
>  		}
> @@ -1115,9 +1112,6 @@ int rtw_check_beacon_data(struct adapter *padapter, u8 *pbuf,  int len)
>  	case WIRELESS_11BG_24N:
>  		pbss_network->NetworkTypeInUse = Ndis802_11OFDM24;
>  		break;
> -	case WIRELESS_11A:
> -		pbss_network->NetworkTypeInUse = Ndis802_11OFDM5;
> -		break;
>  	default:
>  		pbss_network->NetworkTypeInUse = Ndis802_11OFDM24;
>  		break;
> diff --git a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c b/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
> index e341789954b9..0f0fcd9dc652 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_ieee80211.c
> @@ -96,10 +96,7 @@ bool rtw_is_cckratesonly_included(u8 *rate)
>  int rtw_check_network_type(unsigned char *rate, int ratelen, int channel)
>  {
>  	if (channel > 14) {
> -		if (rtw_is_cckrates_included(rate))
> -			return WIRELESS_INVALID;
> -		else
> -			return WIRELESS_11A;
> +		return WIRELESS_INVALID;
>  	} else { /*  could be pure B, pure G, or B/G */
>  		if (rtw_is_cckratesonly_included(rate))
>  			return WIRELESS_11B;
> @@ -266,10 +263,6 @@ void rtw_set_supported_rate(u8 *SupportedRates, uint mode)
>  		break;
>  
>  	case WIRELESS_11G:
> -	case WIRELESS_11A:
> -	case WIRELESS_11_5N:
> -	case WIRELESS_11A_5N:/* Todo: no basic rate for ofdm ? */
> -	case WIRELESS_11_5AC:
>  		memcpy(SupportedRates, WIFI_OFDMRATES, IEEE80211_NUM_OFDM_RATESLEN);
>  		break;
>  
> @@ -327,14 +320,7 @@ int rtw_generate_ie(struct registry_priv *pregistrypriv)
>  	ie = rtw_set_ie(ie, WLAN_EID_SSID, pdev_network->Ssid.SsidLength, pdev_network->Ssid.Ssid, &sz);
>  
>  	/* supported rates */
> -	if (pregistrypriv->wireless_mode == WIRELESS_11ABGN) {
> -		if (pdev_network->Configuration.DSConfig > 14)
> -			wireless_mode = WIRELESS_11A_5N;
> -		else
> -			wireless_mode = WIRELESS_11BG_24N;
> -	} else {
> -		wireless_mode = pregistrypriv->wireless_mode;
> -	}
> +	wireless_mode = pregistrypriv->wireless_mode;
>  
>  	rtw_set_supported_rate(pdev_network->SupportedRates, wireless_mode);
>  
> @@ -359,8 +345,8 @@ int rtw_generate_ie(struct registry_priv *pregistrypriv)
>  	}
>  
>  	/* HT Cap. */
> -	if (((pregistrypriv->wireless_mode&WIRELESS_11_5N) || (pregistrypriv->wireless_mode&WIRELESS_11_24N))
> -		&& (pregistrypriv->ht_enable == true)) {
> +	if ((pregistrypriv->wireless_mode & WIRELESS_11_24N) &&
> +	    (pregistrypriv->ht_enable == true)) {
>  		/* todo: */
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> index 158f92d4d882..2dd75e007239 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> @@ -2184,16 +2184,6 @@ void rtw_update_registrypriv_dev_network(struct adapter *adapter)
>  	case WIRELESS_11BG_24N:
>  		pdev_network->NetworkTypeInUse = (Ndis802_11OFDM24);
>  		break;
> -	case WIRELESS_11A:
> -	case WIRELESS_11A_5N:
> -		pdev_network->NetworkTypeInUse = (Ndis802_11OFDM5);
> -		break;
> -	case WIRELESS_11ABGN:
> -		if (pregistrypriv->channel > 14)
> -			pdev_network->NetworkTypeInUse = (Ndis802_11OFDM5);
> -		else
> -			pdev_network->NetworkTypeInUse = (Ndis802_11OFDM24);
> -		break;
>  	default:
>  		/*  TODO */
>  		break;
> diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> index 4df3cba97f42..106779fb9fef 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> @@ -4478,61 +4478,6 @@ static void process_80211d(struct adapter *padapter, struct wlan_bssid_ex *bssid
>  			}
>  		}
>  
> -		if (pregistrypriv->wireless_mode & WIRELESS_11A) {
> -			do {
> -				if ((i == MAX_CHANNEL_NUM) ||
> -					(chplan_sta[i].ChannelNum == 0))
> -					break;
> -
> -				if ((j == chplan_ap.Len) || (chplan_ap.Channel[j] == 0))
> -					break;
> -
> -				if (chplan_sta[i].ChannelNum == chplan_ap.Channel[j]) {
> -					chplan_new[k].ChannelNum = chplan_ap.Channel[j];
> -					chplan_new[k].ScanType = SCAN_ACTIVE;
> -					i++;
> -					j++;
> -					k++;
> -				} else if (chplan_sta[i].ChannelNum < chplan_ap.Channel[j]) {
> -					chplan_new[k].ChannelNum = chplan_sta[i].ChannelNum;
> -/* 					chplan_new[k].ScanType = chplan_sta[i].ScanType; */
> -					chplan_new[k].ScanType = SCAN_PASSIVE;
> -					i++;
> -					k++;
> -				} else if (chplan_sta[i].ChannelNum > chplan_ap.Channel[j]) {
> -					chplan_new[k].ChannelNum = chplan_ap.Channel[j];
> -					chplan_new[k].ScanType = SCAN_ACTIVE;
> -					j++;
> -					k++;
> -				}
> -			} while (1);
> -
> -			/*  change AP not support channel to Passive scan */
> -			while ((i < MAX_CHANNEL_NUM) && (chplan_sta[i].ChannelNum != 0)) {
> -				chplan_new[k].ChannelNum = chplan_sta[i].ChannelNum;
> -/* 				chplan_new[k].ScanType = chplan_sta[i].ScanType; */
> -				chplan_new[k].ScanType = SCAN_PASSIVE;
> -				i++;
> -				k++;
> -			}
> -
> -			/*  add channel AP supported */
> -			while ((j < chplan_ap.Len) && (chplan_ap.Channel[j] != 0)) {
> -				chplan_new[k].ChannelNum = chplan_ap.Channel[j];
> -				chplan_new[k].ScanType = SCAN_ACTIVE;
> -				j++;
> -				k++;
> -			}
> -		} else {
> -			/*  keep original STA 5G channel plan */
> -			while ((i < MAX_CHANNEL_NUM) && (chplan_sta[i].ChannelNum != 0)) {
> -				chplan_new[k].ChannelNum = chplan_sta[i].ChannelNum;
> -				chplan_new[k].ScanType = chplan_sta[i].ScanType;
> -				i++;
> -				k++;
> -			}
> -		}
> -
>  		pmlmeext->update_channel_plan_by_ap_done = 1;
>  	}
>  
> diff --git a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
> index afabb9d48021..2a47d678de01 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c
> @@ -55,9 +55,6 @@ u8 networktype_to_raid_ex(struct adapter *adapter, struct sta_info *psta)
>  
>  	if (cur_rf_type == RF_1T1R) {
>  		rf_type = RF_1T1R;
> -	} else if (IsSupportedVHT(psta->wireless_mode)) {
> -		if (psta->ra_mask & 0xffc00000)
> -			rf_type = RF_2T2R;
>  	} else if (IsSupportedHT(psta->wireless_mode)) {
>  		if (psta->ra_mask & 0xfff00000)
>  			rf_type = RF_2T2R;
> @@ -67,7 +64,6 @@ u8 networktype_to_raid_ex(struct adapter *adapter, struct sta_info *psta)
>  	case WIRELESS_11B:
>  		raid = RATEID_IDX_B;
>  		break;
> -	case WIRELESS_11A:
>  	case WIRELESS_11G:
>  		raid = RATEID_IDX_G;
>  		break;
> @@ -75,8 +71,6 @@ u8 networktype_to_raid_ex(struct adapter *adapter, struct sta_info *psta)
>  		raid = RATEID_IDX_BG;
>  		break;
>  	case WIRELESS_11_24N:
> -	case WIRELESS_11_5N:
> -	case WIRELESS_11A_5N:
>  	case WIRELESS_11G_24N:
>  		if (rf_type == RF_2T2R)
>  			raid = RATEID_IDX_GN_N2SS;
> @@ -827,7 +821,7 @@ void WMMOnAssocRsp(struct adapter *padapter)
>  
>  		AIFS = aSifsTime + (2 * pmlmeinfo->slotTime);
>  
> -		if (pmlmeext->cur_wireless_mode & (WIRELESS_11G | WIRELESS_11A)) {
> +		if (pmlmeext->cur_wireless_mode & WIRELESS_11G) {
>  			ECWMin = 4;
>  			ECWMax = 10;
>  		} else if (pmlmeext->cur_wireless_mode & WIRELESS_11B) {
> @@ -1623,7 +1617,7 @@ void update_capinfo(struct adapter *Adapter, u16 updateCap)
>  		pmlmeinfo->slotTime = NON_SHORT_SLOT_TIME;
>  	} else {
>  		/* Filen: See 802.11-2007 p.90 */
> -		if (pmlmeext->cur_wireless_mode & (WIRELESS_11_24N | WIRELESS_11A | WIRELESS_11_5N | WIRELESS_11AC)) {
> +		if (pmlmeext->cur_wireless_mode & (WIRELESS_11_24N)) {
>  			pmlmeinfo->slotTime = SHORT_SLOT_TIME;
>  		} else if (pmlmeext->cur_wireless_mode & (WIRELESS_11G)) {
>  			if ((updateCap & cShortSlotTime) /* && (!(pMgntInfo->pHTInfo->RT2RT_HT_Mode & RT_HT_CAP_USE_LONG_PREAMBLE)) */)
> @@ -1653,9 +1647,7 @@ void update_wireless_mode(struct adapter *padapter)
>  	if ((pmlmeinfo->HT_info_enable) && (pmlmeinfo->HT_caps_enable))
>  		pmlmeinfo->HT_enable = 1;
>  
> -	if (pmlmeinfo->VHT_enable)
> -		network_type = WIRELESS_11AC;
> -	else if (pmlmeinfo->HT_enable)
> +	if (pmlmeinfo->HT_enable)
>  		network_type = WIRELESS_11_24N;
>  
>  	if (rtw_is_cckratesonly_included(rate))
> diff --git a/drivers/staging/rtl8723bs/hal/odm.c b/drivers/staging/rtl8723bs/hal/odm.c
> index fa275138c983..bfe0c3a773af 100644
> --- a/drivers/staging/rtl8723bs/hal/odm.c
> +++ b/drivers/staging/rtl8723bs/hal/odm.c
> @@ -377,7 +377,6 @@ u32 ODM_Get_Rate_Bitmap(
>  		break;
>  
>  	case (ODM_WM_G):
> -	case (ODM_WM_A):
>  		if (rssi_level == DM_RATR_STA_HIGH)
>  			rate_bitmap = 0x00000f00;
>  		else
> @@ -396,7 +395,6 @@ u32 ODM_Get_Rate_Bitmap(
>  	case (ODM_WM_B|ODM_WM_G|ODM_WM_N24G):
>  	case (ODM_WM_B|ODM_WM_N24G):
>  	case (ODM_WM_G|ODM_WM_N24G):
> -	case (ODM_WM_A|ODM_WM_N5G):
>  		if (pDM_Odm->RFType == ODM_1T2R || pDM_Odm->RFType == ODM_1T1R) {
>  			if (rssi_level == DM_RATR_STA_HIGH)
>  				rate_bitmap = 0x000f0000;
> @@ -422,34 +420,6 @@ u32 ODM_Get_Rate_Bitmap(
>  		}
>  		break;
>  
> -	case (ODM_WM_AC|ODM_WM_G):
> -		if (rssi_level == 1)
> -			rate_bitmap = 0xfc3f0000;
> -		else if (rssi_level == 2)
> -			rate_bitmap = 0xfffff000;
> -		else
> -			rate_bitmap = 0xffffffff;
> -		break;
> -
> -	case (ODM_WM_AC|ODM_WM_A):
> -
> -		if (pDM_Odm->RFType == RF_1T1R) {
> -			if (rssi_level == 1)				/*  add by Gary for ac-series */
> -				rate_bitmap = 0x003f8000;
> -			else if (rssi_level == 2)
> -				rate_bitmap = 0x003ff000;
> -			else
> -				rate_bitmap = 0x003ff010;
> -		} else {
> -			if (rssi_level == 1)				/*  add by Gary for ac-series */
> -				rate_bitmap = 0xfe3f8000;       /*  VHT 2SS MCS3~9 */
> -			else if (rssi_level == 2)
> -				rate_bitmap = 0xfffff000;       /*  VHT 2SS MCS0~9 */
> -			else
> -				rate_bitmap = 0xfffff010;       /*  All */
> -		}
> -		break;
> -
>  	default:
>  		if (pDM_Odm->RFType == RF_1T2R)
>  			rate_bitmap = 0x000fffff;
> diff --git a/drivers/staging/rtl8723bs/hal/odm.h b/drivers/staging/rtl8723bs/hal/odm.h
> index 950e7850bac2..7e2d3679c7d0 100644
> --- a/drivers/staging/rtl8723bs/hal/odm.h
> +++ b/drivers/staging/rtl8723bs/hal/odm.h
> @@ -455,11 +455,8 @@ enum { /* tag_Wireless_Mode_Definition */
>  	ODM_WM_UNKNOWN    = 0x0,
>  	ODM_WM_B          = BIT0,
>  	ODM_WM_G          = BIT1,
> -	ODM_WM_A          = BIT2,
>  	ODM_WM_N24G       = BIT3,
> -	ODM_WM_N5G        = BIT4,
>  	ODM_WM_AUTO       = BIT5,
> -	ODM_WM_AC         = BIT6,
>  };
>  
>  /*  ODM_CMNINFO_BAND */
> diff --git a/drivers/staging/rtl8723bs/hal/odm_EdcaTurboCheck.c b/drivers/staging/rtl8723bs/hal/odm_EdcaTurboCheck.c
> index f3856e47bfd3..578d5712645c 100644
> --- a/drivers/staging/rtl8723bs/hal/odm_EdcaTurboCheck.c
> +++ b/drivers/staging/rtl8723bs/hal/odm_EdcaTurboCheck.c
> @@ -124,12 +124,10 @@ void odm_EdcaTurboCheckCE(void *pDM_VOID)
>  		} else if ((iot_peer == HT_IOT_PEER_CISCO) &&
>  			   ((wirelessmode == ODM_WM_G) ||
>  			    (wirelessmode == (ODM_WM_B | ODM_WM_G)) ||
> -			    (wirelessmode == ODM_WM_A) ||
>  			    (wirelessmode == ODM_WM_B))) {
>  			EDCA_BE_DL = edca_setting_DL_GMode[iot_peer];
>  		} else if ((iot_peer == HT_IOT_PEER_AIRGO) &&
> -			   ((wirelessmode == ODM_WM_G) ||
> -			    (wirelessmode == ODM_WM_A))) {
> +			   (wirelessmode == ODM_WM_G)) {
>  			EDCA_BE_DL = 0xa630;
>  		} else if (iot_peer == HT_IOT_PEER_MARVELL) {
>  			EDCA_BE_DL = edca_setting_DL[iot_peer];
> diff --git a/drivers/staging/rtl8723bs/hal/sdio_halinit.c b/drivers/staging/rtl8723bs/hal/sdio_halinit.c
> index 3738727feb60..a07a6dacec42 100644
> --- a/drivers/staging/rtl8723bs/hal/sdio_halinit.c
> +++ b/drivers/staging/rtl8723bs/hal/sdio_halinit.c
> @@ -512,8 +512,6 @@ static void _InitOperationMode(struct adapter *padapter)
>  	case WIRELESS_MODE_B:
>  		regBwOpMode = BW_OPMODE_20MHZ;
>  		break;
> -	case WIRELESS_MODE_A:
> -		break;
>  	case WIRELESS_MODE_G:
>  		regBwOpMode = BW_OPMODE_20MHZ;
>  		break;
> @@ -525,9 +523,6 @@ static void _InitOperationMode(struct adapter *padapter)
>  		/*  CCK rate will be filtered out only when associated AP does not support it. */
>  		regBwOpMode = BW_OPMODE_20MHZ;
>  		break;
> -	case WIRELESS_MODE_N_5G:
> -		regBwOpMode = BW_OPMODE_5G;
> -		break;
>  
>  	default: /* for MacOSX compiler warning. */
>  		break;
> diff --git a/drivers/staging/rtl8723bs/include/hal_phy.h b/drivers/staging/rtl8723bs/include/hal_phy.h
> index 19221289b8ce..6417d1b758d2 100644
> --- a/drivers/staging/rtl8723bs/include/hal_phy.h
> +++ b/drivers/staging/rtl8723bs/include/hal_phy.h
> @@ -51,13 +51,10 @@ enum rf_path {
>  
>  enum wireless_mode {
>  	WIRELESS_MODE_UNKNOWN = 0x00,
> -	WIRELESS_MODE_A = 0x01,
>  	WIRELESS_MODE_B = 0x02,
>  	WIRELESS_MODE_G = 0x04,
>  	WIRELESS_MODE_AUTO = 0x08,
>  	WIRELESS_MODE_N_24G = 0x10,
> -	WIRELESS_MODE_N_5G = 0x20,
> -	WIRELESS_MODE_AC_5G = 0x40,
>  	WIRELESS_MODE_AC_24G  = 0x80,
>  	WIRELESS_MODE_AC_ONLY  = 0x100,
>  };
> diff --git a/drivers/staging/rtl8723bs/include/ieee80211.h b/drivers/staging/rtl8723bs/include/ieee80211.h
> index 6540c7a22938..ac88bd30b097 100644
> --- a/drivers/staging/rtl8723bs/include/ieee80211.h
> +++ b/drivers/staging/rtl8723bs/include/ieee80211.h
> @@ -144,33 +144,20 @@ enum network_type {
>  	/* Sub-Element */
>  	WIRELESS_11B = BIT(0), /*  tx: cck only , rx: cck only, hw: cck */
>  	WIRELESS_11G = BIT(1), /*  tx: ofdm only, rx: ofdm & cck, hw: cck & ofdm */
> -	WIRELESS_11A = BIT(2), /*  tx: ofdm only, rx: ofdm only, hw: ofdm only */
>  	WIRELESS_11_24N = BIT(3), /*  tx: MCS only, rx: MCS & cck, hw: MCS & cck */
> -	WIRELESS_11_5N = BIT(4), /*  tx: MCS only, rx: MCS & ofdm, hw: ofdm only */
>  	WIRELESS_AUTO = BIT(5),
> -	WIRELESS_11AC = BIT(6),
>  
>  	/* Combination */
>  	/* Type for current wireless mode */
>  	WIRELESS_11BG = (WIRELESS_11B|WIRELESS_11G), /*  tx: cck & ofdm, rx: cck & ofdm & MCS, hw: cck & ofdm */
>  	WIRELESS_11G_24N = (WIRELESS_11G|WIRELESS_11_24N), /*  tx: ofdm & MCS, rx: ofdm & cck & MCS, hw: cck & ofdm */
> -	WIRELESS_11A_5N = (WIRELESS_11A|WIRELESS_11_5N), /*  tx: ofdm & MCS, rx: ofdm & MCS, hw: ofdm only */
>  	WIRELESS_11B_24N = (WIRELESS_11B|WIRELESS_11_24N), /*  tx: ofdm & cck & MCS, rx: ofdm & cck & MCS, hw: ofdm & cck */
>  	WIRELESS_11BG_24N = (WIRELESS_11B|WIRELESS_11G|WIRELESS_11_24N), /*  tx: ofdm & cck & MCS, rx: ofdm & cck & MCS, hw: ofdm & cck */
> -	WIRELESS_11_24AC = (WIRELESS_11G|WIRELESS_11AC),
> -	WIRELESS_11_5AC = (WIRELESS_11A|WIRELESS_11AC),
> -
> -
> -	/* Type for registry default wireless mode */
> -	WIRELESS_11AGN = (WIRELESS_11A|WIRELESS_11G|WIRELESS_11_24N|WIRELESS_11_5N), /*  tx: ofdm & MCS, rx: ofdm & MCS, hw: ofdm only */
> -	WIRELESS_11ABGN = (WIRELESS_11A|WIRELESS_11B|WIRELESS_11G|WIRELESS_11_24N|WIRELESS_11_5N),
> -	WIRELESS_MODE_24G = (WIRELESS_11B|WIRELESS_11G|WIRELESS_11_24N|WIRELESS_11AC),
> -	WIRELESS_MODE_MAX = (WIRELESS_11A|WIRELESS_11B|WIRELESS_11G|WIRELESS_11_24N|WIRELESS_11_5N|WIRELESS_11AC),
>  };
>  
>  #define SUPPORTED_24G_NETTYPE_MSK (WIRELESS_11B | WIRELESS_11G | WIRELESS_11_24N)
>  
> -#define IsLegacyOnly(NetType)  ((NetType) == ((NetType) & (WIRELESS_11BG|WIRELESS_11A)))
> +#define IsLegacyOnly(NetType)  ((NetType) == ((NetType) & (WIRELESS_11BG)))
>  
>  #define IsSupported24G(NetType) ((NetType) & SUPPORTED_24G_NETTYPE_MSK ? true : false)
>  
> @@ -182,11 +169,8 @@ enum network_type {
>  #define IsSupportedRxHT(NetType) IsEnableHWOFDM(NetType)
>  
>  #define IsSupportedTxCCK(NetType) (((NetType) & (WIRELESS_11B)) ? true : false)
> -#define IsSupportedTxOFDM(NetType) (((NetType) & (WIRELESS_11G|WIRELESS_11A)) ? true : false)
> -#define IsSupportedHT(NetType) (((NetType) & (WIRELESS_11_24N|WIRELESS_11_5N)) ? true : false)
> -
> -#define IsSupportedVHT(NetType) (((NetType) & (WIRELESS_11AC)) ? true : false)
> -
> +#define IsSupportedTxOFDM(NetType) (((NetType) & (WIRELESS_11G) ? true : false)
> +#define IsSupportedHT(NetType) (((NetType) & (WIRELESS_11_24N)) ? true : false)
>  
>  struct ieee_param {
>  	u32 cmd;
> diff --git a/drivers/staging/rtl8723bs/include/wifi.h b/drivers/staging/rtl8723bs/include/wifi.h
> index 23de3ab87f9c..0bd7b662b972 100644
> --- a/drivers/staging/rtl8723bs/include/wifi.h
> +++ b/drivers/staging/rtl8723bs/include/wifi.h
> @@ -336,7 +336,6 @@ static inline int IsFrameTypeCtrl(unsigned char *pframe)
>  #define _PRE_ALLOCICVHDR_		5
>  #define _PRE_ALLOCMICHDR_		6
>  
> -#define _SIFSTIME_				((priv->pmib->dot11BssType.net_work_type&WIRELESS_11A)?16:10)
>  #define _ACKCTSLNG_				14	/* 14 bytes long, including crclng */
>  #define _CRCLNG_				4
>  
> diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> index 11b9a5919da4..4e7c115c8bc1 100644
> --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c
> @@ -22,7 +22,7 @@ static int rtw_lbkmode;/* RTL8712_AIR_TRX; */
>  static int rtw_network_mode = Ndis802_11IBSS;/* Ndis802_11Infrastructure;infra, ad-hoc, auto */
>  /* struct ndis_802_11_ssid	ssid; */
>  static int rtw_channel = 1;/* ad-hoc support requirement */
> -static int rtw_wireless_mode = WIRELESS_MODE_MAX;
> +static int rtw_wireless_mode = WIRELESS_11BG_24N;
>  static int rtw_vrtl_carrier_sense = AUTO_VCS;
>  static int rtw_vcs_type = RTS_CTS;/*  */
>  static int rtw_rts_thresh = 2347;/*  */
> 


  reply	other threads:[~2021-06-21  9:40 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-19 10:47 [PATCH 00/18] staging: rtl8723bs: remove 5Ghz code Fabio Aiuto
2021-06-19 10:47 ` [PATCH 01/18] staging: rtl8723bs: remove all 5Ghz network types Fabio Aiuto
2021-06-21  9:40   ` Hans de Goede [this message]
2021-06-21 11:23     ` Fabio Aiuto
2021-06-19 10:47 ` [PATCH 02/18] staging: rtl8723bs: remove code related to unsupported channel bandwidth Fabio Aiuto
2021-06-19 10:47 ` [PATCH 03/18] staging: rtl8723bs: remove unused enum items related to channel bonding Fabio Aiuto
2021-06-19 10:47 ` [PATCH 04/18] staging: rtl8723bs: rename " Fabio Aiuto
2021-06-19 10:47 ` [PATCH 05/18] staging: rtl8723bs: remove 5Ghz field in struct registry_priv Fabio Aiuto
2021-06-19 10:47 ` [PATCH 06/18] staging: rtl8723bs: remove struct rt_channel_plan_5g Fabio Aiuto
2021-06-19 10:47 ` [PATCH 07/18] staging: rtl8723bs: remove all branchings between 2.4Ghz and 5Ghz band types Fabio Aiuto
2021-06-19 10:47 ` [PATCH 08/18] staging: rtl8723bs: beautify prototypes in include/hal_com_phycfg.h Fabio Aiuto
2021-06-19 10:47 ` [PATCH 09/18] staging: rtl8723bs: remove 5Ghz code related to channel plan definition Fabio Aiuto
2021-06-19 10:47 ` [PATCH 10/18] staging: rtl8723bs: remove some unused 5Ghz macro definitions Fabio Aiuto
2021-06-19 10:47 ` [PATCH 11/18] staging: rtl8723bs: remove 5Ghz code related to RF power calibration Fabio Aiuto
2021-06-19 10:47 ` [PATCH 12/18] staging: rtl8723bs: remove VHT dead code Fabio Aiuto
2021-06-21  9:45   ` Hans de Goede
2021-06-21 12:41     ` Fabio Aiuto
2021-06-21 12:45       ` Hans de Goede
2021-06-21 12:51         ` Fabio Aiuto
2021-06-22  9:16     ` Fabio Aiuto
2021-06-22  9:19       ` Hans de Goede
2021-06-22  9:57         ` Fabio Aiuto
2021-06-22 11:15           ` Hans de Goede
2021-06-19 10:47 ` [PATCH 13/18] staging: rtl8723bs: remove unused ODM_CMNINFO_BOARD_TYPE enum item Fabio Aiuto
2021-06-19 10:47 ` [PATCH 14/18] staging: rtl8723bs: fix macro value for 2.4Ghz only device Fabio Aiuto
2021-06-19 10:47 ` [PATCH 15/18] staging: rtl8723bs: remove register initializations tied to 802.11ac standard Fabio Aiuto
2021-06-21  9:48   ` Hans de Goede
2021-06-21 12:45     ` Fabio Aiuto
2021-06-19 10:47 ` [PATCH 16/18] staging: rtl8723bs: remove obsolete 5Ghz comments Fabio Aiuto
2021-06-19 10:47 ` [PATCH 17/18] staging: rtl8723bs: fix check allowing 5Ghz settings Fabio Aiuto
2021-06-19 10:47 ` [PATCH 18/18] staging: rtl8723bs: remove item from TODO list Fabio Aiuto
2021-06-21  9:49 ` [PATCH 00/18] staging: rtl8723bs: remove 5Ghz code Hans de Goede
2021-06-21 12:47   ` Fabio Aiuto

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7d973fa3-5f2f-ca23-5fd3-5f7e11ce6e98@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=Larry.Finger@lwfinger.net \
    --cc=fabioaiuto83@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-staging@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).