lkml.org 
[lkml]   [2022]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2 0/7] drop some unnecessary wrappers
Date
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



\
 
 \ /
  Last update: 2022-04-11 22:35    [W:1.438 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site