mbox series

[0/4] staging: rtl8192e: remove rf_type from r8192_priv

Message ID 20230325132823.22872-1-straube.linux@gmail.com
Headers
Series staging: rtl8192e: remove rf_type from r8192_priv |

Message

Michael Straube March 25, 2023, 1:28 p.m. UTC 
  This series removes the rf_type field from the r8192_priv structure.
RF type is always RF_1T2R, so we do not need to store it in an extra
variable.

I don't have any hardware to test this, so compile-tested only.

Michael Straube (4):
  staging: rtl8192e: remove RTL819X_DEFAULT_RF_TYPE
  staging: rtl8192e: remove redundant setting of rf_type
  staging: rtl8192e: priv->rf_type is always RF_1T2R
  staging: rtl8192e: remove rf_type from struct r8192_priv

 .../staging/rtl8192e/rtl8192e/r8192E_dev.c    | 19 +++-----------
 .../staging/rtl8192e/rtl8192e/r8192E_phy.c    |  7 ++----
 drivers/staging/rtl8192e/rtl8192e/rtl_core.h  |  3 ---
 drivers/staging/rtl8192e/rtl8192e/rtl_dm.c    | 25 +++++++------------
 4 files changed, 15 insertions(+), 39 deletions(-)

Comments

Philipp Hortmann March 25, 2023, 7:05 p.m. UTC | #1 
On 3/25/23 14:28, Michael Straube wrote:
> This series removes the rf_type field from the r8192_priv structure.
> RF type is always RF_1T2R, so we do not need to store it in an extra
> variable.
> 
> I don't have any hardware to test this, so compile-tested only.
> 
> Michael Straube (4):
>    staging: rtl8192e: remove RTL819X_DEFAULT_RF_TYPE
>    staging: rtl8192e: remove redundant setting of rf_type
>    staging: rtl8192e: priv->rf_type is always RF_1T2R
>    staging: rtl8192e: remove rf_type from struct r8192_priv
> 
>   .../staging/rtl8192e/rtl8192e/r8192E_dev.c    | 19 +++-----------
>   .../staging/rtl8192e/rtl8192e/r8192E_phy.c    |  7 ++----
>   drivers/staging/rtl8192e/rtl8192e/rtl_core.h  |  3 ---
>   drivers/staging/rtl8192e/rtl8192e/rtl_dm.c    | 25 +++++++------------
>   4 files changed, 15 insertions(+), 39 deletions(-)
> 
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>