staging: rtl8723bs: os_dep: remove two set but not used variables
authorYueHaibing <yuehaibing@huawei.com>
Fri, 26 Jul 2019 14:09:59 +0000 (22:09 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 30 Jul 2019 07:52:03 +0000 (09:52 +0200)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/staging/rtl8723bs//os_dep/osdep_service.c: In function 'rtw_buf_free':
drivers/staging/rtl8723bs//os_dep/osdep_service.c:321:6: warning:
 variable 'ori_len' set but not used [-Wunused-but-set-variable]
drivers/staging/rtl8723bs//os_dep/ioctl_linux.c: In function 'rtw_ioctl_wext_private':
drivers/staging/rtl8723bs//os_dep/ioctl_linux.c:4915:6: warning:
 variable 'num_priv' set but not used [-Wunused-but-set-variable]

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Link: https://lore.kernel.org/r/20190726140959.15008-1-yuehaibing@huawei.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
drivers/staging/rtl8723bs/os_dep/osdep_service.c

index 99e6b10..90c2997 100644 (file)
@@ -4912,7 +4912,6 @@ static int rtw_ioctl_wext_private(struct net_device *dev, union iwreq_data *wrq_
        s32 k;
        const iw_handler *priv;         /* Private ioctl */
        const struct iw_priv_args *priv_args;   /* Private ioctl description */
-       u32 num_priv;                           /* Number of ioctl */
        u32 num_priv_args;                      /* Number of descriptions */
        iw_handler handler;
        int temp;
@@ -4948,7 +4947,6 @@ static int rtw_ioctl_wext_private(struct net_device *dev, union iwreq_data *wrq_
 
        priv = rtw_private_handler;
        priv_args = rtw_private_args;
-       num_priv = ARRAY_SIZE(rtw_private_handler);
        num_priv_args = ARRAY_SIZE(rtw_private_args);
 
        if (num_priv_args == 0) {
index 62fdd24..25a8004 100644 (file)
@@ -318,13 +318,9 @@ error:
 
 void rtw_buf_free(u8 **buf, u32 *buf_len)
 {
-       u32 ori_len;
-
        if (!buf || !buf_len)
                return;
 
-       ori_len = *buf_len;
-
        if (*buf) {
                *buf_len = 0;
                kfree(*buf);