staging: r8188eu: remove function _rtw_zvmalloc
authorPhillip Potter <phil@philpotter.co.uk>
Wed, 18 Aug 2021 23:48:52 +0000 (00:48 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Aug 2021 05:42:57 +0000 (07:42 +0200)
Remove the function _rtw_zvmalloc from os_dep/osdep_service.c, as this
function is now unused and is just an inline wrapper around vmalloc
which zeroes out the memory. All previous callers have been converted to
use vzalloc. Also remove the declaration from include/osdep_service.h.

It is considered generally bad practice to declare functions as inline in
the majority of cases, as not only can this qualifier be ignored by the
compiler but the compiler generally makes good decisions about inlining
anyway.

Signed-off-by: Phillip Potter <phil@philpotter.co.uk>
Link: https://lore.kernel.org/r/20210818234853.208448-7-phil@philpotter.co.uk
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/include/osdep_service.h
drivers/staging/r8188eu/os_dep/osdep_service.c

index dba7a7bec9d11b30c9d6ffe3bb8c0b7e8f0b02f8..bda435ca9d080de716e2e9cbef3f11e72c7e4e14 100644 (file)
@@ -171,7 +171,6 @@ extern unsigned char WPA_TKIP_CIPHER[4];
 extern unsigned char RSN_TKIP_CIPHER[4];
 
 #define rtw_update_mem_stat(flag, sz) do {} while (0)
-u8 *_rtw_zvmalloc(u32 sz);
 
 void *rtw_malloc2d(int h, int w, int size);
 
index 8a797bfcb7f50159adfe28f1bcd7e5fbfa3aa599..95ac6086370ba227e2753beb22b958a197122a41 100644 (file)
@@ -36,15 +36,6 @@ u32 rtw_atoi(u8 *s)
        return num;
 }
 
-inline u8 *_rtw_zvmalloc(u32 sz)
-{
-       u8      *pbuf;
-       pbuf = vmalloc(sz);
-       if (pbuf)
-               memset(pbuf, 0, sz);
-       return pbuf;
-}
-
 void *rtw_malloc2d(int h, int w, int size)
 {
        int j;