mbox series

[v3,0/2] staging: rtl8192u: unused code cleanup

Message ID cover.1667398621.git.drv@mailo.com
Headers
Series staging: rtl8192u: unused code cleanup |

Message

Deepak R Varma Nov. 2, 2022, 2:37 p.m. UTC 
  Remove unnecessary or unused code.

Changes in v3:
   Patch-1:
      1. Already accepted and applied to the stating-testing tree. Hence not
         included in this patch set now.
   Patch-2:
      1. The patch caused staging tree build to fail due to EXPORT_SYMBOL
         conflict. Fixed the issue caused due to deletion of 2 macros.
         Correction and re-submission asked by gregkh@linuxfoundation.org

Changes in v2:
   Patch-1:
      1. Revise patch log to explain impact of the patch
         Suggested by gregkh@linuxfoundation.org
   Patch-2:
      1. Review other similar macro defines and clean those up as well.
         Suggested by gregkh@linuxfoundation.org
      2. Revise patch log to explain impact of the patch
         Suggested by julia.lawall@inria.fr


Deepak R Varma (1):
  staging: rtl8192u: remove redundant macro definitions

 .../staging/rtl8192u/ieee80211/ieee80211.h    | 61 -------------------
 1 file changed, 61 deletions(-)

--
2.30.2

Comments

Greg KH Nov. 8, 2022, 3:14 p.m. UTC | #1 
On Wed, Nov 02, 2022 at 08:07:52PM +0530, Deepak R Varma wrote:
> Remove unnecessary or unused code.
> 
> Changes in v3:
>    Patch-1:
>       1. Already accepted and applied to the stating-testing tree. Hence not
>          included in this patch set now.
>    Patch-2:
>       1. The patch caused staging tree build to fail due to EXPORT_SYMBOL
>          conflict. Fixed the issue caused due to deletion of 2 macros.
>          Correction and re-submission asked by gregkh@linuxfoundation.org

But now this patch set looks like it is missing a patch, which confuses
everything.

Please just send a new patch, v4, 1/1, and all will be better.

thanks,

greg k-h