staging: rtl8192e: Remove undefined function LedControlHandler
authorPhilipp Hortmann <philipp.g.hortmann@gmail.com>
Sat, 13 May 2023 18:09:40 +0000 (20:09 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 28 May 2023 09:11:12 +0000 (10:11 +0100)
Remove function LedControlHandler as it is not defined. Remove resulting
unused local variables bLedBlinking, type, fc and LedAction.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/e3c29c3db33d314dffb673f8e563da0fb1f2b22f.1683960685.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8192e/rtl8192e/r8192E_dev.c
drivers/staging/rtl8192e/rtl8192e/rtl_core.c
drivers/staging/rtl8192e/rtl8192e/rtl_wx.c
drivers/staging/rtl8192e/rtllib.h
drivers/staging/rtl8192e/rtllib_softmac.c

index c612e31d1a84fef81a0fee4fc354005c8b80ff79..d9d056d18fa360e27039aacb9fdeef450ece1d0c 100644 (file)
@@ -46,7 +46,6 @@ static void _rtl92e_update_msr(struct net_device *dev)
 {
        struct r8192_priv *priv = rtllib_priv(dev);
        u8 msr;
-       enum led_ctl_mode LedAction = LED_CTL_NO_LINK;
 
        msr  = rtl92e_readb(dev, MSR);
        msr &= ~MSR_LINK_MASK;
@@ -55,7 +54,6 @@ static void _rtl92e_update_msr(struct net_device *dev)
        case IW_MODE_INFRA:
                if (priv->rtllib->state == RTLLIB_LINKED)
                        msr |= MSR_LINK_MANAGED;
-               LedAction = LED_CTL_LINK;
                break;
        case IW_MODE_ADHOC:
                if (priv->rtllib->state == RTLLIB_LINKED)
@@ -70,8 +68,6 @@ static void _rtl92e_update_msr(struct net_device *dev)
        }
 
        rtl92e_writeb(dev, MSR, msr);
-       if (priv->rtllib->LedControlHandler)
-               priv->rtllib->LedControlHandler(dev, LedAction);
 }
 
 void rtl92e_set_reg(struct net_device *dev, u8 variable, u8 *val)
index 5fdee7b54f24be18aabeb42b1e5a6a69a1b392c9..17b70dde7eeb35f19d7b02d2fc7f26cc9f7f723a 100644 (file)
@@ -737,8 +737,6 @@ static void _rtl92e_init_priv_handler(struct net_device *dev)
        priv->rtllib->InitialGainHandler = rtl92e_init_gain;
        priv->rtllib->rtllib_ips_leave_wq = rtl92e_rtllib_ips_leave_wq;
        priv->rtllib->rtllib_ips_leave = rtl92e_rtllib_ips_leave;
-
-       priv->rtllib->LedControlHandler = NULL;
        priv->rtllib->ScanOperationBackupHandler = rtl92e_scan_op_backup;
 }
 
@@ -1272,10 +1270,6 @@ static void _rtl92e_watchdog_wq_cb(void *data)
                        ieee->is_roaming = true;
                        ieee->is_set_key = false;
                        ieee->link_change(dev);
-                       if (ieee->LedControlHandler)
-                               ieee->LedControlHandler(ieee->dev,
-                                                       LED_CTL_START_TO_LINK);
-
                        notify_wx_assoc_event(ieee);
 
                        if (!(ieee->rtllib_ap_sec_type(ieee) &
@@ -1506,7 +1500,6 @@ static short _rtl92e_tx(struct net_device *dev, struct sk_buff *skb)
                                    MAX_DEV_ADDR_SIZE);
        struct tx_desc *pdesc = NULL;
        struct rtllib_hdr_1addr *header = NULL;
-       u16 fc = 0, type = 0;
        u8 *pda_addr = NULL;
        int   idx;
        u32 fwinfo_size = 0;
@@ -1522,8 +1515,6 @@ static short _rtl92e_tx(struct net_device *dev, struct sk_buff *skb)
        fwinfo_size = sizeof(struct tx_fwinfo_8190pci);
 
        header = (struct rtllib_hdr_1addr *)(((u8 *)skb->data) + fwinfo_size);
-       fc = le16_to_cpu(header->frame_ctl);
-       type = WLAN_FC_GET_TYPE(fc);
        pda_addr = header->addr1;
 
        if (!is_broadcast_ether_addr(pda_addr) && !is_multicast_ether_addr(pda_addr))
@@ -1545,11 +1536,6 @@ static short _rtl92e_tx(struct net_device *dev, struct sk_buff *skb)
                spin_unlock_irqrestore(&priv->irq_th_lock, flags);
                return skb->len;
        }
-
-       if (type == RTLLIB_FTYPE_DATA) {
-               if (priv->rtllib->LedControlHandler)
-                       priv->rtllib->LedControlHandler(dev, LED_CTL_TX);
-       }
        rtl92e_fill_tx_desc(dev, pdesc, tcb_desc, skb);
        __skb_queue_tail(&ring->queue, skb);
        pdesc->OWN = 1;
@@ -1776,8 +1762,6 @@ static void _rtl92e_rx_normal(struct net_device *dev)
        struct r8192_priv *priv = rtllib_priv(dev);
        struct rtllib_hdr_1addr *rtllib_hdr = NULL;
        bool unicast_packet = false;
-       bool bLedBlinking = true;
-       u16 fc = 0, type = 0;
        u32 skb_len = 0;
        int rx_queue_idx = RX_MPDU_QUEUE;
 
@@ -1821,16 +1805,6 @@ static void _rtl92e_rx_normal(struct net_device *dev)
                        /* unicast packet */
                        unicast_packet = true;
                }
-               fc = le16_to_cpu(rtllib_hdr->frame_ctl);
-               type = WLAN_FC_GET_TYPE(fc);
-               if (type == RTLLIB_FTYPE_MGMT)
-                       bLedBlinking = false;
-
-               if (bLedBlinking)
-                       if (priv->rtllib->LedControlHandler)
-                               priv->rtllib->LedControlHandler(dev,
-                                                       LED_CTL_RX);
-
                skb_len = skb->len;
 
                if (!rtllib_rx(priv->rtllib, skb, &stats)) {
index 0bb657fda06c01c6472a8ae41296109e796ae553..8b656980ee25fc6d43a743c153d795898a9296ff 100644 (file)
@@ -421,10 +421,6 @@ static int _rtl92e_wx_set_scan(struct net_device *dev,
                        mutex_unlock(&priv->rtllib->ips_mutex);
                }
                rtllib_stop_scan(priv->rtllib);
-               if (priv->rtllib->LedControlHandler)
-                       priv->rtllib->LedControlHandler(dev,
-                                                        LED_CTL_SITE_SURVEY);
-
                if (priv->rtllib->rf_power_state != rf_off) {
                        priv->rtllib->actscanning = true;
 
index 8bafee31e50c8b286837920eae410e4d2a2cd459..4aa5ce9f77928f13d85c12b331ae24b899489da0 100644 (file)
@@ -1716,8 +1716,6 @@ struct rtllib_device {
        void (*InitialGainHandler)(struct net_device *dev, u8 Operation);
        void (*ScanOperationBackupHandler)(struct net_device *dev,
                                           u8 Operation);
-       void (*LedControlHandler)(struct net_device *dev,
-                                 enum led_ctl_mode LedAction);
        void (*SetHwRegHandler)(struct net_device *dev, u8 variable, u8 *val);
 
        void (*AllowAllDestAddrHandler)(struct net_device *dev,
index 1fe0675033cbc272ed82ca4340b67265f9b37061..18885cda60f62367b627ee15a46c67dbc19ab9c6 100644 (file)
@@ -1680,9 +1680,6 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
                                                                 false;
 
                                ieee->state = RTLLIB_ASSOCIATING;
-                               if (ieee->LedControlHandler != NULL)
-                                       ieee->LedControlHandler(ieee->dev,
-                                                        LED_CTL_START_TO_LINK);
                                schedule_delayed_work(
                                           &ieee->associate_procedure_wq, 0);
                        } else {
@@ -2343,10 +2340,6 @@ rtllib_rx_deauth(struct rtllib_device *ieee, struct sk_buff *skb)
                ieee->link_detect_info.bBusyTraffic = false;
                rtllib_disassociate(ieee);
                RemovePeerTS(ieee, header->addr2);
-               if (ieee->LedControlHandler != NULL)
-                       ieee->LedControlHandler(ieee->dev,
-                                               LED_CTL_START_TO_LINK);
-
                if (!(ieee->rtllib_ap_sec_type(ieee) &
                    (SEC_ALG_CCMP|SEC_ALG_TKIP)))
                        schedule_delayed_work(
@@ -2641,9 +2634,6 @@ static void rtllib_start_ibss_wq(void *data)
        ieee->link_change(ieee->dev);
 
        HTSetConnectBwMode(ieee, HT_CHANNEL_WIDTH_20, HT_EXTCHNL_OFFSET_NO_EXT);
-       if (ieee->LedControlHandler != NULL)
-               ieee->LedControlHandler(ieee->dev, LED_CTL_LINK);
-
        rtllib_start_send_beacons(ieee);
 
        notify_wx_assoc_event(ieee);