>> BTW, same can be done in rtw_usb_if1_init(). Yea, I noticed that too. But the NULL check in rtw_usb_if1_init() has a follow up "else if", so I didn't change it. Thanks, Vihas On Tue, Nov 23, 2021 at 2:52 AM Pavel Skripkin <paskripkin@xxxxxxxxx> wrote: > > On 11/22/21 22:53, Vihas Mak wrote: > > remove unnecessary NULL check surrounding rtw_free_netdev(), as the check > > is already performed inside rtw_free_netdev() in > > drivers/staging/r8188eu/os_dep/osdep_service.c. > > > > Signed-off-by: Vihas Mak <makvihas@xxxxxxxxx> > > Reviewed-by: Pavel Skripkin <paskripkin@xxxxxxxxx> > > BTW, same can be done in rtw_usb_if1_init(). > > > > With regards, > Pavel Skripkin