]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
libertas: Remove function entry/exit debugging
authorKees Cook <keescook@chromium.org>
Mon, 15 May 2017 21:33:17 +0000 (14:33 -0700)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 24 May 2017 13:44:28 +0000 (16:44 +0300)
In at least one place, the enter/exit debugging was not being correctly
matched. Based on mailing list feedback, it was desired to drop all of
these in favor of using ftrace instead.

Suggested-by: Joe Perches <joe@perches.com>
Suggested-by: Kalle Valo <kvalo@codeaurora.org>
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
13 files changed:
drivers/net/wireless/marvell/libertas/cfg.c
drivers/net/wireless/marvell/libertas/cmd.c
drivers/net/wireless/marvell/libertas/cmdresp.c
drivers/net/wireless/marvell/libertas/defs.h
drivers/net/wireless/marvell/libertas/ethtool.c
drivers/net/wireless/marvell/libertas/if_cs.c
drivers/net/wireless/marvell/libertas/if_sdio.c
drivers/net/wireless/marvell/libertas/if_spi.c
drivers/net/wireless/marvell/libertas/if_usb.c
drivers/net/wireless/marvell/libertas/main.c
drivers/net/wireless/marvell/libertas/mesh.c
drivers/net/wireless/marvell/libertas/rx.c
drivers/net/wireless/marvell/libertas/tx.c

index a0463fef79b016635b8cbd59978fe4c94b10a735..71ba2c8d09b50b834837cb606024927dd221f8e1 100644 (file)
@@ -443,17 +443,12 @@ static int lbs_cfg_set_monitor_channel(struct wiphy *wiphy,
        struct lbs_private *priv = wiphy_priv(wiphy);
        int ret = -ENOTSUPP;
 
-       lbs_deb_enter_args(LBS_DEB_CFG80211, "freq %d, type %d",
-                          chandef->chan->center_freq,
-                          cfg80211_get_chandef_type(chandef));
-
        if (cfg80211_get_chandef_type(chandef) != NL80211_CHAN_NO_HT)
                goto out;
 
        ret = lbs_set_channel(priv, chandef->chan->hw_value);
 
  out:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -464,16 +459,12 @@ static int lbs_cfg_set_mesh_channel(struct wiphy *wiphy,
        struct lbs_private *priv = wiphy_priv(wiphy);
        int ret = -ENOTSUPP;
 
-       lbs_deb_enter_args(LBS_DEB_CFG80211, "iface %s freq %d",
-                          netdev_name(netdev), channel->center_freq);
-
        if (netdev != priv->mesh_dev)
                goto out;
 
        ret = lbs_mesh_set_channel(priv, channel->hw_value);
 
  out:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -512,8 +503,6 @@ static int lbs_ret_scan(struct lbs_private *priv, unsigned long dummy,
        int i;
        int ret = -EILSEQ;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        bsssize = get_unaligned_le16(&scanresp->bssdescriptsize);
 
        lbs_deb_scan("scan response: %d BSSs (%d bytes); resp size %d bytes\n",
@@ -665,7 +654,6 @@ static int lbs_ret_scan(struct lbs_private *priv, unsigned long dummy,
        ret = 0;
 
  done:
-       lbs_deb_leave_args(LBS_DEB_SCAN, "ret %d", ret);
        return ret;
 }
 
@@ -693,11 +681,9 @@ static void lbs_scan_worker(struct work_struct *work)
        int last_channel;
        int running, carrier;
 
-       lbs_deb_enter(LBS_DEB_SCAN);
-
        scan_cmd = kzalloc(LBS_SCAN_MAX_CMD_SIZE, GFP_KERNEL);
        if (scan_cmd == NULL)
-               goto out_no_scan_cmd;
+               return;
 
        /* prepare fixed part of scan command */
        scan_cmd->bsstype = CMD_BSS_TYPE_ANY;
@@ -766,16 +752,11 @@ static void lbs_scan_worker(struct work_struct *work)
                lbs_deb_scan("scan: waking up waiters\n");
                wake_up_all(&priv->scan_q);
        }
-
- out_no_scan_cmd:
-       lbs_deb_leave(LBS_DEB_SCAN);
 }
 
 static void _internal_start_scan(struct lbs_private *priv, bool internal,
        struct cfg80211_scan_request *request)
 {
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        lbs_deb_scan("scan: ssids %d, channels %d, ie_len %zd\n",
                request->n_ssids, request->n_channels, request->ie_len);
 
@@ -785,8 +766,6 @@ static void _internal_start_scan(struct lbs_private *priv, bool internal,
 
        queue_delayed_work(priv->work_thread, &priv->scan_work,
                msecs_to_jiffies(50));
-
-       lbs_deb_leave(LBS_DEB_CFG80211);
 }
 
 /*
@@ -815,8 +794,6 @@ static int lbs_cfg_scan(struct wiphy *wiphy,
        struct lbs_private *priv = wiphy_priv(wiphy);
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        if (priv->scan_req || delayed_work_pending(&priv->scan_work)) {
                /* old scan request not yet processed */
                ret = -EAGAIN;
@@ -829,7 +806,6 @@ static int lbs_cfg_scan(struct wiphy *wiphy,
                ret = -EIO;
 
  out:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -843,18 +819,12 @@ static int lbs_cfg_scan(struct wiphy *wiphy,
 void lbs_send_disconnect_notification(struct lbs_private *priv,
                                      bool locally_generated)
 {
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        cfg80211_disconnected(priv->dev, 0, NULL, 0, locally_generated,
                              GFP_KERNEL);
-
-       lbs_deb_leave(LBS_DEB_CFG80211);
 }
 
 void lbs_send_mic_failureevent(struct lbs_private *priv, u32 event)
 {
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        cfg80211_michael_mic_failure(priv->dev,
                priv->assoc_bss,
                event == MACREG_INT_CODE_MIC_ERR_MULTICAST ?
@@ -863,8 +833,6 @@ void lbs_send_mic_failureevent(struct lbs_private *priv, u32 event)
                -1,
                NULL,
                GFP_KERNEL);
-
-       lbs_deb_leave(LBS_DEB_CFG80211);
 }
 
 
@@ -883,8 +851,6 @@ static int lbs_remove_wep_keys(struct lbs_private *priv)
        struct cmd_ds_802_11_set_wep cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.keyindex = cpu_to_le16(priv->wep_tx_key);
@@ -892,7 +858,6 @@ static int lbs_remove_wep_keys(struct lbs_private *priv)
 
        ret = lbs_cmd_with_response(priv, CMD_802_11_SET_WEP, &cmd);
 
-       lbs_deb_leave(LBS_DEB_CFG80211);
        return ret;
 }
 
@@ -905,8 +870,6 @@ static int lbs_set_wep_keys(struct lbs_private *priv)
        int i;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        /*
         * command         13 00
         * size            50 00
@@ -956,7 +919,6 @@ static int lbs_set_wep_keys(struct lbs_private *priv)
                ret = lbs_remove_wep_keys(priv);
        }
 
-       lbs_deb_leave(LBS_DEB_CFG80211);
        return ret;
 }
 
@@ -969,8 +931,6 @@ static int lbs_enable_rsn(struct lbs_private *priv, int enable)
        struct cmd_ds_802_11_enable_rsn cmd;
        int ret;
 
-       lbs_deb_enter_args(LBS_DEB_CFG80211, "%d", enable);
-
        /*
         * cmd       2f 00
         * size      0c 00
@@ -986,7 +946,6 @@ static int lbs_enable_rsn(struct lbs_private *priv, int enable)
 
        ret = lbs_cmd_with_response(priv, CMD_802_11_ENABLE_RSN, &cmd);
 
-       lbs_deb_leave(LBS_DEB_CFG80211);
        return ret;
 }
 
@@ -1014,8 +973,6 @@ static int lbs_set_key_material(struct lbs_private *priv,
        struct cmd_key_material cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        /*
         * Example for WPA (TKIP):
         *
@@ -1044,7 +1001,6 @@ static int lbs_set_key_material(struct lbs_private *priv,
 
        ret = lbs_cmd_with_response(priv, CMD_802_11_KEY_MATERIAL, &cmd);
 
-       lbs_deb_leave(LBS_DEB_CFG80211);
        return ret;
 }
 
@@ -1061,8 +1017,6 @@ static int lbs_set_authtype(struct lbs_private *priv,
        struct cmd_ds_802_11_authenticate cmd;
        int ret;
 
-       lbs_deb_enter_args(LBS_DEB_CFG80211, "%d", sme->auth_type);
-
        /*
         * cmd        11 00
         * size       19 00
@@ -1085,7 +1039,6 @@ static int lbs_set_authtype(struct lbs_private *priv,
        ret = lbs_cmd_with_response(priv, CMD_802_11_AUTHENTICATE, &cmd);
 
  done:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -1116,8 +1069,6 @@ static int lbs_associate(struct lbs_private *priv,
        u8 *pos;
        u8 *tmp;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        if (!cmd) {
                ret = -ENOMEM;
                goto done;
@@ -1262,7 +1213,6 @@ static int lbs_associate(struct lbs_private *priv,
 
        kfree(cmd);
 done:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -1329,8 +1279,6 @@ static int lbs_cfg_connect(struct wiphy *wiphy, struct net_device *dev,
        if (dev == priv->mesh_dev)
                return -EOPNOTSUPP;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        if (!sme->bssid) {
                struct cfg80211_scan_request *creq;
 
@@ -1442,7 +1390,6 @@ static int lbs_cfg_connect(struct wiphy *wiphy, struct net_device *dev,
  done:
        if (bss)
                cfg80211_put_bss(wiphy, bss);
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -1478,8 +1425,6 @@ static int lbs_cfg_disconnect(struct wiphy *wiphy, struct net_device *dev,
        if (dev == priv->mesh_dev)
                return -EOPNOTSUPP;
 
-       lbs_deb_enter_args(LBS_DEB_CFG80211, "reason_code %d", reason_code);
-
        /* store for lbs_cfg_ret_disconnect() */
        priv->disassoc_reason = reason_code;
 
@@ -1496,8 +1441,6 @@ static int lbs_cfg_set_default_key(struct wiphy *wiphy,
        if (netdev == priv->mesh_dev)
                return -EOPNOTSUPP;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        if (key_index != priv->wep_tx_key) {
                lbs_deb_assoc("set_default_key: to %d\n", key_index);
                priv->wep_tx_key = key_index;
@@ -1520,8 +1463,6 @@ static int lbs_cfg_add_key(struct wiphy *wiphy, struct net_device *netdev,
        if (netdev == priv->mesh_dev)
                return -EOPNOTSUPP;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        lbs_deb_assoc("add_key: cipher 0x%x, mac_addr %pM\n",
                      params->cipher, mac_addr);
        lbs_deb_assoc("add_key: key index %d, key len %d\n",
@@ -1575,8 +1516,6 @@ static int lbs_cfg_del_key(struct wiphy *wiphy, struct net_device *netdev,
                           u8 key_index, bool pairwise, const u8 *mac_addr)
 {
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        lbs_deb_assoc("del_key: key_idx %d, mac_addr %pM\n",
                      key_index, mac_addr);
 
@@ -1619,8 +1558,6 @@ static int lbs_cfg_get_station(struct wiphy *wiphy, struct net_device *dev,
        int ret;
        size_t i;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        sinfo->filled |= BIT(NL80211_STA_INFO_TX_BYTES) |
                         BIT(NL80211_STA_INFO_TX_PACKETS) |
                         BIT(NL80211_STA_INFO_RX_BYTES) |
@@ -1675,15 +1612,12 @@ static int lbs_change_intf(struct wiphy *wiphy, struct net_device *dev,
                return -EOPNOTSUPP;
        }
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        if (priv->iface_running)
                ret = lbs_set_iface_type(priv, type);
 
        if (!ret)
                priv->wdev->iftype = type;
 
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -1713,8 +1647,6 @@ static void lbs_join_post(struct lbs_private *priv,
        u8 *fake = fake_ie;
        struct cfg80211_bss *bss;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        /*
         * For cfg80211_inform_bss, we'll need a fake IE, as we can't get
         * the real IE from the firmware. So we fabricate a fake IE based on
@@ -1777,8 +1709,6 @@ static void lbs_join_post(struct lbs_private *priv,
        netif_carrier_on(priv->dev);
        if (!priv->tx_pending_len)
                netif_wake_queue(priv->dev);
-
-       lbs_deb_leave(LBS_DEB_CFG80211);
 }
 
 static int lbs_ibss_join_existing(struct lbs_private *priv,
@@ -1790,8 +1720,6 @@ static int lbs_ibss_join_existing(struct lbs_private *priv,
        u8 preamble = RADIO_PREAMBLE_SHORT;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        /* TODO: set preamble based on scan result */
        ret = lbs_set_radio(priv, preamble, 1);
        if (ret)
@@ -1888,7 +1816,6 @@ static int lbs_ibss_join_existing(struct lbs_private *priv,
        lbs_join_post(priv, params, bss->bssid, bss->capability);
 
  out:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -1904,8 +1831,6 @@ static int lbs_ibss_start_new(struct lbs_private *priv,
        int ret = 0;
        u16 capability;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        ret = lbs_set_radio(priv, preamble, 1);
        if (ret)
                goto out;
@@ -1975,7 +1900,6 @@ static int lbs_ibss_start_new(struct lbs_private *priv,
        lbs_join_post(priv, params, resp->bssid, capability);
 
  out:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -1990,8 +1914,6 @@ static int lbs_join_ibss(struct wiphy *wiphy, struct net_device *dev,
        if (dev == priv->mesh_dev)
                return -EOPNOTSUPP;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        if (!params->chandef.chan) {
                ret = -ENOTSUPP;
                goto out;
@@ -2015,7 +1937,6 @@ static int lbs_join_ibss(struct wiphy *wiphy, struct net_device *dev,
 
 
  out:
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -2029,8 +1950,6 @@ static int lbs_leave_ibss(struct wiphy *wiphy, struct net_device *dev)
        if (dev == priv->mesh_dev)
                return -EOPNOTSUPP;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        ret = lbs_cmd_with_response(priv, CMD_802_11_AD_HOC_STOP, &cmd);
@@ -2038,7 +1957,6 @@ static int lbs_leave_ibss(struct wiphy *wiphy, struct net_device *dev)
        /* TODO: consider doing this at MACREG_INT_CODE_ADHOC_BCN_LOST time */
        lbs_mac_event_disconnected(priv, true);
 
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
@@ -2114,8 +2032,6 @@ struct wireless_dev *lbs_cfg_alloc(struct device *dev)
        int ret = 0;
        struct wireless_dev *wdev;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        wdev = kzalloc(sizeof(struct wireless_dev), GFP_KERNEL);
        if (!wdev)
                return ERR_PTR(-ENOMEM);
@@ -2127,12 +2043,10 @@ struct wireless_dev *lbs_cfg_alloc(struct device *dev)
                goto err_wiphy_new;
        }
 
-       lbs_deb_leave(LBS_DEB_CFG80211);
        return wdev;
 
  err_wiphy_new:
        kfree(wdev);
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ERR_PTR(ret);
 }
 
@@ -2155,15 +2069,11 @@ static void lbs_cfg_set_regulatory_hint(struct lbs_private *priv)
        };
        size_t i;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        for (i = 0; i < ARRAY_SIZE(regmap); i++)
                if (regmap[i].code == priv->regioncode) {
                        regulatory_hint(priv->wdev->wiphy, regmap[i].cn);
                        break;
                }
-
-       lbs_deb_leave(LBS_DEB_CFG80211);
 }
 
 static void lbs_reg_notifier(struct wiphy *wiphy,
@@ -2171,15 +2081,9 @@ static void lbs_reg_notifier(struct wiphy *wiphy,
 {
        struct lbs_private *priv = wiphy_priv(wiphy);
 
-       lbs_deb_enter_args(LBS_DEB_CFG80211, "cfg80211 regulatory domain "
-                       "callback for domain %c%c\n", request->alpha2[0],
-                       request->alpha2[1]);
-
        memcpy(priv->country_code, request->alpha2, sizeof(request->alpha2));
        if (lbs_iface_active(priv))
                lbs_set_11d_domain_info(priv);
-
-       lbs_deb_leave(LBS_DEB_CFG80211);
 }
 
 /*
@@ -2192,8 +2096,6 @@ int lbs_cfg_register(struct lbs_private *priv)
        struct wireless_dev *wdev = priv->wdev;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        wdev->wiphy->max_scan_ssids = 1;
        wdev->wiphy->signal_type = CFG80211_SIGNAL_TYPE_MBM;
 
@@ -2229,13 +2131,11 @@ int lbs_cfg_register(struct lbs_private *priv)
 
        lbs_cfg_set_regulatory_hint(priv);
 
-       lbs_deb_leave_args(LBS_DEB_CFG80211, "ret %d", ret);
        return ret;
 }
 
 void lbs_scan_deinit(struct lbs_private *priv)
 {
-       lbs_deb_enter(LBS_DEB_CFG80211);
        cancel_delayed_work_sync(&priv->scan_work);
 }
 
@@ -2244,8 +2144,6 @@ void lbs_cfg_free(struct lbs_private *priv)
 {
        struct wireless_dev *wdev = priv->wdev;
 
-       lbs_deb_enter(LBS_DEB_CFG80211);
-
        if (!wdev)
                return;
 
index 033ff881c7518109cf0278552af223ea24307f79..c1f42291873783cb8968a6075c158f348bd97b04 100644 (file)
@@ -91,8 +91,6 @@ int lbs_update_hw_spec(struct lbs_private *priv)
        int ret = -1;
        u32 i;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        memcpy(cmd.permanentaddr, priv->current_addr, ETH_ALEN);
@@ -159,14 +157,12 @@ int lbs_update_hw_spec(struct lbs_private *priv)
        }
 
 out:
-       lbs_deb_leave(LBS_DEB_CMD);
        return ret;
 }
 
 static int lbs_ret_host_sleep_cfg(struct lbs_private *priv, unsigned long dummy,
                        struct cmd_header *resp)
 {
-       lbs_deb_enter(LBS_DEB_CMD);
        if (priv->is_host_sleep_activated) {
                priv->is_host_sleep_configured = 0;
                if (priv->psstate == PS_STATE_FULL_POWER) {
@@ -176,7 +172,7 @@ static int lbs_ret_host_sleep_cfg(struct lbs_private *priv, unsigned long dummy,
        } else {
                priv->is_host_sleep_configured = 1;
        }
-       lbs_deb_leave(LBS_DEB_CMD);
+
        return 0;
 }
 
@@ -236,8 +232,6 @@ int lbs_set_ps_mode(struct lbs_private *priv, u16 cmd_action, bool block)
        struct cmd_ds_802_11_ps_mode cmd;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(cmd_action);
@@ -262,7 +256,6 @@ int lbs_set_ps_mode(struct lbs_private *priv, u16 cmd_action, bool block)
                lbs_cmd_async(priv, CMD_802_11_PS_MODE, &cmd.hdr, sizeof (cmd));
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -272,8 +265,6 @@ int lbs_cmd_802_11_sleep_params(struct lbs_private *priv, uint16_t cmd_action,
        struct cmd_ds_802_11_sleep_params cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        if (cmd_action == CMD_ACT_GET) {
                memset(&cmd, 0, sizeof(cmd));
        } else {
@@ -304,7 +295,6 @@ int lbs_cmd_802_11_sleep_params(struct lbs_private *priv, uint16_t cmd_action,
                sp->sp_reserved = le16_to_cpu(cmd.reserved);
        }
 
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -312,8 +302,6 @@ static int lbs_wait_for_ds_awake(struct lbs_private *priv)
 {
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        if (priv->is_deep_sleep) {
                if (!wait_event_interruptible_timeout(priv->ds_awake_q,
                                        !priv->is_deep_sleep, (10 * HZ))) {
@@ -322,7 +310,6 @@ static int lbs_wait_for_ds_awake(struct lbs_private *priv)
                }
        }
 
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -330,8 +317,6 @@ int lbs_set_deep_sleep(struct lbs_private *priv, int deep_sleep)
 {
        int ret =  0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        if (deep_sleep) {
                if (priv->is_deep_sleep != 1) {
                        lbs_deb_cmd("deep sleep: sleep\n");
@@ -358,7 +343,6 @@ int lbs_set_deep_sleep(struct lbs_private *priv, int deep_sleep)
                }
        }
 
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -366,10 +350,9 @@ static int lbs_ret_host_sleep_activate(struct lbs_private *priv,
                unsigned long dummy,
                struct cmd_header *cmd)
 {
-       lbs_deb_enter(LBS_DEB_FW);
        priv->is_host_sleep_activated = 1;
        wake_up_interruptible(&priv->host_sleep_q);
-       lbs_deb_leave(LBS_DEB_FW);
+
        return 0;
 }
 
@@ -379,8 +362,6 @@ int lbs_set_host_sleep(struct lbs_private *priv, int host_sleep)
        int ret = 0;
        uint32_t criteria = EHS_REMOVE_WAKEUP;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        if (host_sleep) {
                if (priv->is_host_sleep_activated != 1) {
                        memset(&cmd, 0, sizeof(cmd));
@@ -438,8 +419,6 @@ int lbs_set_snmp_mib(struct lbs_private *priv, u32 oid, u16 val)
        struct cmd_ds_802_11_snmp_mib cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof (cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_ACT_SET);
@@ -470,7 +449,6 @@ int lbs_set_snmp_mib(struct lbs_private *priv, u32 oid, u16 val)
        ret = lbs_cmd_with_response(priv, CMD_802_11_SNMP_MIB, &cmd);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -488,8 +466,6 @@ int lbs_get_snmp_mib(struct lbs_private *priv, u32 oid, u16 *out_val)
        struct cmd_ds_802_11_snmp_mib cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof (cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_ACT_GET);
@@ -513,7 +489,6 @@ int lbs_get_snmp_mib(struct lbs_private *priv, u32 oid, u16 *out_val)
        }
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -533,8 +508,6 @@ int lbs_get_tx_power(struct lbs_private *priv, s16 *curlevel, s16 *minlevel,
        struct cmd_ds_802_11_rf_tx_power cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_ACT_GET);
@@ -548,7 +521,6 @@ int lbs_get_tx_power(struct lbs_private *priv, s16 *curlevel, s16 *minlevel,
                        *maxlevel = cmd.maxlevel;
        }
 
-       lbs_deb_leave(LBS_DEB_CMD);
        return ret;
 }
 
@@ -565,8 +537,6 @@ int lbs_set_tx_power(struct lbs_private *priv, s16 dbm)
        struct cmd_ds_802_11_rf_tx_power cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_ACT_SET);
@@ -576,7 +546,6 @@ int lbs_set_tx_power(struct lbs_private *priv, s16 dbm)
 
        ret = lbs_cmd_with_response(priv, CMD_802_11_RF_TX_POWER, &cmd);
 
-       lbs_deb_leave(LBS_DEB_CMD);
        return ret;
 }
 
@@ -608,7 +577,6 @@ int lbs_set_monitor_mode(struct lbs_private *priv, int enable)
                                                ARPHRD_ETHER;
        }
 
-       lbs_deb_leave(LBS_DEB_CMD);
        return ret;
 }
 
@@ -624,8 +592,6 @@ static int lbs_get_channel(struct lbs_private *priv)
        struct cmd_ds_802_11_rf_channel cmd;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_OPT_802_11_RF_CHANNEL_GET);
@@ -638,7 +604,6 @@ static int lbs_get_channel(struct lbs_private *priv)
        lbs_deb_cmd("current radio channel is %d\n", ret);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -647,14 +612,12 @@ int lbs_update_channel(struct lbs_private *priv)
        int ret;
 
        /* the channel in f/w could be out of sync; get the current channel */
-       lbs_deb_enter(LBS_DEB_ASSOC);
-
        ret = lbs_get_channel(priv);
        if (ret > 0) {
                priv->channel = ret;
                ret = 0;
        }
-       lbs_deb_leave_args(LBS_DEB_ASSOC, "ret %d", ret);
+
        return ret;
 }
 
@@ -674,8 +637,6 @@ int lbs_set_channel(struct lbs_private *priv, u8 channel)
 #endif
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_OPT_802_11_RF_CHANNEL_SET);
@@ -690,7 +651,6 @@ int lbs_set_channel(struct lbs_private *priv, u8 channel)
                priv->channel);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -708,8 +668,6 @@ int lbs_get_rssi(struct lbs_private *priv, s8 *rssi, s8 *nf)
        struct cmd_ds_802_11_rssi cmd;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        BUG_ON(rssi == NULL);
        BUG_ON(nf == NULL);
 
@@ -724,7 +682,6 @@ int lbs_get_rssi(struct lbs_private *priv, s8 *rssi, s8 *nf)
                *rssi = CAL_RSSI(le16_to_cpu(cmd.n_or_snr), le16_to_cpu(cmd.nf));
        }
 
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -752,7 +709,6 @@ int lbs_set_11d_domain_info(struct lbs_private *priv)
        size_t triplet_size;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_11D);
        if (!priv->country_code[0])
                goto out;
 
@@ -849,7 +805,6 @@ int lbs_set_11d_domain_info(struct lbs_private *priv)
        ret = lbs_cmd_with_response(priv, CMD_802_11D_DOMAIN_INFO, &cmd);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_11D, "ret %d", ret);
        return ret;
 }
 
@@ -869,8 +824,6 @@ int lbs_get_reg(struct lbs_private *priv, u16 reg, u16 offset, u32 *value)
        struct cmd_ds_reg_access cmd;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        BUG_ON(value == NULL);
 
        memset(&cmd, 0, sizeof(cmd));
@@ -894,7 +847,6 @@ int lbs_get_reg(struct lbs_private *priv, u16 reg, u16 offset, u32 *value)
        }
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -914,8 +866,6 @@ int lbs_set_reg(struct lbs_private *priv, u16 reg, u16 offset, u32 value)
        struct cmd_ds_reg_access cmd;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        memset(&cmd, 0, sizeof(cmd));
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_ACT_SET);
@@ -933,7 +883,6 @@ int lbs_set_reg(struct lbs_private *priv, u16 reg, u16 offset, u32 value)
        ret = lbs_cmd_with_response(priv, reg, &cmd);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -943,15 +892,13 @@ static void lbs_queue_cmd(struct lbs_private *priv,
        unsigned long flags;
        int addtail = 1;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        if (!cmdnode) {
                lbs_deb_host("QUEUE_CMD: cmdnode is NULL\n");
-               goto done;
+               return;
        }
        if (!cmdnode->cmdbuf->size) {
                lbs_deb_host("DNLD_CMD: cmd size is zero\n");
-               goto done;
+               return;
        }
        cmdnode->result = 0;
 
@@ -979,9 +926,6 @@ static void lbs_queue_cmd(struct lbs_private *priv,
 
        lbs_deb_host("QUEUE_CMD: inserted command 0x%04x into cmdpendingq\n",
                     le16_to_cpu(cmdnode->cmdbuf->command));
-
-done:
-       lbs_deb_leave(LBS_DEB_HOST);
 }
 
 static void lbs_submit_command(struct lbs_private *priv,
@@ -994,8 +938,6 @@ static void lbs_submit_command(struct lbs_private *priv,
        int timeo = 3 * HZ;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        cmd = cmdnode->cmdbuf;
 
        spin_lock_irqsave(&priv->driver_lock, flags);
@@ -1036,8 +978,6 @@ static void lbs_submit_command(struct lbs_private *priv,
                /* Setup the timer after transmit command */
                mod_timer(&priv->command_timer, jiffies + timeo);
        }
-
-       lbs_deb_leave(LBS_DEB_HOST);
 }
 
 /*
@@ -1047,10 +987,8 @@ static void lbs_submit_command(struct lbs_private *priv,
 static void __lbs_cleanup_and_insert_cmd(struct lbs_private *priv,
                                         struct cmd_ctrl_node *cmdnode)
 {
-       lbs_deb_enter(LBS_DEB_HOST);
-
        if (!cmdnode)
-               goto out;
+               return;
 
        cmdnode->callback = NULL;
        cmdnode->callback_arg = 0;
@@ -1058,8 +996,6 @@ static void __lbs_cleanup_and_insert_cmd(struct lbs_private *priv,
        memset(cmdnode->cmdbuf, 0, LBS_CMD_BUFFER_SIZE);
 
        list_add_tail(&cmdnode->list, &priv->cmdfreeq);
- out:
-       lbs_deb_leave(LBS_DEB_HOST);
 }
 
 static void lbs_cleanup_and_insert_cmd(struct lbs_private *priv,
@@ -1107,8 +1043,6 @@ int lbs_set_radio(struct lbs_private *priv, u8 preamble, u8 radio_on)
        struct cmd_ds_802_11_radio_control cmd;
        int ret = -EINVAL;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(CMD_ACT_SET);
        cmd.control = 0;
@@ -1141,7 +1075,6 @@ int lbs_set_radio(struct lbs_private *priv, u8 preamble, u8 radio_on)
        ret = lbs_cmd_with_response(priv, CMD_802_11_RADIO_CONTROL, &cmd);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
 
@@ -1149,15 +1082,11 @@ void lbs_set_mac_control(struct lbs_private *priv)
 {
        struct cmd_ds_mac_control cmd;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(priv->mac_control);
        cmd.reserved = 0;
 
        lbs_cmd_async(priv, CMD_MAC_CONTROL, &cmd.hdr, sizeof(cmd));
-
-       lbs_deb_leave(LBS_DEB_CMD);
 }
 
 int lbs_set_mac_control_sync(struct lbs_private *priv)
@@ -1165,14 +1094,11 @@ int lbs_set_mac_control_sync(struct lbs_private *priv)
        struct cmd_ds_mac_control cmd;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        cmd.hdr.size = cpu_to_le16(sizeof(cmd));
        cmd.action = cpu_to_le16(priv->mac_control);
        cmd.reserved = 0;
        ret = lbs_cmd_with_response(priv, CMD_MAC_CONTROL, &cmd);
 
-       lbs_deb_leave(LBS_DEB_CMD);
        return ret;
 }
 
@@ -1191,8 +1117,6 @@ int lbs_allocate_cmd_buffer(struct lbs_private *priv)
        u32 i;
        struct cmd_ctrl_node *cmdarray;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        /* Allocate and initialize the command array */
        bufsize = sizeof(struct cmd_ctrl_node) * LBS_NUM_CMD_BUFFERS;
        if (!(cmdarray = kzalloc(bufsize, GFP_KERNEL))) {
@@ -1219,7 +1143,6 @@ int lbs_allocate_cmd_buffer(struct lbs_private *priv)
        ret = 0;
 
 done:
-       lbs_deb_leave_args(LBS_DEB_HOST, "ret %d", ret);
        return ret;
 }
 
@@ -1235,8 +1158,6 @@ int lbs_free_cmd_buffer(struct lbs_private *priv)
        struct cmd_ctrl_node *cmdarray;
        unsigned int i;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        /* need to check if cmd array is allocated or not */
        if (priv->cmd_array == NULL) {
                lbs_deb_host("FREE_CMD_BUF: cmd_array is NULL\n");
@@ -1260,7 +1181,6 @@ int lbs_free_cmd_buffer(struct lbs_private *priv)
        }
 
 done:
-       lbs_deb_leave(LBS_DEB_HOST);
        return 0;
 }
 
@@ -1278,8 +1198,6 @@ static struct cmd_ctrl_node *lbs_get_free_cmd_node(struct lbs_private *priv)
        struct cmd_ctrl_node *tempnode;
        unsigned long flags;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        if (!priv)
                return NULL;
 
@@ -1296,7 +1214,6 @@ static struct cmd_ctrl_node *lbs_get_free_cmd_node(struct lbs_private *priv)
 
        spin_unlock_irqrestore(&priv->driver_lock, flags);
 
-       lbs_deb_leave(LBS_DEB_HOST);
        return tempnode;
 }
 
@@ -1318,8 +1235,6 @@ int lbs_execute_next_command(struct lbs_private *priv)
        /* Debug group is LBS_DEB_THREAD and not LBS_DEB_HOST, because the
         * only caller to us is lbs_thread() and we get even when a
         * data packet is received */
-       lbs_deb_enter(LBS_DEB_THREAD);
-
        spin_lock_irqsave(&priv->driver_lock, flags);
 
        if (priv->cur_cmd) {
@@ -1440,7 +1355,6 @@ int lbs_execute_next_command(struct lbs_private *priv)
 
        ret = 0;
 done:
-       lbs_deb_leave(LBS_DEB_THREAD);
        return ret;
 }
 
@@ -1449,7 +1363,6 @@ static void lbs_send_confirmsleep(struct lbs_private *priv)
        unsigned long flags;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_HOST);
        lbs_deb_hex(LBS_DEB_HOST, "sleep confirm", (u8 *) &confirm_sleep,
                sizeof(confirm_sleep));
 
@@ -1457,7 +1370,7 @@ static void lbs_send_confirmsleep(struct lbs_private *priv)
                sizeof(confirm_sleep));
        if (ret) {
                netdev_alert(priv->dev, "confirm_sleep failed\n");
-               goto out;
+               return;
        }
 
        spin_lock_irqsave(&priv->driver_lock, flags);
@@ -1475,9 +1388,6 @@ static void lbs_send_confirmsleep(struct lbs_private *priv)
                priv->psstate = PS_STATE_SLEEP;
 
        spin_unlock_irqrestore(&priv->driver_lock, flags);
-
-out:
-       lbs_deb_leave(LBS_DEB_HOST);
 }
 
 /**
@@ -1493,8 +1403,6 @@ void lbs_ps_confirm_sleep(struct lbs_private *priv)
        unsigned long flags =0;
        int allowed = 1;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        spin_lock_irqsave(&priv->driver_lock, flags);
        if (priv->dnld_sent) {
                allowed = 0;
@@ -1520,8 +1428,6 @@ void lbs_ps_confirm_sleep(struct lbs_private *priv)
        } else {
                lbs_deb_host("sleep confirm has been delayed\n");
        }
-
-       lbs_deb_leave(LBS_DEB_HOST);
 }
 
 
@@ -1596,8 +1502,6 @@ struct cmd_ctrl_node *__lbs_cmd_async(struct lbs_private *priv,
 {
        struct cmd_ctrl_node *cmdnode;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        if (priv->surpriseremoved) {
                lbs_deb_host("PREP_CMD: card removed\n");
                cmdnode = ERR_PTR(-ENOENT);
@@ -1643,17 +1547,14 @@ struct cmd_ctrl_node *__lbs_cmd_async(struct lbs_private *priv,
        wake_up(&priv->waitq);
 
  done:
-       lbs_deb_leave_args(LBS_DEB_HOST, "ret %p", cmdnode);
        return cmdnode;
 }
 
 void lbs_cmd_async(struct lbs_private *priv, uint16_t command,
        struct cmd_header *in_cmd, int in_cmd_size)
 {
-       lbs_deb_enter(LBS_DEB_CMD);
        __lbs_cmd_async(priv, command, in_cmd, in_cmd_size,
                lbs_cmd_async_callback, 0);
-       lbs_deb_leave(LBS_DEB_CMD);
 }
 
 int __lbs_cmd(struct lbs_private *priv, uint16_t command,
@@ -1665,8 +1566,6 @@ int __lbs_cmd(struct lbs_private *priv, uint16_t command,
        unsigned long flags;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        cmdnode = __lbs_cmd_async(priv, command, in_cmd, in_cmd_size,
                                  callback, callback_arg);
        if (IS_ERR(cmdnode)) {
@@ -1693,7 +1592,6 @@ int __lbs_cmd(struct lbs_private *priv, uint16_t command,
        spin_unlock_irqrestore(&priv->driver_lock, flags);
 
 done:
-       lbs_deb_leave_args(LBS_DEB_HOST, "ret %d", ret);
        return ret;
 }
 EXPORT_SYMBOL_GPL(__lbs_cmd);
index c753e36c2c0e191f7990268caf801122287796c7..aaf01619de59e2033cc7d3428807f4ece87f413a 100644 (file)
@@ -32,8 +32,6 @@ void lbs_mac_event_disconnected(struct lbs_private *priv,
        if (priv->connect_status != LBS_CONNECTED)
                return;
 
-       lbs_deb_enter(LBS_DEB_ASSOC);
-
        /*
         * Cisco AP sends EAP failure and de-auth in less than 0.5 ms.
         * It causes problem in the Supplicant
@@ -61,7 +59,6 @@ void lbs_mac_event_disconnected(struct lbs_private *priv,
                lbs_deb_cmd("disconnected, so exit PS mode\n");
                lbs_set_ps_mode(priv, PS_MODE_ACTION_EXIT_PS, false);
        }
-       lbs_deb_leave(LBS_DEB_ASSOC);
 }
 
 int lbs_process_command_response(struct lbs_private *priv, u8 *data, u32 len)
@@ -72,8 +69,6 @@ int lbs_process_command_response(struct lbs_private *priv, u8 *data, u32 len)
        unsigned long flags;
        uint16_t result;
 
-       lbs_deb_enter(LBS_DEB_HOST);
-
        mutex_lock(&priv->lock);
        spin_lock_irqsave(&priv->driver_lock, flags);
 
@@ -221,7 +216,6 @@ int lbs_process_command_response(struct lbs_private *priv, u8 *data, u32 len)
 
 done:
        mutex_unlock(&priv->lock);
-       lbs_deb_leave_args(LBS_DEB_HOST, "ret %d", ret);
        return ret;
 }
 
@@ -230,8 +224,6 @@ int lbs_process_event(struct lbs_private *priv, u32 event)
        int ret = 0;
        struct cmd_header cmd;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        switch (event) {
        case MACREG_INT_CODE_LINK_SENSED:
                lbs_deb_cmd("EVENT: link sensed\n");
@@ -359,6 +351,5 @@ int lbs_process_event(struct lbs_private *priv, u32 event)
                break;
        }
 
-       lbs_deb_leave_args(LBS_DEB_CMD, "ret %d", ret);
        return ret;
 }
index 407784aca627bcd69b8a8632b45be53390a17481..d3221444e51c3f8dda0132a52235717968a5e630 100644 (file)
@@ -55,15 +55,6 @@ do { if ((lbs_debug & (grp)) == (grp)) \
 #define LBS_DEB_LL(grp, grpnam, fmt, args...) do {} while (0)
 #endif
 
-#define lbs_deb_enter(grp) \
-  LBS_DEB_LL(grp | LBS_DEB_ENTER, " enter", "%s()\n", __func__);
-#define lbs_deb_enter_args(grp, fmt, args...) \
-  LBS_DEB_LL(grp | LBS_DEB_ENTER, " enter", "%s(" fmt ")\n", __func__, ## args);
-#define lbs_deb_leave(grp) \
-  LBS_DEB_LL(grp | LBS_DEB_LEAVE, " leave", "%s()\n", __func__);
-#define lbs_deb_leave_args(grp, fmt, args...) \
-  LBS_DEB_LL(grp | LBS_DEB_LEAVE, " leave", "%s(), " fmt "\n", \
-  __func__, ##args);
 #define lbs_deb_main(fmt, args...)      LBS_DEB_LL(LBS_DEB_MAIN, " main", fmt, ##args)
 #define lbs_deb_net(fmt, args...)       LBS_DEB_LL(LBS_DEB_NET, " net", fmt, ##args)
 #define lbs_deb_mesh(fmt, args...)      LBS_DEB_LL(LBS_DEB_MESH, " mesh", fmt, ##args)
index f955b2d66ed6e7609836e919678fc8c79d479b4b..693868f16921da0c0fd7421e63e4d7f4bf39fb18 100644 (file)
@@ -41,8 +41,6 @@ static int lbs_ethtool_get_eeprom(struct net_device *dev,
        struct cmd_ds_802_11_eeprom_access cmd;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_ETHTOOL);
-
        if (eeprom->offset + eeprom->len > LBS_EEPROM_LEN ||
            eeprom->len > LBS_EEPROM_READ_LEN) {
                ret = -EINVAL;
@@ -59,7 +57,6 @@ static int lbs_ethtool_get_eeprom(struct net_device *dev,
                memcpy(bytes, cmd.value, eeprom->len);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_ETHTOOL, "ret %d", ret);
         return ret;
 }
 
index f499efc6abcf8bed36c46c7aaa342c8b8be3c878..7d88223f890baf6020cdbf2a2417babd2f8c7e8c 100644 (file)
@@ -336,13 +336,11 @@ static inline u32 get_model(u16 manf_id, u16 card_id)
 
 static inline void if_cs_enable_ints(struct if_cs_card *card)
 {
-       lbs_deb_enter(LBS_DEB_CS);
        if_cs_write16(card, IF_CS_HOST_INT_MASK, 0);
 }
 
 static inline void if_cs_disable_ints(struct if_cs_card *card)
 {
-       lbs_deb_enter(LBS_DEB_CS);
        if_cs_write16(card, IF_CS_HOST_INT_MASK, IF_CS_BIT_MASK);
 }
 
@@ -355,7 +353,6 @@ static int if_cs_send_cmd(struct lbs_private *priv, u8 *buf, u16 nb)
        int ret = -1;
        int loops = 0;
 
-       lbs_deb_enter(LBS_DEB_CS);
        if_cs_disable_ints(card);
 
        /* Is hardware ready? */
@@ -388,7 +385,6 @@ static int if_cs_send_cmd(struct lbs_private *priv, u8 *buf, u16 nb)
 
 done:
        if_cs_enable_ints(card);
-       lbs_deb_leave_args(LBS_DEB_CS, "ret %d", ret);
        return ret;
 }
 
@@ -400,7 +396,6 @@ static void if_cs_send_data(struct lbs_private *priv, u8 *buf, u16 nb)
        struct if_cs_card *card = (struct if_cs_card *)priv->card;
        u16 status;
 
-       lbs_deb_enter(LBS_DEB_CS);
        if_cs_disable_ints(card);
 
        status = if_cs_read16(card, IF_CS_CARD_STATUS);
@@ -416,8 +411,6 @@ static void if_cs_send_data(struct lbs_private *priv, u8 *buf, u16 nb)
        if_cs_write16(card, IF_CS_HOST_STATUS, IF_CS_BIT_TX);
        if_cs_write16(card, IF_CS_HOST_INT_CAUSE, IF_CS_BIT_TX);
        if_cs_enable_ints(card);
-
-       lbs_deb_leave(LBS_DEB_CS);
 }
 
 /*
@@ -429,8 +422,6 @@ static int if_cs_receive_cmdres(struct lbs_private *priv, u8 *data, u32 *len)
        int ret = -1;
        u16 status;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        /* is hardware ready? */
        status = if_cs_read16(priv->card, IF_CS_CARD_STATUS);
        if ((status & IF_CS_BIT_RESP) == 0) {
@@ -463,7 +454,6 @@ static int if_cs_receive_cmdres(struct lbs_private *priv, u8 *data, u32 *len)
        spin_unlock_irqrestore(&priv->driver_lock, flags);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CS, "ret %d, len %d", ret, *len);
        return ret;
 }
 
@@ -473,8 +463,6 @@ static struct sk_buff *if_cs_receive_data(struct lbs_private *priv)
        u16 len;
        u8 *data;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        len = if_cs_read16(priv->card, IF_CS_READ_LEN);
        if (len == 0 || len > MRVDRV_ETH_RX_PACKET_BUFFER_SIZE) {
                netdev_err(priv->dev,
@@ -501,7 +489,6 @@ dat_err:
        if_cs_write16(priv->card, IF_CS_HOST_INT_CAUSE, IF_CS_BIT_RX);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_CS, "ret %p", skb);
        return skb;
 }
 
@@ -511,8 +498,6 @@ static irqreturn_t if_cs_interrupt(int irq, void *data)
        struct lbs_private *priv = card->priv;
        u16 cause;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        /* Ask card interrupt cause register if there is something for us */
        cause = if_cs_read16(card, IF_CS_CARD_INT_CAUSE);
        lbs_deb_cs("cause 0x%04x\n", cause);
@@ -569,7 +554,6 @@ static irqreturn_t if_cs_interrupt(int irq, void *data)
        /* Clear interrupt cause */
        if_cs_write16(card, IF_CS_CARD_INT_CAUSE, cause & IF_CS_BIT_MASK);
 
-       lbs_deb_leave(LBS_DEB_CS);
        return IRQ_HANDLED;
 }
 
@@ -591,8 +575,6 @@ static int if_cs_prog_helper(struct if_cs_card *card, const struct firmware *fw)
        int sent = 0;
        u8  scratch;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        /*
         * This is the only place where an unaligned register access happens on
         * the CF8305 card, therefore for the sake of speed of the driver, we do
@@ -671,7 +653,6 @@ static int if_cs_prog_helper(struct if_cs_card *card, const struct firmware *fw)
        }
 
 done:
-       lbs_deb_leave_args(LBS_DEB_CS, "ret %d", ret);
        return ret;
 }
 
@@ -683,8 +664,6 @@ static int if_cs_prog_real(struct if_cs_card *card, const struct firmware *fw)
        int len = 0;
        int sent;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        lbs_deb_cs("fw size %td\n", fw->size);
 
        ret = if_cs_poll_while_fw_download(card, IF_CS_SQ_READ_LOW,
@@ -734,7 +713,6 @@ static int if_cs_prog_real(struct if_cs_card *card, const struct firmware *fw)
                pr_err("firmware download failed\n");
 
 done:
-       lbs_deb_leave_args(LBS_DEB_CS, "ret %d", ret);
        return ret;
 }
 
@@ -792,8 +770,6 @@ static int if_cs_host_to_card(struct lbs_private *priv,
 {
        int ret = -1;
 
-       lbs_deb_enter_args(LBS_DEB_CS, "type %d, bytes %d", type, nb);
-
        switch (type) {
        case MVMS_DAT:
                priv->dnld_sent = DNLD_DATA_SENT;
@@ -809,7 +785,6 @@ static int if_cs_host_to_card(struct lbs_private *priv,
                           __func__, type);
        }
 
-       lbs_deb_leave_args(LBS_DEB_CS, "ret %d", ret);
        return ret;
 }
 
@@ -818,14 +793,10 @@ static void if_cs_release(struct pcmcia_device *p_dev)
 {
        struct if_cs_card *card = p_dev->priv;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        free_irq(p_dev->irq, card);
        pcmcia_disable_device(p_dev);
        if (card->iobase)
                ioport_unmap(card->iobase);
-
-       lbs_deb_leave(LBS_DEB_CS);
 }
 
 
@@ -850,8 +821,6 @@ static int if_cs_probe(struct pcmcia_device *p_dev)
        struct lbs_private *priv;
        struct if_cs_card *card;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        card = kzalloc(sizeof(struct if_cs_card), GFP_KERNEL);
        if (!card)
                goto out;
@@ -961,7 +930,6 @@ out2:
 out1:
        pcmcia_disable_device(p_dev);
 out:
-       lbs_deb_leave_args(LBS_DEB_CS, "ret %d", ret);
        return ret;
 }
 
@@ -970,15 +938,11 @@ static void if_cs_detach(struct pcmcia_device *p_dev)
 {
        struct if_cs_card *card = p_dev->priv;
 
-       lbs_deb_enter(LBS_DEB_CS);
-
        lbs_stop_card(card->priv);
        lbs_remove_card(card->priv);
        if_cs_disable_ints(card);
        if_cs_release(p_dev);
        kfree(card);
-
-       lbs_deb_leave(LBS_DEB_CS);
 }
 
 
index 47f4a14c84fedb037bbbe12479e43f915829afaa..e0196208ab0d57bb0b2cfecf38382bfba0ec2838 100644 (file)
@@ -211,8 +211,6 @@ static int if_sdio_handle_cmd(struct if_sdio_card *card,
        unsigned long flags;
        u8 i;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        if (size > LBS_CMD_BUFFER_SIZE) {
                lbs_deb_sdio("response packet too large (%d bytes)\n",
                        (int)size);
@@ -233,7 +231,6 @@ static int if_sdio_handle_cmd(struct if_sdio_card *card,
        ret = 0;
 
 out:
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
        return ret;
 }
 
@@ -244,8 +241,6 @@ static int if_sdio_handle_data(struct if_sdio_card *card,
        struct sk_buff *skb;
        char *data;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        if (size > MRVDRV_ETH_RX_PACKET_BUFFER_SIZE) {
                lbs_deb_sdio("response packet too large (%d bytes)\n",
                        (int)size);
@@ -270,8 +265,6 @@ static int if_sdio_handle_data(struct if_sdio_card *card,
        ret = 0;
 
 out:
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
-
        return ret;
 }
 
@@ -281,8 +274,6 @@ static int if_sdio_handle_event(struct if_sdio_card *card,
        int ret;
        u32 event;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        if (card->model == MODEL_8385) {
                event = sdio_readb(card->func, IF_SDIO_EVENT, &ret);
                if (ret)
@@ -307,8 +298,6 @@ static int if_sdio_handle_event(struct if_sdio_card *card,
        ret = 0;
 
 out:
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
-
        return ret;
 }
 
@@ -337,8 +326,6 @@ static int if_sdio_card_to_host(struct if_sdio_card *card)
        int ret;
        u16 size, type, chunk;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        size = if_sdio_read_rx_len(card, &ret);
        if (ret)
                goto out;
@@ -410,8 +397,6 @@ out:
        if (ret)
                pr_err("problem fetching packet from firmware\n");
 
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
-
        return ret;
 }
 
@@ -422,8 +407,6 @@ static void if_sdio_host_to_card_worker(struct work_struct *work)
        int ret;
        unsigned long flags;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        card = container_of(work, struct if_sdio_card, packet_worker);
 
        while (1) {
@@ -451,8 +434,6 @@ static void if_sdio_host_to_card_worker(struct work_struct *work)
 
                kfree(packet);
        }
-
-       lbs_deb_leave(LBS_DEB_SDIO);
 }
 
 /********************************************************************/
@@ -471,8 +452,6 @@ static int if_sdio_prog_helper(struct if_sdio_card *card,
        const u8 *firmware;
        size_t size;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        chunk_buffer = kzalloc(64, GFP_KERNEL);
        if (!chunk_buffer) {
                ret = -ENOMEM;
@@ -556,7 +535,6 @@ out:
        if (ret)
                pr_err("failed to load helper firmware\n");
 
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
        return ret;
 }
 
@@ -570,8 +548,6 @@ static int if_sdio_prog_real(struct if_sdio_card *card,
        const u8 *firmware;
        size_t size, req_size;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        chunk_buffer = kzalloc(512, GFP_KERNEL);
        if (!chunk_buffer) {
                ret = -ENOMEM;
@@ -691,7 +667,6 @@ out:
        if (ret)
                pr_err("failed to load firmware\n");
 
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
        return ret;
 }
 
@@ -725,8 +700,6 @@ static int if_sdio_prog_firmware(struct if_sdio_card *card)
        int ret;
        u16 scratch;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        /*
         * Disable interrupts
         */
@@ -769,7 +742,6 @@ static int if_sdio_prog_firmware(struct if_sdio_card *card)
                                     fw_table, if_sdio_do_prog_firmware);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
        return ret;
 }
 
@@ -948,8 +920,6 @@ static int if_sdio_host_to_card(struct lbs_private *priv,
        u16 size;
        unsigned long flags;
 
-       lbs_deb_enter_args(LBS_DEB_SDIO, "type %d, bytes %d", type, nb);
-
        card = priv->card;
 
        if (nb > (65536 - sizeof(struct if_sdio_packet) - 4)) {
@@ -1013,8 +983,6 @@ static int if_sdio_host_to_card(struct lbs_private *priv,
        ret = 0;
 
 out:
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
-
        return ret;
 }
 
@@ -1040,7 +1008,6 @@ static int if_sdio_exit_deep_sleep(struct lbs_private *priv)
        struct if_sdio_card *card = priv->card;
        int ret = -1;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
        sdio_claim_host(card->func);
 
        sdio_writeb(card->func, HOST_POWER_UP, CONFIGURATION_REG, &ret);
@@ -1048,7 +1015,7 @@ static int if_sdio_exit_deep_sleep(struct lbs_private *priv)
                netdev_err(priv->dev, "sdio_writeb failed!\n");
 
        sdio_release_host(card->func);
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
+
        return ret;
 }
 
@@ -1057,7 +1024,6 @@ static int if_sdio_reset_deep_sleep_wakeup(struct lbs_private *priv)
        struct if_sdio_card *card = priv->card;
        int ret = -1;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
        sdio_claim_host(card->func);
 
        sdio_writeb(card->func, 0, CONFIGURATION_REG, &ret);
@@ -1065,7 +1031,7 @@ static int if_sdio_reset_deep_sleep_wakeup(struct lbs_private *priv)
                netdev_err(priv->dev, "sdio_writeb failed!\n");
 
        sdio_release_host(card->func);
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
+
        return ret;
 
 }
@@ -1143,19 +1109,17 @@ static void if_sdio_interrupt(struct sdio_func *func)
        struct if_sdio_card *card;
        u8 cause;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        card = sdio_get_drvdata(func);
 
        cause = sdio_readb(card->func, IF_SDIO_H_INT_STATUS, &ret);
        if (ret || !cause)
-               goto out;
+               return;
 
        lbs_deb_sdio("interrupt: 0x%X\n", (unsigned)cause);
 
        sdio_writeb(card->func, ~cause, IF_SDIO_H_INT_STATUS, &ret);
        if (ret)
-               goto out;
+               return;
 
        /*
         * Ignore the define name, this really means the card has
@@ -1169,13 +1133,8 @@ static void if_sdio_interrupt(struct sdio_func *func)
        if (cause & IF_SDIO_H_INT_UPLD) {
                ret = if_sdio_card_to_host(card);
                if (ret)
-                       goto out;
+                       return;
        }
-
-       ret = 0;
-
-out:
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
 }
 
 static int if_sdio_probe(struct sdio_func *func,
@@ -1187,8 +1146,6 @@ static int if_sdio_probe(struct sdio_func *func,
        unsigned int model;
        struct if_sdio_packet *packet;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        for (i = 0;i < func->card->num_info;i++) {
                if (sscanf(func->card->info[i],
                                "802.11 SDIO ID: %x", &model) == 1)
@@ -1273,8 +1230,6 @@ static int if_sdio_probe(struct sdio_func *func,
                goto err_activate_card;
 
 out:
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
-
        return ret;
 
 err_activate_card:
@@ -1298,8 +1253,6 @@ static void if_sdio_remove(struct sdio_func *func)
        struct if_sdio_card *card;
        struct if_sdio_packet *packet;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        card = sdio_get_drvdata(func);
 
        /* Undo decrement done above in if_sdio_probe */
@@ -1335,7 +1288,6 @@ static void if_sdio_remove(struct sdio_func *func)
        }
 
        kfree(card);
-       lbs_deb_leave(LBS_DEB_SDIO);
 }
 
 static int if_sdio_suspend(struct device *dev)
@@ -1415,8 +1367,6 @@ static int __init if_sdio_init_module(void)
 {
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        printk(KERN_INFO "libertas_sdio: Libertas SDIO driver\n");
        printk(KERN_INFO "libertas_sdio: Copyright Pierre Ossman\n");
 
@@ -1425,23 +1375,17 @@ static int __init if_sdio_init_module(void)
        /* Clear the flag in case user removes the card. */
        user_rmmod = 0;
 
-       lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
-
        return ret;
 }
 
 static void __exit if_sdio_exit_module(void)
 {
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        /* Set the flag as user is removing this module. */
        user_rmmod = 1;
 
        cancel_work_sync(&card_reset_work);
 
        sdio_unregister_driver(&if_sdio_driver);
-
-       lbs_deb_leave(LBS_DEB_SDIO);
 }
 
 module_init(if_sdio_init_module);
index 7b4955cc38db3d9b19376d0fde19c1ada14139f9..e9aec6cb110546521e46836085114f07605e7cfe 100644 (file)
@@ -466,8 +466,6 @@ static int if_spi_prog_helper_firmware(struct if_spi_card *card,
        const u8 *fw;
        u8 temp[HELPER_FW_LOAD_CHUNK_SZ];
 
-       lbs_deb_enter(LBS_DEB_SPI);
-
        err = spu_set_interrupt_mode(card, 1, 0);
        if (err)
                goto out;
@@ -533,7 +531,7 @@ static int if_spi_prog_helper_firmware(struct if_spi_card *card,
 out:
        if (err)
                pr_err("failed to load helper firmware (err=%d)\n", err);
-       lbs_deb_leave_args(LBS_DEB_SPI, "err %d", err);
+
        return err;
 }
 
@@ -588,8 +586,6 @@ static int if_spi_prog_main_firmware(struct if_spi_card *card,
        const u8 *fw;
        u16 num_crc_errs;
 
-       lbs_deb_enter(LBS_DEB_SPI);
-
        err = spu_set_interrupt_mode(card, 1, 0);
        if (err)
                goto out;
@@ -666,7 +662,7 @@ static int if_spi_prog_main_firmware(struct if_spi_card *card,
 out:
        if (err)
                pr_err("failed to load firmware (err=%d)\n", err);
-       lbs_deb_leave_args(LBS_DEB_SPI, "err %d", err);
+
        return err;
 }
 
@@ -699,8 +695,6 @@ static int if_spi_c2h_cmd(struct if_spi_card *card)
         */
        BUILD_BUG_ON(IF_SPI_CMD_BUF_SIZE % 4 != 0);
 
-       lbs_deb_enter(LBS_DEB_SPI);
-
        /* How many bytes are there to read? */
        err = spu_read_u16(card, IF_SPI_SCRATCH_2_REG, &len);
        if (err)
@@ -735,7 +729,7 @@ static int if_spi_c2h_cmd(struct if_spi_card *card)
 out:
        if (err)
                netdev_err(priv->dev, "%s: err=%d\n", __func__, err);
-       lbs_deb_leave(LBS_DEB_SPI);
+
        return err;
 }
 
@@ -748,8 +742,6 @@ static int if_spi_c2h_data(struct if_spi_card *card)
        u16 len;
        int err = 0;
 
-       lbs_deb_enter(LBS_DEB_SPI);
-
        /* How many bytes are there to read? */
        err = spu_read_u16(card, IF_SPI_SCRATCH_1_REG, &len);
        if (err)
@@ -794,7 +786,7 @@ free_skb:
 out:
        if (err)
                netdev_err(priv->dev, "%s: err=%d\n", __func__, err);
-       lbs_deb_leave(LBS_DEB_SPI);
+
        return err;
 }
 
@@ -870,8 +862,6 @@ static void if_spi_host_to_card_worker(struct work_struct *work)
        card = container_of(work, struct if_spi_card, packet_work);
        priv = card->priv;
 
-       lbs_deb_enter(LBS_DEB_SPI);
-
        /*
         * Read the host interrupt status register to see what we
         * can do.
@@ -943,8 +933,6 @@ static void if_spi_host_to_card_worker(struct work_struct *work)
 err:
        if (err)
                netdev_err(priv->dev, "%s: got error %d\n", __func__, err);
-
-       lbs_deb_leave(LBS_DEB_SPI);
 }
 
 /*
@@ -962,8 +950,6 @@ static int if_spi_host_to_card(struct lbs_private *priv,
        struct if_spi_packet *packet;
        u16 blen;
 
-       lbs_deb_enter_args(LBS_DEB_SPI, "type %d, bytes %d", type, nb);
-
        if (nb == 0) {
                netdev_err(priv->dev, "%s: invalid size requested: %d\n",
                           __func__, nb);
@@ -1004,7 +990,6 @@ static int if_spi_host_to_card(struct lbs_private *priv,
        /* Queue spi xfer work */
        queue_work(card->workqueue, &card->packet_work);
 out:
-       lbs_deb_leave_args(LBS_DEB_SPI, "err=%d", err);
        return err;
 }
 
@@ -1035,8 +1020,6 @@ static int if_spi_init_card(struct if_spi_card *card)
        const struct firmware *helper = NULL;
        const struct firmware *mainfw = NULL;
 
-       lbs_deb_enter(LBS_DEB_SPI);
-
        err = spu_init(card, card->pdata->use_dummy_writes);
        if (err)
                goto out;
@@ -1093,7 +1076,6 @@ static int if_spi_init_card(struct if_spi_card *card)
                goto out;
 
 out:
-       lbs_deb_leave_args(LBS_DEB_SPI, "err %d\n", err);
        return err;
 }
 
@@ -1126,8 +1108,6 @@ static int if_spi_probe(struct spi_device *spi)
        struct libertas_spi_platform_data *pdata = dev_get_platdata(&spi->dev);
        int err = 0;
 
-       lbs_deb_enter(LBS_DEB_SPI);
-
        if (!pdata) {
                err = -EINVAL;
                goto out;
@@ -1221,7 +1201,6 @@ teardown:
        if (pdata->teardown)
                pdata->teardown(spi);
 out:
-       lbs_deb_leave_args(LBS_DEB_SPI, "err %d\n", err);
        return err;
 }
 
@@ -1231,7 +1210,6 @@ static int libertas_spi_remove(struct spi_device *spi)
        struct lbs_private *priv = card->priv;
 
        lbs_deb_spi("libertas_spi_remove\n");
-       lbs_deb_enter(LBS_DEB_SPI);
 
        cancel_work_sync(&card->resume_work);
 
@@ -1243,7 +1221,7 @@ static int libertas_spi_remove(struct spi_device *spi)
        if (card->pdata->teardown)
                card->pdata->teardown(spi);
        free_if_spi_card(card);
-       lbs_deb_leave(LBS_DEB_SPI);
+
        return 0;
 }
 
@@ -1297,18 +1275,16 @@ static struct spi_driver libertas_spi_driver = {
 static int __init if_spi_init_module(void)
 {
        int ret = 0;
-       lbs_deb_enter(LBS_DEB_SPI);
+
        printk(KERN_INFO "libertas_spi: Libertas SPI driver\n");
        ret = spi_register_driver(&libertas_spi_driver);
-       lbs_deb_leave(LBS_DEB_SPI);
+
        return ret;
 }
 
 static void __exit if_spi_exit_module(void)
 {
-       lbs_deb_enter(LBS_DEB_SPI);
        spi_unregister_driver(&libertas_spi_driver);
-       lbs_deb_leave(LBS_DEB_SPI);
 }
 
 module_init(if_spi_init_module);
index aba0c9995b14b143f716d4a83c0a695f0cc628eb..e53025ea6689c77af8f95b5454533f6819901172 100644 (file)
@@ -111,8 +111,6 @@ static void if_usb_write_bulk_callback(struct urb *urb)
  */
 static void if_usb_free(struct if_usb_card *cardp)
 {
-       lbs_deb_enter(LBS_DEB_USB);
-
        /* Unlink tx & rx urb */
        usb_kill_urb(cardp->tx_urb);
        usb_kill_urb(cardp->rx_urb);
@@ -125,8 +123,6 @@ static void if_usb_free(struct if_usb_card *cardp)
 
        kfree(cardp->ep_out_buf);
        cardp->ep_out_buf = NULL;
-
-       lbs_deb_leave(LBS_DEB_USB);
 }
 
 static void if_usb_setup_firmware(struct lbs_private *priv)
@@ -306,8 +302,6 @@ static void if_usb_disconnect(struct usb_interface *intf)
        struct if_usb_card *cardp = usb_get_intfdata(intf);
        struct lbs_private *priv = cardp->priv;
 
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        cardp->surprise_removed = 1;
 
        if (priv) {
@@ -320,8 +314,6 @@ static void if_usb_disconnect(struct usb_interface *intf)
 
        usb_set_intfdata(intf, NULL);
        usb_put_dev(interface_to_usbdev(intf));
-
-       lbs_deb_leave(LBS_DEB_MAIN);
 }
 
 /**
@@ -388,8 +380,6 @@ static int if_usb_reset_device(struct if_usb_card *cardp)
        struct cmd_header *cmd = cardp->ep_out_buf + 4;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_USB);
-
        *(__le32 *)cardp->ep_out_buf = cpu_to_le32(CMD_TYPE_REQUEST);
 
        cmd->command = cpu_to_le16(CMD_802_11_RESET);
@@ -407,8 +397,6 @@ static int if_usb_reset_device(struct if_usb_card *cardp)
                if_usb_reset_olpc_card(NULL);
 #endif
 
-       lbs_deb_leave_args(LBS_DEB_USB, "ret %d", ret);
-
        return ret;
 }
 
@@ -671,8 +659,6 @@ static void if_usb_receive(struct urb *urb)
        __le32 *pkt = (__le32 *)(skb->data + IPFIELD_ALIGN_OFFSET);
        uint32_t event;
 
-       lbs_deb_enter(LBS_DEB_USB);
-
        if (recvlength) {
                if (urb->status) {
                        lbs_deb_usbd(&cardp->udev->dev, "RX URB failed: %d\n",
@@ -688,7 +674,7 @@ static void if_usb_receive(struct urb *urb)
                            recvlength, recvtype);
        } else if (urb->status) {
                kfree_skb(skb);
-               goto rx_exit;
+               return;
        }
 
        switch (recvtype) {
@@ -724,8 +710,6 @@ static void if_usb_receive(struct urb *urb)
 
 setup_for_next:
        if_usb_submit_rx_urb(cardp);
-rx_exit:
-       lbs_deb_leave(LBS_DEB_USB);
 }
 
 /**
@@ -835,8 +819,6 @@ static void if_usb_prog_firmware(struct lbs_private *priv, int ret,
        int i = 0;
        static int reset_count = 10;
 
-       lbs_deb_enter(LBS_DEB_USB);
-
        if (ret) {
                pr_err("failed to find firmware (%d)\n", ret);
                goto done;
@@ -942,7 +924,6 @@ restart:
 
  done:
        cardp->fw = NULL;
-       lbs_deb_leave(LBS_DEB_USB);
 }
 
 
@@ -953,8 +934,6 @@ static int if_usb_suspend(struct usb_interface *intf, pm_message_t message)
        struct lbs_private *priv = cardp->priv;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_USB);
-
        if (priv->psstate != PS_STATE_FULL_POWER) {
                ret = -1;
                goto out;
@@ -978,7 +957,6 @@ static int if_usb_suspend(struct usb_interface *intf, pm_message_t message)
        usb_kill_urb(cardp->rx_urb);
 
  out:
-       lbs_deb_leave(LBS_DEB_USB);
        return ret;
 }
 
@@ -987,13 +965,10 @@ static int if_usb_resume(struct usb_interface *intf)
        struct if_usb_card *cardp = usb_get_intfdata(intf);
        struct lbs_private *priv = cardp->priv;
 
-       lbs_deb_enter(LBS_DEB_USB);
-
        if_usb_submit_rx_urb(cardp);
 
        lbs_resume(priv);
 
-       lbs_deb_leave(LBS_DEB_USB);
        return 0;
 }
 #else
index e3500203715cf6d1f42a7243460ad1878ca10c1d..55f8c997e5cbbff495889d8ff59c0abd0c92d1d0 100644 (file)
@@ -180,7 +180,6 @@ static int lbs_dev_open(struct net_device *dev)
        struct lbs_private *priv = dev->ml_priv;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_NET);
        if (!priv->iface_running) {
                ret = lbs_start_iface(priv);
                if (ret)
@@ -197,7 +196,6 @@ static int lbs_dev_open(struct net_device *dev)
        spin_unlock_irq(&priv->driver_lock);
 
 out:
-       lbs_deb_leave_args(LBS_DEB_NET, "ret %d", ret);
        return ret;
 }
 
@@ -216,8 +214,6 @@ int lbs_stop_iface(struct lbs_private *priv)
        unsigned long flags;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        spin_lock_irqsave(&priv->driver_lock, flags);
        priv->iface_running = false;
        kfree_skb(priv->currenttxskb);
@@ -236,7 +232,6 @@ int lbs_stop_iface(struct lbs_private *priv)
        if (priv->power_save)
                ret = priv->power_save(priv);
 
-       lbs_deb_leave(LBS_DEB_MAIN);
        return ret;
 }
 
@@ -250,8 +245,6 @@ static int lbs_eth_stop(struct net_device *dev)
 {
        struct lbs_private *priv = dev->ml_priv;
 
-       lbs_deb_enter(LBS_DEB_NET);
-
        if (priv->connect_status == LBS_CONNECTED)
                lbs_disconnect(priv, WLAN_REASON_DEAUTH_LEAVING);
 
@@ -269,7 +262,6 @@ static int lbs_eth_stop(struct net_device *dev)
        if (!lbs_iface_active(priv))
                lbs_stop_iface(priv);
 
-       lbs_deb_leave(LBS_DEB_NET);
        return 0;
 }
 
@@ -277,8 +269,6 @@ void lbs_host_to_card_done(struct lbs_private *priv)
 {
        unsigned long flags;
 
-       lbs_deb_enter(LBS_DEB_THREAD);
-
        spin_lock_irqsave(&priv->driver_lock, flags);
        del_timer(&priv->tx_lockup_timer);
 
@@ -291,7 +281,6 @@ void lbs_host_to_card_done(struct lbs_private *priv)
        }
 
        spin_unlock_irqrestore(&priv->driver_lock, flags);
-       lbs_deb_leave(LBS_DEB_THREAD);
 }
 EXPORT_SYMBOL_GPL(lbs_host_to_card_done);
 
@@ -301,8 +290,6 @@ int lbs_set_mac_address(struct net_device *dev, void *addr)
        struct lbs_private *priv = dev->ml_priv;
        struct sockaddr *phwaddr = addr;
 
-       lbs_deb_enter(LBS_DEB_NET);
-
        /*
         * Can only set MAC address when all interfaces are down, to be written
         * to the hardware when one of them is brought up.
@@ -318,7 +305,6 @@ int lbs_set_mac_address(struct net_device *dev, void *addr)
        if (priv->mesh_dev)
                memcpy(priv->mesh_dev->dev_addr, phwaddr->sa_data, ETH_ALEN);
 
-       lbs_deb_leave_args(LBS_DEB_NET, "ret %d", ret);
        return ret;
 }
 
@@ -378,8 +364,6 @@ void lbs_update_mcast(struct lbs_private *priv)
        int nr_addrs;
        int old_mac_control = priv->mac_control;
 
-       lbs_deb_enter(LBS_DEB_NET);
-
        if (netif_running(priv->dev))
                dev_flags |= priv->dev->flags;
        if (priv->mesh_dev && netif_running(priv->mesh_dev))
@@ -424,8 +408,6 @@ void lbs_update_mcast(struct lbs_private *priv)
  out_set_mac_control:
        if (priv->mac_control != old_mac_control)
                lbs_set_mac_control(priv);
-
-       lbs_deb_leave(LBS_DEB_NET);
 }
 
 static void lbs_set_mcast_worker(struct work_struct *work)
@@ -455,8 +437,6 @@ static int lbs_thread(void *data)
        struct lbs_private *priv = dev->ml_priv;
        wait_queue_t wait;
 
-       lbs_deb_enter(LBS_DEB_THREAD);
-
        init_waitqueue_entry(&wait, current);
 
        for (;;) {
@@ -648,7 +628,6 @@ static int lbs_thread(void *data)
        del_timer(&priv->tx_lockup_timer);
        del_timer(&priv->auto_deepsleep_timer);
 
-       lbs_deb_leave(LBS_DEB_THREAD);
        return 0;
 }
 
@@ -664,8 +643,6 @@ static int lbs_setup_firmware(struct lbs_private *priv)
        int ret = -1;
        s16 curlevel = 0, minlevel = 0, maxlevel = 0;
 
-       lbs_deb_enter(LBS_DEB_FW);
-
        /* Read MAC address from firmware */
        eth_broadcast_addr(priv->current_addr);
        ret = lbs_update_hw_spec(priv);
@@ -687,7 +664,6 @@ static int lbs_setup_firmware(struct lbs_private *priv)
 
        ret = lbs_set_mac_control_sync(priv);
 done:
-       lbs_deb_leave_args(LBS_DEB_FW, "ret %d", ret);
        return ret;
 }
 
@@ -695,8 +671,6 @@ int lbs_suspend(struct lbs_private *priv)
 {
        int ret;
 
-       lbs_deb_enter(LBS_DEB_FW);
-
        if (priv->is_deep_sleep) {
                ret = lbs_set_deep_sleep(priv, 0);
                if (ret) {
@@ -713,7 +687,6 @@ int lbs_suspend(struct lbs_private *priv)
        if (priv->mesh_dev)
                netif_device_detach(priv->mesh_dev);
 
-       lbs_deb_leave_args(LBS_DEB_FW, "ret %d", ret);
        return ret;
 }
 EXPORT_SYMBOL_GPL(lbs_suspend);
@@ -722,8 +695,6 @@ int lbs_resume(struct lbs_private *priv)
 {
        int ret;
 
-       lbs_deb_enter(LBS_DEB_FW);
-
        ret = lbs_set_host_sleep(priv, 0);
 
        netif_device_attach(priv->dev);
@@ -741,7 +712,6 @@ int lbs_resume(struct lbs_private *priv)
        if (priv->setup_fw_on_resume)
                ret = lbs_setup_firmware(priv);
 
-       lbs_deb_leave_args(LBS_DEB_FW, "ret %d", ret);
        return ret;
 }
 EXPORT_SYMBOL_GPL(lbs_resume);
@@ -757,7 +727,6 @@ static void lbs_cmd_timeout_handler(unsigned long data)
        struct lbs_private *priv = (struct lbs_private *)data;
        unsigned long flags;
 
-       lbs_deb_enter(LBS_DEB_CMD);
        spin_lock_irqsave(&priv->driver_lock, flags);
 
        if (!priv->cur_cmd)
@@ -778,7 +747,6 @@ static void lbs_cmd_timeout_handler(unsigned long data)
        wake_up(&priv->waitq);
 out:
        spin_unlock_irqrestore(&priv->driver_lock, flags);
-       lbs_deb_leave(LBS_DEB_CMD);
 }
 
 /**
@@ -793,7 +761,6 @@ static void lbs_tx_lockup_handler(unsigned long data)
        struct lbs_private *priv = (struct lbs_private *)data;
        unsigned long flags;
 
-       lbs_deb_enter(LBS_DEB_TX);
        spin_lock_irqsave(&priv->driver_lock, flags);
 
        netdev_info(priv->dev, "TX lockup detected\n");
@@ -804,7 +771,6 @@ static void lbs_tx_lockup_handler(unsigned long data)
        wake_up_interruptible(&priv->waitq);
 
        spin_unlock_irqrestore(&priv->driver_lock, flags);
-       lbs_deb_leave(LBS_DEB_TX);
 }
 
 /**
@@ -817,8 +783,6 @@ static void auto_deepsleep_timer_fn(unsigned long data)
 {
        struct lbs_private *priv = (struct lbs_private *)data;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        if (priv->is_activity_detected) {
                priv->is_activity_detected = 0;
        } else {
@@ -836,32 +800,25 @@ static void auto_deepsleep_timer_fn(unsigned long data)
        }
        mod_timer(&priv->auto_deepsleep_timer , jiffies +
                                (priv->auto_deep_sleep_timeout * HZ)/1000);
-       lbs_deb_leave(LBS_DEB_CMD);
 }
 
 int lbs_enter_auto_deep_sleep(struct lbs_private *priv)
 {
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        priv->is_auto_deep_sleep_enabled = 1;
        if (priv->is_deep_sleep)
                priv->wakeup_dev_required = 1;
        mod_timer(&priv->auto_deepsleep_timer ,
                        jiffies + (priv->auto_deep_sleep_timeout * HZ)/1000);
 
-       lbs_deb_leave(LBS_DEB_SDIO);
        return 0;
 }
 
 int lbs_exit_auto_deep_sleep(struct lbs_private *priv)
 {
-       lbs_deb_enter(LBS_DEB_SDIO);
-
        priv->is_auto_deep_sleep_enabled = 0;
        priv->auto_deep_sleep_timeout = 0;
        del_timer(&priv->auto_deepsleep_timer);
 
-       lbs_deb_leave(LBS_DEB_SDIO);
        return 0;
 }
 
@@ -869,8 +826,6 @@ static int lbs_init_adapter(struct lbs_private *priv)
 {
        int ret;
 
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        eth_broadcast_addr(priv->current_addr);
 
        priv->connect_status = LBS_DISCONNECTED;
@@ -921,22 +876,16 @@ static int lbs_init_adapter(struct lbs_private *priv)
        }
 
 out:
-       lbs_deb_leave_args(LBS_DEB_MAIN, "ret %d", ret);
-
        return ret;
 }
 
 static void lbs_free_adapter(struct lbs_private *priv)
 {
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        lbs_free_cmd_buffer(priv);
        kfifo_free(&priv->event_fifo);
        del_timer(&priv->command_timer);
        del_timer(&priv->tx_lockup_timer);
        del_timer(&priv->auto_deepsleep_timer);
-
-       lbs_deb_leave(LBS_DEB_MAIN);
 }
 
 static const struct net_device_ops lbs_netdev_ops = {
@@ -962,8 +911,6 @@ struct lbs_private *lbs_add_card(void *card, struct device *dmdev)
        struct wireless_dev *wdev;
        struct lbs_private *priv = NULL;
 
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        /* Allocate an Ethernet device and register it */
        wdev = lbs_cfg_alloc(dmdev);
        if (IS_ERR(wdev)) {
@@ -1031,7 +978,6 @@ struct lbs_private *lbs_add_card(void *card, struct device *dmdev)
        priv = NULL;
 
 done:
-       lbs_deb_leave_args(LBS_DEB_MAIN, "priv %p", priv);
        return priv;
 }
 EXPORT_SYMBOL_GPL(lbs_add_card);
@@ -1041,8 +987,6 @@ void lbs_remove_card(struct lbs_private *priv)
 {
        struct net_device *dev = priv->dev;
 
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        lbs_remove_mesh(priv);
 
        if (priv->wiphy_registered)
@@ -1083,8 +1027,6 @@ void lbs_remove_card(struct lbs_private *priv)
        lbs_free_adapter(priv);
        lbs_cfg_free(priv);
        free_netdev(dev);
-
-       lbs_deb_leave(LBS_DEB_MAIN);
 }
 EXPORT_SYMBOL_GPL(lbs_remove_card);
 
@@ -1105,8 +1047,6 @@ int lbs_start_card(struct lbs_private *priv)
        struct net_device *dev = priv->dev;
        int ret = -1;
 
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        /* poke the firmware */
        ret = lbs_setup_firmware(priv);
        if (ret)
@@ -1133,7 +1073,6 @@ int lbs_start_card(struct lbs_private *priv)
        ret = 0;
 
 done:
-       lbs_deb_leave_args(LBS_DEB_MAIN, "ret %d", ret);
        return ret;
 }
 EXPORT_SYMBOL_GPL(lbs_start_card);
@@ -1143,16 +1082,14 @@ void lbs_stop_card(struct lbs_private *priv)
 {
        struct net_device *dev;
 
-       lbs_deb_enter(LBS_DEB_MAIN);
-
        if (!priv)
-               goto out;
+               return;
        dev = priv->dev;
 
        /* If the netdev isn't registered, it means that lbs_start_card() was
         * never called so we have nothing to do here. */
        if (dev->reg_state != NETREG_REGISTERED)
-               goto out;
+               return;
 
        netif_stop_queue(dev);
        netif_carrier_off(dev);
@@ -1160,9 +1097,6 @@ void lbs_stop_card(struct lbs_private *priv)
        lbs_debugfs_remove_one(priv);
        lbs_deinit_mesh(priv);
        unregister_netdev(dev);
-
-out:
-       lbs_deb_leave(LBS_DEB_MAIN);
 }
 EXPORT_SYMBOL_GPL(lbs_stop_card);
 
@@ -1171,7 +1105,6 @@ void lbs_queue_event(struct lbs_private *priv, u32 event)
 {
        unsigned long flags;
 
-       lbs_deb_enter(LBS_DEB_THREAD);
        spin_lock_irqsave(&priv->driver_lock, flags);
 
        if (priv->psstate == PS_STATE_SLEEP)
@@ -1182,14 +1115,11 @@ void lbs_queue_event(struct lbs_private *priv, u32 event)
        wake_up(&priv->waitq);
 
        spin_unlock_irqrestore(&priv->driver_lock, flags);
-       lbs_deb_leave(LBS_DEB_THREAD);
 }
 EXPORT_SYMBOL_GPL(lbs_queue_event);
 
 void lbs_notify_command_response(struct lbs_private *priv, u8 resp_idx)
 {
-       lbs_deb_enter(LBS_DEB_THREAD);
-
        if (priv->psstate == PS_STATE_SLEEP)
                priv->psstate = PS_STATE_AWAKE;
 
@@ -1198,28 +1128,23 @@ void lbs_notify_command_response(struct lbs_private *priv, u8 resp_idx)
        priv->resp_idx = resp_idx;
 
        wake_up(&priv->waitq);
-
-       lbs_deb_leave(LBS_DEB_THREAD);
 }
 EXPORT_SYMBOL_GPL(lbs_notify_command_response);
 
 static int __init lbs_init_module(void)
 {
-       lbs_deb_enter(LBS_DEB_MAIN);
        memset(&confirm_sleep, 0, sizeof(confirm_sleep));
        confirm_sleep.hdr.command = cpu_to_le16(CMD_802_11_PS_MODE);
        confirm_sleep.hdr.size = cpu_to_le16(sizeof(confirm_sleep));
        confirm_sleep.action = cpu_to_le16(PS_MODE_ACTION_SLEEP_CONFIRMED);
        lbs_debugfs_init();
-       lbs_deb_leave(LBS_DEB_MAIN);
+
        return 0;
 }
 
 static void __exit lbs_exit_module(void)
 {
-       lbs_deb_enter(LBS_DEB_MAIN);
        lbs_debugfs_remove();
-       lbs_deb_leave(LBS_DEB_MAIN);
 }
 
 module_init(lbs_init_module);
index 2229fb4481895d26bb911291092953c463c3122a..eeeb892219aa4c50fb154207d2b8d773b085fd77 100644 (file)
@@ -26,8 +26,6 @@ static int lbs_mesh_access(struct lbs_private *priv, uint16_t cmd_action,
 {
        int ret;
 
-       lbs_deb_enter_args(LBS_DEB_CMD, "action %d", cmd_action);
-
        cmd->hdr.command = cpu_to_le16(CMD_MESH_ACCESS);
        cmd->hdr.size = cpu_to_le16(sizeof(*cmd));
        cmd->hdr.result = 0;
@@ -36,7 +34,6 @@ static int lbs_mesh_access(struct lbs_private *priv, uint16_t cmd_action,
 
        ret = lbs_cmd_with_response(priv, CMD_MESH_ACCESS, cmd);
 
-       lbs_deb_leave(LBS_DEB_CMD);
        return ret;
 }
 
@@ -47,8 +44,6 @@ static int __lbs_mesh_config_send(struct lbs_private *priv,
        int ret;
        u16 command = CMD_MESH_CONFIG_OLD;
 
-       lbs_deb_enter(LBS_DEB_CMD);
-
        /*
         * Command id is 0xac for v10 FW along with mesh interface
         * id in bits 14-13-12.
@@ -66,7 +61,6 @@ static int __lbs_mesh_config_send(struct lbs_private *priv,
 
        ret = lbs_cmd_with_response(priv, command, cmd);
 
-       lbs_deb_leave(LBS_DEB_CMD);
        return ret;
 }
 
@@ -823,8 +817,6 @@ int lbs_init_mesh(struct lbs_private *priv)
 {
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_MESH);
-
        /* Determine mesh_fw_ver from fwrelease and fwcapinfo */
        /* 5.0.16p0 9.0.0.p0 is known to NOT support any mesh */
        /* 5.110.22 have mesh command with 0xa3 command id */
@@ -870,7 +862,6 @@ int lbs_init_mesh(struct lbs_private *priv)
                ret = 1;
        }
 
-       lbs_deb_leave_args(LBS_DEB_MESH, "ret %d", ret);
        return ret;
 }
 
@@ -887,14 +878,11 @@ int lbs_deinit_mesh(struct lbs_private *priv)
        struct net_device *dev = priv->dev;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_MESH);
-
        if (priv->mesh_tlv) {
                device_remove_file(&dev->dev, &dev_attr_lbs_mesh);
                ret = 1;
        }
 
-       lbs_deb_leave_args(LBS_DEB_MESH, "ret %d", ret);
        return ret;
 }
 
@@ -909,7 +897,6 @@ static int lbs_mesh_stop(struct net_device *dev)
 {
        struct lbs_private *priv = dev->ml_priv;
 
-       lbs_deb_enter(LBS_DEB_MESH);
        lbs_mesh_config(priv, CMD_ACT_MESH_CONFIG_STOP,
                lbs_mesh_get_channel(priv));
 
@@ -924,7 +911,6 @@ static int lbs_mesh_stop(struct net_device *dev)
        if (!lbs_iface_active(priv))
                lbs_stop_iface(priv);
 
-       lbs_deb_leave(LBS_DEB_MESH);
        return 0;
 }
 
@@ -939,7 +925,6 @@ static int lbs_mesh_dev_open(struct net_device *dev)
        struct lbs_private *priv = dev->ml_priv;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_NET);
        if (!priv->iface_running) {
                ret = lbs_start_iface(priv);
                if (ret)
@@ -965,7 +950,6 @@ static int lbs_mesh_dev_open(struct net_device *dev)
                lbs_mesh_get_channel(priv));
 
 out:
-       lbs_deb_leave_args(LBS_DEB_NET, "ret %d", ret);
        return ret;
 }
 
@@ -989,8 +973,6 @@ static int lbs_add_mesh(struct lbs_private *priv)
        struct wireless_dev *mesh_wdev;
        int ret = 0;
 
-       lbs_deb_enter(LBS_DEB_MESH);
-
        /* Allocate a virtual mesh device */
        mesh_wdev = kzalloc(sizeof(struct wireless_dev), GFP_KERNEL);
        if (!mesh_wdev) {
@@ -1048,7 +1030,6 @@ err_free_wdev:
        kfree(mesh_wdev);
 
 done:
-       lbs_deb_leave_args(LBS_DEB_MESH, "ret %d", ret);
        return ret;
 }
 
@@ -1060,7 +1041,6 @@ void lbs_remove_mesh(struct lbs_private *priv)
        if (!mesh_dev)
                return;
 
-       lbs_deb_enter(LBS_DEB_MESH);
        netif_stop_queue(mesh_dev);
        netif_carrier_off(mesh_dev);
        sysfs_remove_group(&(mesh_dev->dev.kobj), &lbs_mesh_attr_group);
@@ -1069,7 +1049,6 @@ void lbs_remove_mesh(struct lbs_private *priv)
        priv->mesh_dev = NULL;
        kfree(mesh_dev->ieee80211_ptr);
        free_netdev(mesh_dev);
-       lbs_deb_leave(LBS_DEB_MESH);
 }
 
 
@@ -1126,8 +1105,6 @@ void lbs_mesh_ethtool_get_stats(struct net_device *dev,
        struct cmd_ds_mesh_access mesh_access;
        int ret;
 
-       lbs_deb_enter(LBS_DEB_ETHTOOL);
-
        /* Get Mesh Statistics */
        ret = lbs_mesh_access(priv, CMD_ACT_MESH_GET_STATS, &mesh_access);
 
@@ -1153,8 +1130,6 @@ void lbs_mesh_ethtool_get_stats(struct net_device *dev,
        data[5] = priv->mstats.fwd_bcast_cnt;
        data[6] = priv->mstats.drop_blind;
        data[7] = priv->mstats.tx_failed_cnt;
-
-       lbs_deb_enter(LBS_DEB_ETHTOOL);
 }
 
 int lbs_mesh_ethtool_get_sset_count(struct net_device *dev, int sset)
@@ -1170,12 +1145,9 @@ int lbs_mesh_ethtool_get_sset_count(struct net_device *dev, int sset)
 void lbs_mesh_ethtool_get_strings(struct net_device *dev,
        uint32_t stringset, uint8_t *s)
 {
-       lbs_deb_enter(LBS_DEB_ETHTOOL);
-
        switch (stringset) {
        case ETH_SS_STATS:
                memcpy(s, mesh_stat_strings, sizeof(mesh_stat_strings));
                break;
        }
-       lbs_deb_enter(LBS_DEB_ETHTOOL);
 }
index e446fed7b3459b854b05ab7728c0c15c3ac51e7e..a18bb7a9889ce7989d7a8d80de13cd425b345b8b 100644 (file)
@@ -65,8 +65,6 @@ int lbs_process_rxed_packet(struct lbs_private *priv, struct sk_buff *skb)
                0xaa, 0xaa, 0x03, 0x00, 0x00, 0x00
        };
 
-       lbs_deb_enter(LBS_DEB_RX);
-
        BUG_ON(!skb);
 
        skb->ip_summed = CHECKSUM_NONE;
@@ -158,7 +156,6 @@ int lbs_process_rxed_packet(struct lbs_private *priv, struct sk_buff *skb)
 
        ret = 0;
 done:
-       lbs_deb_leave_args(LBS_DEB_RX, "ret %d", ret);
        return ret;
 }
 EXPORT_SYMBOL_GPL(lbs_process_rxed_packet);
@@ -221,8 +218,6 @@ static int process_rxed_802_11_packet(struct lbs_private *priv,
        struct rx_radiotap_hdr radiotap_hdr;
        struct rx_radiotap_hdr *pradiotap_hdr;
 
-       lbs_deb_enter(LBS_DEB_RX);
-
        p_rx_pkt = (struct rx80211packethdr *) skb->data;
        prxpd = &p_rx_pkt->rx_pd;
 
@@ -281,6 +276,5 @@ static int process_rxed_802_11_packet(struct lbs_private *priv,
        ret = 0;
 
 done:
-       lbs_deb_leave_args(LBS_DEB_RX, "ret %d", ret);
        return ret;
 }
index c025f9c1828234ebbac20396c76ae69c05af05da..723ba5fd0bfe2be2c7635fbe19c08a8359571330 100644 (file)
@@ -70,8 +70,6 @@ netdev_tx_t lbs_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)
        uint16_t pkt_len;
        netdev_tx_t ret = NETDEV_TX_OK;
 
-       lbs_deb_enter(LBS_DEB_TX);
-
        /* We need to protect against the queues being restarted before
           we get round to stopping them */
        spin_lock_irqsave(&priv->driver_lock, flags);
@@ -166,7 +164,6 @@ netdev_tx_t lbs_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)
        spin_unlock_irqrestore(&priv->driver_lock, flags);
        wake_up(&priv->waitq);
 
-       lbs_deb_leave_args(LBS_DEB_TX, "ret %d", ret);
        return ret;
 }