From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f42.google.com (mail-lf1-f42.google.com [209.85.167.42]) (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 540BC41DE06 for ; Wed, 15 Jul 2026 12:50:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784119821; cv=none; b=mXVsMqvR92LC8vN+s9xyz4wUVT83IIp3Q8yQVHGYS7KIGbJrNePhZ2JPsIWhKlYNkgSSjd5dvRhcibl0XdT71sgMDgxKywjBkrVVEFPBET84VgiRvKqq059dA4q7hPlfFoTI7iJ+fVbZsOBU5HjTZnFe6Ie/GHZKarZjV6upcBE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784119821; c=relaxed/simple; bh=BM5yn8aa6nsA0MchY4B51U8iZ7cExTIA1lfWS+CBrDE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=hYKLV5bZQyg6NAOcLf9CfcAfo4/Sp/tyFTV4o8CyWBrN1GmzwiJl6hPOWsLGtMM6+vvsRL7lhYIrvTl9Iwv3p1An4ExwnRlBt1YDnIRC/bVZUzcD5vJhdgpVNS/LCDuK4wfSkMdupOqkKJz1e8dlWBIfAPJ7nZjlececj45LtGg= 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=YMRl/7Q4; arc=none smtp.client-ip=209.85.167.42 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="YMRl/7Q4" Received: by mail-lf1-f42.google.com with SMTP id 2adb3069b0e04-5b01146b205so3898740e87.2 for ; Wed, 15 Jul 2026 05:50:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784119818; x=1784724618; 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:content-type; bh=P6gSPxNU6kjOmC+K/JcDDEl1A8+wRRo1W8eJFJVQ8aU=; b=YMRl/7Q4c58i+OzyonlCRljUVCI4AMOPpYi7QR8dRdx4yF7DzxaqHBByvKXdTj9LB8 Oc6/1FBUyzNKBhF49NEb4cLM4T8WD+g3FDYNmhsZnIsLY/WCTRP5B18f5Fcp9ZP0kyN2 i9socs4Umbur1KN0sSrMdS7V/fNbnkL5ARFc/c98dVidM0Cl+M/4O5BpHPRFSvUBGyI1 NLUoZnYI7q6pWSHgAlbNi/IK3YCF0jKd/QZiuodp28EzCqA0jmp4ea7jXSSxyIy1KJnW yjJyBAb4CICnHi1gUFJ7OiRHQH5dsUFCX9NqwimXCpGGca48x4VVFdStOVO7U9t3zdrj 3u/w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784119818; x=1784724618; 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:content-type; bh=P6gSPxNU6kjOmC+K/JcDDEl1A8+wRRo1W8eJFJVQ8aU=; b=VbnIdudo4Nr7Wjz7MNRMlWo+zGEKHjE9UA3s/SnDoBFhmmrFjU29UPq1cihq/8WpMK VO6q4/pAHe2MqaO+RbfuwNXMU8VcPZ3APnP+cTXfn3PZUmfoKVTZK8W5BqXXVB/vc4zQ xQ58sJe2vxwWURwjqW7cu5s0rPYJ29O1NW0DMXyi2NScAnWN9fsrYjAnm/VBATFymwmV XUp93BOG0em/DhAgJMrBf9DaeJW+I2wPLMJZAV5FG1/jNFMF/RkPW2UcPGHKC0/MRB5C CLdLHjo5XssHwwnA5vRJ0SHhi9WaBlQsZasRNw5ngiyFV0ULjoWJeiyRKnZehBeIrVZ1 nO9g== X-Forwarded-Encrypted: i=1; AHgh+RoyFpRmqOZZuct0bNgsNSvvPck3SugQNnhCRop20wFpOvvZrMjtHXoiN67TsSfWULA7avpwEA0Te3ff0sk=@vger.kernel.org X-Gm-Message-State: AOJu0YxZn1zZZYhOuEjzpV3sRAjFHan1IQ/cSElHOS28rBZcw63MOa6d hpLYr2rrTKr+UdnDqPXS1N6WxmBQx2YuhFOI8PAbWBg7706ER2YIcTae X-Gm-Gg: AfdE7cl9hjpovwnHun/0px6TiuY6n5e3ZEEghVa/qxL0Bx7nIixuuQbi+Gs6gfaZE4F iilWO3cqF/7aQ1SIV3sWvNl/jUaozpBvzXwhkoFSupmSfRj041Fv2ARQzqIkrB0s4Da0Az6Wp+P WwKzV7qVoeuerioY2/GN4YY2VBggom9TTWUDkwWcsg/5UpxoXHJ7JU5AM6cZMbgxfH6KQl8e72K 7urKeymglSlfJ/UO1TAzK/IPKjXtjP8Jy4OLPCzF/z1gErY4C6hveOnn/V+BWod9VUwlA8C5KCV CSkY8BxzTk72FLPlmgoII6/Rz5T6S0IMYhmX655glRXp5MWJ3E3EEOXtriU9bIK3JvzhR98h0Rb R7dSxYG0SgFXihhUTeDuMDGDnuXwy+QSJSfB34uS1cec8PFB63M8tGdUoL9IYr20egzG7CakbQ9 TkobitUUdd91ivBgjDk6BAb2DQo0f/zTbeeOZRPY5FV6j+4aBRt+sOmHqNlyh1mY1RkS6RgqzQQ xye9xzplItba6CCIrRA0OCL X-Received: by 2002:a05:6512:2c8c:b0:5ae:b0ec:b024 with SMTP id 2adb3069b0e04-5b15d7b5373mr638834e87.53.1784119818261; Wed, 15 Jul 2026 05:50:18 -0700 (PDT) Received: from archlinux ([94.180.44.28]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b022fac999sm3404042e87.55.2026.07.15.05.50.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 15 Jul 2026 05:50:17 -0700 (PDT) From: Arsenii Pashchenko To: Greg Kroah-Hartman Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, ulijg308@gmail.com Subject: [PATCH 04/61] staging: rtl8723bs: remove redundant rtw_dec_to_roam helper Date: Wed, 15 Jul 2026 19:47:32 +0700 Message-ID: <20260715124829.90290-5-ulijg308@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260715124829.90290-1-ulijg308@gmail.com> References: <20260715124829.90290-1-ulijg308@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: 8bit Eliminate the redundant rtw_dec_to_roam() inline helper function by replacing it with the open-coded prefix decrement operator '--' at all call sites. Since the original helper merely decrements the mlmepriv.to_roam field and returns its updated value, using '--adapter->mlmepriv.to_roam' acts as a direct, native language equivalent. This reduces boilerplate and improves code scannability. Signed-off-by: Arsenii Pashchenko --- drivers/staging/rtl8723bs/core/rtw_mlme.c | 15 +++++---------- drivers/staging/rtl8723bs/include/rtw_mlme.h | 1 - 2 files changed, 5 insertions(+), 11 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c index 6c042de11..b038233af 100644 --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c @@ -745,7 +745,7 @@ void rtw_surveydone_event_callback(struct adapter *adapter, u8 *pbuf) rtw_indicate_connect(adapter); } else { if (adapter->mlmepriv.to_roam != 0) { - if (rtw_dec_to_roam(adapter) == 0 || + if (--adapter->mlmepriv.to_roam == 0 || rtw_sitesurvey_cmd(adapter, &pmlmepriv->assoc_ssid, 1, NULL, 0) != _SUCCESS) { rtw_set_to_roam(adapter, 0); @@ -1421,7 +1421,8 @@ void rtw_stadel_event_callback(struct adapter *adapter, u8 *pbuf) if (roam) { if (adapter->mlmepriv.to_roam > 0) - rtw_dec_to_roam(adapter); /* this stadel_event is caused by roaming, decrease to_roam */ + /* this stadel_event is caused by roaming, decrease to_roam */ + --adapter->mlmepriv.to_roam; else if (adapter->mlmepriv.to_roam == 0) rtw_set_to_roam(adapter, adapter->registrypriv.max_roaming_times); } else { @@ -1518,7 +1519,7 @@ void _rtw_join_timeout_handler(struct timer_list *t) if (adapter->mlmepriv.to_roam > 0) { /* join timeout caused by roaming */ while (1) { - rtw_dec_to_roam(adapter); + --adapter->mlmepriv.to_roam; if (adapter->mlmepriv.to_roam != 0) { /* try another */ int do_join_r; @@ -2534,12 +2535,6 @@ inline void rtw_set_to_roam(struct adapter *adapter, u8 to_roam) adapter->mlmepriv.to_roam = to_roam; } -inline u8 rtw_dec_to_roam(struct adapter *adapter) -{ - adapter->mlmepriv.to_roam--; - return adapter->mlmepriv.to_roam; -} - void rtw_roaming(struct adapter *padapter, struct wlan_network *tgt_network) { @@ -2561,7 +2556,7 @@ void _rtw_roaming(struct adapter *padapter, struct wlan_network *tgt_network) pmlmepriv->assoc_by_bssid = false; while (rtw_do_join(padapter) != _SUCCESS) { - rtw_dec_to_roam(padapter); + --padapter->mlmepriv.to_roam; if (padapter->mlmepriv.to_roam <= 0) { rtw_indicate_disconnect(padapter); break; diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme.h b/drivers/staging/rtl8723bs/include/rtw_mlme.h index b5276c429..233bf98e2 100644 --- a/drivers/staging/rtl8723bs/include/rtw_mlme.h +++ b/drivers/staging/rtl8723bs/include/rtw_mlme.h @@ -388,7 +388,6 @@ int is_same_network(struct wlan_bssid_ex *src, struct wlan_bssid_ex *dst, u8 fea void _rtw_roaming(struct adapter *adapter, struct wlan_network *tgt_network); void rtw_roaming(struct adapter *adapter, struct wlan_network *tgt_network); void rtw_set_to_roam(struct adapter *adapter, u8 to_roam); -u8 rtw_dec_to_roam(struct adapter *adapter); int rtw_select_roaming_candidate(struct mlme_priv *pmlmepriv); void rtw_sta_media_status_rpt(struct adapter *adapter, struct sta_info *psta, u32 mstatus); -- 2.55.0