staging: rtl8712: Removed redundant code from function oid_rt_pro_write_register_hdl
authorDeepak Mishra <linux.dkm@gmail.com>
Sun, 9 Jun 2019 12:31:41 +0000 (18:01 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 10 Jun 2019 15:57:21 +0000 (17:57 +0200)
In function oid_rt_pro_write_register_hdl, Adapter->ImrContent is
assigned with RegRWStruct->value but Adapter->ImrContent is never used
anywhere else. So those lines has no impact and are removed removed.

As that was the only place where ImrContent was used, so the member
variable is removed from the structure _adapter

Signed-off-by: Deepak Mishra <linux.dkm@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8712/drv_types.h
drivers/staging/rtl8712/rtl871x_mp_ioctl.c

index d4262a6..9fbd19f 100644 (file)
@@ -149,7 +149,6 @@ struct _adapter {
        bool    surprise_removed;
        bool    suspended;
        u32     IsrContent;
-       u32     ImrContent;
        u8      eeprom_address_size;
        u8      hw_init_completed;
        struct task_struct *cmdThread;
index 588346d..add6c18 100644 (file)
@@ -661,11 +661,6 @@ uint oid_rt_pro_write_register_hdl(struct oid_par_priv *poid_par_priv)
                        status = RNDIS_STATUS_NOT_ACCEPTED;
                        break;
                }
-
-               if ((status == RNDIS_STATUS_SUCCESS) &&
-                   (RegRWStruct->offset == HIMR) &&
-                   (RegRWStruct->width == 4))
-                       Adapter->ImrContent = RegRWStruct->value;
        }
        return status;
 }