staging: rtl8723bs: convert private allocation functions to return void *
authorJoe Perches <joe@perches.com>
Sat, 9 Sep 2017 20:14:52 +0000 (13:14 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 17 Sep 2017 14:35:26 +0000 (16:35 +0200)
Using char * for a return from allocation functions means the
code has to cast generic allocations to specific types.

Allow the compiler to not need casts on the allocations.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/include/osdep_service.h
drivers/staging/rtl8723bs/os_dep/osdep_service.c

index ac9ffe0e3b8487b116fde36692ac3fcb8abcb76c..57523baa4517c704fba3283e92d7992ed9311b41 100644 (file)
@@ -96,8 +96,8 @@ typedef enum mstat_status{
 
 #define rtw_mstat_update(flag, status, sz) do {} while (0)
 #define rtw_mstat_dump(sel) do {} while (0)
-u8*_rtw_zmalloc(u32 sz);
-u8*_rtw_malloc(u32 sz);
+void *_rtw_zmalloc(u32 sz);
+void *_rtw_malloc(u32 sz);
 void _kfree(u8 *pbuf, u32 sz);
 
 struct sk_buff *_rtw_skb_alloc(u32 sz);
index a05daf06a870a92ae1b575937149c3edd9a97f2e..93747c2bce526cd9b3133ab62b3e4831c0b94a2c 100644 (file)
@@ -30,22 +30,17 @@ inline int RTW_STATUS_CODE(int error_code)
        return _FAIL;
 }
 
-u8 *_rtw_malloc(u32 sz)
+void *_rtw_malloc(u32 sz)
 {
-       u8 *pbuf = NULL;
-
-       pbuf = kmalloc(sz, in_interrupt() ? GFP_ATOMIC : GFP_KERNEL);
-
-       return pbuf;
+       return kmalloc(sz, in_interrupt() ? GFP_ATOMIC : GFP_KERNEL);
 }
 
-u8 *_rtw_zmalloc(u32 sz)
+void *_rtw_zmalloc(u32 sz)
 {
-       u8 *pbuf = _rtw_malloc(sz);
+       void *pbuf = _rtw_malloc(sz);
 
-       if (pbuf != NULL) {
+       if (pbuf)
                memset(pbuf, 0, sz);
-       }
 
        return pbuf;
 }