From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f177.google.com (mail-pg1-f177.google.com [209.85.215.177]) (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 6581D3B635B for ; Wed, 17 Jun 2026 06:43:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781678586; cv=none; b=h7zQqI+5VSK4tNkqLKMMfjZhmJtMSRzjO2VeqzbEcJndS2ORV/aEfxUAT7eQ/R+ilnQuv6l7oZQ0AxvzaRyWFz2x0KmOuGGhxb0LbQcBPlwal0x5l0vGUa2ySqfgpggiDILxOZ+CFc9a//JH/JMCEbnS+P6NjfTI3qDzD9ryq/k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781678586; c=relaxed/simple; bh=vvDwO2bDCySlMPvDLNBWcqqr1m7dAUbx9wxY2Rzs2VM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=fdfv9rYH5DgCUPDF7cbhYrZJwkvIpFcVusGOWLtpQQm65fW6OPJ4cbSglYRb61q8KhOZtF7ZrnMLUsMejcDDKORweMRiU+L8Ri6OGQY57JQXlhN7W+8lk4pIQEjQzxJUzzRw+9Hvz4GNb8owzIqUtb8yGNgdWcTk2+QROLANvBo= 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=ByieQeqF; arc=none smtp.client-ip=209.85.215.177 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="ByieQeqF" Received: by mail-pg1-f177.google.com with SMTP id 41be03b00d2f7-c8583b8fd89so3292240a12.0 for ; Tue, 16 Jun 2026 23:43:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781678585; x=1782283385; 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=FqWyG36ay5LrncKhXJx65qkxjWOz+jvOne9pjPptPCk=; b=ByieQeqF/qB6XSTDRrw7ndH62aIb22zVyGeUvGfS9457pHpkDZX3gQaioYZDwVGy+N FTtYbJxB0yRAkek7NI7UkC8Lnz5i4Qs0MoXsvlDiF4zSf5QJ6XPRPJGVDWt4rDxavuW+ +UsHOP1bQvp+6Ld8horx+XADvb0MUjK8Ad6Bz869sfYjvT5CWUnKTVc7ao/9/in6y5qG OrfBqdJj4HL1nY9B5rTz3WxM8h4ZK61WLTykENJBe1VpcjV1pJwgSelZ+WAqlT4NL0+y ZXyoevHwzOA6kuovAGYMk1RGfefWSDNjgldaCv+SQsjU+JQ4bE1whTqi/pAScMSP3afJ 42VA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781678585; x=1782283385; 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=FqWyG36ay5LrncKhXJx65qkxjWOz+jvOne9pjPptPCk=; b=r9j79QbERKZLEGVJArJHzqGQvnExu5F4AyAD+Qnaw1ipITo8wGLVrSY0BT62NJ8kD4 eFKO0I/yQf7cdXqRbQWuTD0RbWDCHVN0mMJvbHJqcVvq/RMaoYbuZmwOXsYJaGzQTqPB lJiLiQkMXSp8jmGDD1Q4xhcIyYmDO7idDDKZP1gBRPXYchaT+yNfl302sEHWxl/DWdXE 83zIUqx76QlPRXPX+g+SGWATIOgg/n+2uDKowMrTmLnwxGKNtm6zFHE79TYTWixVEGGo I8RP645f0m0O8Z1yL0KgMC5Af3EVEZ9CcqACqFZUOFnXY30cRu7M83lE37GYJqUcByDI qJFw== X-Forwarded-Encrypted: i=1; AFNElJ9KemGJSEaep2Le2kAtDIrBCe20xXtj9HFm21QROEJzd+D+c0l2VycB8CXsUBlmBpg6ZQ6EZoq7rMp9UN4=@vger.kernel.org X-Gm-Message-State: AOJu0YzHdqM4YmwB8Gezt5yHIluhexX5Vbr6GUsdbUpzYdvqiFSct4sO NDNiolLTeB4t1CaHbH3EZUHG7/4skBw2/zFxwC/x1ZninypttUHAjKxlfbghaR8u X-Gm-Gg: Acq92OERWxoHLyXjCJ6NT9Hwlp5qMd0MN7LU39TQ8xQ75TRk2HAICFY5CQcUq4AS/Lh vYsVJLpafXEVvtD/Kzz5XBKiJ7cjTGAyn2RqFoWrbUgL2HYahlrWXFjaVIe9cRyPRgvkVnS0jPX cY+tMv8eeN+Ot1Z1LwGAPzzKWnepbFkrfdG0LGbZOO6TAD3vdPDqMRFonRm++2m1VxTiT+RgDFs ZsOewpPJu3SVGiU1GYE27A5oB9qaSYYzf2tuvJTMeXqrJH9vMorMF+ulRLaOd5uhbhVtTqBAJJo AiBRlAxgseOGF2+9JUV97x6AQyh5Sw35FXnqzxqx3osyBcfj4Fo26Dtl0jfDnmYA6WbkIIvnCsj aqRpkQPUB6XlvhFpg9aJVrOR7TGr2jek2j3BBpThpt+teGcXAr9JPt1/DmU6O/s0tosXCUWYTlW gB8OFSnp9E563LApiDJfJD97tnULafC+gDA09NtdTd7tVSTqEyvDebUYWB5ZvZylfBuc6XeJC3q A== X-Received: by 2002:a05:6a21:8cc2:b0:3a2:d838:bfc6 with SMTP id adf61e73a8af0-3b8be9afe66mr2720398637.29.1781678584724; Tue, 16 Jun 2026 23:43:04 -0700 (PDT) Received: from fedora ([103.51.149.90]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c866325d156sm13738985a12.13.2026.06.16.23.43.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 16 Jun 2026 23:43:04 -0700 (PDT) From: Subhrojyoti Bala To: Greg Kroah-Hartman Cc: Dan Carpenter , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Subhrojyoti Bala Subject: [PATCH v2 2/2] staging: rtl8723bs: Fix comment style issues in header files Date: Wed, 17 Jun 2026 12:11:50 +0530 Message-ID: <20260617064150.16078-3-subhrojyoti0609@gmail.com> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260617064150.16078-1-subhrojyoti0609@gmail.com> References: <20260617064150.16078-1-subhrojyoti0609@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 Fix various comment style issues in wlan_bssdef.h and rtw_mlme.h to comply with kernel coding style: - Use proper multi-line comment format with trailing */ on its own line - Remove extra spaces after /* in single-line comments No functional change. Signed-off-by: Subhrojyoti Bala --- v2: Combined four separate comment style patches from v1 into this single patch per Dan Carpenter's feedback. drivers/staging/rtl8723bs/include/rtw_mlme.h | 44 +++++++++---------- .../staging/rtl8723bs/include/wlan_bssdef.h | 13 +++--- 2 files changed, 29 insertions(+), 28 deletions(-) diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme.h b/drivers/staging/rtl8723bs/include/rtw_mlme.h index 403c097b46ef..950761d4740c 100644 --- a/drivers/staging/rtl8723bs/include/rtw_mlme.h +++ b/drivers/staging/rtl8723bs/include/rtw_mlme.h @@ -13,8 +13,8 @@ /* define MAX_JOIN_TIMEOUT 2500 */ #define MAX_JOIN_TIMEOUT 6500 -/* Commented by Albert 20101105 */ -/* Increase the scanning timeout because of increasing the SURVEY_TO value. */ +/* Commented by Albert 20101105 */ +/* Increase the scanning timeout because of increasing the SURVEY_TO value. */ #define SCANNING_TIMEOUT 8000 @@ -52,7 +52,7 @@ enum { dot11AuthAlgrthm_MaxNum }; -/* Scan type including active and passive scan. */ +/* Scan type including active and passive scan. */ enum rt_scan_type { SCAN_PASSIVE, SCAN_ACTIVE, @@ -67,23 +67,22 @@ enum { }; /* - -there are several "locks" in mlme_priv, -since mlme_priv is a shared resource between many threads, -like ISR/Call-Back functions, the OID handlers, and even timer functions. - -Each struct __queue has its own locks, already. -Other items in mlme_priv are protected by mlme_priv.lock, while items in -xmit_priv are protected by xmit_priv.lock. - -To avoid possible dead lock, any thread trying to modifying mlme_priv -SHALL not lock up more than one locks at a time! - -The only exception is that queue functions which take the __queue.lock -may be called with the xmit_priv.lock held. In this case the order -MUST always be first lock xmit_priv.lock and then call any queue functions -which take __queue.lock. -*/ + * There are several "locks" in mlme_priv, + * since mlme_priv is a shared resource between many threads, + * like ISR/Call-Back functions, the OID handlers, and even timer functions. + * + * Each struct __queue has its own locks, already. + * Other items in mlme_priv are protected by mlme_priv.lock, while items in + * xmit_priv are protected by xmit_priv.lock. + * + * To avoid possible dead lock, any thread trying to modifying mlme_priv + * SHALL not lock up more than one locks at a time! + * + * The only exception is that queue functions which take the __queue.lock + * may be called with the xmit_priv.lock held. In this case the order + * MUST always be first lock xmit_priv.lock and then call any queue functions + * which take __queue.lock. + */ struct sitesurvey_ctrl { u64 last_tx_pkts; @@ -184,7 +183,8 @@ struct mlme_priv { u32 wps_probe_req_ie_len; /* Number of associated Non-ERP stations (i.e., stations using 802.11b - * in 802.11g BSS) */ + * in 802.11g BSS) + */ int num_sta_non_erp; /* Number of associated stations that do not support Short Slot Time */ @@ -279,7 +279,7 @@ extern signed int rtw_set_auth(struct adapter *adapter, struct security_priv *ps static inline u8 *get_bssid(struct mlme_priv *pmlmepriv) { /* if sta_mode:pmlmepriv->cur_network.network.mac_address => bssid */ - /* if adhoc_mode:pmlmepriv->cur_network.network.mac_address => ibss mac address */ + /* if adhoc_mode:pmlmepriv->cur_network.network.mac_address => ibss mac address */ return pmlmepriv->cur_network.network.mac_address; } diff --git a/drivers/staging/rtl8723bs/include/wlan_bssdef.h b/drivers/staging/rtl8723bs/include/wlan_bssdef.h index 812a68394268..26d166f087df 100644 --- a/drivers/staging/rtl8723bs/include/wlan_bssdef.h +++ b/drivers/staging/rtl8723bs/include/wlan_bssdef.h @@ -32,9 +32,9 @@ enum ndis_802_11_network_type { }; /* - FW will only save the channel number in DSConfig. - ODI Handler will convert the channel number to freq. number. -*/ + * FW will only save the channel number in DSConfig. + * ODI Handler will convert the channel number to freq. number. + */ struct ndis_802_11_conf { u32 length; /* Length of structure */ u32 beacon_period; /* units are Kusec */ @@ -138,7 +138,8 @@ struct wlan_phy_info { struct wlan_bcn_info { /* these infor get from rtw_get_encrypt_info when - * * translate scan to UI */ + * translate scan to UI + */ u8 encryp_protocol;/* ENCRYP_PROTOCOL_E: OPEN/WEP/WPA/WPA2/WAPI */ int group_cipher; /* WPA/WPA2 group cipher */ int pairwise_cipher;/* WPA/WPA2/WEP pairwise cipher */ @@ -150,8 +151,8 @@ struct wlan_bcn_info { }; /* temporally add #pragma pack for structure alignment issue of -* struct wlan_bssid_ex and get_wlan_bssid_ex_sz() -*/ + * struct wlan_bssid_ex and get_wlan_bssid_ex_sz() + */ struct wlan_bssid_ex { u32 length; u8 mac_address[ETH_ALEN]; -- 2.54.0