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 d4262a68dd4df8f24609d829c6b259d6d3f7d165..9fbd19f03ca9a6c7e36d75ce1230f60bad73a4c0 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 588346da1412fc02c284a0a6b8ca871890bb292b..add6c18195d6291c46f54ac5bda27a69891e808a 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;
 }