Re: [PATCH v2 0/5] Trivial code cleanup patches

Greg Kroah-Hartman
2023-06-15T10:36:47Z

On Wed, Jun 07, 2023 at 08:31:19PM +0530, Yogesh Hegde wrote:
> These patches remove variables in the struct rtllib_device that were
> set only once throughout the driver. Instead of using these variables,
> the patches modify the relevant code to directly call the corresponding
> functions, eliminating unnecessary indirection and removing CamelCase.
> The patches are required to be applied in sequence.
>
> Yogesh Hegde (5):
> staging: rtl8192e: Remove variable SetWirelessMode
> staging: rtl8192e: Remove variable SetBWModeHandler
> staging: rtl8192e: Remove variable LeisurePSLeave
> staging: rtl8192e: Remove variable InitialGainHandler
> staging: rtl8192e: Remove DRV_NAME definition in rtllib_debug.h
>
> drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c | 4 ++--
> drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 4 ----
> drivers/staging/rtl8192e/rtl819x_HTProc.c | 8 ++++----
> drivers/staging/rtl8192e/rtllib.h | 6 ------
> drivers/staging/rtl8192e/rtllib_debug.h | 5 -----
> drivers/staging/rtl8192e/rtllib_rx.c | 3 ++-
> drivers/staging/rtl8192e/rtllib_softmac.c | 16 +++++++---------
> drivers/staging/rtl8192e/rtllib_softmac_wx.c | 9 +++++----
> 8 files changed, 20 insertions(+), 35 deletions(-)
>
> --
> 2.25.1
>
>

These break the build, as reported by the test robot, please fix up.

thanks,

greg k-h