mbox series

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

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

Message

Tree Davies Jan. 24, 2024, 10:44 p.m. UTC 
  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


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. 26, 2024, 6:23 p.m. UTC | #1 
On 1/24/24 23:44, Tree Davies wrote:
> 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
> 

On Patch 2, 5, 6, 17
WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit 
description?)

On Patch 13
WARNING: 'unecessary' may be misspelled - perhaps 'unnecessary'?

Is working fine on hardware.

Bye Philipp
Tree Davies" <tdavies@darkphysics.net Jan. 26, 2024, 10:05 p.m. UTC | #2 
On Fri, Jan 26, 2024 at 07:23:35PM +0100, Philipp Hortmann wrote:
> On 1/24/24 23:44, Tree Davies wrote:
> > 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
> > 
> 
> On Patch 2, 5, 6, 17
> WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit
> description?)
> 
> On Patch 13
> WARNING: 'unecessary' may be misspelled - perhaps 'unnecessary'?
> 
> Is working fine on hardware.
> 
> Bye Philipp

Ah, Thank you. Will send a v2.
~Tree