Re: [PATCH 0/7] staging: r8188eu: more odm cleanups

From: Martin Kaiser
Date: Tue Oct 19 2021 - 16:33:18 EST


Thus wrote Michael Straube (straube.linux@xxxxxxxxx):

> This series removes more unneeded and/or dead odm code.

> Tested on x86_64 with Inter-Tech DMG-02.

> Michael Straube (7):
> staging: r8188eu: remove duplicate structure
> staging: r8188eu: BTRxRSSIPercentage is set but never used
> staging: r8188eu: rename ODM_PhyStatusQuery_92CSeries()
> staging: r8188eu: remove unused cases from ODM_CmnInfo{Hook,Update}
> staging: r8188eu: remove unused fields from enum odm_common_info_def
> staging: r8188eu: remove unused enums and defines from odm.h
> staging: r8188eu: RFType type is always ODM_1T1R

> drivers/staging/r8188eu/hal/HalPhyRf_8188e.c | 82 +---------
> drivers/staging/r8188eu/hal/odm.c | 70 ++-------
> drivers/staging/r8188eu/hal/odm_HWConfig.c | 31 ++--
> drivers/staging/r8188eu/hal/rtl8188e_dm.c | 7 -
> drivers/staging/r8188eu/hal/rtl8188e_rxdesc.c | 2 +-
> drivers/staging/r8188eu/include/odm.h | 142 ------------------
> .../staging/r8188eu/include/odm_HWConfig.h | 2 +-
> drivers/staging/r8188eu/include/rtw_recv.h | 1 -
> 8 files changed, 26 insertions(+), 311 deletions(-)

> --
> 2.33.1

Hi Michael,

Looks good, thanks. For all patches:

Acked-by: Martin Kaiser <martin@xxxxxxxxx>