From nobody Sat Feb 7 17:41:59 2026 Received: from mail-yw1-f172.google.com (mail-yw1-f172.google.com [209.85.128.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7DB0933DEF5 for ; Thu, 29 Jan 2026 16:34:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769704500; cv=none; b=LNF0NiNRL/ZqB1RJx3zdymRenIxDwIiDVgzvQogERjjOMV6jMtO3yTd50iiENnJLHe+53by5L7RE74seFb7zOuNi8alDiExJcpTfrv63jhsimYI+AEO+IQylheMMgNNel7Uv0Pek+/AwY8PA52B7Cn0tXOMtP2sprBllh2BlV38= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769704500; c=relaxed/simple; bh=KaDyhB2+fBVUTje2U5MuzVD336RxMpwA6+FfBdxO3ug=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=NHkhybWcyoTmWe8XwZ71U/lHU1DVOhpOdrmLxW1e+StfmPM18J75CIEFaMA/Vxlvax3hRTHehrE9pmGtNG0mPGVrdr0eAxriDUzS/5YSlZgAIFVk4tlYZECEQvGl9HLxM2Nj9EKbtDLMLwnyciPbgI+fkDwJ+l8k1pL+qPQyOtk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=dK3o1pjv; arc=none smtp.client-ip=209.85.128.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="dK3o1pjv" Received: by mail-yw1-f172.google.com with SMTP id 00721157ae682-79430ef54c3so11721037b3.2 for ; Thu, 29 Jan 2026 08:34:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1769704495; x=1770309295; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=NaJpzcO7ytgqonRt37w37bsyHyjw9mdrz8DqSdQn0a0=; b=dK3o1pjvzJDHWKvVD/pX1xOuPBe9e4YQqpZfmuPg05K3CvP8pQ049sSC3kfSObtdM5 Am+sNYdXfLeT/CinPjLztThyCh3iIbmy8iEvglIlK6KboLPYjB/2SVtR1GIcEdB0q3ym EZlLhZNYOIjxQZ4LHpdwEjqIO2YbbTkrM1EsHPmSvmnS+xGrJSbiteG287o6QoWlshL1 QsWmkNHrdQAOa13KWr7T7asjaxoer/V3I53YjjntCXf70B+MUS+l12Z8HjZCldexZmMg BH7NEmJWdiiG43Yx8gLsN3yxShLqND24XOhFvGa7lBE0+fro+houyEeNycBoaUMPE+i2 wRyw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769704495; x=1770309295; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=NaJpzcO7ytgqonRt37w37bsyHyjw9mdrz8DqSdQn0a0=; b=YrnxzfzWEDx2nzbvh2y0QyvlNtKUKT1ZYzsIMvDoYz+EBfmMy3J3rtMxbJrBJoce6H x6ZkL0EpCh1ka9c+zNndkAZvYyZ3haGrzeQf8Lkup3uUlmKq9C7uUmQMiT1wrdifxW8k Nf2o+SX/+CVCTrBEp2ftpjj1X93ukOVNTJ80T/Kb7QvUTB2xWnMa9N7aw5mEWqC2op5O sP2iBxtnfmmKGYTWVONiklW+dqao0XxpqXCeylGG+ubtPwn1lqnVQAl8rYvAG05bmlCZ 7USNnEUlQB2UiqRLku7YGA5e3YnMtHkwd4cNlIyqLWH+TmO0jRgA0FjEPODCmrQlJcRc 20mg== X-Forwarded-Encrypted: i=1; AJvYcCUbypAnvj/KvXwZPyMUNwCPgydc7/nUax8k0OaenH0va140hTa8GKvTDDjYabx67Syh+w22j7bH5P3/H2Y=@vger.kernel.org X-Gm-Message-State: AOJu0YwkzxMQjne2vd9ZiY5Ewe4kvg32gQksMTn/fx9WWa6f87D6mi9G jkoFbA7YQ6sQWP0DDDzEwmbizI6X/lz9oqcZkhcETDfv067OudWvBkUg X-Gm-Gg: AZuq6aItqmkeTpkMXYbDOYhklpdjB9vuKORjF/QcQrNWZZuR2JdmcN7ih9KFMU25heO NbIsV5eL8bfLNE7BIeTDWRWt6PmXmClfayrImBgHNha91epk9uuJKrP9akuJsElBO2UMG7SqWJA fyFCgISqKdS8SJHxhS3oYaj/NbTZVZoordHKe8DF1MgkiYdz3vhWtDRu9pUPaaDBpY78nlQwL3U UpsqLhC2eSva5FD3CPWURfH9l3o0YfH70dOz+cBf9nktEly+doORa2THX0qwQNVGYf3/5lekLCv h1LpQnROcz2iG1BQMzM8Z2BGE1JP2iPuWWB1jJvxZ7jQs40wSR1Z5h0stthJoQwly+3btGcdBW2 PpUfrzHqzOns0/r4YOzxfblr1+teP+gCCCpc2qm6oK+Y/uq0Brm0NWrb1gNrhbYwm/oY= X-Received: by 2002:a05:690c:e3e5:b0:794:77da:aa57 with SMTP id 00721157ae682-7949dea7e82mr2110727b3.19.1769704494704; Thu, 29 Jan 2026 08:34:54 -0800 (PST) Received: from tux ([2601:7c0:c37c:4c00::9944]) by smtp.gmail.com with ESMTPSA id 00721157ae682-794828ae4basm26257987b3.27.2026.01.29.08.34.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Jan 2026 08:34:54 -0800 (PST) From: Ethan Tidmore To: gregkh@linuxfoundation.org Cc: dan.carpenter@linaro.org, straube.linux@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Ethan Tidmore Subject: [PATCH v3] staging: rtl8723bs: remove unused private debug counters Date: Thu, 29 Jan 2026 10:34:38 -0600 Message-ID: <20260129163438.35749-1-ethantidmore06@gmail.com> X-Mailer: git-send-email 2.52.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The driver maintains a private `struct debug_priv` embedded within `struct dvobj_priv` to track various error counters (e.g., suspend errors, alloc failures, RX drops). These counters are incremented in various files but the data is never read or exposed to userspace. Remove the unused `debug_priv` structure definition, the instance in `dvobj_priv`, and all associated increment operations across the driver to clean up the code. This also removes the following helper functions which were used solely to update these counters: - rtw_reset_rx_info() - recv_indicatepkts_pkt_loss_cnt() - rtw_hal_check_rxfifo_full() --- v3: - Squashed previous series into a single atomic patch to remove definitions and usages simultaneously (Feedback from Dan Carpenter). - Removed stray blank line in rtw_mlme.c (Feedback from Dan Carpenter). - Removed unused variable 'fw_current_in_ps_mode' in rtl8723b_dm.c to fix -Wunused-but-set-variable warning (Fixes kbuild bot failure). Signed-off-by: Ethan Tidmore --- drivers/staging/rtl8723bs/core/rtw_mlme.c | 13 ------ drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 3 -- drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 6 +-- drivers/staging/rtl8723bs/core/rtw_recv.c | 24 +--------- drivers/staging/rtl8723bs/hal/hal_com.c | 19 -------- drivers/staging/rtl8723bs/hal/rtl8723b_dm.c | 9 ---- .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 4 -- drivers/staging/rtl8723bs/hal/sdio_halinit.c | 8 +--- drivers/staging/rtl8723bs/include/drv_types.h | 41 ----------------- drivers/staging/rtl8723bs/include/hal_com.h | 1 - drivers/staging/rtl8723bs/os_dep/os_intfs.c | 15 +------ drivers/staging/rtl8723bs/os_dep/sdio_intf.c | 45 +++++-------------- 12 files changed, 15 insertions(+), 173 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rt= l8723bs/core/rtw_mlme.c index f81a29cd6a78..2364709595b1 100644 --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c @@ -819,15 +819,6 @@ static void free_scanqueue(struct mlme_priv *pmlmepriv) spin_unlock_bh(&scan_queue->lock); } =20 -static void rtw_reset_rx_info(struct debug_priv *pdbgpriv) -{ - pdbgpriv->dbg_rx_ampdu_drop_count =3D 0; - pdbgpriv->dbg_rx_ampdu_forced_indicate_count =3D 0; - pdbgpriv->dbg_rx_ampdu_loss_count =3D 0; - pdbgpriv->dbg_rx_dup_mgt_frame_drop_count =3D 0; - pdbgpriv->dbg_rx_ampdu_window_shift_cnt =3D 0; -} - static void find_network(struct adapter *adapter) { struct wlan_network *pwlan =3D NULL; @@ -848,8 +839,6 @@ void rtw_free_assoc_resources(struct adapter *adapter, = int lock_scanned_queue) { struct mlme_priv *pmlmepriv =3D &adapter->mlmepriv; struct wlan_network *tgt_network =3D &pmlmepriv->cur_network; - struct dvobj_priv *psdpriv =3D adapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; =20 if (check_fwstate(pmlmepriv, WIFI_STATION_STATE | WIFI_AP_STATE)) { struct sta_info *psta; @@ -873,8 +862,6 @@ void rtw_free_assoc_resources(struct adapter *adapter, = int lock_scanned_queue) =20 if (lock_scanned_queue) adapter->securitypriv.key_mask =3D 0; - - rtw_reset_rx_info(pdbgpriv); } =20 /* rtw_indicate_connect: the caller has to lock pmlmepriv->lock */ diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/stagin= g/rtl8723bs/core/rtw_mlme_ext.c index fa1e3ad59254..f4b50bfdb9d5 100644 --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c @@ -438,8 +438,6 @@ void mgt_dispatcher(struct adapter *padapter, union rec= v_frame *precv_frame) struct mlme_priv *pmlmepriv =3D &padapter->mlmepriv; u8 *pframe =3D precv_frame->u.hdr.rx_data; struct sta_info *psta =3D rtw_get_stainfo(&padapter->stapriv, GetAddr2Ptr= (pframe)); - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; =20 if (GetFrameType(pframe) !=3D WIFI_MGT_TYPE) return; @@ -463,7 +461,6 @@ void mgt_dispatcher(struct adapter *padapter, union rec= v_frame *precv_frame) if (GetRetry(pframe)) { if (precv_frame->u.hdr.attrib.seq_num =3D=3D psta->RxMgmtFrameSeqNum) { /* drop the duplicate management frame */ - pdbgpriv->dbg_rx_dup_mgt_frame_drop_count++; return; } } diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging= /rtl8723bs/core/rtw_pwrctrl.c index 0ef788abf403..1c9e02732687 100644 --- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c +++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c @@ -139,8 +139,6 @@ static bool rtw_pwr_unassociated_idle(struct adapter *a= dapter) void rtw_ps_processor(struct adapter *padapter) { struct pwrctrl_priv *pwrpriv =3D adapter_to_pwrctl(padapter); - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; u32 ps_deny =3D 0; =20 mutex_lock(&adapter_to_pwrctl(padapter)->lock); @@ -149,10 +147,8 @@ void rtw_ps_processor(struct adapter *padapter) if (ps_deny !=3D 0) goto exit; =20 - if (pwrpriv->bInSuspend) {/* system suspend or autosuspend */ - pdbgpriv->dbg_ps_insuspend_cnt++; + if (pwrpriv->bInSuspend) /* system suspend or autosuspend */ return; - } =20 pwrpriv->ps_processing =3D true; =20 diff --git a/drivers/staging/rtl8723bs/core/rtw_recv.c b/drivers/staging/rt= l8723bs/core/rtw_recv.c index 24cc4a9e0445..16645d0cd67b 100644 --- a/drivers/staging/rtl8723bs/core/rtw_recv.c +++ b/drivers/staging/rtl8723bs/core/rtw_recv.c @@ -1782,9 +1782,6 @@ static int amsdu_to_msdu(struct adapter *padapter, un= ion recv_frame *prframe) =20 static int check_indicate_seq(struct recv_reorder_ctrl *preorder_ctrl, u16= seq_num) { - struct adapter *padapter =3D preorder_ctrl->padapter; - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; u8 wsize =3D preorder_ctrl->wsize_b; u16 wend =3D (preorder_ctrl->indicate_seq + wsize - 1) % 4096u; =20 @@ -1810,7 +1807,6 @@ static int check_indicate_seq(struct recv_reorder_ctr= l *preorder_ctrl, u16 seq_n preorder_ctrl->indicate_seq =3D seq_num + 1 - wsize; else preorder_ctrl->indicate_seq =3D 0xFFF - (wsize - (seq_num + 1)) + 1; - pdbgpriv->dbg_rx_ampdu_window_shift_cnt++; } =20 return true; @@ -1861,15 +1857,6 @@ static int enqueue_reorder_recvframe(struct recv_reo= rder_ctrl *preorder_ctrl, un =20 } =20 -static void recv_indicatepkts_pkt_loss_cnt(struct debug_priv *pdbgpriv, u6= 4 prev_seq, u64 current_seq) -{ - if (current_seq < prev_seq) - pdbgpriv->dbg_rx_ampdu_loss_count +=3D (4096 + current_seq - prev_seq); - else - pdbgpriv->dbg_rx_ampdu_loss_count +=3D (current_seq - prev_seq); - -} - static int rtw_recv_indicatepkt(struct adapter *padapter, union recv_frame= *precv_frame) { struct recv_priv *precvpriv; @@ -1916,8 +1903,6 @@ static int recv_indicatepkts_in_order(struct adapter = *padapter, struct recv_reor int bPktInBuf =3D false; struct recv_priv *precvpriv =3D &padapter->recvpriv; struct __queue *ppending_recvframe_queue =3D &preorder_ctrl->pending_recv= frame_queue; - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; =20 /* spin_lock_irqsave(&ppending_recvframe_queue->lock, irql); */ /* spin_lock(&ppending_recvframe_queue->lock); */ @@ -1927,7 +1912,6 @@ static int recv_indicatepkts_in_order(struct adapter = *padapter, struct recv_reor =20 /* Handling some condition for forced indicate case. */ if (bforced =3D=3D true) { - pdbgpriv->dbg_rx_ampdu_forced_indicate_count++; if (list_empty(phead)) { /* spin_unlock_irqrestore(&ppending_recvframe_queue->lock, irql); */ /* spin_unlock(&ppending_recvframe_queue->lock); */ @@ -1937,7 +1921,6 @@ static int recv_indicatepkts_in_order(struct adapter = *padapter, struct recv_reor prframe =3D (union recv_frame *)plist; pattrib =3D &prframe->u.hdr.attrib; =20 - recv_indicatepkts_pkt_loss_cnt(pdbgpriv, preorder_ctrl->indicate_seq, pa= ttrib->seq_num); preorder_ctrl->indicate_seq =3D pattrib->seq_num; =20 } @@ -1998,8 +1981,6 @@ static int recv_indicatepkt_reorder(struct adapter *p= adapter, union recv_frame * struct rx_pkt_attrib *pattrib =3D &prframe->u.hdr.attrib; struct recv_reorder_ctrl *preorder_ctrl =3D prframe->u.hdr.preorder_ctrl; struct __queue *ppending_recvframe_queue =3D &preorder_ctrl->pending_recv= frame_queue; - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; =20 if (!pattrib->amsdu) { /* s1. */ @@ -2042,11 +2023,8 @@ static int recv_indicatepkt_reorder(struct adapter *= padapter, union recv_frame * spin_lock_bh(&ppending_recvframe_queue->lock); =20 /* s2. check if winstart_b(indicate_seq) needs to been updated */ - if (!check_indicate_seq(preorder_ctrl, pattrib->seq_num)) { - pdbgpriv->dbg_rx_ampdu_drop_count++; + if (!check_indicate_seq(preorder_ctrl, pattrib->seq_num)) goto _err_exit; - } - =20 /* s3. Insert all packet into Reorder Queue to maintain its ordering. */ if (!enqueue_reorder_recvframe(preorder_ctrl, prframe)) { diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8= 723bs/hal/hal_com.c index 70b5b289f9cb..fae251c332d3 100644 --- a/drivers/staging/rtl8723bs/hal/hal_com.c +++ b/drivers/staging/rtl8723bs/hal/hal_com.c @@ -780,25 +780,6 @@ bool GetU1ByteIntegerFromStringInDecimal(char *Str, u8= *pInt) return true; } =20 -void rtw_hal_check_rxfifo_full(struct adapter *adapter) -{ - struct dvobj_priv *psdpriv =3D adapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; - int save_cnt =3D false; - - /* switch counter to RX fifo */ - rtw_write8(adapter, REG_RXERR_RPT+3, rtw_read8(adapter, REG_RXERR_RPT+3)|= 0xf0); - save_cnt =3D true; - /* todo: other chips */ - - if (save_cnt) { - /* rtw_write8(adapter, REG_RXERR_RPT+3, rtw_read8(adapter, REG_RXERR_RPT= +3)|0xa0); */ - pdbgpriv->dbg_rx_fifo_last_overflow =3D pdbgpriv->dbg_rx_fifo_curr_overf= low; - pdbgpriv->dbg_rx_fifo_curr_overflow =3D rtw_read16(adapter, REG_RXERR_RP= T); - pdbgpriv->dbg_rx_fifo_diff_overflow =3D pdbgpriv->dbg_rx_fifo_curr_overf= low-pdbgpriv->dbg_rx_fifo_last_overflow; - } -} - static u32 Array_kfreemap[] =3D { 0xf8, 0xe, 0xf6, 0xc, diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c b/drivers/staging/= rtl8723bs/hal/rtl8723b_dm.c index 928226679ab4..e4d1b54607c8 100644 --- a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c @@ -124,7 +124,6 @@ void rtl8723b_InitHalDm(struct adapter *Adapter) =20 void rtl8723b_HalDmWatchDog(struct adapter *Adapter) { - bool fw_current_in_ps_mode =3D false; bool bFwPSAwake =3D true; u8 hw_init_completed =3D false; struct hal_com_data *pHalData =3D GET_HAL_DATA(Adapter); @@ -134,16 +133,8 @@ void rtl8723b_HalDmWatchDog(struct adapter *Adapter) if (hw_init_completed =3D=3D false) goto skip_dm; =20 - fw_current_in_ps_mode =3D adapter_to_pwrctl(Adapter)->fw_current_in_ps_mo= de; rtw_hal_get_hwreg(Adapter, HW_VAR_FWLPS_RF_ON, (u8 *)(&bFwPSAwake)); =20 - if ( - (hw_init_completed =3D=3D true) && - ((!fw_current_in_ps_mode) && bFwPSAwake) - ) { - rtw_hal_check_rxfifo_full(Adapter); - } - /* ODM */ if (hw_init_completed =3D=3D true) { u8 bLinked =3D false; diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c b/drivers/st= aging/rtl8723bs/hal/rtl8723b_hal_init.c index 054e2c2eab02..0527f36c6bfa 100644 --- a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c @@ -304,8 +304,6 @@ s32 rtl8723b_FirmwareDownload(struct adapter *padapter,= bool bUsedWoWLANFw) const struct firmware *fw; struct device *device =3D dvobj_to_dev(padapter->dvobj); u8 *fwfilepath; - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; u8 tmp_ps; =20 pFirmware =3D kzalloc(sizeof(struct rt_firmware), GFP_KERNEL); @@ -324,8 +322,6 @@ s32 rtl8723b_FirmwareDownload(struct adapter *padapter,= bool bUsedWoWLANFw) /* 2. read power_state =3D 0xA0[1:0] */ tmp_ps =3D rtw_read8(padapter, 0xa0); tmp_ps &=3D 0x03; - if (tmp_ps !=3D 0x01) - pdbgpriv->dbg_downloadfw_pwr_state_cnt++; =20 fwfilepath =3D "rtlwifi/rtl8723bs_nic.bin"; =20 diff --git a/drivers/staging/rtl8723bs/hal/sdio_halinit.c b/drivers/staging= /rtl8723bs/hal/sdio_halinit.c index 4e81ef53dc47..a3940b61e177 100644 --- a/drivers/staging/rtl8723bs/hal/sdio_halinit.c +++ b/drivers/staging/rtl8723bs/hal/sdio_halinit.c @@ -886,9 +886,6 @@ static void CardDisableRTL8723BSdio(struct adapter *pad= apter) =20 u32 rtl8723bs_hal_deinit(struct adapter *padapter) { - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; - if (padapter->hw_init_completed) { if (adapter_to_pwrctl(padapter)->bips_processing) { if (padapter->netif_up) { @@ -921,18 +918,15 @@ u32 rtl8723bs_hal_deinit(struct adapter *padapter) adapter_to_pwrctl(padapter)->pre_ips_type =3D 0; =20 } else { - pdbgpriv->dbg_carddisable_cnt++; CardDisableRTL8723BSdio(padapter); =20 adapter_to_pwrctl(padapter)->pre_ips_type =3D 1; } =20 } else { - pdbgpriv->dbg_carddisable_cnt++; CardDisableRTL8723BSdio(padapter); } - } else - pdbgpriv->dbg_deinit_fail_cnt++; + } =20 return _SUCCESS; } diff --git a/drivers/staging/rtl8723bs/include/drv_types.h b/drivers/stagin= g/rtl8723bs/include/drv_types.h index f86180dc350c..d07f04d51e55 100644 --- a/drivers/staging/rtl8723bs/include/drv_types.h +++ b/drivers/staging/rtl8723bs/include/drv_types.h @@ -177,45 +177,6 @@ struct registry_priv { #define GET_IFACE_NUMS(padapter) (((struct adapter *)padapter)->dvobj->ifa= ce_nums) #define GET_ADAPTER(padapter, iface_id) (((struct adapter *)padapter)->dvo= bj->padapters[iface_id]) =20 -struct debug_priv { - u32 dbg_sdio_free_irq_error_cnt; - u32 dbg_sdio_alloc_irq_error_cnt; - u32 dbg_sdio_free_irq_cnt; - u32 dbg_sdio_alloc_irq_cnt; - u32 dbg_sdio_deinit_error_cnt; - u32 dbg_sdio_init_error_cnt; - u32 dbg_suspend_error_cnt; - u32 dbg_suspend_cnt; - u32 dbg_resume_cnt; - u32 dbg_resume_error_cnt; - u32 dbg_deinit_fail_cnt; - u32 dbg_carddisable_cnt; - u32 dbg_carddisable_error_cnt; - u32 dbg_ps_insuspend_cnt; - u32 dbg_dev_unload_inIPS_cnt; - u32 dbg_wow_leave_ps_fail_cnt; - u32 dbg_scan_pwr_state_cnt; - u32 dbg_downloadfw_pwr_state_cnt; - u32 dbg_fw_read_ps_state_fail_cnt; - u32 dbg_leave_ips_fail_cnt; - u32 dbg_leave_lps_fail_cnt; - u32 dbg_h2c_leave32k_fail_cnt; - u32 dbg_diswow_dload_fw_fail_cnt; - u32 dbg_enwow_dload_fw_fail_cnt; - u32 dbg_ips_drvopen_fail_cnt; - u32 dbg_poll_fail_cnt; - u32 dbg_rpwm_toggle_cnt; - u32 dbg_rpwm_timeout_fail_cnt; - u64 dbg_rx_fifo_last_overflow; - u64 dbg_rx_fifo_curr_overflow; - u64 dbg_rx_fifo_diff_overflow; - u64 dbg_rx_ampdu_drop_count; - u64 dbg_rx_ampdu_forced_indicate_count; - u64 dbg_rx_ampdu_loss_count; - u64 dbg_rx_dup_mgt_frame_drop_count; - u64 dbg_rx_ampdu_window_shift_cnt; -}; - struct rtw_traffic_statistics { /* tx statistics */ u64 tx_bytes; @@ -251,8 +212,6 @@ struct dvobj_priv { =20 s32 processing_dev_remove; =20 - struct debug_priv drv_dbg; - /* for local/global synchronization */ /* */ spinlock_t lock; diff --git a/drivers/staging/rtl8723bs/include/hal_com.h b/drivers/staging/= rtl8723bs/include/hal_com.h index 74d6c892c401..24e7fd8ee4dc 100644 --- a/drivers/staging/rtl8723bs/include/hal_com.h +++ b/drivers/staging/rtl8723bs/include/hal_com.h @@ -136,7 +136,6 @@ void rtw_hal_update_sta_rate_mask(struct adapter *padap= ter, struct sta_info *pst =20 void SetHwReg(struct adapter *padapter, u8 variable, u8 *val); void GetHwReg(struct adapter *padapter, u8 variable, u8 *val); -void rtw_hal_check_rxfifo_full(struct adapter *adapter); =20 u8 GetHalDefVar(struct adapter *adapter, enum hal_def_variable variable, void *value); diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/= rtl8723bs/os_dep/os_intfs.c index bc02db13781c..dd70309016e7 100644 --- a/drivers/staging/rtl8723bs/os_dep/os_intfs.c +++ b/drivers/staging/rtl8723bs/os_dep/os_intfs.c @@ -964,8 +964,6 @@ void rtw_ndev_destructor(struct net_device *ndev) void rtw_dev_unload(struct adapter *padapter) { struct pwrctrl_priv *pwrctl =3D adapter_to_pwrctl(padapter); - struct dvobj_priv *pobjpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &pobjpriv->drv_dbg; struct cmd_priv *pcmdpriv =3D &padapter->cmdpriv; u8 cnt =3D 0; =20 @@ -994,7 +992,6 @@ void rtw_dev_unload(struct adapter *padapter) /* check HW status and SW state */ netdev_dbg(padapter->pnetdev, "%s: driver in IPS-FWLPS\n", __func__); - pdbgpriv->dbg_dev_unload_inIPS_cnt++; LeaveAllPowerSaveMode(padapter); } else { netdev_dbg(padapter->pnetdev, @@ -1077,24 +1074,21 @@ static void rtw_suspend_normal(struct adapter *pada= pter) void rtw_suspend_common(struct adapter *padapter) { struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; struct pwrctrl_priv *pwrpriv =3D dvobj_to_pwrctl(psdpriv); struct mlme_priv *pmlmepriv =3D &padapter->mlmepriv; =20 unsigned long start_time =3D jiffies; =20 netdev_dbg(padapter->pnetdev, " suspend start\n"); - pdbgpriv->dbg_suspend_cnt++; =20 pwrpriv->bInSuspend =3D true; =20 while (pwrpriv->bips_processing) msleep(1); =20 - if ((!padapter->bup) || (padapter->bDriverStopped) || (padapter->bSurpris= eRemoved)) { - pdbgpriv->dbg_suspend_error_cnt++; + if ((!padapter->bup) || (padapter->bDriverStopped) || (padapter->bSurpris= eRemoved)) return; - } + rtw_ps_deny(padapter, PS_DENY_SUSPEND); =20 rtw_cancel_all_timer(padapter); @@ -1122,8 +1116,6 @@ static int rtw_resume_process_normal(struct adapter *= padapter) struct net_device *pnetdev; struct pwrctrl_priv *pwrpriv; struct mlme_priv *pmlmepriv; - struct dvobj_priv *psdpriv; - struct debug_priv *pdbgpriv; =20 int ret =3D _SUCCESS; =20 @@ -1135,8 +1127,6 @@ static int rtw_resume_process_normal(struct adapter *= padapter) pnetdev =3D padapter->pnetdev; pwrpriv =3D adapter_to_pwrctl(padapter); pmlmepriv =3D &padapter->mlmepriv; - psdpriv =3D padapter->dvobj; - pdbgpriv =3D &psdpriv->drv_dbg; /* interface init */ /* if (sdio_init(adapter_to_dvobj(padapter)) !=3D _SUCCESS) */ if ((padapter->intf_init) && (padapter->intf_init(adapter_to_dvobj(padapt= er)) !=3D _SUCCESS)) { @@ -1155,7 +1145,6 @@ static int rtw_resume_process_normal(struct adapter *= padapter) =20 if (pm_netdev_open(pnetdev, true) !=3D 0) { ret =3D -1; - pdbgpriv->dbg_resume_error_cnt++; goto exit; } =20 diff --git a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c b/drivers/staging= /rtl8723bs/os_dep/sdio_intf.c index dc787954126f..d664e254912c 100644 --- a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c +++ b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c @@ -70,13 +70,10 @@ static int sdio_alloc_irq(struct dvobj_priv *dvobj) sdio_claim_host(func); =20 err =3D sdio_claim_irq(func, &sd_sync_int_hdl); - if (err) { - dvobj->drv_dbg.dbg_sdio_alloc_irq_error_cnt++; + if (err) netdev_crit(dvobj->if1->pnetdev, "%s: sdio_claim_irq FAIL(%d)!\n", __fun= c__, err); - } else { - dvobj->drv_dbg.dbg_sdio_alloc_irq_cnt++; + else dvobj->irq_alloc =3D 1; - } =20 sdio_release_host(func); =20 @@ -97,12 +94,10 @@ static void sdio_free_irq(struct dvobj_priv *dvobj) sdio_claim_host(func); err =3D sdio_release_irq(func); if (err) { - dvobj->drv_dbg.dbg_sdio_free_irq_error_cnt++; netdev_err(dvobj->if1->pnetdev, "%s: sdio_release_irq FAIL(%d)!\n", __func__, err); - } else - dvobj->drv_dbg.dbg_sdio_free_irq_cnt++; + } sdio_release_host(func); } dvobj->irq_alloc =3D 0; @@ -122,16 +117,13 @@ static u32 sdio_init(struct dvobj_priv *dvobj) sdio_claim_host(func); =20 err =3D sdio_enable_func(func); - if (err) { - dvobj->drv_dbg.dbg_sdio_init_error_cnt++; + if (err) goto release; - } =20 err =3D sdio_set_block_size(func, 512); - if (err) { - dvobj->drv_dbg.dbg_sdio_init_error_cnt++; + if (err) goto release; - } + psdio_data->block_transfer_len =3D 512; psdio_data->tx_block_mode =3D 1; psdio_data->rx_block_mode =3D 1; @@ -147,22 +139,15 @@ static u32 sdio_init(struct dvobj_priv *dvobj) static void sdio_deinit(struct dvobj_priv *dvobj) { struct sdio_func *func; - int err; =20 func =3D dvobj->intf_data.func; =20 if (func) { sdio_claim_host(func); - err =3D sdio_disable_func(func); - if (err) - dvobj->drv_dbg.dbg_sdio_deinit_error_cnt++; + sdio_disable_func(func); =20 if (dvobj->irq_alloc) { - err =3D sdio_release_irq(func); - if (err) - dvobj->drv_dbg.dbg_sdio_free_irq_error_cnt++; - else - dvobj->drv_dbg.dbg_sdio_free_irq_cnt++; + sdio_release_irq(func); } =20 sdio_release_host(func); @@ -434,15 +419,12 @@ static int rtw_sdio_suspend(struct device *dev) struct dvobj_priv *psdpriv =3D sdio_get_drvdata(func); struct pwrctrl_priv *pwrpriv =3D dvobj_to_pwrctl(psdpriv); struct adapter *padapter =3D psdpriv->if1; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; =20 if (padapter->bDriverStopped) return 0; =20 - if (pwrpriv->bInSuspend) { - pdbgpriv->dbg_suspend_error_cnt++; + if (pwrpriv->bInSuspend) return 0; - } =20 rtw_suspend_common(padapter); =20 @@ -452,13 +434,9 @@ static int rtw_sdio_suspend(struct device *dev) static int rtw_resume_process(struct adapter *padapter) { struct pwrctrl_priv *pwrpriv =3D adapter_to_pwrctl(padapter); - struct dvobj_priv *psdpriv =3D padapter->dvobj; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; =20 - if (!pwrpriv->bInSuspend) { - pdbgpriv->dbg_resume_error_cnt++; + if (!pwrpriv->bInSuspend) return -1; - } =20 return rtw_resume_common(padapter); } @@ -470,9 +448,6 @@ static int rtw_sdio_resume(struct device *dev) struct adapter *padapter =3D psdpriv->if1; struct mlme_ext_priv *pmlmeext =3D &padapter->mlmeextpriv; int ret =3D 0; - struct debug_priv *pdbgpriv =3D &psdpriv->drv_dbg; - - pdbgpriv->dbg_resume_cnt++; =20 ret =3D rtw_resume_process(padapter); =20 --=20 2.52.0