mbox series

[v2,00/18] Staging: rtl8192e: 18 Additional checkpatch fixes for rtllib_softmac.c

Message ID 20240126223106.986093-1-tdavies@darkphysics.net
Headers
Series Staging: rtl8192e: 18 Additional checkpatch fixes for rtllib_softmac.c |

Message

Tree Davies Jan. 26, 2024, 10:30 p.m. UTC 
  Version 2 

Another checkpatch fix series to be applied after the series titled:
'checkpatch fixes for rtllib_softmac.c' submitted by me, to the list, 
on 1/5/2024

Thank you in advance to all reviewers
regards,
Tree

Changes since v1:                                                               
- On Patches 2, 5, 6, 17                                                        
Update commit message, to fix max chars per line.                               
                                                                                
- On Patch 13                                                                   
Fixed spelling mistake in Subject line

Tree Davies (18):
  Staging: rtl8192e: Rename variable FirstIe_InScan
  Staging: rtl8192e: Rename function rtllib_rx_ADDBARsp()
  Staging: rtl8192e: Rename variable LPSAwakeIntvl_tmp
  Staging: rtl8192e: Rename variable LPSDelayCnt
  Staging: rtl8192e: Rename function pointer SetHwRegHandler()
  Staging: rtl8192e: Rename function MgntQuery_TxRateExcludeCCKRates()
  Staging: rtl8192e: Rename variable PeerHTCapBuf
  Staging: rtl8192e: Rename variable PeerHTInfoBuf
  Staging: rtl8192e: Rename variable LPSAwakeIntvl
  Staging: rtl8192e: Rename variable SelfHTCap
  Staging: rtl8192e: Fix paren alignment for
    rtllib_disable_net_monitor_mode()
  Staging: rtl8192e: Fixup if statement broken across multiple lines.
  Staging: rtl8192e: Remove unecessary blank line
  Staging: rtl8192e: Fix if statement alignment with open parenthesis
  Staging: rtl8192e: Rename variable Octet
  Staging: rtl8192e: Rename variable LpsIdleCount
  Staging: rtl8192e: Rename variable NumRecvBcnInPeriod
  Staging: rtl8192e: Rename variable bForcedBgMode

 .../staging/rtl8192e/rtl8192e/r8192E_dev.c    |  4 +-
 drivers/staging/rtl8192e/rtl8192e/rtl_core.c  | 18 ++--
 drivers/staging/rtl8192e/rtl8192e/rtl_dm.c    |  6 +-
 drivers/staging/rtl8192e/rtl8192e/rtl_ps.c    |  4 +-
 drivers/staging/rtl8192e/rtl8192e/rtl_wx.c    |  2 +-
 drivers/staging/rtl8192e/rtl819x_BAProc.c     |  2 +-
 drivers/staging/rtl8192e/rtl819x_HT.h         |  6 +-
 drivers/staging/rtl8192e/rtl819x_HTProc.c     | 32 +++----
 drivers/staging/rtl8192e/rtl819x_Qos.h        |  2 +-
 drivers/staging/rtl8192e/rtllib.h             | 18 ++--
 drivers/staging/rtl8192e/rtllib_rx.c          |  2 +-
 drivers/staging/rtl8192e/rtllib_softmac.c     | 96 +++++++++----------
 drivers/staging/rtl8192e/rtllib_softmac_wx.c  |  4 +-
 drivers/staging/rtl8192e/rtllib_tx.c          |  8 +-
 14 files changed, 100 insertions(+), 104 deletions(-)

Comments

Philipp Hortmann Jan. 27, 2024, 4:44 p.m. UTC | #1 
On 1/26/24 23:30, Tree Davies wrote:
> Version 2
> 
> Another checkpatch fix series to be applied after the series titled:
> 'checkpatch fixes for rtllib_softmac.c' submitted by me, to the list,
> on 1/5/2024
> 
> Thank you in advance to all reviewers
> regards,
> Tree


Is working fine on hardware.

Bye Philipp
Greg KH Jan. 27, 2024, 8:24 p.m. UTC | #2 
On Sat, Jan 27, 2024 at 05:44:58PM +0100, Philipp Hortmann wrote:
> On 1/26/24 23:30, Tree Davies wrote:
> > Version 2
> > 
> > Another checkpatch fix series to be applied after the series titled:
> > 'checkpatch fixes for rtllib_softmac.c' submitted by me, to the list,
> > on 1/5/2024
> > 
> > Thank you in advance to all reviewers
> > regards,
> > Tree
> 
> 
> Is working fine on hardware.

You might want to respond with:
	Tested-by: ....
so that our tools pick it up and you get added to the commit logs, if
you want that.

thanks,

greg k-h
Philipp Hortmann Jan. 29, 2024, 7:36 p.m. UTC | #3 
On 1/27/24 21:24, Greg KH wrote:
> You might want to respond with:
> 	Tested-by: ....
> so that our tools pick it up and you get added to the commit logs, if
> you want that.
> 
> thanks,
> 
> greg k-h


Hi Greg,

thanks for your hint. I am already on Position 6 on the world rank list 
for "Tested-by:..."

No.1	 Daniel Wheeler <daniel ! wheeler () amd ! com> 
2417(3.51%)	@AMD                             @Unknown
No.2	 Andrew Bowers <andrewx ! bowers () intel ! com> 
2386(3.46%)	@Intel                           @Unknown
No.3	 Arnaldo Carvalho de Melo <acme () redhat ! com> 
1300(1.89%)	@Red Hat                         @Brazilian
No.4	 Aaron Brown <aaron ! f ! brown () intel ! com> 
1206(1.75%)	@Intel                           @Unknown
No.5	 Phil Schmitt <phillip ! j ! schmitt () intel ! com> 
566(0.82%)	@Intel                           @Unknown
No.6	 Philipp Hortmann <philipp ! g ! hortmann () gmail ! com> 
537(0.78%)	@Hobbyists                       @German

I wonder if I deserve so much recognition. I thought you do not like me 
hunting for so easy credits. But it seems that I was wrong with this 
assumption.

Will use "Tested-by:..." in the future.

Thanks for your support.

Bye Philipp