linux-kernel - Re: [PATCH] staging: rtl8192e: remove 5G related code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <6411c15d-9e22-5a58-0e1b-d18654acc127@gmail.com>
Date:   Mon, 19 Jun 2023 22:04:41 +0200
From:   Philipp Hortmann <philipp.g.hortmann@...il.com>
To:     Michael Straube <straube.linux@...il.com>,
        gregkh@...uxfoundation.org
Cc:     linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: rtl8192e: remove 5G related code

On 6/19/23 17:52, Michael Straube wrote:
> In previous patches we removed 5G code since the hardware that uses this
> driver does not support 5G. There is still some 5G related code, remove it.
> All the removed defines are unused and we can safely remove "N-5G" from the
> rtllib_modes array.
> 
> Signed-off-by: Michael Straube <straube.linux@...il.com>
> ---
> Compile-tested only, due to lack of hardware.
> 
>   drivers/staging/rtl8192e/rtl8192e/r8192E_hw.h     | 1 -
>   drivers/staging/rtl8192e/rtl8192e/r8192E_phyreg.h | 2 --
>   drivers/staging/rtl8192e/rtllib.h                 | 1 -
>   drivers/staging/rtl8192e/rtllib_wx.c              | 2 +-
>   4 files changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/drivers/staging/rtl8192e/rtl8192e/r8192E_hw.h b/drivers/staging/rtl8192e/rtl8192e/r8192E_hw.h
> index f4d4b01630a2..09f8c76b7e65 100644
> --- a/drivers/staging/rtl8192e/rtl8192e/r8192E_hw.h
> +++ b/drivers/staging/rtl8192e/rtl8192e/r8192E_hw.h
> @@ -159,7 +159,6 @@ enum _RTL8192PCI_HW {
>   	WFCRC2		  = 0x2f8,
>   
>   	BW_OPMODE		= 0x300,
> -#define	BW_OPMODE_5G			BIT1
>   #define	BW_OPMODE_20MHZ			BIT2
>   	IC_VERRSION		= 0x301,
>   	MSR			= 0x303,
> diff --git a/drivers/staging/rtl8192e/rtl8192e/r8192E_phyreg.h b/drivers/staging/rtl8192e/rtl8192e/r8192E_phyreg.h
> index 65365ebe4e85..24fb0ca539ea 100644
> --- a/drivers/staging/rtl8192e/rtl8192e/r8192E_phyreg.h
> +++ b/drivers/staging/rtl8192e/rtl8192e/r8192E_phyreg.h
> @@ -266,7 +266,6 @@
>   #define b3WireAddressLength		0x400
>   #define b3WireRFPowerDown		0x1
>   /*#define bHWSISelect			0x8 */
> -#define b5GPAPEPolarity			0x40000000
>   #define b2GPAPEPolarity			0x80000000
>   #define bRFSW_TxDefaultAnt		0x3
>   #define bRFSW_TxOptionAnt		0x30
> @@ -284,7 +283,6 @@
>   #define bRFSI_ANTSW			0x100
>   #define bRFSI_ANTSWB			0x200
>   #define bRFSI_PAPE			0x400
> -#define bRFSI_PAPE5G			0x800
>   #define bBandSelect			0x1
>   #define bHTSIG2_GI			0x80
>   #define bHTSIG2_Smoothing		0x01
> diff --git a/drivers/staging/rtl8192e/rtllib.h b/drivers/staging/rtl8192e/rtllib.h
> index dbf78b60dd78..e3ce4431d460 100644
> --- a/drivers/staging/rtl8192e/rtllib.h
> +++ b/drivers/staging/rtl8192e/rtllib.h
> @@ -427,7 +427,6 @@ enum wireless_mode {
>   	WIRELESS_MODE_G = 0x04,
>   	WIRELESS_MODE_AUTO = 0x08,
>   	WIRELESS_MODE_N_24G = 0x10,
> -	WIRELESS_MODE_N_5G = 0x20
>   };
>   
>   #ifndef ETH_P_PAE
> diff --git a/drivers/staging/rtl8192e/rtllib_wx.c b/drivers/staging/rtl8192e/rtllib_wx.c
> index 538cedb1dc02..e9469bfef3dd 100644
> --- a/drivers/staging/rtl8192e/rtllib_wx.c
> +++ b/drivers/staging/rtl8192e/rtllib_wx.c
> @@ -19,7 +19,7 @@
>   #include "rtllib.h"
>   
>   static const char * const rtllib_modes[] = {
> -	"a", "b", "g", "?", "N-24G", "N-5G"
> +	"a", "b", "g", "?", "N-24G"
>   };
>   
>   #define MAX_CUSTOM_LEN 64

Tested-by: Philipp Hortmann <philipp.g.hortmann@...il.com>