From nobody Tue Dec 2 02:05:01 2025 Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) (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 3759933E7 for ; Thu, 20 Nov 2025 00:29:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.174 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763598594; cv=none; b=SOPE8o1uY9/LwpbOxifpUm87uISqDE/3dbUZnCGe7B1LSdvWVZQLIQ7LQQplfDgz/pXP/nh6DHmMPQhAqiQW4HnUvvunrgXbHKoDzXr7DuLyT5nYpEp7kfGQySq86CrNyaKQrD75RK7uqKPFEWbpG6FDb5uYnszC6TAMvillczM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763598594; c=relaxed/simple; bh=WyhterUIUeasTQj0gimyhixGcq4sH6V97ZkOD5Er1No=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=pDmQfx9rub6XrQUPIGxjhgh/OePHC7eCg5AuyxETV7f/qaProUbFhGk8HvZwu11P0Dk7kpkmkSlJDdar/uymuWtDJ8E4mB+r1GJjeJvIiH2E3M5LwfS54fFC3jCBHNqzvIcr2EwIpHyA0o5Y3/BTKRBFX1VqXap9GbaAE50umAY= 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=J56ILwyX; arc=none smtp.client-ip=209.85.214.174 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="J56ILwyX" Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-295548467c7so3583575ad.2 for ; Wed, 19 Nov 2025 16:29:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1763598591; x=1764203391; 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=fRrGUmaovt0R97si4Ukyb1q4iVfc43FtiDhdXgbGoYI=; b=J56ILwyXdlipDFfi+/+xvS01G/qGaqrQaOC6RFSnGLA/EH7rMD/RobQbJZcLMh2aZE OkpWxWy06UWAqvfAUrSxGCdzisveo2XYzM/WpU+zlPk8S2eO817NfcKJD7iVE30waGVd I3rWmU6S1LanFdA48lkZDOgSlYM+/zepgoEE93bdNWhCKhT0VC/aYqBR24wgU2+zEsmy s9IH1a32F1LgGFEPzgKB08ENXiJY7wFXIijL+c3Fh51SR5DSmP2Vg5stbyYbBsmlcgSR Eo7v5eRHlbatSOhlafV4tpX2sk7vrm/0+N843x3e7NoCCFaqc3j7BXeVyoxepnE/mUX0 SLuQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763598591; x=1764203391; 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=fRrGUmaovt0R97si4Ukyb1q4iVfc43FtiDhdXgbGoYI=; b=iIZNfuP72aQgT/Vh/zJuFFiZgV/bJSj2ZUtiWVNBFLZp8EZxpe9i5B/eL3GJ2zz/Nj JMLtKwyPst2m7s3GhjMQ4R/OZJ2QnPofSXwkyU/HbGdBez7XDjW9XX0gmgRlDxqygSbI Frxo9rQwix/968Ct+9dDVISTJbfQ1tFZ5L+S51xDhFFdjpAN1hz9ezTbPgTzuiGmU0gd shJBFVaeBHTsuK0kwhTX4ntCxHPdTj/l9V16ZIpsgbfpqpZZX4YK7oZ0mwQE9Blv/UKU bauw2RR+Otat6ASQx7nT+hCqB1+96BlI5Olwn7otcPX+y+I38e4yysyXgXCZ598/dGuT ifag== X-Forwarded-Encrypted: i=1; AJvYcCUIxT2I6WkWET63eZ1ySgy+uLaRbdRxrSnvlSNk761aKDPgGGsZuODn6BbRd6+2k5PWJ6KhgP9dk7/cE3s=@vger.kernel.org X-Gm-Message-State: AOJu0Yy6nJ30kLa1cLg9vXvdMtO01+gC+IEu+wPa3AW+L7Y6tMmGo1Zm rmr5i4nQLn2gbxY7xo98FTTuHBlp8K/aLmRV5b79dhggblVF7y5st+O2 X-Gm-Gg: ASbGncu/jz8m3lQVCsKbPPrKO0v5zUZKnDlLDYw/SFzarHq5ANXXJ0/mUTGNeJZ5AAy AZGNvc2kcDF9KqeuUCAagAkNcRdIg3OWunCXejz4R8ZukYgetcGGNieUMWtDABgHk9HjWCHpO4D YtZMRV6FompPL0fvwp0KHrprMV2R4PaEpdkxnhKvJj8URX4+bVpCOHM6ZxTcESU5suMj3I1oH45 ijopSZNrEPivTOKkLNlRSpqNiQfbtug7++V5+1aTygUxn8U4kHvIz4tHea4a/52yss5uCxh4jVw 9MZOsDyr0Fz7XOKeTIdaUGaLs7iY82MMznD3yVGcqDXcFnMwbpsQEUIXqHjKDIYZbUvOpYY1tVz 8U6QHDqTpZMCC+NZNFDcYOPDkti/5433gwk41qSTx49TzGL+BPFVSx3V4UjVvWGK4MWP5PA9XNv lu6PZARfFJ6p954zr5fU/GvHqx/MofiQLLHTMzGbnh96ZOkXoMDwA93kK+QMyNbdLdIIb8xRfq4 BB7cKljYiqDl8xUF3VX730BMrBIRb4= X-Google-Smtp-Source: AGHT+IF1tdHuWiDx7UE0cag+8oLtqzPGUiXWjyLf4ihC3TB/qGGKm5C39cH2S9YsHt/qrXbmkvV7cA== X-Received: by 2002:a17:903:3843:b0:297:c272:80ec with SMTP id d9443c01a7336-29b5ec70596mr2736075ad.42.1763598591384; Wed, 19 Nov 2025 16:29:51 -0800 (PST) Received: from main.. (c-76-146-85-33.hsd1.wa.comcast.net. [76.146.85.33]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-29b5b25e579sm6245665ad.51.2025.11.19.16.29.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Nov 2025 16:29:50 -0800 (PST) From: Artur Stupa To: gregkh@linuxfoundation.org Cc: linux-staging@lists.linux.dev, dan.carpenter@linaro.org, linux-kernel@vger.kernel.org, arthur.stupa@gmail.com Subject: [PATCH v2] staging: rtl8723bs: core: fix block comment style issues Date: Wed, 19 Nov 2025 16:29:21 -0800 Message-ID: <20251120002948.23153-1-arthur.stupa@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20251119062701.38279-1-arthur.stupa@gmail.com> References: <20251119062701.38279-1-arthur.stupa@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" Fix the following checkpatch warnings for block comments: * Block comments should align the * on each line * Block comments use * on subsequent lines * Block comments use a trailing */ on a separate line Also delete dead commented-out code. No functional changes. Signed-off-by: Artur Stupa --- Changes in v2: * Fixed "trailing */" checkpatch warnings * Deleted dead commented-out code --- .../staging/rtl8723bs/core/rtw_ieee80211.c | 22 +++-- drivers/staging/rtl8723bs/core/rtw_io.c | 48 +++++------ drivers/staging/rtl8723bs/core/rtw_mlme.c | 76 ++++++++--------- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 59 +++---------- drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 10 +-- drivers/staging/rtl8723bs/core/rtw_security.c | 85 +++++++++---------- drivers/staging/rtl8723bs/core/rtw_sta_mgt.c | 8 +- .../staging/rtl8723bs/core/rtw_wlan_util.c | 15 +--- 8 files changed, 138 insertions(+), 185 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c b/drivers/stagi= ng/rtl8723bs/core/rtw_ieee80211.c index 027b2dd0a4b9..10911a661223 100644 --- a/drivers/staging/rtl8723bs/core/rtw_ieee80211.c +++ b/drivers/staging/rtl8723bs/core/rtw_ieee80211.c @@ -132,9 +132,7 @@ u8 *rtw_set_ie(u8 *pbuf, return pbuf + len + 2; } =20 -/*------------------------------------------------------------------------= ---- -index: the information element id index, limit is the limit for search ---------------------------------------------------------------------------= ---*/ +/* index: the information element id index, limit is the limit for search = */ u8 *rtw_get_ie(u8 *pbuf, signed int index, signed int *len, signed int lim= it) { signed int tmp, i; @@ -767,21 +765,27 @@ static int rtw_ieee802_11_parse_vendor_specific(u8 *p= os, uint elen, { unsigned int oui; =20 - /* first 3 bytes in vendor specific information element are the IEEE + /* + * first 3 bytes in vendor specific information element are the IEEE * OUI of the vendor. The following byte is used a vendor specific - * sub-type. */ + * sub-type. + */ if (elen < 4) return -1; =20 oui =3D get_unaligned_be24(pos); switch (oui) { case OUI_MICROSOFT: - /* Microsoft/Wi-Fi information elements are further typed and - * subtyped */ + /* + * Microsoft/Wi-Fi information elements are further typed and + * subtyped + */ switch (pos[3]) { case 1: - /* Microsoft OUI (00:50:F2) with OUI Type 1: - * real WPA information element */ + /* + * Microsoft OUI (00:50:F2) with OUI Type 1: + * real WPA information element + */ elems->wpa_ie =3D pos; elems->wpa_ie_len =3D elen; break; diff --git a/drivers/staging/rtl8723bs/core/rtw_io.c b/drivers/staging/rtl8= 723bs/core/rtw_io.c index 79d543d88278..fe9f94001eed 100644 --- a/drivers/staging/rtl8723bs/core/rtw_io.c +++ b/drivers/staging/rtl8723bs/core/rtw_io.c @@ -5,25 +5,23 @@ * *************************************************************************= *****/ /* - -The purpose of rtw_io.c - -a. provides the API - -b. provides the protocol engine - -c. provides the software interface between caller and the hardware interfa= ce - - -Compiler Flag Option: - -1. CONFIG_SDIO_HCI: - a. USE_SYNC_IRP: Only sync operations are provided. - b. USE_ASYNC_IRP:Both sync/async operations are provided. - -jackson@realtek.com.tw - -*/ + * The purpose of rtw_io.c + * + * a. provides the API + * + * b. provides the protocol engine + * + * c. provides the software interface between caller and the hardware inte= rface + * + * + * Compiler Flag Option: + * + * 1. CONFIG_SDIO_HCI: + * a. USE_SYNC_IRP: Only sync operations are provided. + * b. USE_ASYNC_IRP:Both sync/async operations are provided. + * + * jackson@realtek.com.tw + */ =20 #include =20 @@ -135,10 +133,10 @@ int rtw_init_io_priv(struct adapter *padapter, void (= *set_intf_ops)(struct adapt } =20 /* -* Increase and check if the continual_io_error of this @param dvobjprive i= s larger than MAX_CONTINUAL_IO_ERR -* @return true: -* @return false: -*/ + * Increase and check if the continual_io_error of this @param dvobjprive = is larger than MAX_CONTINUAL_IO_ERR + * @return true: + * @return false: + */ int rtw_inc_and_chk_continual_io_error(struct dvobj_priv *dvobj) { int error_count =3D atomic_inc_return(&dvobj->continual_io_error); @@ -149,9 +147,7 @@ int rtw_inc_and_chk_continual_io_error(struct dvobj_pri= v *dvobj) return false; } =20 -/* -* Set the continual_io_error of this @param dvobjprive to 0 -*/ +/* Set the continual_io_error of this @param dvobjprive to 0 */ void rtw_reset_continual_io_error(struct dvobj_priv *dvobj) { atomic_set(&dvobj->continual_io_error, 0); diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rt= l8723bs/core/rtw_mlme.c index c06d990350e6..98704179ad35 100644 --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c @@ -214,10 +214,10 @@ void _rtw_free_network_nolock(struct mlme_priv *pmlme= priv, struct wlan_network * } =20 /* - return the wlan_network with the matching addr - - Shall be called under atomic context... to avoid possible racing conditio= n... -*/ + * return the wlan_network with the matching addr + * + * Shall be called under atomic context... to avoid possible racing condit= ion... + */ struct wlan_network *_rtw_find_network(struct __queue *scanned_queue, u8 *= addr) { struct list_head *phead, *plist; @@ -319,10 +319,10 @@ void rtw_free_network_nolock(struct adapter *padapter= , struct wlan_network *pnet } =20 /* - return the wlan_network with the matching addr - - Shall be called under atomic context... to avoid possible racing conditio= n... -*/ + * return the wlan_network with the matching addr + * + * Shall be called under atomic context... to avoid possible racing condit= ion... + */ struct wlan_network *rtw_find_network(struct __queue *scanned_queue, u8 *a= ddr) { struct wlan_network *pnetwork =3D _rtw_find_network(scanned_queue, addr); @@ -476,9 +476,7 @@ static void update_current_network(struct adapter *adap= ter, struct wlan_bssid_ex } } =20 -/* -Caller must hold pmlmepriv->lock first. -*/ +/* Caller must hold pmlmepriv->lock first. */ void rtw_update_scanned_network(struct adapter *adapter, struct wlan_bssid= _ex *target) { struct list_head *plist, *phead; @@ -510,8 +508,10 @@ void rtw_update_scanned_network(struct adapter *adapte= r, struct wlan_bssid_ex *t oldest =3D pnetwork; } =20 - /* If we didn't find a match, then get a new network slot to initialize - * with this beacon's information */ + /* + * If we didn't find a match, then get a new network slot to initialize + * with this beacon's information + */ if (!target_find) { if (list_empty(&pmlmepriv->free_bss_pool.queue)) { /* If there are no more slots, expire the oldest */ @@ -843,9 +843,7 @@ static void find_network(struct adapter *adapter) rtw_free_network_nolock(adapter, pwlan); } =20 -/* -*rtw_free_assoc_resources: the caller has to lock pmlmepriv->lock -*/ +/* rtw_free_assoc_resources: the caller has to lock pmlmepriv->lock */ void rtw_free_assoc_resources(struct adapter *adapter, int lock_scanned_qu= eue) { struct mlme_priv *pmlmepriv =3D &adapter->mlmepriv; @@ -879,9 +877,7 @@ void rtw_free_assoc_resources(struct adapter *adapter, = int lock_scanned_queue) rtw_reset_rx_info(pdbgpriv); } =20 -/* -*rtw_indicate_connect: the caller has to lock pmlmepriv->lock -*/ +/* rtw_indicate_connect: the caller has to lock pmlmepriv->lock */ void rtw_indicate_connect(struct adapter *padapter) { struct mlme_priv *pmlmepriv =3D &padapter->mlmepriv; @@ -908,9 +904,7 @@ void rtw_indicate_connect(struct adapter *padapter) rtw_set_scan_deny(padapter, 3000); } =20 -/* -*rtw_indicate_disconnect: the caller has to lock pmlmepriv->lock -*/ +/* rtw_indicate_disconnect: the caller has to lock pmlmepriv->lock */ void rtw_indicate_disconnect(struct adapter *padapter) { struct mlme_priv *pmlmepriv =3D &padapter->mlmepriv; @@ -1543,9 +1537,9 @@ void rtw_wmm_event_callback(struct adapter *padapter,= u8 *pbuf) } =20 /* -* _rtw_join_timeout_handler - Timeout/failure handler for CMD JoinBss -* @adapter: pointer to struct adapter structure -*/ + * _rtw_join_timeout_handler - Timeout/failure handler for CMD JoinBss + * @adapter: pointer to struct adapter structure + */ void _rtw_join_timeout_handler(struct timer_list *t) { struct adapter *adapter =3D timer_container_of(adapter, t, @@ -1586,9 +1580,9 @@ void _rtw_join_timeout_handler(struct timer_list *t) } =20 /* -* rtw_scan_timeout_handler - Timeout/Failure handler for CMD SiteSurvey -* @adapter: pointer to struct adapter structure -*/ + * rtw_scan_timeout_handler - Timeout/Failure handler for CMD SiteSurvey + * @adapter: pointer to struct adapter structure + */ void rtw_scan_timeout_handler(struct timer_list *t) { struct adapter *adapter =3D timer_container_of(adapter, t, @@ -1704,10 +1698,10 @@ void rtw_set_scan_deny(struct adapter *adapter, u32= ms) } =20 /* -* Select a new roaming candidate from the original @param candidate and @p= aram competitor -* @return true: candidate is updated -* @return false: candidate is not updated -*/ + * Select a new roaming candidate from the original @param candidate and @= param competitor + * @return true: candidate is updated + * @return false: candidate is not updated + */ static int rtw_check_roaming_candidate(struct mlme_priv *mlme , struct wlan_network **candidate, struct wlan_network *competitor) { @@ -1785,10 +1779,10 @@ int rtw_select_roaming_candidate(struct mlme_priv *= mlme) } =20 /* -* Select a new join candidate from the original @param candidate and @para= m competitor -* @return true: candidate is updated -* @return false: candidate is not updated -*/ + * Select a new join candidate from the original @param candidate and @par= am competitor + * @return true: candidate is updated + * @return false: candidate is not updated + */ static int rtw_check_join_candidate(struct mlme_priv *mlme , struct wlan_network **candidate, struct wlan_network *competitor) { @@ -1829,11 +1823,11 @@ static int rtw_check_join_candidate(struct mlme_pri= v *mlme } =20 /* -Calling context: -The caller of the sub-routine will be in critical section... -The caller must hold the following spinlock -pmlmepriv->lock -*/ + * Calling context: + * The caller of the sub-routine will be in critical section... + * The caller must hold the following spinlock + * pmlmepriv->lock + */ =20 int rtw_select_and_join_from_scanned_queue(struct mlme_priv *pmlmepriv) { diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/stagin= g/rtl8723bs/core/rtw_mlme_ext.c index 8fe0555ab033..016609cd1292 100644 --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c @@ -18,9 +18,7 @@ static struct mlme_handler mlme_sta_tbl[] =3D { {WIFI_PROBEREQ, "OnProbeReq", &OnProbeReq}, {WIFI_PROBERSP, "OnProbeRsp", &OnProbeRsp}, =20 - /*---------------------------------------------------------- - below 2 are reserved - -----------------------------------------------------------*/ + /* below 2 are reserved */ {0, "DoReserved", &DoReserved}, {0, "DoReserved", &DoReserved}, {WIFI_BEACON, "OnBeacon", &OnBeacon}, @@ -50,9 +48,7 @@ static struct action_handler OnAction_tbl[] =3D { =20 static u8 null_addr[ETH_ALEN] =3D {0, 0, 0, 0, 0, 0}; =20 -/************************************************** -OUI definitions for the vendor specific IE -***************************************************/ +/* OUI definitions for the vendor specific IE */ unsigned char RTW_WPA_OUI[] =3D {0x00, 0x50, 0xf2, 0x01}; unsigned char WMM_OUI[] =3D {0x00, 0x50, 0xf2, 0x02}; unsigned char WPS_OUI[] =3D {0x00, 0x50, 0xf2, 0x04}; @@ -64,9 +60,7 @@ unsigned char WMM_PARA_OUI[] =3D {0x00, 0x50, 0xf2, 0x02,= 0x01, 0x01}; =20 static unsigned char REALTEK_96B_IE[] =3D {0x00, 0xe0, 0x4c, 0x02, 0x01, 0= x20}; =20 -/******************************************************** -ChannelPlan definitions -*********************************************************/ +/* ChannelPlan definitions */ static struct rt_channel_plan_2g RTW_ChannelPlan2G[RT_CHANNEL_DOMAIN_2G_MA= X] =3D { {{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13}, 13}, /* 0x00, RT_CHANNEL_= DOMAIN_2G_WORLD , Passive scan CH 12, 13 */ {{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13}, 13}, /* 0x01, RT_CHANNEL_= DOMAIN_2G_ETSI1 */ @@ -187,11 +181,7 @@ int rtw_ch_set_search_ch(struct rt_channel_info *ch_se= t, const u32 ch) return i; } =20 -/*************************************************************************= *** - -Following are the initialization functions for WiFi MLME - -**************************************************************************= ***/ +/* Following are the initialization functions for WiFi MLME */ =20 int init_hw_mlme_ext(struct adapter *padapter) { @@ -507,11 +497,7 @@ void mgt_dispatcher(struct adapter *padapter, union re= cv_frame *precv_frame) } } =20 -/*************************************************************************= *** - -Following are the callback functions for each subtype of the management fr= ames - -**************************************************************************= ***/ +/* Following are the callback functions for each subtype of the management= frames */ =20 unsigned int OnProbeReq(struct adapter *padapter, union recv_frame *precv_= frame) { @@ -1950,11 +1936,7 @@ inline struct xmit_frame *alloc_mgtxmitframe(struct = xmit_priv *pxmitpriv) return _alloc_mgtxmitframe(pxmitpriv, false); } =20 -/*************************************************************************= *** - -Following are some TX functions for WiFi MLME - -**************************************************************************= ***/ +/* Following are some TX functions for WiFi MLME */ =20 void update_mgnt_tx_rate(struct adapter *padapter, u8 rate) { @@ -3797,11 +3779,7 @@ unsigned int send_beacon(struct adapter *padapter) return _SUCCESS; } =20 -/*************************************************************************= *** - -Following are some utility functions for WiFi MLME - -**************************************************************************= ***/ +/* Following are some utility functions for WiFi MLME */ =20 void site_survey(struct adapter *padapter) { @@ -4392,11 +4370,7 @@ static void process_80211d(struct adapter *padapter,= struct wlan_bssid_ex *bssid } } =20 -/*************************************************************************= *** - -Following are the functions to report events - -**************************************************************************= ***/ +/* Following are the functions to report events */ =20 void report_survey_event(struct adapter *padapter, union recv_frame *precv= _frame) { @@ -4692,11 +4666,7 @@ void report_add_sta_event(struct adapter *padapter, = unsigned char *MacAddr, int rtw_enqueue_cmd(pcmdpriv, pcmd_obj); } =20 -/*************************************************************************= *** - -Following are the event callback functions - -**************************************************************************= ***/ +/* Following are the event callback functions */ =20 /* for sta/adhoc mode */ void update_sta_info(struct adapter *padapter, struct sta_info *psta) @@ -4863,8 +4833,10 @@ void mlmeext_joinbss_event_callback(struct adapter *= padapter, int join_res) =20 rtw_sta_media_status_rpt(padapter, psta, 1); =20 - /* wakeup macid after join bss successfully to ensure - the subsequent data frames can be sent out normally */ + /* + * wakeup macid after join bss successfully to ensure + * the subsequent data frames can be sent out normally + */ rtw_hal_macid_wakeup(padapter, psta->mac_id); } =20 @@ -4940,11 +4912,8 @@ void mlmeext_sta_del_event_callback(struct adapter *= padapter) rtw_mlmeext_disconnect(padapter); } =20 -/*************************************************************************= *** - -Following are the functions for the timer handlers +/* Following are the functions for the timer handlers */ =20 -**************************************************************************= ***/ void _linked_info_dump(struct adapter *padapter) { int i; diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging= /rtl8723bs/core/rtw_pwrctrl.c index 7b643ac320f0..0ef788abf403 100644 --- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c +++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c @@ -999,11 +999,11 @@ inline void rtw_set_ips_deny(struct adapter *padapter= , u32 ms) } =20 /* -* rtw_pwr_wakeup - Wake the NIC up from: 1)IPS. 2)USB autosuspend -* @adapter: pointer to struct adapter structure -* @ips_deffer_ms: the ms will prevent from falling into IPS after wakeup -* Return _SUCCESS or _FAIL -*/ + * rtw_pwr_wakeup - Wake the NIC up from: 1)IPS. 2)USB autosuspend + * @adapter: pointer to struct adapter structure + * @ips_deffer_ms: the ms will prevent from falling into IPS after wakeup + * Return _SUCCESS or _FAIL + */ =20 int _rtw_pwr_wakeup(struct adapter *padapter, u32 ips_deffer_ms, const cha= r *caller) { diff --git a/drivers/staging/rtl8723bs/core/rtw_security.c b/drivers/stagin= g/rtl8723bs/core/rtw_security.c index b2c783ada49b..9a74f90e1d3c 100644 --- a/drivers/staging/rtl8723bs/core/rtw_security.c +++ b/drivers/staging/rtl8723bs/core/rtw_security.c @@ -30,9 +30,7 @@ const char *security_type_str(u8 value) =20 /* WEP related =3D=3D=3D=3D=3D */ =20 -/* - Need to consider the fragment situation -*/ +/* Need to consider the fragment situation */ void rtw_wep_encrypt(struct adapter *padapter, u8 *pxmitframe) { /* exclude ICV */ union { @@ -343,23 +341,23 @@ static const unsigned short Sbox1[2][256] =3D { = /* Sbox for hash (can be in R } }; =20 - /* -********************************************************************** -* Routine: Phase 1 -- generate P1K, given TA, TK, IV32 -* -* Inputs: -* tk[] =3D temporal key [128 bits] -* ta[] =3D transmitter's MAC address [ 48 bits] -* iv32 =3D upper 32 bits of IV [ 32 bits] -* Output: -* p1k[] =3D Phase 1 key [ 80 bits] -* -* Note: -* This function only needs to be called every 2**16 packets, -* although in theory it could be called every packet. -* -********************************************************************** -*/ +/* + ********************************************************************** + * Routine: Phase 1 -- generate P1K, given TA, TK, IV32 + * + * Inputs: + * tk[] =3D temporal key [128 bits] + * ta[] =3D transmitter's MAC address [ 48 bits] + * iv32 =3D upper 32 bits of IV [ 32 bits] + * Output: + * p1k[] =3D Phase 1 key [ 80 bits] + * + * Note: + * This function only needs to be called every 2**16 packets, + * although in theory it could be called every packet. + * + ********************************************************************** + */ static void phase1(u16 *p1k, const u8 *tk, const u8 *ta, u32 iv32) { signed int i; @@ -386,28 +384,28 @@ static void phase1(u16 *p1k, const u8 *tk, const u8 *= ta, u32 iv32) =20 =20 /* -********************************************************************** -* Routine: Phase 2 -- generate RC4KEY, given TK, P1K, IV16 -* -* Inputs: -* tk[] =3D Temporal key [128 bits] -* p1k[] =3D Phase 1 output key [ 80 bits] -* iv16 =3D low 16 bits of IV counter [ 16 bits] -* Output: -* rc4key[] =3D the key used to encrypt the packet [128 bits] -* -* Note: -* The value {TA, IV32, IV16} for Phase1/Phase2 must be unique -* across all packets using the same key TK value. Then, for a -* given value of TK[], this TKIP48 construction guarantees that -* the final RC4KEY value is unique across all packets. -* -* Suggested implementation optimization: if PPK[] is "overlaid" -* appropriately on RC4KEY[], there is no need for the final -* for loop below that copies the PPK[] result into RC4KEY[]. -* -********************************************************************** -*/ + ********************************************************************** + * Routine: Phase 2 -- generate RC4KEY, given TK, P1K, IV16 + * + * Inputs: + * tk[] =3D Temporal key [128 bits] + * p1k[] =3D Phase 1 output key [ 80 bits] + * iv16 =3D low 16 bits of IV counter [ 16 bits] + * Output: + * rc4key[] =3D the key used to encrypt the packet [128 bits] + * + * Note: + * The value {TA, IV32, IV16} for Phase1/Phase2 must be unique + * across all packets using the same key TK value. Then, for a + * given value of TK[], this TKIP48 construction guarantees that + * the final RC4KEY value is unique across all packets. + * + * Suggested implementation optimization: if PPK[] is "overlaid" + * appropriately on RC4KEY[], there is no need for the final + * for loop below that copies the PPK[] result into RC4KEY[]. + * + ********************************************************************** + */ static void phase2(u8 *rc4key, const u8 *tk, const u16 *p1k, u16 iv16) { signed int i; @@ -1483,7 +1481,8 @@ static int omac1_aes_128_vector(u8 *key, size_t num_e= lem, * This is a mode for using block cipher (AES in this case) for authentica= tion. * OMAC1 was standardized with the name CMAC by NIST in a Special Publicat= ion * (SP) 800-38B. - * modify for CONFIG_IEEE80211W */ + * modify for CONFIG_IEEE80211W + */ int omac1_aes_128(u8 *key, u8 *data, size_t data_len, u8 *mac) { return omac1_aes_128_vector(key, 1, &data, &data_len, mac); diff --git a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c b/drivers/staging= /rtl8723bs/core/rtw_sta_mgt.c index d1f6030799cb..2014998245db 100644 --- a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c +++ b/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c @@ -385,10 +385,10 @@ u32 rtw_free_stainfo(struct adapter *padapter, struct= sta_info *psta) rtw_release_macid(pstapriv->padapter, psta); =20 /* - spin_lock_bh(&pstapriv->asoc_list_lock); - list_del_init(&psta->asoc_list); - spin_unlock_bh(&pstapriv->asoc_list_lock); -*/ + * spin_lock_bh(&pstapriv->asoc_list_lock); + * list_del_init(&psta->asoc_list); + * spin_unlock_bh(&pstapriv->asoc_list_lock); + */ spin_lock_bh(&pstapriv->auth_list_lock); if (!list_empty(&psta->auth_list)) { list_del_init(&psta->auth_list); diff --git a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c b/drivers/stagi= ng/rtl8723bs/core/rtw_wlan_util.c index 74d3f2de360d..5ffefa50699e 100644 --- a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c +++ b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c @@ -1021,9 +1021,9 @@ void HTOnAssocRsp(struct adapter *padapter) =20 /* handle A-MPDU parameter field */ /* - AMPDU_para [1:0]:Max AMPDU Len =3D> 0:8k , 1:16k, 2:32k, 3:64k - AMPDU_para [4:2]:Min MPDU Start Spacing - */ + * AMPDU_para [1:0]:Max AMPDU Len =3D> 0:8k , 1:16k, 2:32k, 3:64k + * AMPDU_para [4:2]:Min MPDU Start Spacing + */ max_AMPDU_len =3D pmlmeinfo->HT_caps.u.HT_cap_element.AMPDU_para & 0x03; =20 min_MPDU_spacing =3D (pmlmeinfo->HT_caps.u.HT_cap_element.AMPDU_para & 0x= 1c) >> 2; @@ -1689,15 +1689,6 @@ void adaptive_early_32k(struct mlme_ext_priv *pmlmee= xt, u8 *pframe, uint len) else pmlmeext->bcn_delay_cnt[delay_ms]++; /* pmlmeext->bcn_delay_ratio[delay_ms] =3D (pmlmeext->bcn_delay_cnt[dela= y_ms] * 100) /pmlmeext->bcn_cnt; */ - -/* - - for (i =3D 0; i<9; i++) - { - pmlmeext->bcn_delay_cnt[i] , i, pmlmeext->bcn_delay_ratio[i]); - } -*/ - /* dump for adaptive_early_32k */ if (pmlmeext->bcn_cnt > 100 && (pmlmeext->adaptive_tsf_done =3D=3D true))= { u8 ratio_20_delay, ratio_80_delay; --=20 2.43.0