Re: [PATCH 00/33] Staging: rtl8723bs: remove unnecessary enum names and definitions

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Fri, Mar 26, 2021 at 07:09:03PM +0300, Dan Carpenter wrote:
> On Fri, Mar 26, 2021 at 04:37:54PM +0100, Marco Cesati wrote:
> > On Fri, Mar 26, 2021 at 03:04:08PM +0100, Greg Kroah-Hartman wrote:
> > > On Wed, Mar 24, 2021 at 01:44:23PM +0100, Marco Cesati wrote:
> > > > The staging/rtl8723bs driver includes 82 enum definitions whose
> > > > names are not referenced in the code. Furthermore, 57 enum definitions
> > > > are useless, because the enum elements do not appear in the code.
> > > > This patchset transforms 82 'named' enums in 'anonymous' enums and
> > > > removes 57 enum definitions.
> > > > 
> > > > 
> > > > [PATCH 01/33] Staging: rtl8723bs: remove named enums in rtw_mlme.h
> > > > [PATCH 02/33] Staging: rtl8723bs: remove named enums in rtw_cmd.h
> > > > [PATCH 03/33] Staging: rtl8723bs: remove named enums in rtw_eeprom.h
> > > > [PATCH 04/33] Staging: rtl8723bs: remove named enums in hal_com.h
> > > > [PATCH 05/33] Staging: rtl8723bs: remove named enums in rtw_recv.h
> > > > [PATCH 06/33] Staging: rtl8723bs: remove named enums in drv_types.h
> > > > [PATCH 07/33] Staging: rtl8723bs: remove named enums in rtw_ht.h
> > > > [PATCH 08/33] Staging: rtl8723bs: remove named enums in wlan_bssdef.h
> > > > [PATCH 09/33] Staging: rtl8723bs: remove named enums in rtw_mp.h
> > > > [PATCH 10/33] Staging: rtl8723bs: remove named enums in osdep_service.h
> > > > [PATCH 11/33] Staging: rtl8723bs: remove named enums in hal_intf.h
> > > > [PATCH 12/33] Staging: rtl8723bs: remove named enums in rtw_pwrctrl.h
> > > > [PATCH 13/33] Staging: rtl8723bs: remove named enums in rtl8723b_cmd.h
> > > > [PATCH 14/33] Staging: rtl8723bs: remove named enums in hal_com_h2c.h
> > > > [PATCH 15/33] Staging: rtl8723bs: remove named enums in rtw_wifi_regd.h
> > > > [PATCH 16/33] Staging: rtl8723bs: remove named enums in wifi.h
> > > > [PATCH 17/33] Staging: rtl8723bs: remove named enums in rtl8723b_hal.h
> > > > [PATCH 18/33] Staging: rtl8723bs: remove named enums in hal_phy.h
> > > > [PATCH 19/33] Staging: rtl8723bs: remove named enums in rtw_mlme_ext.h
> > > > [PATCH 20/33] Staging: rtl8723bs: remove named enums in rtw_xmit.h
> > > > [PATCH 21/33] Staging: rtl8723bs: remove named enums in rtw_rf.h
> > > > [PATCH 22/33] Staging: rtl8723bs: remove named enums in HalPwrSeqCmd.h
> > > > [PATCH 23/33] Staging: rtl8723bs: remove named enums in hal_com_phycfg.h
> > > > [PATCH 24/33] Staging: rtl8723bs: remove named enums in ieee80211.h
> > > > [PATCH 25/33] Staging: rtl8723bs: remove named enums in rtw_efuse.h
> > > > [PATCH 26/33] Staging: rtl8723bs: remove named enums in odm_interface.h
> > > > [PATCH 27/33] Staging: rtl8723bs: remove named enums in HalBtcOutSrc.h
> > > > [PATCH 28/33] Staging: rtl8723bs: remove named enums in HalBtc8723b1Ant.h
> > > > [PATCH 29/33] Staging: rtl8723bs: remove named enums in odm_types.h
> > > > [PATCH 30/33] Staging: rtl8723bs: remove named enums in odm_DIG.h
> > > > [PATCH 31/33] Staging: rtl8723bs: remove named enums in HalPhyRf.h
> > > > [PATCH 32/33] Staging: rtl8723bs: remove named enums in odm.h
> > > > [PATCH 33/33] Staging: rtl8723bs: remove named enums in HalBtc8723b2Ant.h
> > > 
> > > Was this a "v2" of the series?  Or does it not conflict with your other
> > > series?
> > > 
> > > I'm getting confused...  Please check my tree and see what I have not
> > > applied from you and resend...
> > > 
> > > thanks,
> > > greg k-h
> > 
> > This was the first, original patch set. However, I got a comment from
> > Dan Carpenter from which I inferred that transforming named enums like
> > "enum XXX { ... };" into anonymous enums like "enum { ...  };", when
> > "XXX" is not referenced elsewhere, was not desiderable. Therefore, I
> > submitted another version of the patch set ("[PATCH 00/21] Staging:
> > rtl8723bs: remove unreferenced enum definitions"), which just removes
> > the enums having no reference _at_all_ in the code. To be clear, the 
> > patch set ("remove unreferenced enums") removes a strict subset of the
> > enums touched by this patch set ("remove named enums").
> > 
> > This patch set ("remove named enums") has been properly applied to your
> > tree, so for me all is fine.  I guess the decision is up to you: do you
> > prefer to _not_ remove the useless enum names? In this case, this patch
> > set has to be reverted.
> 
> Forget about it.  I didn't really endorse removing the enum names
> because I thought they were harmless and had some documentation value.
> But the value was very minimal...  Let's not revert anything.  Let's
> just move forward.
> 
> You're over apologizing.  No one is upset with you if your patch doesn't
> apply.  Greg just deletes it and moves on.  No one invests any time into
> trying to figure out what happened.
> 
> Now that Greg has applied whatever can apply he has deleted all the rest
> of your patches.  Just rebase and resend.

Yup, nothing to worry about here, I took this patchset as it seemed
semi-sane and cleaned up the code.  Please rebase and resend your other
work as well and I will be glad to review them.

thanks,

greg k-h