Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp3614679pxj; Mon, 21 Jun 2021 02:46:31 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz1ijPgXywyhPH9LRcsjx12G54dVgAQ28eTVQ/+HOHQLbL1Hsc0KPHksTFoblTH25lToM3V X-Received: by 2002:a92:7d08:: with SMTP id y8mr11927267ilc.43.1624268791504; Mon, 21 Jun 2021 02:46:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1624268791; cv=none; d=google.com; s=arc-20160816; b=NdPpeWmh2PD5OurJ/IdDEVdWzdDAo52DLRdbTbEyDrTa+2ZOdgaB0IttpzxMxMBy5n DSB211PojTOPAClphCU6uFR4RylTedkGWUke8V/DUZ3exECJ+HgF8cLzBNbV+4e3JfvH IXf2cSTXjYkpX9YSdSVdh1Z6fuDZsOP/B16OhwNM5f7cxL+4jLAjGIf7kzpmm5/0DcJv c2n9GQ7HE4AUUN1bMhhETZXHPNFS3BocuTgCfyXuLPOFojDhdhekU3MNyRpPB2XLLK1c If8UD06Snt8ctJjlbdlP0EUFmqXD+N5OngGmTUe+maStx2shWomQjDRP0qH9ZY7PRaMo vF8g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=G9SU4BRTXIiy90uMM9jwTSei/FzgXFcCjI/JzEBv3fM=; b=o6lAZpRTsO3UOkT+Q49Zcqu9XqU2JvUddKgDNHumlTvtAYs4/SOp80JP3VEVWzNQxk sXpPR8i742SViB1fK8Mu5uoED4royhqUdh+5fnRXbusvMB7g6Jh0uzwj6a5N6TVZ7t0H f9MCCGjm6K9g3f11akx3UiCX1emNxjIp4t2/bdQ5wzEPr2x2kzVTuW96Kq4bnflq/vHT yWghdSUG+FNRlq65XAzHeXhk5+LdOtE/cTtA8U9wyJUmwkkznUf7pm9Lt5hTaVn8HBOy e6ATG72ENRL0a5uO3WSKqF4K4ajuUXqhWdofKJ1A3ArsldZQ2u/fVMY7LV1h5edegvis OndA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=BqAHgCFt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id l4si4151206ilo.32.2021.06.21.02.46.18; Mon, 21 Jun 2021 02:46:31 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=BqAHgCFt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229890AbhFUJsA (ORCPT + 99 others); Mon, 21 Jun 2021 05:48:00 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:57126 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229576AbhFUJr6 (ORCPT ); Mon, 21 Jun 2021 05:47:58 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1624268744; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=G9SU4BRTXIiy90uMM9jwTSei/FzgXFcCjI/JzEBv3fM=; b=BqAHgCFtWSB8hCOynqE8/HkeNe9+fovZ6AfEvGLsrcT5zJn+7c0muzYUkwOjww0mgDsNYA hkBVwn6mLGV9T/q/DNYccg0AG8ULkS4ww2QByM7QS81ZIwc9ftWInTUe2Q20DMQoDJljqL gBNhBWCZTAchKHCLdnKKpkz/BkBOMvE= Received: from mail-ej1-f71.google.com (mail-ej1-f71.google.com [209.85.218.71]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-169-0EREtDvUMBSAuh-isn-p-g-1; Mon, 21 Jun 2021 05:45:42 -0400 X-MC-Unique: 0EREtDvUMBSAuh-isn-p-g-1 Received: by mail-ej1-f71.google.com with SMTP id l6-20020a1709062a86b029046ec0ceaf5cso3714743eje.8 for ; Mon, 21 Jun 2021 02:45:42 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=G9SU4BRTXIiy90uMM9jwTSei/FzgXFcCjI/JzEBv3fM=; b=TuvYts5+OdVhYv41DOVilNyQeZqFmwQ8vDH8NJAtSZnDBjPafVUyLHC2C+T2fHdn5Z 0hnd4pW04O1E9r+FUCMgW1Htld61qQ9fgiO00gvDGBwKS5TSn8B2s9gNole6biL3v3aM L9rV1AgAHZssY/dCGhamJ61ZjJN91waSkO1byAiQKOPzfTx+3UonKcRzR1U5xUmFwxX/ 6iiEmPUv0IQ3ymcr7c+A2IO2LYtGJNcQf2oPeVQlogH+uIAHovZW950HB3nOZ3COzqSe q/C3+UleKHd7l9YMzMA2r0TTuaPlqxG/CX08qQtYtFWWPMM/o82Zu51PGKlU6KHjfv4w Sf8A== X-Gm-Message-State: AOAM532rSKxdny+ZUKNCVH1soSdpicyrjKmksnLd5gpg0IhopjDOhcuA zXj9G3fWHxechR2+BPNkSwGabbFKHT+5mhjk+wOalH0OHgyaHp8newPj5RCYuOdQhXZfFlM8j4X EMvIQ0h31nWQ7CcLtgeQM5oNEUadiMbOhvzMaHUTvojUAn7fDvZeJDd+1IhhocPOPobiLM5XLV5 Td X-Received: by 2002:a17:906:17d8:: with SMTP id u24mr23967925eje.106.1624268741072; Mon, 21 Jun 2021 02:45:41 -0700 (PDT) X-Received: by 2002:a17:906:17d8:: with SMTP id u24mr23967904eje.106.1624268740767; Mon, 21 Jun 2021 02:45:40 -0700 (PDT) Received: from x1.localdomain (2001-1c00-0c1e-bf00-1054-9d19-e0f0-8214.cable.dynamic.v6.ziggo.nl. [2001:1c00:c1e:bf00:1054:9d19:e0f0:8214]) by smtp.gmail.com with ESMTPSA id og26sm4611724ejc.52.2021.06.21.02.45.39 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 21 Jun 2021 02:45:39 -0700 (PDT) Subject: Re: [PATCH 12/18] staging: rtl8723bs: remove VHT dead code To: Fabio Aiuto , gregkh@linuxfoundation.org Cc: Larry.Finger@lwfinger.net, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org References: From: Hans de Goede Message-ID: Date: Mon, 21 Jun 2021 11:45:39 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 6/19/21 12:47 PM, Fabio Aiuto wrote: > remove VHT dead code, as the device doesn't support > VHT (which is a 802.11ac capability). > > Signed-off-by: Fabio Aiuto > --- > drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 1 - > drivers/staging/rtl8723bs/hal/hal_com.c | 241 ----------- > .../staging/rtl8723bs/hal/hal_com_phycfg.c | 383 +----------------- > drivers/staging/rtl8723bs/include/hal_com.h | 62 +-- > .../rtl8723bs/include/hal_com_phycfg.h | 4 - > drivers/staging/rtl8723bs/include/ieee80211.h | 45 -- > .../staging/rtl8723bs/include/rtl8723b_xmit.h | 21 - > drivers/staging/rtl8723bs/include/rtw_ht.h | 4 - > 8 files changed, 9 insertions(+), 752 deletions(-) > > diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c > index 285acd3d843b..c128d462c6c7 100644 > --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c > +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c > @@ -46,7 +46,6 @@ static struct action_handler OnAction_tbl[] = { > {RTW_WLAN_CATEGORY_UNPROTECTED_WNM, "ACTION_UNPROTECTED_WNM", &DoReserved}, > {RTW_WLAN_CATEGORY_SELF_PROTECTED, "ACTION_SELF_PROTECTED", &DoReserved}, > {RTW_WLAN_CATEGORY_WMM, "ACTION_WMM", &DoReserved}, > - {RTW_WLAN_CATEGORY_VHT, "ACTION_VHT", &DoReserved}, > {RTW_WLAN_CATEGORY_P2P, "ACTION_P2P", &DoReserved}, > }; > > diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c > index 7a88447f8294..eebd48438733 100644 > --- a/drivers/staging/rtl8723bs/hal/hal_com.c > +++ b/drivers/staging/rtl8723bs/hal/hal_com.c > @@ -295,126 +295,6 @@ u8 MRateToHwRate(u8 rate) > case MGN_MCS31: > ret = DESC_RATEMCS31; > break; > - case MGN_VHT1SS_MCS0: > - ret = DESC_RATEVHTSS1MCS0; > - break; > - case MGN_VHT1SS_MCS1: > - ret = DESC_RATEVHTSS1MCS1; > - break; > - case MGN_VHT1SS_MCS2: > - ret = DESC_RATEVHTSS1MCS2; > - break; > - case MGN_VHT1SS_MCS3: > - ret = DESC_RATEVHTSS1MCS3; > - break; > - case MGN_VHT1SS_MCS4: > - ret = DESC_RATEVHTSS1MCS4; > - break; > - case MGN_VHT1SS_MCS5: > - ret = DESC_RATEVHTSS1MCS5; > - break; > - case MGN_VHT1SS_MCS6: > - ret = DESC_RATEVHTSS1MCS6; > - break; > - case MGN_VHT1SS_MCS7: > - ret = DESC_RATEVHTSS1MCS7; > - break; > - case MGN_VHT1SS_MCS8: > - ret = DESC_RATEVHTSS1MCS8; > - break; > - case MGN_VHT1SS_MCS9: > - ret = DESC_RATEVHTSS1MCS9; > - break; > - case MGN_VHT2SS_MCS0: > - ret = DESC_RATEVHTSS2MCS0; > - break; > - case MGN_VHT2SS_MCS1: > - ret = DESC_RATEVHTSS2MCS1; > - break; > - case MGN_VHT2SS_MCS2: > - ret = DESC_RATEVHTSS2MCS2; > - break; > - case MGN_VHT2SS_MCS3: > - ret = DESC_RATEVHTSS2MCS3; > - break; > - case MGN_VHT2SS_MCS4: > - ret = DESC_RATEVHTSS2MCS4; > - break; > - case MGN_VHT2SS_MCS5: > - ret = DESC_RATEVHTSS2MCS5; > - break; > - case MGN_VHT2SS_MCS6: > - ret = DESC_RATEVHTSS2MCS6; > - break; > - case MGN_VHT2SS_MCS7: > - ret = DESC_RATEVHTSS2MCS7; > - break; > - case MGN_VHT2SS_MCS8: > - ret = DESC_RATEVHTSS2MCS8; > - break; > - case MGN_VHT2SS_MCS9: > - ret = DESC_RATEVHTSS2MCS9; > - break; > - case MGN_VHT3SS_MCS0: > - ret = DESC_RATEVHTSS3MCS0; > - break; > - case MGN_VHT3SS_MCS1: > - ret = DESC_RATEVHTSS3MCS1; > - break; > - case MGN_VHT3SS_MCS2: > - ret = DESC_RATEVHTSS3MCS2; > - break; > - case MGN_VHT3SS_MCS3: > - ret = DESC_RATEVHTSS3MCS3; > - break; > - case MGN_VHT3SS_MCS4: > - ret = DESC_RATEVHTSS3MCS4; > - break; > - case MGN_VHT3SS_MCS5: > - ret = DESC_RATEVHTSS3MCS5; > - break; > - case MGN_VHT3SS_MCS6: > - ret = DESC_RATEVHTSS3MCS6; > - break; > - case MGN_VHT3SS_MCS7: > - ret = DESC_RATEVHTSS3MCS7; > - break; > - case MGN_VHT3SS_MCS8: > - ret = DESC_RATEVHTSS3MCS8; > - break; > - case MGN_VHT3SS_MCS9: > - ret = DESC_RATEVHTSS3MCS9; > - break; > - case MGN_VHT4SS_MCS0: > - ret = DESC_RATEVHTSS4MCS0; > - break; > - case MGN_VHT4SS_MCS1: > - ret = DESC_RATEVHTSS4MCS1; > - break; > - case MGN_VHT4SS_MCS2: > - ret = DESC_RATEVHTSS4MCS2; > - break; > - case MGN_VHT4SS_MCS3: > - ret = DESC_RATEVHTSS4MCS3; > - break; > - case MGN_VHT4SS_MCS4: > - ret = DESC_RATEVHTSS4MCS4; > - break; > - case MGN_VHT4SS_MCS5: > - ret = DESC_RATEVHTSS4MCS5; > - break; > - case MGN_VHT4SS_MCS6: > - ret = DESC_RATEVHTSS4MCS6; > - break; > - case MGN_VHT4SS_MCS7: > - ret = DESC_RATEVHTSS4MCS7; > - break; > - case MGN_VHT4SS_MCS8: > - ret = DESC_RATEVHTSS4MCS8; > - break; > - case MGN_VHT4SS_MCS9: > - ret = DESC_RATEVHTSS4MCS9; > - break; > default: > break; > } > @@ -559,127 +439,6 @@ u8 HwRateToMRate(u8 rate) > case DESC_RATEMCS31: > ret_rate = MGN_MCS31; > break; > - case DESC_RATEVHTSS1MCS0: > - ret_rate = MGN_VHT1SS_MCS0; > - break; > - case DESC_RATEVHTSS1MCS1: > - ret_rate = MGN_VHT1SS_MCS1; > - break; > - case DESC_RATEVHTSS1MCS2: > - ret_rate = MGN_VHT1SS_MCS2; > - break; > - case DESC_RATEVHTSS1MCS3: > - ret_rate = MGN_VHT1SS_MCS3; > - break; > - case DESC_RATEVHTSS1MCS4: > - ret_rate = MGN_VHT1SS_MCS4; > - break; > - case DESC_RATEVHTSS1MCS5: > - ret_rate = MGN_VHT1SS_MCS5; > - break; > - case DESC_RATEVHTSS1MCS6: > - ret_rate = MGN_VHT1SS_MCS6; > - break; > - case DESC_RATEVHTSS1MCS7: > - ret_rate = MGN_VHT1SS_MCS7; > - break; > - case DESC_RATEVHTSS1MCS8: > - ret_rate = MGN_VHT1SS_MCS8; > - break; > - case DESC_RATEVHTSS1MCS9: > - ret_rate = MGN_VHT1SS_MCS9; > - break; > - case DESC_RATEVHTSS2MCS0: > - ret_rate = MGN_VHT2SS_MCS0; > - break; > - case DESC_RATEVHTSS2MCS1: > - ret_rate = MGN_VHT2SS_MCS1; > - break; > - case DESC_RATEVHTSS2MCS2: > - ret_rate = MGN_VHT2SS_MCS2; > - break; > - case DESC_RATEVHTSS2MCS3: > - ret_rate = MGN_VHT2SS_MCS3; > - break; > - case DESC_RATEVHTSS2MCS4: > - ret_rate = MGN_VHT2SS_MCS4; > - break; > - case DESC_RATEVHTSS2MCS5: > - ret_rate = MGN_VHT2SS_MCS5; > - break; > - case DESC_RATEVHTSS2MCS6: > - ret_rate = MGN_VHT2SS_MCS6; > - break; > - case DESC_RATEVHTSS2MCS7: > - ret_rate = MGN_VHT2SS_MCS7; > - break; > - case DESC_RATEVHTSS2MCS8: > - ret_rate = MGN_VHT2SS_MCS8; > - break; > - case DESC_RATEVHTSS2MCS9: > - ret_rate = MGN_VHT2SS_MCS9; > - break; > - case DESC_RATEVHTSS3MCS0: > - ret_rate = MGN_VHT3SS_MCS0; > - break; > - case DESC_RATEVHTSS3MCS1: > - ret_rate = MGN_VHT3SS_MCS1; > - break; > - case DESC_RATEVHTSS3MCS2: > - ret_rate = MGN_VHT3SS_MCS2; > - break; > - case DESC_RATEVHTSS3MCS3: > - ret_rate = MGN_VHT3SS_MCS3; > - break; > - case DESC_RATEVHTSS3MCS4: > - ret_rate = MGN_VHT3SS_MCS4; > - break; > - case DESC_RATEVHTSS3MCS5: > - ret_rate = MGN_VHT3SS_MCS5; > - break; > - case DESC_RATEVHTSS3MCS6: > - ret_rate = MGN_VHT3SS_MCS6; > - break; > - case DESC_RATEVHTSS3MCS7: > - ret_rate = MGN_VHT3SS_MCS7; > - break; > - case DESC_RATEVHTSS3MCS8: > - ret_rate = MGN_VHT3SS_MCS8; > - break; > - case DESC_RATEVHTSS3MCS9: > - ret_rate = MGN_VHT3SS_MCS9; > - break; > - case DESC_RATEVHTSS4MCS0: > - ret_rate = MGN_VHT4SS_MCS0; > - break; > - case DESC_RATEVHTSS4MCS1: > - ret_rate = MGN_VHT4SS_MCS1; > - break; > - case DESC_RATEVHTSS4MCS2: > - ret_rate = MGN_VHT4SS_MCS2; > - break; > - case DESC_RATEVHTSS4MCS3: > - ret_rate = MGN_VHT4SS_MCS3; > - break; > - case DESC_RATEVHTSS4MCS4: > - ret_rate = MGN_VHT4SS_MCS4; > - break; > - case DESC_RATEVHTSS4MCS5: > - ret_rate = MGN_VHT4SS_MCS5; > - break; > - case DESC_RATEVHTSS4MCS6: > - ret_rate = MGN_VHT4SS_MCS6; > - break; > - case DESC_RATEVHTSS4MCS7: > - ret_rate = MGN_VHT4SS_MCS7; > - break; > - case DESC_RATEVHTSS4MCS8: > - ret_rate = MGN_VHT4SS_MCS8; > - break; > - case DESC_RATEVHTSS4MCS9: > - ret_rate = MGN_VHT4SS_MCS9; > - break; > - > default: > break; > } > diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c > index 395eb3b5af71..bb7941aee0c4 100644 > --- a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c > +++ b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c > @@ -39,18 +39,6 @@ u8 PHY_GetTxPowerByRateBase(struct adapter *Adapter, u8 RfPath, > case HT_MCS24_MCS31: > value = pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][5]; > break; > - case VHT_1SSMCS0_1SSMCS9: > - value = pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][6]; > - break; > - case VHT_2SSMCS0_2SSMCS9: > - value = pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][7]; > - break; > - case VHT_3SSMCS0_3SSMCS9: > - value = pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][8]; > - break; > - case VHT_4SSMCS0_4SSMCS9: > - value = pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][9]; > - break; > default: > break; > } > @@ -91,18 +79,6 @@ phy_SetTxPowerByRateBase( > case HT_MCS24_MCS31: > pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][5] = Value; > break; > - case VHT_1SSMCS0_1SSMCS9: > - pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][6] = Value; > - break; > - case VHT_2SSMCS0_2SSMCS9: > - pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][7] = Value; > - break; > - case VHT_3SSMCS0_3SSMCS9: > - pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][8] = Value; > - break; > - case VHT_4SSMCS0_4SSMCS9: > - pHalData->TxPwrByRateBase2_4G[RfPath][TxNum][9] = Value; > - break; > default: > break; > } > @@ -131,14 +107,6 @@ struct adapter *padapter > base = PHY_GetTxPowerByRate(padapter, path, RF_3TX, MGN_MCS23); > phy_SetTxPowerByRateBase(padapter, path, HT_MCS16_MCS23, RF_3TX, base); > > - base = PHY_GetTxPowerByRate(padapter, path, RF_1TX, MGN_VHT1SS_MCS7); > - phy_SetTxPowerByRateBase(padapter, path, VHT_1SSMCS0_1SSMCS9, RF_1TX, base); > - > - base = PHY_GetTxPowerByRate(padapter, path, RF_2TX, MGN_VHT2SS_MCS7); > - phy_SetTxPowerByRateBase(padapter, path, VHT_2SSMCS0_2SSMCS9, RF_2TX, base); > - > - base = PHY_GetTxPowerByRate(padapter, path, RF_3TX, MGN_VHT3SS_MCS7); > - phy_SetTxPowerByRateBase(padapter, path, VHT_3SSMCS0_3SSMCS9, RF_3TX, base); You are removing a bunch of register writes from a block here, while keeping others, so this does not seem like it is just dead-code removal it feels like you are actually making a functional change. Perhaps this entire block can never be executed ? Or maybe the VHT name used here is misleading and this is also relevant for 2.4 GHZ 40Mhz use ? Did you test with an access point configured to use 40 MHz channels and then checked that you were actually using a 40MHz channel ? Note these are quite rare AFAIK because 40 MHz tends to cause even more collosion issues in the already quite full 2.4G spectrum, still if possible you should test this and make sure that it does not regress (with out specifically testing this we might not note regressions for a long time because of 40MHz usage being rare). > } > } > > @@ -442,81 +410,6 @@ PHY_GetRateValuesOfTxPowerByRate( > *RateNum = 4; > break; > > - case 0xC3C: > - case 0xE3C: > - case 0x183C: > - case 0x1a3C: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS0); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS1); > - RateIndex[2] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS2); > - RateIndex[3] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS3); > - for (i = 0; i < 4; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - > - case 0xC40: > - case 0xE40: > - case 0x1840: > - case 0x1a40: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS4); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS5); > - RateIndex[2] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS6); > - RateIndex[3] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS7); > - for (i = 0; i < 4; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - > - case 0xC44: > - case 0xE44: > - case 0x1844: > - case 0x1a44: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS8); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT1SS_MCS9); > - RateIndex[2] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS0); > - RateIndex[3] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS1); > - for (i = 0; i < 4; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - > - case 0xC48: > - case 0xE48: > - case 0x1848: > - case 0x1a48: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS2); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS3); > - RateIndex[2] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS4); > - RateIndex[3] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS5); > - for (i = 0; i < 4; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - > - case 0xC4C: > - case 0xE4C: > - case 0x184C: > - case 0x1a4C: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS6); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS7); > - RateIndex[2] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS8); > - RateIndex[3] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS9); > - for (i = 0; i < 4; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - > case 0xCD8: > case 0xED8: > case 0x18D8: > @@ -547,49 +440,6 @@ PHY_GetRateValuesOfTxPowerByRate( > *RateNum = 4; > break; > > - case 0xCE0: > - case 0xEE0: > - case 0x18E0: > - case 0x1aE0: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS0); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS1); > - RateIndex[2] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS2); > - RateIndex[3] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS3); > - for (i = 0; i < 4; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - > - case 0xCE4: > - case 0xEE4: > - case 0x18E4: > - case 0x1aE4: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS4); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS5); > - RateIndex[2] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS6); > - RateIndex[3] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS7); > - for (i = 0; i < 4; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - > - case 0xCE8: > - case 0xEE8: > - case 0x18E8: > - case 0x1aE8: > - RateIndex[0] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS8); > - RateIndex[1] = PHY_GetRateIndexOfTxPowerByRate(MGN_VHT3SS_MCS9); > - for (i = 0; i < 2; ++i) { > - PwrByRateVal[i] = (s8) ((((Value >> (i * 8 + 4)) & 0xF)) * 10 + > - ((Value >> (i * 8)) & 0xF)); > - } > - *RateNum = 4; > - break; > - Here only 2 rates are set, so this feels like it is a 40MHz case, are you sure this can be removed (maybe it is 40MHz channel use at 5GHz ? The rest of the patch looks ok. Regards, Hans > default: > break; > } > @@ -617,10 +467,6 @@ static void PHY_StoreTxPowerByRateNew( > return; > > for (i = 0; i < rateNum; ++i) { > - if (rateIndex[i] == PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS0) || > - rateIndex[i] == PHY_GetRateIndexOfTxPowerByRate(MGN_VHT2SS_MCS1)) > - TxNum = RF_2TX; > - > pHalData->TxPwrByRateOffset[RfPath][TxNum][rateIndex[i]] = PwrByRateVal[i]; > } > } > @@ -691,18 +537,6 @@ struct adapter *padapter > u8 mcs16_23Rates[8] = { > MGN_MCS16, MGN_MCS17, MGN_MCS18, MGN_MCS19, MGN_MCS20, MGN_MCS21, MGN_MCS22, MGN_MCS23 > }; > - u8 vht1ssRates[10] = { > - MGN_VHT1SS_MCS0, MGN_VHT1SS_MCS1, MGN_VHT1SS_MCS2, MGN_VHT1SS_MCS3, MGN_VHT1SS_MCS4, > - MGN_VHT1SS_MCS5, MGN_VHT1SS_MCS6, MGN_VHT1SS_MCS7, MGN_VHT1SS_MCS8, MGN_VHT1SS_MCS9 > - }; > - u8 vht2ssRates[10] = { > - MGN_VHT2SS_MCS0, MGN_VHT2SS_MCS1, MGN_VHT2SS_MCS2, MGN_VHT2SS_MCS3, MGN_VHT2SS_MCS4, > - MGN_VHT2SS_MCS5, MGN_VHT2SS_MCS6, MGN_VHT2SS_MCS7, MGN_VHT2SS_MCS8, MGN_VHT2SS_MCS9 > - }; > - u8 vht3ssRates[10] = { > - MGN_VHT3SS_MCS0, MGN_VHT3SS_MCS1, MGN_VHT3SS_MCS2, MGN_VHT3SS_MCS3, MGN_VHT3SS_MCS4, > - MGN_VHT3SS_MCS5, MGN_VHT3SS_MCS6, MGN_VHT3SS_MCS7, MGN_VHT3SS_MCS8, MGN_VHT3SS_MCS9 > - }; > > for (path = ODM_RF_PATH_A; path <= ODM_RF_PATH_D; ++path) { > for (txNum = RF_1TX; txNum < RF_MAX_TX_NUM; ++txNum) { > @@ -740,27 +574,6 @@ struct adapter *padapter > value = PHY_GetTxPowerByRate(padapter, path, txNum, mcs16_23Rates[i]); > PHY_SetTxPowerByRate(padapter, path, txNum, mcs16_23Rates[i], value - base); > } > - > - /* VHT 1SS */ > - base = PHY_GetTxPowerByRate(padapter, path, txNum, MGN_VHT1SS_MCS7); > - for (i = 0; i < sizeof(vht1ssRates); ++i) { > - value = PHY_GetTxPowerByRate(padapter, path, txNum, vht1ssRates[i]); > - PHY_SetTxPowerByRate(padapter, path, txNum, vht1ssRates[i], value - base); > - } > - > - /* VHT 2SS */ > - base = PHY_GetTxPowerByRate(padapter, path, txNum, MGN_VHT2SS_MCS7); > - for (i = 0; i < sizeof(vht2ssRates); ++i) { > - value = PHY_GetTxPowerByRate(padapter, path, txNum, vht2ssRates[i]); > - PHY_SetTxPowerByRate(padapter, path, txNum, vht2ssRates[i], value - base); > - } > - > - /* VHT 3SS */ > - base = PHY_GetTxPowerByRate(padapter, path, txNum, MGN_VHT3SS_MCS7); > - for (i = 0; i < sizeof(vht3ssRates); ++i) { > - value = PHY_GetTxPowerByRate(padapter, path, txNum, vht3ssRates[i]); > - PHY_SetTxPowerByRate(padapter, path, txNum, vht3ssRates[i], value - base); > - } > } > } > } > @@ -823,38 +636,6 @@ void PHY_SetTxPowerIndexByRateSection( > Channel, htRates4T, > ARRAY_SIZE(htRates4T)); > > - } else if (RateSection == VHT_1SSMCS0_1SSMCS9) { > - u8 vhtRates1T[] = {MGN_VHT1SS_MCS0, MGN_VHT1SS_MCS1, MGN_VHT1SS_MCS2, MGN_VHT1SS_MCS3, MGN_VHT1SS_MCS4, > - MGN_VHT1SS_MCS5, MGN_VHT1SS_MCS6, MGN_VHT1SS_MCS7, MGN_VHT1SS_MCS8, MGN_VHT1SS_MCS9}; > - PHY_SetTxPowerIndexByRateArray(padapter, RFPath, > - pHalData->CurrentChannelBW, > - Channel, vhtRates1T, > - ARRAY_SIZE(vhtRates1T)); > - > - } else if (RateSection == VHT_2SSMCS0_2SSMCS9) { > - u8 vhtRates2T[] = {MGN_VHT2SS_MCS0, MGN_VHT2SS_MCS1, MGN_VHT2SS_MCS2, MGN_VHT2SS_MCS3, MGN_VHT2SS_MCS4, > - MGN_VHT2SS_MCS5, MGN_VHT2SS_MCS6, MGN_VHT2SS_MCS7, MGN_VHT2SS_MCS8, MGN_VHT2SS_MCS9}; > - > - PHY_SetTxPowerIndexByRateArray(padapter, RFPath, > - pHalData->CurrentChannelBW, > - Channel, vhtRates2T, > - ARRAY_SIZE(vhtRates2T)); > - } else if (RateSection == VHT_3SSMCS0_3SSMCS9) { > - u8 vhtRates3T[] = {MGN_VHT3SS_MCS0, MGN_VHT3SS_MCS1, MGN_VHT3SS_MCS2, MGN_VHT3SS_MCS3, MGN_VHT3SS_MCS4, > - MGN_VHT3SS_MCS5, MGN_VHT3SS_MCS6, MGN_VHT3SS_MCS7, MGN_VHT3SS_MCS8, MGN_VHT3SS_MCS9}; > - > - PHY_SetTxPowerIndexByRateArray(padapter, RFPath, > - pHalData->CurrentChannelBW, > - Channel, vhtRates3T, > - ARRAY_SIZE(vhtRates3T)); > - } else if (RateSection == VHT_4SSMCS0_4SSMCS9) { > - u8 vhtRates4T[] = {MGN_VHT4SS_MCS0, MGN_VHT4SS_MCS1, MGN_VHT4SS_MCS2, MGN_VHT4SS_MCS3, MGN_VHT4SS_MCS4, > - MGN_VHT4SS_MCS5, MGN_VHT4SS_MCS6, MGN_VHT4SS_MCS7, MGN_VHT4SS_MCS8, MGN_VHT4SS_MCS9}; > - > - PHY_SetTxPowerIndexByRateArray(padapter, RFPath, > - pHalData->CurrentChannelBW, > - Channel, vhtRates4T, > - ARRAY_SIZE(vhtRates4T)); > } > } > > @@ -883,23 +664,23 @@ u8 PHY_GetTxPowerIndexBase( > txPower += pHalData->OFDM_24G_Diff[RFPath][TX_1S]; > > if (BandWidth == CHANNEL_WIDTH_20) { /* BW20-1S, BW20-2S */ > - if ((MGN_MCS0 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT1SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS0 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW20_24G_Diff[RFPath][TX_1S]; > - if ((MGN_MCS8 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT2SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS8 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW20_24G_Diff[RFPath][TX_2S]; > - if ((MGN_MCS16 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT3SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS16 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW20_24G_Diff[RFPath][TX_3S]; > - if ((MGN_MCS24 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT4SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS24 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW20_24G_Diff[RFPath][TX_4S]; > > } else if (BandWidth == CHANNEL_WIDTH_40) { /* BW40-1S, BW40-2S */ > - if ((MGN_MCS0 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT1SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS0 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW40_24G_Diff[RFPath][TX_1S]; > - if ((MGN_MCS8 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT2SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS8 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW40_24G_Diff[RFPath][TX_2S]; > - if ((MGN_MCS16 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT3SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS16 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW40_24G_Diff[RFPath][TX_3S]; > - if ((MGN_MCS24 <= Rate && Rate <= MGN_MCS31) || (MGN_VHT4SS_MCS0 <= Rate && Rate <= MGN_VHT4SS_MCS9)) > + if (MGN_MCS24 <= Rate && Rate <= MGN_MCS31) > txPower += pHalData->BW40_24G_Diff[RFPath][TX_4S]; > > } > @@ -1060,126 +841,6 @@ u8 PHY_GetRateIndexOfTxPowerByRate(u8 Rate) > case MGN_MCS31: > index = 43; > break; > - case MGN_VHT1SS_MCS0: > - index = 44; > - break; > - case MGN_VHT1SS_MCS1: > - index = 45; > - break; > - case MGN_VHT1SS_MCS2: > - index = 46; > - break; > - case MGN_VHT1SS_MCS3: > - index = 47; > - break; > - case MGN_VHT1SS_MCS4: > - index = 48; > - break; > - case MGN_VHT1SS_MCS5: > - index = 49; > - break; > - case MGN_VHT1SS_MCS6: > - index = 50; > - break; > - case MGN_VHT1SS_MCS7: > - index = 51; > - break; > - case MGN_VHT1SS_MCS8: > - index = 52; > - break; > - case MGN_VHT1SS_MCS9: > - index = 53; > - break; > - case MGN_VHT2SS_MCS0: > - index = 54; > - break; > - case MGN_VHT2SS_MCS1: > - index = 55; > - break; > - case MGN_VHT2SS_MCS2: > - index = 56; > - break; > - case MGN_VHT2SS_MCS3: > - index = 57; > - break; > - case MGN_VHT2SS_MCS4: > - index = 58; > - break; > - case MGN_VHT2SS_MCS5: > - index = 59; > - break; > - case MGN_VHT2SS_MCS6: > - index = 60; > - break; > - case MGN_VHT2SS_MCS7: > - index = 61; > - break; > - case MGN_VHT2SS_MCS8: > - index = 62; > - break; > - case MGN_VHT2SS_MCS9: > - index = 63; > - break; > - case MGN_VHT3SS_MCS0: > - index = 64; > - break; > - case MGN_VHT3SS_MCS1: > - index = 65; > - break; > - case MGN_VHT3SS_MCS2: > - index = 66; > - break; > - case MGN_VHT3SS_MCS3: > - index = 67; > - break; > - case MGN_VHT3SS_MCS4: > - index = 68; > - break; > - case MGN_VHT3SS_MCS5: > - index = 69; > - break; > - case MGN_VHT3SS_MCS6: > - index = 70; > - break; > - case MGN_VHT3SS_MCS7: > - index = 71; > - break; > - case MGN_VHT3SS_MCS8: > - index = 72; > - break; > - case MGN_VHT3SS_MCS9: > - index = 73; > - break; > - case MGN_VHT4SS_MCS0: > - index = 74; > - break; > - case MGN_VHT4SS_MCS1: > - index = 75; > - break; > - case MGN_VHT4SS_MCS2: > - index = 76; > - break; > - case MGN_VHT4SS_MCS3: > - index = 77; > - break; > - case MGN_VHT4SS_MCS4: > - index = 78; > - break; > - case MGN_VHT4SS_MCS5: > - index = 79; > - break; > - case MGN_VHT4SS_MCS6: > - index = 80; > - break; > - case MGN_VHT4SS_MCS7: > - index = 81; > - break; > - case MGN_VHT4SS_MCS8: > - index = 82; > - break; > - case MGN_VHT4SS_MCS9: > - index = 83; > - break; > default: > break; > } > @@ -1319,26 +980,6 @@ static s16 get_rate_sctn_idx(const u8 rate) > case MGN_MCS24: case MGN_MCS25: case MGN_MCS26: case MGN_MCS27: > case MGN_MCS28: case MGN_MCS29: case MGN_MCS30: case MGN_MCS31: > return 5; > - case MGN_VHT1SS_MCS0: case MGN_VHT1SS_MCS1: case MGN_VHT1SS_MCS2: > - case MGN_VHT1SS_MCS3: case MGN_VHT1SS_MCS4: case MGN_VHT1SS_MCS5: > - case MGN_VHT1SS_MCS6: case MGN_VHT1SS_MCS7: case MGN_VHT1SS_MCS8: > - case MGN_VHT1SS_MCS9: > - return 6; > - case MGN_VHT2SS_MCS0: case MGN_VHT2SS_MCS1: case MGN_VHT2SS_MCS2: > - case MGN_VHT2SS_MCS3: case MGN_VHT2SS_MCS4: case MGN_VHT2SS_MCS5: > - case MGN_VHT2SS_MCS6: case MGN_VHT2SS_MCS7: case MGN_VHT2SS_MCS8: > - case MGN_VHT2SS_MCS9: > - return 7; > - case MGN_VHT3SS_MCS0: case MGN_VHT3SS_MCS1: case MGN_VHT3SS_MCS2: > - case MGN_VHT3SS_MCS3: case MGN_VHT3SS_MCS4: case MGN_VHT3SS_MCS5: > - case MGN_VHT3SS_MCS6: case MGN_VHT3SS_MCS7: case MGN_VHT3SS_MCS8: > - case MGN_VHT3SS_MCS9: > - return 8; > - case MGN_VHT4SS_MCS0: case MGN_VHT4SS_MCS1: case MGN_VHT4SS_MCS2: > - case MGN_VHT4SS_MCS3: case MGN_VHT4SS_MCS4: case MGN_VHT4SS_MCS5: > - case MGN_VHT4SS_MCS6: case MGN_VHT4SS_MCS7: case MGN_VHT4SS_MCS8: > - case MGN_VHT4SS_MCS9: > - return 9; > default: > return -1; > } > @@ -1510,14 +1151,6 @@ void PHY_SetTxPowerLimit( > rateSection = 4; > else if (eqNByte(RateSection, (u8 *)("HT"), 2) && eqNByte(RfPath, (u8 *)("4T"), 2)) > rateSection = 5; > - else if (eqNByte(RateSection, (u8 *)("VHT"), 3) && eqNByte(RfPath, (u8 *)("1T"), 2)) > - rateSection = 6; > - else if (eqNByte(RateSection, (u8 *)("VHT"), 3) && eqNByte(RfPath, (u8 *)("2T"), 2)) > - rateSection = 7; > - else if (eqNByte(RateSection, (u8 *)("VHT"), 3) && eqNByte(RfPath, (u8 *)("3T"), 2)) > - rateSection = 8; > - else if (eqNByte(RateSection, (u8 *)("VHT"), 3) && eqNByte(RfPath, (u8 *)("4T"), 2)) > - rateSection = 9; > else > return; > > diff --git a/drivers/staging/rtl8723bs/include/hal_com.h b/drivers/staging/rtl8723bs/include/hal_com.h > index 6bcc443d59fb..1bc332261b2a 100644 > --- a/drivers/staging/rtl8723bs/include/hal_com.h > +++ b/drivers/staging/rtl8723bs/include/hal_com.h > @@ -69,46 +69,6 @@ > #define DESC_RATEMCS29 0x29 > #define DESC_RATEMCS30 0x2A > #define DESC_RATEMCS31 0x2B > -#define DESC_RATEVHTSS1MCS0 0x2C > -#define DESC_RATEVHTSS1MCS1 0x2D > -#define DESC_RATEVHTSS1MCS2 0x2E > -#define DESC_RATEVHTSS1MCS3 0x2F > -#define DESC_RATEVHTSS1MCS4 0x30 > -#define DESC_RATEVHTSS1MCS5 0x31 > -#define DESC_RATEVHTSS1MCS6 0x32 > -#define DESC_RATEVHTSS1MCS7 0x33 > -#define DESC_RATEVHTSS1MCS8 0x34 > -#define DESC_RATEVHTSS1MCS9 0x35 > -#define DESC_RATEVHTSS2MCS0 0x36 > -#define DESC_RATEVHTSS2MCS1 0x37 > -#define DESC_RATEVHTSS2MCS2 0x38 > -#define DESC_RATEVHTSS2MCS3 0x39 > -#define DESC_RATEVHTSS2MCS4 0x3A > -#define DESC_RATEVHTSS2MCS5 0x3B > -#define DESC_RATEVHTSS2MCS6 0x3C > -#define DESC_RATEVHTSS2MCS7 0x3D > -#define DESC_RATEVHTSS2MCS8 0x3E > -#define DESC_RATEVHTSS2MCS9 0x3F > -#define DESC_RATEVHTSS3MCS0 0x40 > -#define DESC_RATEVHTSS3MCS1 0x41 > -#define DESC_RATEVHTSS3MCS2 0x42 > -#define DESC_RATEVHTSS3MCS3 0x43 > -#define DESC_RATEVHTSS3MCS4 0x44 > -#define DESC_RATEVHTSS3MCS5 0x45 > -#define DESC_RATEVHTSS3MCS6 0x46 > -#define DESC_RATEVHTSS3MCS7 0x47 > -#define DESC_RATEVHTSS3MCS8 0x48 > -#define DESC_RATEVHTSS3MCS9 0x49 > -#define DESC_RATEVHTSS4MCS0 0x4A > -#define DESC_RATEVHTSS4MCS1 0x4B > -#define DESC_RATEVHTSS4MCS2 0x4C > -#define DESC_RATEVHTSS4MCS3 0x4D > -#define DESC_RATEVHTSS4MCS4 0x4E > -#define DESC_RATEVHTSS4MCS5 0x4F > -#define DESC_RATEVHTSS4MCS6 0x50 > -#define DESC_RATEVHTSS4MCS7 0x51 > -#define DESC_RATEVHTSS4MCS8 0x52 > -#define DESC_RATEVHTSS4MCS9 0x53 > > #define HDATA_RATE(rate)\ > (rate == DESC_RATE1M) ? "CCK_1M" : \ > @@ -138,27 +98,7 @@ > (rate == DESC_RATEMCS12) ? "MCS12" : \ > (rate == DESC_RATEMCS13) ? "MCS13" : \ > (rate == DESC_RATEMCS14) ? "MCS14" : \ > -(rate == DESC_RATEMCS15) ? "MCS15" : \ > -(rate == DESC_RATEVHTSS1MCS0) ? "VHTSS1MCS0" : \ > -(rate == DESC_RATEVHTSS1MCS1) ? "VHTSS1MCS1" : \ > -(rate == DESC_RATEVHTSS1MCS2) ? "VHTSS1MCS2" : \ > -(rate == DESC_RATEVHTSS1MCS3) ? "VHTSS1MCS3" : \ > -(rate == DESC_RATEVHTSS1MCS4) ? "VHTSS1MCS4" : \ > -(rate == DESC_RATEVHTSS1MCS5) ? "VHTSS1MCS5" : \ > -(rate == DESC_RATEVHTSS1MCS6) ? "VHTSS1MCS6" : \ > -(rate == DESC_RATEVHTSS1MCS7) ? "VHTSS1MCS7" : \ > -(rate == DESC_RATEVHTSS1MCS8) ? "VHTSS1MCS8" : \ > -(rate == DESC_RATEVHTSS1MCS9) ? "VHTSS1MCS9" : \ > -(rate == DESC_RATEVHTSS2MCS0) ? "VHTSS2MCS0" : \ > -(rate == DESC_RATEVHTSS2MCS1) ? "VHTSS2MCS1" : \ > -(rate == DESC_RATEVHTSS2MCS2) ? "VHTSS2MCS2" : \ > -(rate == DESC_RATEVHTSS2MCS3) ? "VHTSS2MCS3" : \ > -(rate == DESC_RATEVHTSS2MCS4) ? "VHTSS2MCS4" : \ > -(rate == DESC_RATEVHTSS2MCS5) ? "VHTSS2MCS5" : \ > -(rate == DESC_RATEVHTSS2MCS6) ? "VHTSS2MCS6" : \ > -(rate == DESC_RATEVHTSS2MCS7) ? "VHTSS2MCS7" : \ > -(rate == DESC_RATEVHTSS2MCS8) ? "VHTSS2MCS8" : \ > -(rate == DESC_RATEVHTSS2MCS9) ? "VHTSS2MCS9" : "UNKNOWN" > +(rate == DESC_RATEMCS15) ? "MCS15" : "UNKNOWN" > > > enum{ > diff --git a/drivers/staging/rtl8723bs/include/hal_com_phycfg.h b/drivers/staging/rtl8723bs/include/hal_com_phycfg.h > index 3d95ab1986b1..c966d0e3e5ae 100644 > --- a/drivers/staging/rtl8723bs/include/hal_com_phycfg.h > +++ b/drivers/staging/rtl8723bs/include/hal_com_phycfg.h > @@ -19,10 +19,6 @@ enum rate_section { > HT_MCS8_MCS15, > HT_MCS16_MCS23, > HT_MCS24_MCS31, > - VHT_1SSMCS0_1SSMCS9, > - VHT_2SSMCS0_2SSMCS9, > - VHT_3SSMCS0_3SSMCS9, > - VHT_4SSMCS0_4SSMCS9, > }; > > enum { > diff --git a/drivers/staging/rtl8723bs/include/ieee80211.h b/drivers/staging/rtl8723bs/include/ieee80211.h > index ac88bd30b097..378c21595e05 100644 > --- a/drivers/staging/rtl8723bs/include/ieee80211.h > +++ b/drivers/staging/rtl8723bs/include/ieee80211.h > @@ -60,7 +60,6 @@ enum { > #define WLAN_STA_HT BIT(11) > #define WLAN_STA_WPS BIT(12) > #define WLAN_STA_MAYBE_WPS BIT(13) > -#define WLAN_STA_VHT BIT(14) > #define WLAN_STA_NONERP BIT(31) > > #define IEEE_CMD_SET_WPA_PARAM 1 > @@ -135,8 +134,6 @@ enum { > RATEID_IDX_BG = 6, > RATEID_IDX_G = 7, > RATEID_IDX_B = 8, > - RATEID_IDX_VHT_2SS = 9, > - RATEID_IDX_VHT_1SS = 10, > }; > > enum network_type { > @@ -424,51 +421,10 @@ enum { > MGN_MCS29, > MGN_MCS30, > MGN_MCS31, > - MGN_VHT1SS_MCS0, > - MGN_VHT1SS_MCS1, > - MGN_VHT1SS_MCS2, > - MGN_VHT1SS_MCS3, > - MGN_VHT1SS_MCS4, > - MGN_VHT1SS_MCS5, > - MGN_VHT1SS_MCS6, > - MGN_VHT1SS_MCS7, > - MGN_VHT1SS_MCS8, > - MGN_VHT1SS_MCS9, > - MGN_VHT2SS_MCS0, > - MGN_VHT2SS_MCS1, > - MGN_VHT2SS_MCS2, > - MGN_VHT2SS_MCS3, > - MGN_VHT2SS_MCS4, > - MGN_VHT2SS_MCS5, > - MGN_VHT2SS_MCS6, > - MGN_VHT2SS_MCS7, > - MGN_VHT2SS_MCS8, > - MGN_VHT2SS_MCS9, > - MGN_VHT3SS_MCS0, > - MGN_VHT3SS_MCS1, > - MGN_VHT3SS_MCS2, > - MGN_VHT3SS_MCS3, > - MGN_VHT3SS_MCS4, > - MGN_VHT3SS_MCS5, > - MGN_VHT3SS_MCS6, > - MGN_VHT3SS_MCS7, > - MGN_VHT3SS_MCS8, > - MGN_VHT3SS_MCS9, > - MGN_VHT4SS_MCS0, > - MGN_VHT4SS_MCS1, > - MGN_VHT4SS_MCS2, > - MGN_VHT4SS_MCS3, > - MGN_VHT4SS_MCS4, > - MGN_VHT4SS_MCS5, > - MGN_VHT4SS_MCS6, > - MGN_VHT4SS_MCS7, > - MGN_VHT4SS_MCS8, > - MGN_VHT4SS_MCS9, > MGN_UNKNOWN > }; > > #define IS_HT_RATE(_rate) (_rate >= MGN_MCS0 && _rate <= MGN_MCS31) > -#define IS_VHT_RATE(_rate) (_rate >= MGN_VHT1SS_MCS0 && _rate <= MGN_VHT4SS_MCS9) > #define IS_CCK_RATE(_rate) (MGN_1M == _rate || _rate == MGN_2M || _rate == MGN_5_5M || _rate == MGN_11M) > #define IS_OFDM_RATE(_rate) (MGN_6M <= _rate && _rate <= MGN_54M && _rate != MGN_11M) > > @@ -625,7 +581,6 @@ enum { > RTW_WLAN_CATEGORY_TDLS = 12, > RTW_WLAN_CATEGORY_SELF_PROTECTED = 15, /* add for CONFIG_IEEE80211W, none 11w also can use */ > RTW_WLAN_CATEGORY_WMM = 17, > - RTW_WLAN_CATEGORY_VHT = 21, > RTW_WLAN_CATEGORY_P2P = 0x7f,/* P2P action frames */ > }; > > diff --git a/drivers/staging/rtl8723bs/include/rtl8723b_xmit.h b/drivers/staging/rtl8723bs/include/rtl8723b_xmit.h > index 56bdc14af47d..9dd329a5208a 100644 > --- a/drivers/staging/rtl8723bs/include/rtl8723b_xmit.h > +++ b/drivers/staging/rtl8723bs/include/rtl8723b_xmit.h > @@ -402,27 +402,6 @@ struct txdesc_8723b { > #define DESC8723B_RATEMCS13 0x19 > #define DESC8723B_RATEMCS14 0x1a > #define DESC8723B_RATEMCS15 0x1b > -#define DESC8723B_RATEVHTSS1MCS0 0x2c > -#define DESC8723B_RATEVHTSS1MCS1 0x2d > -#define DESC8723B_RATEVHTSS1MCS2 0x2e > -#define DESC8723B_RATEVHTSS1MCS3 0x2f > -#define DESC8723B_RATEVHTSS1MCS4 0x30 > -#define DESC8723B_RATEVHTSS1MCS5 0x31 > -#define DESC8723B_RATEVHTSS1MCS6 0x32 > -#define DESC8723B_RATEVHTSS1MCS7 0x33 > -#define DESC8723B_RATEVHTSS1MCS8 0x34 > -#define DESC8723B_RATEVHTSS1MCS9 0x35 > -#define DESC8723B_RATEVHTSS2MCS0 0x36 > -#define DESC8723B_RATEVHTSS2MCS1 0x37 > -#define DESC8723B_RATEVHTSS2MCS2 0x38 > -#define DESC8723B_RATEVHTSS2MCS3 0x39 > -#define DESC8723B_RATEVHTSS2MCS4 0x3a > -#define DESC8723B_RATEVHTSS2MCS5 0x3b > -#define DESC8723B_RATEVHTSS2MCS6 0x3c > -#define DESC8723B_RATEVHTSS2MCS7 0x3d > -#define DESC8723B_RATEVHTSS2MCS8 0x3e > -#define DESC8723B_RATEVHTSS2MCS9 0x3f > - > > #define RX_HAL_IS_CCK_RATE_8723B(pDesc)\ > (GET_RX_STATUS_DESC_RX_RATE_8723B(pDesc) == DESC8723B_RATE1M ||\ > diff --git a/drivers/staging/rtl8723bs/include/rtw_ht.h b/drivers/staging/rtl8723bs/include/rtw_ht.h > index e3f353fe1e47..1527d8be2d7a 100644 > --- a/drivers/staging/rtl8723bs/include/rtw_ht.h > +++ b/drivers/staging/rtl8723bs/include/rtw_ht.h > @@ -42,10 +42,6 @@ enum { > HT_AGG_SIZE_16K = 1, > HT_AGG_SIZE_32K = 2, > HT_AGG_SIZE_64K = 3, > - VHT_AGG_SIZE_128K = 4, > - VHT_AGG_SIZE_256K = 5, > - VHT_AGG_SIZE_512K = 6, > - VHT_AGG_SIZE_1024K = 7, > }; > > enum { >