> On Sun, 2022-04-10 at 22:01 +0530, Vihas Makwana wrote: > > Remove the unnecessary wrapper. The logic of rtw_free_cmd_priv can > > be moved to _rtw_free_cmd_priv. > > If you are going to drop wrappers, my preference would be > to drop the underscore prefixed ones. > Yes, that was my initial thought too. But there are already some init and free functions in the module with underscores prefixed. So I didn't do it that way. Maybe the developers kept it that way for some reason. On Sun, Apr 10, 2022 at 10:38 PM Joe Perches <joe@xxxxxxxxxxx> wrote: > > On Sun, 2022-04-10 at 22:01 +0530, Vihas Makwana wrote: > > Remove the unnecessary wrapper. The logic of rtw_free_cmd_priv can > > be moved to _rtw_free_cmd_priv. > > If you are going to drop wrappers, my preference would be > to drop the underscore prefixed ones. > > -- Thanks, Vihas