Re: [PATCH 00/28] staging: rtl8188eu: remove DBG_88E callers and macro

From: Phillip Potter <phil@philpotter.co.uk>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Larry.Finger@lwfinger.net, dan.carpenter@oracle.com,
	linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev
Subject: Re: [PATCH 00/28] staging: rtl8188eu: remove DBG_88E callers and macro
Date: Tue, 15 Jun 2021 22:05:54 +0100	[thread overview]
Message-ID: <YMkWMsE/m99cM47/@KernelVM> (raw)
In-Reply-To: <YMiVtgVTGQSb+iC2@kroah.com>

On Tue, Jun 15, 2021 at 01:57:42PM +0200, Greg KH wrote:
> On Tue, Jun 15, 2021 at 01:14:39AM +0100, Phillip Potter wrote:
> > This patch series removes DBG_88E calls from all places in the driver,
> > and then removes the DBG_88E macro definition itself. It also takes two
> > prior patches where I convered DBG_88E calls into netdev_dbg calls, and
> > removes these calls too.
> > 
> > It cleans up subsuently unused local variables along the way, removes an
> > unused static function too, and also contains a reissue of my previous
> > patch to remove core/rtw_debug.c which is entirely unused by the driver
> > and thus is dead code.
> > 
> > Phillip Potter (28):
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_efuse.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_xmit.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_mlme_ext.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_wlan_util.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_ap.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_pwrctrl.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_ieee80211.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_security.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_ioctl_set.c
> >   staging: rtl8188eu: remove all DBG_88E calls from core/rtw_mlme.c
> >   staging: rtl8188eu: remove all DBG_88E calls from os_dep/ioctl_linux.c
> >   staging: rtl8188eu: remove all DBG_88E calls from os_dep/usb_intf.c
> >   staging: rtl8188eu: remove all DBG_88E calls from os_dep/os_intfs.c
> >   staging: rtl8188eu: remove all DBG_88E calls from os_dep/mlme_linux.c
> >   staging: rtl8188eu: remove all DBG_88E calls from os_dep/xmit_linux.c
> >   staging: rtl8188eu: remove all DBG_88E calls from os_dep/rtw_android.c
> >   staging: rtl8188eu: remove all DBG_88E calls from hal/hal_intf.c
> >   staging: rtl8188eu: remove all DBG_88E calls from hal/rtl8188e_cmd.c
> >   staging: rtl8188eu: remove all DBG_88E calls from hal/pwrseqcmd.c
> >   staging: rtl8188eu: remove all DBG_88E calls from
> >     hal/rtl8188e_hal_init.c
> >   staging: rtl8188eu: remove all DBG_88E calls from hal/rtl8188e_xmit.c
> >   staging: rtl8188eu: remove all DBG_88E calls from hal/usb_halinit.c
> >   staging: rtl8188eu: remove all DBG_88E calls from hal/rtl8188eu_recv.c
> >   staging: rtl8188eu: remove all DBG_88E calls from hal/rtl8188eu_xmit.c
> >   staging: rtl8188eu: remove converted netdev_dbg calls from
> >     core/rtw_recv.c
> >   staging: rtl8188eu: remove converted netdev_dbg calls from
> >     core/rtw_sta_mgt.c
> >   staging: rtl8188eu: remove core/rtw_debug.c
> >   staging: rtl8188eu: remove DBG_88E macro definition
> > 
> >  drivers/staging/rtl8188eu/Makefile            |   1 -
> >  drivers/staging/rtl8188eu/core/rtw_ap.c       |  78 +-----
> >  drivers/staging/rtl8188eu/core/rtw_debug.c    | 187 -------------
> >  drivers/staging/rtl8188eu/core/rtw_efuse.c    |  15 +-
> >  .../staging/rtl8188eu/core/rtw_ieee80211.c    |  38 +--
> >  .../staging/rtl8188eu/core/rtw_ioctl_set.c    |   8 -
> >  drivers/staging/rtl8188eu/core/rtw_mlme.c     |  54 +---
> >  drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 262 ++----------------
> >  drivers/staging/rtl8188eu/core/rtw_pwrctrl.c  |  40 +--
> >  drivers/staging/rtl8188eu/core/rtw_recv.c     |  66 +----
> >  drivers/staging/rtl8188eu/core/rtw_security.c |   2 -
> >  drivers/staging/rtl8188eu/core/rtw_sta_mgt.c  |   9 -
> >  .../staging/rtl8188eu/core/rtw_wlan_util.c    |  80 ++----
> >  drivers/staging/rtl8188eu/core/rtw_xmit.c     |  40 +--
> >  drivers/staging/rtl8188eu/hal/hal_intf.c      |   3 -
> >  drivers/staging/rtl8188eu/hal/pwrseqcmd.c     |   4 +-
> >  drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c  |  42 +--
> >  .../staging/rtl8188eu/hal/rtl8188e_hal_init.c |  45 +--
> >  drivers/staging/rtl8188eu/hal/rtl8188e_xmit.c |  15 -
> >  .../staging/rtl8188eu/hal/rtl8188eu_recv.c    |   8 -
> >  .../staging/rtl8188eu/hal/rtl8188eu_xmit.c    |   5 -
> >  drivers/staging/rtl8188eu/hal/usb_halinit.c   |  51 +---
> >  drivers/staging/rtl8188eu/include/rtw_debug.h |  31 ---
> >  .../staging/rtl8188eu/os_dep/ioctl_linux.c    | 144 +---------
> >  drivers/staging/rtl8188eu/os_dep/mlme_linux.c |   4 -
> >  drivers/staging/rtl8188eu/os_dep/os_intfs.c   |  26 --
> >  .../staging/rtl8188eu/os_dep/rtw_android.c    |  15 +-
> >  drivers/staging/rtl8188eu/os_dep/usb_intf.c   |   1 -
> >  drivers/staging/rtl8188eu/os_dep/xmit_linux.c |   8 +-
> >  29 files changed, 86 insertions(+), 1196 deletions(-)
> >  delete mode 100644 drivers/staging/rtl8188eu/core/rtw_debug.c
> 
> Deleting that many lines of code is always nice, thanks for doing this,
> now all applied.
> 
> greg k-h

Dear Greg,

Thanks for taking the patches, happy to help.

Regards,
Phil