All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/56] staging: r8188eu: wifi.h macros refactoring
@ 2022-01-03 19:01 ` Alberto Merciai
  0 siblings, 0 replies; 118+ messages in thread
From: Alberto Merciai @ 2022-01-03 19:01 UTC (permalink / raw)
  Cc: alb3rt0.m3rciai, linuxfancy, Larry Finger, Phillip Potter,
	Greg Kroah-Hartman, Michael Straube, Martin Kaiser,
	Fabio M. De Francesco, Nathan Chancellor, Dan Carpenter,
	Saurav Girepunje, Ivan Safonov, Christophe JAILLET, Yang Li,
	Zameer Manji, linux-staging, linux-kernel

This patch series refactors most of the macros inside
include/wifi.h making them compliant with linux coding style.

Main changes are related to the following issues:
- Avoid CamelCase
- Remove dead code
- Macros with complex values should be enclosed in parentheses

Alberto Merciai (56):
  staging: r8188eu: add parenthesis to macro SetToDs
  staging: r8188eu: rename camelcase SetToDs to set_to_ds
  staging: r8188eu: remove dead macro ClearToDs
  staging: r8188eu: add parenthesis to macro SetFrDs
  staging: r8188eu: rename camelcase SetFrDs to set_fr_ds
  staging: r8188eu: remove dead macro ClearFrDs
  staging: r8188eu: add parenthesis to macro SetMFrag
  staging: r8188eu: rename camelcase SetMFrag to set_m_frag
  staging: r8188eu: rename camelcase GetToDs to get_to_ds
  staging: r8188eu: rename camelcase GetFrDs to get_fr_ds
  staging: r8188eu: rename camelcase GetMFrag to get_m_frag
  staging: r8188eu: rename camelcase ClearMFrag to clear_m_frag
  staging: r8188eu: add parenthesis to macro clear_m_frag
  staging: r8188eu: remove dead macro SetRetry
  staging: r8188eu: rename camelcase GetRetry to get_retry
  staging: r8188eu: remove dead macro ClearRetry
  staging: r8188eu: rename camelcase SetPwrMgt to set_pwr_mgt
  staging: r8188eu: add parenthesis to macro set_pwr_mgt
  staging: r8188eu: rename camelcase GetPwrMgt to get_pwr_mgt
  staging: r8188eu: remove dead macro ClearPwrMgt
  staging: r8188eu: rename camelcase SetMData to set_m_data
  staging: r8188eu: add parenthesis to macro set_m_data
  staging: r8188eu: rename camelcase GetMData to get_m_data
  staging: r8188eu: remove dead macro ClearMData
  staging: r8188eu: rename camelcase SetPrivacy to set_privacy
  staging: r8188eu: add parenthesis to macro set_privacy
  staging: r8188eu: rename camelcase GetPrivacy to get_privacy
  staging: r8188eu: remove dead macro ClearPrivacy
  staging: r8188eu: rename camelcase GetOrder to get_order
  staging: r8188eu: rename camelcase GetFrameType to get_frame_type
  staging: r8188eu: remove dead macro SetFrameType
  staging: r8188eu: rename camelcase GetFrameSubType to
    get_frame_subtype
  staging: r8188eu: rename camelcase SetFrameSubType to
    set_frame_subtype
  staging: r8188eu: rename camelcase GetSequence to get_sequence
  staging: r8188eu: rename camelcase GetFragNum to get_frag_num
  staging: r8188eu: remove dead macro GetTupleCache
  staging: r8188eu: remove dead macro SetFragNum
  staging: r8188eu: rename camelcase SetSeqNum to set_seq_num
  staging: r8188eu: rename camelcase SetDuration to set_duration
  staging: r8188eu: rename camelcase SetPriority to set_priority
  staging: r8188eu: add parenthesis to macro set_duration
  staging: r8188eu: add parenthesis to macro set_priority
  staging: r8188eu: rename camelcase GetPriority to get_priority
  staging: r8188eu: rename camelcase SetEOSP to set_eosp
  staging: r8188eu: add parenthesis to macro set_eosp
  staging: r8188eu: remove dead macro GetTid
  staging: r8188eu: rename camelcase SetAckpolicy to set_ack_policy
  staging: r8188eu: add parenthesis to macro set_ack_policy
  staging: r8188eu: rename camelcase GetAckpolicy into get_ack_policy
  staging: r8188eu: rename camelcase GetAMsdu to get_a_msdu
  staging: r8188eu: remove dead macro SetAMsdu
  staging: r8188eu: rename camelcase GetAid to get_aid
  staging: r8188eu: rename camelcase GetAddr1Ptr to get_addr_1_ptr
  staging: r8188eu: rename camelcase GetAddr2Ptr to get_addr_2_ptr
  staging: r8188eu: rename camelcase GetAddr3Ptr to get_addr_3_ptr
  staging: r8188eu: rename camelcase GetAddr4Ptr to get_addr_4_ptr

 drivers/staging/r8188eu/core/rtw_mlme_ext.c   | 190 +++++++++---------
 drivers/staging/r8188eu/core/rtw_p2p.c        |  24 +--
 drivers/staging/r8188eu/core/rtw_recv.c       |  78 +++----
 drivers/staging/r8188eu/core/rtw_security.c   |   8 +-
 drivers/staging/r8188eu/core/rtw_wlan_util.c  |   4 +-
 drivers/staging/r8188eu/core/rtw_xmit.c       |  22 +-
 drivers/staging/r8188eu/hal/rtl8188e_cmd.c    |  30 +--
 drivers/staging/r8188eu/hal/rtl8188e_rxdesc.c |   2 +-
 drivers/staging/r8188eu/include/wifi.h        | 167 ++++++---------
 9 files changed, 240 insertions(+), 285 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 118+ messages in thread

end of thread, other threads:[~2022-01-08 13:53 UTC | newest]

Thread overview: 118+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-03 19:01 [PATCH 00/56] staging: r8188eu: wifi.h macros refactoring Alberto Merciai
2022-01-03 19:01 ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 01/56] staging: r8188eu: add parenthesis to macro SetToDs Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-06 14:12   ` Greg Kroah-Hartman
2022-01-03 19:01 ` [PATCH 02/56] staging: r8188eu: rename camelcase SetToDs to set_to_ds Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-06 14:13   ` Greg Kroah-Hartman
2022-01-03 19:01 ` [PATCH 03/56] staging: r8188eu: remove dead macro ClearToDs Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-06 14:14   ` Greg Kroah-Hartman
2022-01-08 13:53     ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 04/56] staging: r8188eu: add parenthesis to macro SetFrDs Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 05/56] staging: r8188eu: rename camelcase SetFrDs to set_fr_ds Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 06/56] staging: r8188eu: remove dead macro ClearFrDs Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 07/56] staging: r8188eu: add parenthesis to macro SetMFrag Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 08/56] staging: r8188eu: rename camelcase SetMFrag to set_m_frag Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 09/56] staging: r8188eu: rename camelcase GetToDs to get_to_ds Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 10/56] staging: r8188eu: rename camelcase GetFrDs to get_fr_ds Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 11/56] staging: r8188eu: rename camelcase GetMFrag to get_m_frag Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 12/56] staging: r8188eu: rename camelcase ClearMFrag to clear_m_frag Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 13/56] staging: r8188eu: add parenthesis to macro clear_m_frag Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 14/56] staging: r8188eu: remove dead macro SetRetry Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 15/56] staging: r8188eu: rename camelcase GetRetry to get_retry Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 16/56] staging: r8188eu: remove dead macro ClearRetry Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 17/56] staging: r8188eu: rename camelcase SetPwrMgt to set_pwr_mgt Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 18/56] staging: r8188eu: add parenthesis to macro set_pwr_mgt Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 19/56] staging: r8188eu: rename camelcase GetPwrMgt to get_pwr_mgt Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 20/56] staging: r8188eu: remove dead macro ClearPwrMgt Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 21/56] staging: r8188eu: rename camelcase SetMData to set_m_data Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 22/56] staging: r8188eu: add parenthesis to macro set_m_data Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 23/56] staging: r8188eu: rename camelcase GetMData to get_m_data Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:01 ` [PATCH 24/56] staging: r8188eu: remove dead macro ClearMData Alberto Merciai
2022-01-03 19:01   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 25/56] staging: r8188eu: rename camelcase SetPrivacy to set_privacy Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 26/56] staging: r8188eu: add parenthesis to macro set_privacy Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 27/56] staging: r8188eu: rename camelcase GetPrivacy to get_privacy Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 28/56] staging: r8188eu: remove dead macro ClearPrivacy Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 29/56] staging: r8188eu: rename camelcase GetOrder to get_order Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 30/56] staging: r8188eu: rename camelcase GetFrameType to get_frame_type Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 31/56] staging: r8188eu: remove dead macro SetFrameType Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 32/56] staging: r8188eu: rename camelcase GetFrameSubType to get_frame_subtype Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 33/56] staging: r8188eu: rename camelcase SetFrameSubType to set_frame_subtype Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 34/56] staging: r8188eu: rename camelcase GetSequence to get_sequence Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 35/56] staging: r8188eu: rename camelcase GetFragNum to get_frag_num Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 36/56] staging: r8188eu: remove dead macro GetTupleCache Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 37/56] staging: r8188eu: remove dead macro SetFragNum Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 38/56] staging: r8188eu: rename camelcase SetSeqNum to set_seq_num Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 39/56] staging: r8188eu: rename camelcase SetDuration to set_duration Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 40/56] staging: r8188eu: rename camelcase SetPriority to set_priority Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 41/56] staging: r8188eu: add parenthesis to macro set_duration Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 42/56] staging: r8188eu: add parenthesis to macro set_priority Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 43/56] staging: r8188eu: rename camelcase GetPriority to get_priority Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 44/56] staging: r8188eu: rename camelcase SetEOSP to set_eosp Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 45/56] staging: r8188eu: add parenthesis to macro set_eosp Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 46/56] staging: r8188eu: remove dead macro GetTid Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 47/56] staging: r8188eu: rename camelcase SetAckpolicy to set_ack_policy Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 48/56] staging: r8188eu: add parenthesis to macro set_ack_policy Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 49/56] staging: r8188eu: rename camelcase GetAckpolicy into get_ack_policy Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 50/56] staging: r8188eu: rename camelcase GetAMsdu to get_a_msdu Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 51/56] staging: r8188eu: remove dead macro SetAMsdu Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 52/56] staging: r8188eu: rename camelcase GetAid to get_aid Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 53/56] staging: r8188eu: rename camelcase GetAddr1Ptr to get_addr_1_ptr Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 54/56] staging: r8188eu: rename camelcase GetAddr2Ptr to get_addr_2_ptr Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 55/56] staging: r8188eu: rename camelcase GetAddr3Ptr to get_addr_3_ptr Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai
2022-01-03 19:02 ` [PATCH 56/56] staging: r8188eu: rename camelcase GetAddr4Ptr to get_addr_4_ptr Alberto Merciai
2022-01-03 19:02   ` Alberto Merciai

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.