On lunedì 11 aprile 2022 12:21:29 CEST Vihas Makwana wrote: > Drop some unnecessary wrappers and update all the references > accordingly. > Tested on Comfast CF-WU810N RTL8188EUS wireless adapter. > > v1 -> v2: > Drop the wrapper functions with underscores prefixed. > > Vihas Makwana (7): > staging: r8188eu: drop unnecessary wrapper _rtw_free_cmd_priv > staging: r8188eu: drop unnecessary wrapper _rtw_init_cmd_priv > staging: r8188eu: drop unnecessary wrapper _rtw_init_evt_priv > staging: r8188eu: drop unnecessary wrapper _rtw_init_mlme_priv > staging: r8188eu: drop unnecessary wrapper _rtw_free_mlme_priv > staging: r8188eu: drop unnecessary wrapper _rtw_alloc_network > staging: r8188eu: drop unnecessary wrapper _rtw_dequeue_cmd > > drivers/staging/r8188eu/core/rtw_cmd.c | 145 +++++++---------- > drivers/staging/r8188eu/core/rtw_mlme.c | 179 +++++++++------------ > drivers/staging/r8188eu/include/rtw_mlme.h | 4 +- > 3 files changed, 135 insertions(+), 193 deletions(-) > > -- > 2.30.2 > Formally, you are removing the wrapped functions (or helpers, if you prefer) by moving their code into the wrappers. To say that you are removing the wrappers is not correct. However, I'm sure this kind of formal mistake won't prevent your patches to be accepted. I wanted to point this out to you in case you decide to submit similar patches in the future and perhaps want to describe your changes a little more accurately. Thanks, Fabio M. De Francesco