From nobody Sat Feb 7 15:40:34 2026 Received: from mail-yx1-f47.google.com (mail-yx1-f47.google.com [74.125.224.47]) (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 1D7C327B32C for ; Tue, 27 Jan 2026 22:48:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.224.47 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769554085; cv=none; b=gEtLAv9lN/Xp13NuMRdz48nnQ/yt5HA2UNktSq0cEbihmZIfwy9xHNmyritmb1ovOmq1qNY3z4Lhj+L76IwibasFKCSyRzGaCkqPZG35ImW6zyw36uvrG3embN1AhqEBthPnoLVeoDY2lToedaBMo1+6V+hGugQGpWKAePzPDHY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769554085; c=relaxed/simple; bh=2jjo84vXTrnJiSIC/k59vzt29lAphMXqdSh0WJr1nj8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=IOEfvtdkE30Vqd1OYyS/Nh3t0z4h8lh8kI4Yp4chQA/5vHmuidpHkMGTRbqVroTDh2uQoqRgvtaMQsAdyQwORyx4Uz5+Qt6wIfh95vW8XS/DyLbXNK695mAkqIO/c7Z+J3Y+WT9iRhJH85fsSd+8jqtxjmiLWLHWEHh6mUZuF6o= 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=MOTS//Dt; arc=none smtp.client-ip=74.125.224.47 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="MOTS//Dt" Received: by mail-yx1-f47.google.com with SMTP id 956f58d0204a3-64937edbc9eso5669418d50.2 for ; Tue, 27 Jan 2026 14:48:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1769554083; x=1770158883; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=wrbgHaXp5Vm0Dw8q8qO/OWNKLShSjn5LLCMy6DMK5W4=; b=MOTS//Dtw7CrhuH70fZLbwdBx/ldddtnqKpRhPLcQKP7snFphmWDpcu6m8z8w9qxXD teAaCa76ZArhrH7SxlZZxGmCYUAk8UagfCplREG+o+nXdWBlCkKaRGn26GB9TyrAfsoP wTkKbEelk2TByHl63/5vPXEJNa0AxewmUw3jq4dFxyanpo5qtOewXEIl8isTirJToisM 8eULEovwn1NmyvIX7XJfgY8zEeH/E0Uq94DjdAxzqxnxeQT4G4M+KJzThgeO4M4GXllO teXQN3QpMlc1L0F0sj3ZIQmoE2S+3fqnmUTJkahR78Vipr821J6R+/bx2ZM1OJxYR+Ei Ra6w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769554083; x=1770158883; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=wrbgHaXp5Vm0Dw8q8qO/OWNKLShSjn5LLCMy6DMK5W4=; b=LsFKZGPanfhuDT+DodHOd00Yu1PWUz0dXF+kiRI3OkypmfgCKtzYXeqLuMan07rDu8 TzEnH4C9sE+3Q6m0byKYzCF6TgO/jAWGKXvqh8dlZ61AAtd2KPeV64KNSm0oxb+S+vsA zYHWNLBQ9oqT8U/H3UT3/I/Ptl4I3IblhQyhKlY+ta6W9lrTl8Imd5cgLeoYekU5/Hjk yHWnTL5AzHkdFM4immrMX93GYutQuwQsd4FKtxYDPNXlJAD7aPodyThRp2CXwydPQVGC vuPFyucpPP2qK+C0qMFIv7Kqwgup4K9/0TIUUlRD26AUNVl0X1e660EeVNgIJj+TKt0D 0qhw== X-Forwarded-Encrypted: i=1; AJvYcCXxtSZ1YjeboPq4o0YTW8KnH7rgsfboaC5xlcKhHzMXg6u/TxikXNXKqvNApl50gokTU6CAr5k5QccWV90=@vger.kernel.org X-Gm-Message-State: AOJu0YwkNrgnfyIdOL5IROtUluZbId38z31MbVQn34bTJdsyr/5pz1vh QYGhcaK/MGSt3vU3CkriQn5/fTVMcj1iAoG+rEPYp3J9/edVzL7HlDWV X-Gm-Gg: AZuq6aKyqMT2Kbzsugxiajpxj0Q9fwlXrH0Q5be8n9LZmXnRmZORIyNzbmLlz91xYW7 lp7j08JkwoDsv2qeDHh3j6QcRClCPY9JjvAD22jD/417qs6tOLjfb1S6N2fXBBvRb+/ffixru+A 97XM2pjrRuNTGHh84Iy5bOK9EseZ4q+k7qlFzMBKw+8xw22WFfwja8HbaeFx+9n1Tsd8WPuAoE6 S+qS469U8CwyxDrsdxiloPT/7pdO6TAlYK4uipTH53uB3NlHGtNqf5dQvUDgBlVfHQjq3LT9oI/ ZDhpE3eRHGZCnt9MSS4h6FrtuEQLKTNp5vCRWEKuFCQY9Shr2+igYGQqm5KP8ekiY3IFzWGBoWA z98r+qm4Nubb3AEHh+EQFNz0pgPRDivMFwQKYcfrHayntFZrU86aFClHCtA9hkk7dWto= X-Received: by 2002:a05:690e:1698:b0:649:5f25:3f71 with SMTP id 956f58d0204a3-6498fbe834cmr2503075d50.14.1769554082990; Tue, 27 Jan 2026 14:48:02 -0800 (PST) Received: from tux ([2601:7c0:c37c:4c00::9944]) by smtp.gmail.com with ESMTPSA id 956f58d0204a3-649960788bbsm55250d50.9.2026.01.27.14.48.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 Jan 2026 14:48:02 -0800 (PST) From: Ethan Tidmore To: Greg Kroah-Hartman Cc: Michael Straube , Dan Carpenter , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Ethan Tidmore Subject: [PATCH v1 1/3] staging: rtl8723bs: remove dead RX info reset logic Date: Tue, 27 Jan 2026 16:47:36 -0600 Message-ID: <20260127224747.40305-2-ethantidmore06@gmail.com> X-Mailer: git-send-email 2.52.0 In-Reply-To: <20260127224747.40305-1-ethantidmore06@gmail.com> References: <20260127224747.40305-1-ethantidmore06@gmail.com> 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 function rtw_reset_rx_info() and its associated counters in struct debug_priv are used to track AMPDU and management frame statistics that are never read by the driver. Remove the unused function, the write-only struct members, and the now-unused pdbgpriv/psdpriv pointers in rtw_free_assoc_resources() to clean up the MLME code. Signed-off-by: Ethan Tidmore --- drivers/staging/rtl8723bs/core/rtw_mlme.c | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rt= l8723bs/core/rtw_mlme.c index f81a29cd6a78..411339ebab7e 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; @@ -874,7 +863,6 @@ void rtw_free_assoc_resources(struct adapter *adapter, = int lock_scanned_queue) if (lock_scanned_queue) adapter->securitypriv.key_mask =3D 0; =20 - rtw_reset_rx_info(pdbgpriv); } =20 /* rtw_indicate_connect: the caller has to lock pmlmepriv->lock */ --=20 2.52.0 From nobody Sat Feb 7 15:40:34 2026 Received: from mail-yx1-f47.google.com (mail-yx1-f47.google.com [74.125.224.47]) (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 5D5CB2868B4 for ; Tue, 27 Jan 2026 22:48:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.224.47 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769554085; cv=none; b=nLTmNrSi2Sj7PBjQsSMSQYwbmqZj7EzJQb+o6cnTuJhS1UStAdLMHwPROJisXKfVItqkR21NuvKnt9HIuRG2q/wk5nDt4reAubr0YP/AF+VqGDTL/nY+KuIjWxzPmG1zLK4Id5tpdZ1yO75QpQTOIAsrhTsak2NLH+trlQlEbjU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769554085; c=relaxed/simple; bh=tL/wQ3w50xZUcCVSNUiUlEs+YdKe8EgtJihMrXX7ylQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=HWDwp6iNcQlEe0LQ+q7/LwpIxIYzDQNXIGk2NaQ46vsHXJ0/qPVZ2KqvSJrsF8YodYonaY5SktSwqJ53ZO44LiPQYI/KfXiZmRsqaOz1Mml/XB++B/aoMcTvx4qiakkctwPRk3JFd4uAsmUXtkXDgXXks8wzkdrPEj41YzNA9u0= 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=Xn6sLfjp; arc=none smtp.client-ip=74.125.224.47 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="Xn6sLfjp" Received: by mail-yx1-f47.google.com with SMTP id 956f58d0204a3-6493937c208so5727548d50.2 for ; Tue, 27 Jan 2026 14:48:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1769554083; x=1770158883; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=MCz5fhHGNzoQjNWokMKF0CEanvAxqViLRBCIm5sjxKU=; b=Xn6sLfjpOLu+tlzPzqHv8B+yzySMAq/ZTp2W9NMxTZRrfjokXk++8m+vHeVwtEGtuO hhKzDXErkiu/GE0kThMfMizKv3QCj703c7PQqBKwVHOnyKfVfD/DCFyOINLsxSyyZ+PW XDwk1aCjk6hsVNldvAsBjNV+LzZkLlSAi+COxzjC8R5T5f4NiCEk/tp7MdKNGH1eeIME MrNZ9P0gZrhPua5gkz2grIxemBWNyV1PjYc5HWdtbcBtpCss6HVRu5J9BdbAnA+GJZIu CzVCEZx7t7qf2y0wKXOjAVZeR2SGUfAQWpmmGVS+iHPNipVCAZTGYAQMFomN4OLD1sF6 G1Bg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769554083; x=1770158883; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=MCz5fhHGNzoQjNWokMKF0CEanvAxqViLRBCIm5sjxKU=; b=gxa6Zu3fO/vmLPmDUWwXeyjhf6JDJwM0xlUWRuconYgOVztqcxfvA3hkzZhNYVbvuH 5lFkBRkFtcZO+dP9dUeBqb5+33LRb+3objJLN4iL8fKFBdV4XmpUtFuYvsI3cGYzUDmW ICYDE5JVCVlR+NiLcF5iLy/qmOAUpLmwDEaqHfngg/fJW7hRSAY/STq8XztU5eYLtpq2 QhFhIl2x3M6FE1bYdNNNdmg7e5kXJhSqD+LekDlGdzz2KYTFUzL0Q1Y41cYZ3h/qGEgT Th3y9oXor6rcaKc8ZgB1v7Jaz26C+iSTUfucN6/QPr3TTpFhr4q+D0vmzoODppDcjzZc SM+Q== X-Forwarded-Encrypted: i=1; AJvYcCWPz10Qg5t1LNWiaOuqiATNVGYeOKjdQiwlA8CSABxa1YAwCuymvVQMxq85qUcvAQXmsjjw3DUoLim1ztA=@vger.kernel.org X-Gm-Message-State: AOJu0Ywi1Wv1RBv0KT6mz2uU7TawwGOcRc8mzVzoEuHLI786cGz9lU3R WQUoQwdxAW6Pe8sEiyr2UCyzNjbzziscHOHWwYvWdH3lNcuVPPugzQgAgWWwqw== X-Gm-Gg: AZuq6aKG6Zd/h27qFfY4Fr0jRPpf+EQfgsuTJSADGZdBtzIlw7fSblYCb+Db9i7xcop AsfkO/JdA2Ayek8FBYVmVJtLBWr/qCLuBl4xFDSHY+JHDMVYDGGBKpAC7fKmPh9ok9LH0D6zU0L gEJtEl70y6gM2cgADEhjC93ncXbeoTzR3E7doT1ACJGWJC5TUyYFxsEqrBTIppBgXaSN15BBwwp swABeyDWxD7nRV5A0ZPLdAY1wrBBDSESSIxBP/a2qsVLpBodCZru3VnmPRp+8ru1z8vO3ZWnNHV VcPdz6rDuOUipdE/mAh14f4Mz0R33AnemSJpacBvzc6BnuapHF3kuFqUY3CGys1cRjKciicSGcp N3XzGbGUG6NhUSWAe7M/yAPGVLnh0l1IoU/v3PD5Z+YPaafb+XXCw3MFxWFe3fA3j7rtND48yEQ uomw== X-Received: by 2002:a05:690e:130c:b0:649:49ff:3a45 with SMTP id 956f58d0204a3-6498fbf9b64mr2467236d50.38.1769554083496; Tue, 27 Jan 2026 14:48:03 -0800 (PST) Received: from tux ([2601:7c0:c37c:4c00::9944]) by smtp.gmail.com with ESMTPSA id 956f58d0204a3-649960788bbsm55250d50.9.2026.01.27.14.48.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 Jan 2026 14:48:03 -0800 (PST) From: Ethan Tidmore To: Greg Kroah-Hartman Cc: Michael Straube , Dan Carpenter , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Ethan Tidmore Subject: [PATCH v1 2/3] staging: rtl8723bs: remove dead RX sequence logic Date: Tue, 27 Jan 2026 16:47:37 -0600 Message-ID: <20260127224747.40305-3-ethantidmore06@gmail.com> X-Mailer: git-send-email 2.52.0 In-Reply-To: <20260127224747.40305-1-ethantidmore06@gmail.com> References: <20260127224747.40305-1-ethantidmore06@gmail.com> 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 function recv_indicatepkts_pkt_loss_cnt() is used to calculate values for dbg_rx_ampdu_loss_count, but this variable is never used anywhere in the driver. Remove the function and its call site. Signed-off-by: Ethan Tidmore --- drivers/staging/rtl8723bs/core/rtw_recv.c | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_recv.c b/drivers/staging/rt= l8723bs/core/rtw_recv.c index 24cc4a9e0445..cc824427f9a8 100644 --- a/drivers/staging/rtl8723bs/core/rtw_recv.c +++ b/drivers/staging/rtl8723bs/core/rtw_recv.c @@ -1861,15 +1861,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; @@ -1937,7 +1928,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 } --=20 2.52.0 From nobody Sat Feb 7 15:40:34 2026 Received: from mail-yw1-f169.google.com (mail-yw1-f169.google.com [209.85.128.169]) (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 52AE929C33F for ; Tue, 27 Jan 2026 22:48:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.169 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769554088; cv=none; b=epg/jk31DFANyVlLUxo8Q53pQmlkr1BVKdDlYXBZMUffqZMXUzLwGdltXsD57CrIjJyA2nxTN9Z9295Az1aViLZYhPFICNeWegZuV8AqhcjIqXU4U1q/u/KDSA3xSwMfczzRrn4lIKc71qsRwAgUXl4CYUTkbL62B/UL1+vd0fY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769554088; c=relaxed/simple; bh=EDN0QGTFDvm8ggC9Q6gvIRT036y5U8fQ0O9YIfrFmi8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=p+Owan0TqOG5Te5hJxF6Xo6oBLpzvUtMs7Hleqsg+C3qenZDqGVTROS1RVuyXMeplfWfJIONc8ikfNMe2P7HaBG3OxI37aAQEo5HOgtPpfqoDnaNAA/dYEuOEaWqttw+MBdjKILEYvPKDm5LFXjiRL/WxxrkDnDruANh98ohffQ= 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=nUqQONWX; arc=none smtp.client-ip=209.85.128.169 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="nUqQONWX" Received: by mail-yw1-f169.google.com with SMTP id 00721157ae682-79456d5dda4so31266397b3.2 for ; Tue, 27 Jan 2026 14:48:05 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1769554084; x=1770158884; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=HCQDQqxS7QhTVFDcAi3eDL0L7CLRtmR3GtbKlv50k3U=; b=nUqQONWXnCqpFDjemtgQdGStjmvc6ZxZeUVmdn6VjnHa9PoLvuHOZDglyviFypYiLz wuk5RJaMWW0Ng/dJ4cm/BxcrnFp9wprYMukdUPo/QeioWYsXx2MOl3HQrEw0v59QnsP2 4rBYXsYF6+UL9CC/IrO3HEenghXhrMvn4Gdd7s8m4X9corzoDRR1aNCvuoVbYiQfgD2W /7/0L6LIOfUUgSnNrn2jyqTCtTy9lEMLI1TwusztaDg3DLIQDCjff8A6u8X8O0Xx6X4N vKlmeIpC4fJ0fpaR7ADyDKtF+XXICKiwd9Wwu3d1nQ3r70d9TbmSE/yYe4BIXg44higr 31nA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769554084; x=1770158884; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=HCQDQqxS7QhTVFDcAi3eDL0L7CLRtmR3GtbKlv50k3U=; b=RdbE8DsTP1KuIjlC1liZaMAhuMoZseNrc474LUt3e1ozWLB0MDnRIqxAXwTg7tK5f5 2AwyEvY4fxAaKvVcr+AymVjXNif5Ynfa78tqo7C3J1+iRInciF8AwkvW3zhAmCdySNxV t0yCciH8sJIuJ10lrvsNpu6m2+iaYME1DABXJR/aD1HkEcYIO0vJldUKfSZHTjdV/kg4 ryfvNiTYL0qad3xmM5j3NJZUAmyk1MnIEzbs9ZAGSaUCKxPPAYbTd7hk5UMFbFAw7YpJ iT22AvKjKJw8N8LeBE5ecfOlaXZWzsu+zX2qBxWyoqco8RlowLxdwp4gmrJCgQau4Kfl CvNQ== X-Forwarded-Encrypted: i=1; AJvYcCXgH/1LOePXlyDi2A0nSVQR18Ilqm05BhNtoUDzLTQZOXo7N/FSKypruKqmi5vPiJkddJ5Dx9hH6EQzqBA=@vger.kernel.org X-Gm-Message-State: AOJu0YzMZpESRQD8ixxjD4Sc9UsRkDvW06XuCmuahH7WdwhJwTSPIehD 0vcNyeCGBDxWzyoNfo/ZAAGOGHhS+enAGeaz5IX9bAydmNUMxCTW+tcr X-Gm-Gg: AZuq6aKSSp+xl1RBmpq7G4b1mVn/DlejH36J2VBkH06lGpGQUy7OjYGSvaOEy0oqsH0 AoQlD4o9bV6N9q7tZdbY4aRhKF6ezLSq3Cav/yG+r0Nize5HOuw2LPTp//yTTM/k7pDqsbgMD2i jaxLnnb55SKVYGFtlo9LYqDaJ1i8q670v3nISg9eNFb9CjmkdXB51kXwZV9oQ+t0nD49kKnJmVs 94urdOcVrcjRpvShARImzGhMAvMHRuXjpRYCNe1g1gabnb6d9EdVYBzog7WmgPG464/W//xrkqb /10AN3oM8q8raFqRqwuL8eSo2yRedfn5+yQqVtNa+tkf/bikvQPjAFuY+cGGE1p4zhxnTRwuSCT HTs9RbmVB0c4dcOc5TEXXU5O4N79X/MK/m7hScR3tT76TpsKvk6OfmRLb4nzd5NatFbxY6JNNTj JAVg== X-Received: by 2002:a05:690c:fd1:b0:794:7324:dd9 with SMTP id 00721157ae682-7947ab3b960mr61767387b3.19.1769554084150; Tue, 27 Jan 2026 14:48:04 -0800 (PST) Received: from tux ([2601:7c0:c37c:4c00::9944]) by smtp.gmail.com with ESMTPSA id 956f58d0204a3-649960788bbsm55250d50.9.2026.01.27.14.48.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 Jan 2026 14:48:03 -0800 (PST) From: Ethan Tidmore To: Greg Kroah-Hartman Cc: Michael Straube , Dan Carpenter , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Ethan Tidmore Subject: [PATCH v1 3/3] staging: rtl8723bs: remove unused struct debug_priv and all counters Date: Tue, 27 Jan 2026 16:47:38 -0600 Message-ID: <20260127224747.40305-4-ethantidmore06@gmail.com> X-Mailer: git-send-email 2.52.0 In-Reply-To: <20260127224747.40305-1-ethantidmore06@gmail.com> References: <20260127224747.40305-1-ethantidmore06@gmail.com> 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 struct debug_priv is a collection of diagnostic counters that are updated throughout the driver's hot paths (RX/TX, MLME, Power Control, and SDIO) but are never read, exposed via debugfs, or used for logic. This cleanup removes: - The entire struct debug_priv definition and its instance in dvobj_priv. - All write-only counter increments across the core, hal, and os_dep. - Unused local pointers (psdpriv, pdbgpriv) that were only serving the dead diagnostic logic. - Redundant logic blocks that only existed to update these counters. Signed-off-by: Ethan Tidmore --- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 3 -- drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 6 +-- drivers/staging/rtl8723bs/core/rtw_recv.c | 13 +---- .../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/os_dep/os_intfs.c | 14 ++---- drivers/staging/rtl8723bs/os_dep/sdio_intf.c | 48 +++++-------------- 8 files changed, 17 insertions(+), 120 deletions(-) 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..f06d8bc6dac5 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) 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 cc824427f9a8..409dab008a1d 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; @@ -1907,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); */ @@ -1918,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); */ @@ -1988,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. */ @@ -2032,10 +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 =20 /* s3. Insert all packet into Reorder Queue to maintain its ordering. */ 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/os_dep/os_intfs.c b/drivers/staging/= rtl8723bs/os_dep/os_intfs.c index bc02db13781c..fd47f324a8bc 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); @@ -1123,7 +1117,6 @@ static int rtw_resume_process_normal(struct adapter *= padapter) struct pwrctrl_priv *pwrpriv; struct mlme_priv *pmlmepriv; struct dvobj_priv *psdpriv; - struct debug_priv *pdbgpriv; =20 int ret =3D _SUCCESS; =20 @@ -1136,7 +1129,7 @@ static int rtw_resume_process_normal(struct adapter *= padapter) 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 +1148,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..a86252ccac76 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,23 +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++; - } + if (dvobj->irq_alloc) + sdio_release_irq(func); =20 sdio_release_host(func); } @@ -434,15 +418,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 +433,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 +447,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