From nobody Sun Jun 14 08:34:36 2026 Received: from mail-qk1-f177.google.com (mail-qk1-f177.google.com [209.85.222.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 2E6072D94B0 for ; Thu, 2 Apr 2026 02:37:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.177 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775097446; cv=none; b=qjoihDQ7mmqspdTRhZxUE42SUeZj+8hCWbKNI6o/B7dLV/jov3QsgIxeBhRDucfNtOkE0ydzc5jlWn9sT8iIlE6eTgXvvROSJsfl3oI7DzQFblCmrOAMmsug3jhX/5nNYft7u3wHBepPxPSe5wdHXvVPtb3kCSHQrkCNa6Cr20M= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775097446; c=relaxed/simple; bh=LnSdXU0cUk6dvOdLB2FKma0Z0V95s6PSoTm9de7MefQ=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=T2CgGA07IWpwsFbOHss7lzJJpZZtSbBk6UJc+qVIBUSi6OU77NHY62Put+x6AFmWqSY9ArIJVExjAR+/x4J9Gr+SEzKGiMMKDAhSaJSecewckV2GqYFGq31cUX3Bw2c2C7lUEMNIXuod2xrnUlhZRWTEx+PNmviC+qXTvV1sAN8= 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=cnvfXo2R; arc=none smtp.client-ip=209.85.222.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="cnvfXo2R" Received: by mail-qk1-f177.google.com with SMTP id af79cd13be357-8cd8a189f44so42543085a.0 for ; Wed, 01 Apr 2026 19:37:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775097443; x=1775702243; 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=pR+OpKAh2Pk4S3UPukURdxFWxo7C+WaI0tvEXW5llSE=; b=cnvfXo2RaQVV5bc/VdUV/QuaBfUBitoiG/15gTkCwVOaLRV3AS/KU188c6d7kFDjrd 5dXstnnKJo+dHfTifnrUwq/3VRJvAtguEXyMtTw6YXQE/VvWeXWqFdZESKzYf2yI1sAv L6K/+V4Qqj8snb69qzs13utIxIwOFqWtHAFZNR/2Ak3scNU7lLz08wui25SYP1YFMpmc me0k9cKBKAw0d0Sf3t1vX5WaJdh8nRYMzArCOune3FouWASz1aYIM6xkntQHp3Z6j3uv GXWRbV2FZ6CK1nwUZbh3xZliSkMuJaG0NmgJmcSr/i8MEmoApn47LFFT+55lWkCygzGB qIxg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775097443; x=1775702243; 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=pR+OpKAh2Pk4S3UPukURdxFWxo7C+WaI0tvEXW5llSE=; b=AwS0SZQj4l2EzOQgyfWSwe3fu0rssAKAsrgckyWkpAMVZ3UYOMyDjLvvmX0qf+6De1 isHuiXjMY5ey9p3LNYIqdfKVjodgUEA0/tU6D85sQOtLzPvioOs/uIyc7JYnp5kOylWq 2dqENbIae+XcxvAssN04n+nfvqqJks0he4vADQNOZlo37rEEhEWybvCJk+YsiUYXtysa nKI/XoAfnodmIKev72iq4tnz+lsWPhjilnyQsoLE0/0+fdUGrovS7xPM9+lZdfT+PqCX IS2Tu+DhxaRxXLUjivx39h9FLE/2Ri2J9OhoUbEW8vNiRTAAMs4MNQf8gp94O4OnHIgN xc9w== X-Forwarded-Encrypted: i=1; AJvYcCUTYP8H7GgEXPV0yIL4PX2I0ZHDM98Fqo/TQrfLKVLjx6g+59FBufGG5EwV4CMeFKjuLPdt1rLZRlnRAKI=@vger.kernel.org X-Gm-Message-State: AOJu0YyRgPjAi5O47a7o10W5FqUtATO5J5WMvQnNnIlvjoCLvRb2NFBP D208SSbgX+7ETiTvNFP/+XVaR82yc2EwYf4XFJv1mKr9Zo6HwmGQ8Jlj X-Gm-Gg: ATEYQzwN6DMHExF/mNIOHDKhdtnWGTb/2vji61gg2kY2qz7uhwgp+nqncKQC/nM/nPn ig00A3+HvOyvPbU5XQ6w1kaW4xks/tyXf4g3i4stbgkkLW01mTg7vHBJFlqOZoJMIcK3kXVpfPR G2KlNNgdUxf/Grn4fqTxnh0Fuy7KrN6tWCNr/BG738oxcNyqX5zskDguckQJl+f+scQRhaxk1X9 ZfpubhGT0iNIYmPYHM3O77mYZx1ymRZc3GrmiiYGibiHGfKWxrXLVVoZAqXzHQiMSKB8SrU7KYL tmjfR4PQHLNc6Tab4aGQpi8YDYe6c8dnWiM9Vt1ThRHyxC8Q/Cxeky9G3gkBhgM2CweMPiq++1l cTvI50NYFMlpSIEJfuP84cQbaY+ncFOZXg3hsxEuSiT/RcGuFPBSiILylJlii0kPNUUWD2fOH X-Received: by 2002:a05:620a:d82:b0:8cf:c1c2:90f with SMTP id af79cd13be357-8d1b5ac5811mr909059785a.7.1775097442909; Wed, 01 Apr 2026 19:37:22 -0700 (PDT) Received: from r9 ([2601:5c2:100:2f60::418e]) by smtp.gmail.com with ESMTPSA id af79cd13be357-8d2a5392ce8sm139210185a.7.2026.04.01.19.37.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Apr 2026 19:37:22 -0700 (PDT) From: Xiyuan Guo To: gregkh@linuxfoundation.org Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, b9788213@gmail.com, Xiyuan Guo Subject: [PATCH v2] staging: rtl8723bs: remove GEN_CMD_CODE macro and callback array Date: Wed, 1 Apr 2026 22:35:59 -0400 Message-ID: <20260402023621.142194-1-tommyguo039@gmail.com> X-Mailer: git-send-email 2.53.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 GEN_CMD_CODE macro and the rtw_cmd_callback function pointer array add unnecessary layers of indirection and make the code difficult to follow. Remove the GEN_CMD_CODE macro and replace the function pointer array dispatcher in rtw_cmd_thread() with a direct switch statement. Additionally, rename the associated command enums from CamelCase to UPPERCASE_SNAKE_CASE to comply with the standard Linux kernel coding style. Signed-off-by: Xiyuan Guo --- Changes in v2: - Fixed compilation error caused by missed CamelCase conversions. - Restored accidentally deleted todo comment in rtw_cmd_thread(). - Renamed command enums from CamelCase to UPPERCASE_SNAKE_CASE. drivers/staging/rtl8723bs/core/rtw_ap.c | 4 +- drivers/staging/rtl8723bs/core/rtw_cmd.c | 157 ++++++----------- drivers/staging/rtl8723bs/core/rtw_mlme.c | 4 +- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 20 +-- drivers/staging/rtl8723bs/include/rtw_cmd.h | 164 +++++++++--------- .../staging/rtl8723bs/include/rtw_mlme_ext.h | 14 +- 6 files changed, 148 insertions(+), 215 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c b/drivers/staging/rtl8= 723bs/core/rtw_ap.c index 4cdcdddf6b33..b9586b339095 100644 --- a/drivers/staging/rtl8723bs/core/rtw_ap.c +++ b/drivers/staging/rtl8723bs/core/rtw_ap.c @@ -1237,7 +1237,7 @@ u8 rtw_ap_set_pairwise_key(struct adapter *padapter, = struct sta_info *psta) goto exit; } =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, psetstakey_para, _SetStaKey_CMD_); + init_h2fwcmd_w_parm_no_rsp(ph2c, psetstakey_para, SET_STA_KEY); =20 psetstakey_para->algorithm =3D (u8)psta->dot118021XPrivacy; =20 @@ -1300,7 +1300,7 @@ static int rtw_ap_set_key(struct adapter *padapter, =20 memcpy(&psetkeyparm->key[0], key, keylen); =20 - pcmd->cmdcode =3D _SetKey_CMD_; + pcmd->cmdcode =3D SET_KEY; pcmd->parmbuf =3D (u8 *)psetkeyparm; pcmd->cmdsz =3D (sizeof(struct setkey_parm)); pcmd->rsp =3D NULL; diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c b/drivers/staging/rtl= 8723bs/core/rtw_cmd.c index c1185c25ed36..d1e290daae0d 100644 --- a/drivers/staging/rtl8723bs/core/rtw_cmd.c +++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c @@ -10,81 +10,6 @@ #include #include =20 -static struct _cmd_callback rtw_cmd_callback[] =3D { - {GEN_CMD_CODE(_Read_MACREG), NULL}, /*0*/ - {GEN_CMD_CODE(_Write_MACREG), NULL}, - {GEN_CMD_CODE(_Read_BBREG), &rtw_getbbrfreg_cmdrsp_callback}, - {GEN_CMD_CODE(_Write_BBREG), NULL}, - {GEN_CMD_CODE(_Read_RFREG), &rtw_getbbrfreg_cmdrsp_callback}, - {GEN_CMD_CODE(_Write_RFREG), NULL}, /*5*/ - {GEN_CMD_CODE(_Read_EEPROM), NULL}, - {GEN_CMD_CODE(_Write_EEPROM), NULL}, - {GEN_CMD_CODE(_Read_EFUSE), NULL}, - {GEN_CMD_CODE(_Write_EFUSE), NULL}, - - {GEN_CMD_CODE(_Read_CAM), NULL}, /*10*/ - {GEN_CMD_CODE(_Write_CAM), NULL}, - {GEN_CMD_CODE(_setBCNITV), NULL}, - {GEN_CMD_CODE(_setMBIDCFG), NULL}, - {GEN_CMD_CODE(_JoinBss), &rtw_joinbss_cmd_callback}, /*14*/ - {GEN_CMD_CODE(_DisConnect), &rtw_disassoc_cmd_callback}, /*15*/ - {GEN_CMD_CODE(_CreateBss), &rtw_createbss_cmd_callback}, - {GEN_CMD_CODE(_SetOpMode), NULL}, - {GEN_CMD_CODE(_SiteSurvey), &rtw_survey_cmd_callback}, /*18*/ - {GEN_CMD_CODE(_SetAuth), NULL}, - - {GEN_CMD_CODE(_SetKey), NULL}, /*20*/ - {GEN_CMD_CODE(_SetStaKey), &rtw_setstaKey_cmdrsp_callback}, - {GEN_CMD_CODE(_SetAssocSta), &rtw_setassocsta_cmdrsp_callback}, - {GEN_CMD_CODE(_DelAssocSta), NULL}, - {GEN_CMD_CODE(_SetStaPwrState), NULL}, - {GEN_CMD_CODE(_SetBasicRate), NULL}, /*25*/ - {GEN_CMD_CODE(_GetBasicRate), NULL}, - {GEN_CMD_CODE(_SetDataRate), NULL}, - {GEN_CMD_CODE(_GetDataRate), NULL}, - {GEN_CMD_CODE(_SetPhyInfo), NULL}, - - {GEN_CMD_CODE(_GetPhyInfo), NULL}, /*30*/ - {GEN_CMD_CODE(_SetPhy), NULL}, - {GEN_CMD_CODE(_GetPhy), NULL}, - {GEN_CMD_CODE(_readRssi), NULL}, - {GEN_CMD_CODE(_readGain), NULL}, - {GEN_CMD_CODE(_SetAtim), NULL}, /*35*/ - {GEN_CMD_CODE(_SetPwrMode), NULL}, - {GEN_CMD_CODE(_JoinbssRpt), NULL}, - {GEN_CMD_CODE(_SetRaTable), NULL}, - {GEN_CMD_CODE(_GetRaTable), NULL}, - - {GEN_CMD_CODE(_GetCCXReport), NULL}, /*40*/ - {GEN_CMD_CODE(_GetDTMReport), NULL}, - {GEN_CMD_CODE(_GetTXRateStatistics), NULL}, - {GEN_CMD_CODE(_SetUsbSuspend), NULL}, - {GEN_CMD_CODE(_SetH2cLbk), NULL}, - {GEN_CMD_CODE(_AddBAReq), NULL}, /*45*/ - {GEN_CMD_CODE(_SetChannel), NULL}, /*46*/ - {GEN_CMD_CODE(_SetTxPower), NULL}, - {GEN_CMD_CODE(_SwitchAntenna), NULL}, - {GEN_CMD_CODE(_SetCrystalCap), NULL}, - {GEN_CMD_CODE(_SetSingleCarrierTx), NULL}, /*50*/ - - {GEN_CMD_CODE(_SetSingleToneTx), NULL}, /*51*/ - {GEN_CMD_CODE(_SetCarrierSuppressionTx), NULL}, - {GEN_CMD_CODE(_SetContinuousTx), NULL}, - {GEN_CMD_CODE(_SwitchBandwidth), NULL}, /*54*/ - {GEN_CMD_CODE(_TX_Beacon), NULL},/*55*/ - - {GEN_CMD_CODE(_Set_MLME_EVT), NULL},/*56*/ - {GEN_CMD_CODE(_Set_Drv_Extra), NULL},/*57*/ - {GEN_CMD_CODE(_Set_H2C_MSG), NULL},/*58*/ - {GEN_CMD_CODE(_SetChannelPlan), NULL},/*59*/ - - {GEN_CMD_CODE(_SetChannelSwitch), NULL},/*60*/ - {GEN_CMD_CODE(_TDLS), NULL},/*61*/ - {GEN_CMD_CODE(_ChkBMCSleepq), NULL}, /*62*/ - - {GEN_CMD_CODE(_RunInThreadCMD), NULL},/*63*/ -}; - static struct cmd_hdl wlancmds[] =3D { GEN_DRV_CMD_HANDLER(0, NULL) /*0*/ GEN_DRV_CMD_HANDLER(0, NULL) @@ -301,7 +226,7 @@ int rtw_cmd_filter(struct cmd_priv *pcmdpriv, struct cm= d_obj *cmd_obj) { u8 bAllow =3D false; /* set to true to allow enqueuing cmd when hw_init_c= ompleted is false */ =20 - if (cmd_obj->cmdcode =3D=3D GEN_CMD_CODE(_SetChannelPlan)) + if (cmd_obj->cmdcode =3D=3D SET_CHANNEL_PLAN) bAllow =3D true; =20 if ((!pcmdpriv->padapter->hw_init_completed && !bAllow) || @@ -343,8 +268,8 @@ struct cmd_obj *rtw_dequeue_cmd(struct cmd_priv *pcmdpr= iv) =20 void rtw_free_cmd_obj(struct cmd_obj *pcmd) { - if ((pcmd->cmdcode !=3D _JoinBss_CMD_) && - (pcmd->cmdcode !=3D _CreateBss_CMD_)) { + if ((pcmd->cmdcode !=3D JOIN_BSS) && + (pcmd->cmdcode !=3D CREATE_BSS)) { /* free parmbuf in cmd_obj */ kfree(pcmd->parmbuf); } @@ -377,7 +302,6 @@ int rtw_cmd_thread(void *context) struct cmd_obj *pcmd; u8 *pcmdbuf; u8 (*cmd_hdl)(struct adapter *padapter, u8 *pbuf); - void (*pcmd_callback)(struct adapter *dev, struct cmd_obj *pcmd); struct adapter *padapter =3D context; struct cmd_priv *pcmdpriv =3D &padapter->cmdpriv; struct drvextra_cmd_parm *extra_parm =3D NULL; @@ -478,16 +402,33 @@ int rtw_cmd_thread(void *context) } =20 /* call callback function for post-processed */ - if (pcmd->cmdcode < ARRAY_SIZE(rtw_cmd_callback)) { - pcmd_callback =3D rtw_cmd_callback[pcmd->cmdcode].callback; - if (!pcmd_callback) { - rtw_free_cmd_obj(pcmd); - } else { - /* todo: !!! fill rsp_buf to pcmd->rsp if (pcmd->rsp!=3D NULL) */ - pcmd_callback(pcmd->padapter, pcmd);/* need consider that free cmd_obj= in rtw_cmd_callback */ - } - } else { + /* todo: !!! fill rsp_buf to pcmd->rsp if (pcmd->rsp !=3D NULL) */ + switch (pcmd->cmdcode) { + case READ_BBREG: + case READ_RFREG: + rtw_getbbrfreg_cmdrsp_callback(padapter, pcmd); + break; + case JOIN_BSS: + rtw_joinbss_cmd_callback(padapter, pcmd); + break; + case DISCONNECT: + rtw_disassoc_cmd_callback(padapter, pcmd); + break; + case CREATE_BSS: + rtw_createbss_cmd_callback(padapter, pcmd); + break; + case SITE_SURVEY: + rtw_survey_cmd_callback(padapter, pcmd); + break; + case SET_STA_KEY: + rtw_setstaKey_cmdrsp_callback(padapter, pcmd); + break; + case SET_ASSOC_STA: + rtw_setassocsta_cmdrsp_callback(padapter, pcmd); + break; + default: rtw_free_cmd_obj(pcmd); + break; } flush_signals_thread(); goto _next; @@ -501,7 +442,7 @@ int rtw_cmd_thread(void *context) break; } =20 - if (pcmd->cmdcode =3D=3D GEN_CMD_CODE(_Set_Drv_Extra)) { + if (pcmd->cmdcode =3D=3D SET_DRV_EXTRA) { extra_parm =3D (struct drvextra_cmd_parm *)pcmd->parmbuf; if (extra_parm->pbuf && extra_parm->size > 0) kfree(extra_parm->pbuf); @@ -546,7 +487,7 @@ u8 rtw_sitesurvey_cmd(struct adapter *padapter, struct= ndis_802_11_ssid *ssid, =20 rtw_free_network_queue(padapter, false); =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, psurveyPara, GEN_CMD_CODE(_SiteSurvey)); + init_h2fwcmd_w_parm_no_rsp(ph2c, psurveyPara, SITE_SURVEY); =20 /* psurveyPara->bsslimit =3D 48; */ psurveyPara->scan_mode =3D pmlmepriv->scan_mode; @@ -609,7 +550,7 @@ u8 rtw_createbss_cmd(struct adapter *padapter) } =20 INIT_LIST_HEAD(&pcmd->list); - pcmd->cmdcode =3D _CreateBss_CMD_; + pcmd->cmdcode =3D CREATE_BSS; pcmd->parmbuf =3D (unsigned char *)pdev_network; pcmd->cmdsz =3D get_wlan_bssid_ex_sz((struct wlan_bssid_ex *)pdev_network= ); pcmd->rsp =3D NULL; @@ -642,7 +583,7 @@ int rtw_startbss_cmd(struct adapter *padapter, int fla= gs) } =20 INIT_LIST_HEAD(&pcmd->list); - pcmd->cmdcode =3D GEN_CMD_CODE(_CreateBss); + pcmd->cmdcode =3D CREATE_BSS; pcmd->parmbuf =3D NULL; pcmd->cmdsz =3D 0; pcmd->rsp =3D NULL; @@ -777,7 +718,7 @@ u8 rtw_joinbss_cmd(struct adapter *padapter, struct wl= an_network *pnetwork) pcmd->cmdsz =3D get_wlan_bssid_ex_sz(psecnetwork);/* get cmdsz before end= ian conversion */ =20 INIT_LIST_HEAD(&pcmd->list); - pcmd->cmdcode =3D _JoinBss_CMD_;/* GEN_CMD_CODE(_JoinBss) */ + pcmd->cmdcode =3D JOIN_BSS; pcmd->parmbuf =3D (unsigned char *)psecnetwork; pcmd->rsp =3D NULL; pcmd->rspsz =3D 0; @@ -811,7 +752,7 @@ u8 rtw_disassoc_cmd(struct adapter *padapter, u32 deaut= h_timeout_ms, bool enqueu kfree(param); goto exit; } - init_h2fwcmd_w_parm_no_rsp(cmdobj, param, _DisConnect_CMD_); + init_h2fwcmd_w_parm_no_rsp(cmdobj, param, DISCONNECT); res =3D rtw_enqueue_cmd(cmdpriv, cmdobj); } else { /* no need to enqueue, do the cmd hdl directly and free cmd parameter */ @@ -847,7 +788,7 @@ u8 rtw_setopmode_cmd(struct adapter *padapter, enum nd= is_802_11_network_infrast goto exit; } =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, psetop, _SetOpMode_CMD_); + init_h2fwcmd_w_parm_no_rsp(ph2c, psetop, SET_OP_MODE); res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); } else { setopmode_hdl(padapter, (u8 *)psetop); @@ -904,7 +845,7 @@ u8 rtw_setstakey_cmd(struct adapter *padapter, struct s= ta_info *sta, u8 unicast_ goto exit; } =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, psetstakey_para, _SetStaKey_CMD_); + init_h2fwcmd_w_parm_no_rsp(ph2c, psetstakey_para, SET_STA_KEY); ph2c->rsp =3D (u8 *)psetstakey_rsp; ph2c->rspsz =3D sizeof(struct set_stakey_rsp); res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); @@ -955,7 +896,7 @@ u8 rtw_clearstakey_cmd(struct adapter *padapter, struct= sta_info *sta, u8 enqueu goto exit; } =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, psetstakey_para, _SetStaKey_CMD_); + init_h2fwcmd_w_parm_no_rsp(ph2c, psetstakey_para, SET_STA_KEY); ph2c->rsp =3D (u8 *)psetstakey_rsp; ph2c->rspsz =3D sizeof(struct set_stakey_rsp); =20 @@ -993,7 +934,7 @@ u8 rtw_addbareq_cmd(struct adapter *padapter, u8 tid, u= 8 *addr) paddbareq_parm->tid =3D tid; memcpy(paddbareq_parm->addr, addr, ETH_ALEN); =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, paddbareq_parm, GEN_CMD_CODE(_AddBAReq)); + init_h2fwcmd_w_parm_no_rsp(ph2c, paddbareq_parm, ADD_BA_REQ); =20 /* rtw_enqueue_cmd(pcmdpriv, ph2c); */ res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); @@ -1028,7 +969,7 @@ u8 rtw_reset_securitypriv_cmd(struct adapter *padapter) pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D NULL; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 /* rtw_enqueue_cmd(pcmdpriv, ph2c); */ res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); @@ -1061,7 +1002,7 @@ u8 rtw_free_assoc_resources_cmd(struct adapter *padap= ter) pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D NULL; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 /* rtw_enqueue_cmd(pcmdpriv, ph2c); */ res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); @@ -1094,7 +1035,7 @@ u8 rtw_dynamic_chk_wk_cmd(struct adapter *padapter) pdrvextra_cmd_parm->type =3D 0; pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D NULL; - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 /* rtw_enqueue_cmd(pcmdpriv, ph2c); */ res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); @@ -1343,7 +1284,7 @@ u8 rtw_lps_ctrl_wk_cmd(struct adapter *padapter, u8 l= ps_ctrl_type, u8 enqueue) pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D NULL; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_D= rv_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); } else { @@ -1384,7 +1325,7 @@ u8 rtw_dm_in_lps_wk_cmd(struct adapter *padapter) pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D NULL; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); =20 @@ -1446,7 +1387,7 @@ u8 rtw_dm_ra_mask_wk_cmd(struct adapter *padapter, u8= *psta) pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D psta; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); =20 @@ -1479,7 +1420,7 @@ u8 rtw_ps_cmd(struct adapter *padapter) pdrvextra_cmd_parm->type =3D 0; pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D NULL; - init_h2fwcmd_w_parm_no_rsp(ppscmd, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_= Drv_Extra)); + init_h2fwcmd_w_parm_no_rsp(ppscmd, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ppscmd); =20 @@ -1550,7 +1491,7 @@ u8 rtw_chk_hi_queue_cmd(struct adapter *padapter) pdrvextra_cmd_parm->size =3D 0; pdrvextra_cmd_parm->pbuf =3D NULL; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); =20 @@ -1642,7 +1583,7 @@ u8 rtw_c2h_packet_wk_cmd(struct adapter *padapter, u8= *pbuf, u16 length) pdrvextra_cmd_parm->size =3D length; pdrvextra_cmd_parm->pbuf =3D pbuf; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); =20 @@ -1677,7 +1618,7 @@ u8 rtw_c2h_wk_cmd(struct adapter *padapter, u8 *c2h_e= vt) pdrvextra_cmd_parm->size =3D c2h_evt ? 16 : 0; pdrvextra_cmd_parm->pbuf =3D c2h_evt; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, GEN_CMD_CODE(_Set_Dr= v_Extra)); + init_h2fwcmd_w_parm_no_rsp(ph2c, pdrvextra_cmd_parm, SET_DRV_EXTRA); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); =20 diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rt= l8723bs/core/rtw_mlme.c index 5516593bc8ab..6e2b4fc2f096 100644 --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c @@ -1881,7 +1881,7 @@ signed int rtw_set_auth(struct adapter *adapter, stru= ct security_priv *psecurity =20 psetauthparm->mode =3D (unsigned char)psecuritypriv->dot11AuthAlgrthm; =20 - pcmd->cmdcode =3D _SetAuth_CMD_; + pcmd->cmdcode =3D SET_AUTH; pcmd->parmbuf =3D (unsigned char *)psetauthparm; pcmd->cmdsz =3D (sizeof(struct setauth_parm)); pcmd->rsp =3D NULL; @@ -1952,7 +1952,7 @@ signed int rtw_set_key(struct adapter *adapter, struc= t security_priv *psecurityp goto exit; } =20 - pcmd->cmdcode =3D _SetKey_CMD_; + pcmd->cmdcode =3D SET_KEY; pcmd->parmbuf =3D (u8 *)psetkeyparm; pcmd->cmdsz =3D (sizeof(struct setkey_parm)); pcmd->rsp =3D NULL; diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/stagin= g/rtl8723bs/core/rtw_mlme_ext.c index 5f00fe282d1b..3e71021bb8c8 100644 --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c @@ -4391,7 +4391,7 @@ void report_survey_event(struct adapter *padapter, un= ion recv_frame *precv_frame =20 INIT_LIST_HEAD(&pcmd_obj->list); =20 - pcmd_obj->cmdcode =3D GEN_CMD_CODE(_Set_MLME_EVT); + pcmd_obj->cmdcode =3D SET_MLME_EVT; pcmd_obj->cmdsz =3D cmdsz; pcmd_obj->parmbuf =3D pevtcmd; =20 @@ -4444,7 +4444,7 @@ void report_surveydone_event(struct adapter *padapter) =20 INIT_LIST_HEAD(&pcmd_obj->list); =20 - pcmd_obj->cmdcode =3D GEN_CMD_CODE(_Set_MLME_EVT); + pcmd_obj->cmdcode =3D SET_MLME_EVT; pcmd_obj->cmdsz =3D cmdsz; pcmd_obj->parmbuf =3D pevtcmd; =20 @@ -4489,7 +4489,7 @@ void report_join_res(struct adapter *padapter, int re= s) =20 INIT_LIST_HEAD(&pcmd_obj->list); =20 - pcmd_obj->cmdcode =3D GEN_CMD_CODE(_Set_MLME_EVT); + pcmd_obj->cmdcode =3D SET_MLME_EVT; pcmd_obj->cmdsz =3D cmdsz; pcmd_obj->parmbuf =3D pevtcmd; =20 @@ -4498,7 +4498,7 @@ void report_join_res(struct adapter *padapter, int re= s) =20 pc2h_evt_hdr =3D (struct C2HEvent_Header *)(pevtcmd); pc2h_evt_hdr->len =3D sizeof(struct joinbss_event); - pc2h_evt_hdr->ID =3D GEN_EVT_CODE(_JoinBss); + pc2h_evt_hdr->ID =3D GEN_EVT_CODE(JOIN_BSS); pc2h_evt_hdr->seq =3D atomic_inc_return(&pmlmeext->event_seq); =20 pjoinbss_evt =3D (struct joinbss_event *)(pevtcmd + sizeof(struct C2HEven= t_Header)); @@ -4538,7 +4538,7 @@ void report_wmm_edca_update(struct adapter *padapter) =20 INIT_LIST_HEAD(&pcmd_obj->list); =20 - pcmd_obj->cmdcode =3D GEN_CMD_CODE(_Set_MLME_EVT); + pcmd_obj->cmdcode =3D SET_MLME_EVT; pcmd_obj->cmdsz =3D cmdsz; pcmd_obj->parmbuf =3D pevtcmd; =20 @@ -4584,7 +4584,7 @@ void report_del_sta_event(struct adapter *padapter, u= nsigned char *MacAddr, unsi =20 INIT_LIST_HEAD(&pcmd_obj->list); =20 - pcmd_obj->cmdcode =3D GEN_CMD_CODE(_Set_MLME_EVT); + pcmd_obj->cmdcode =3D SET_MLME_EVT; pcmd_obj->cmdsz =3D cmdsz; pcmd_obj->parmbuf =3D pevtcmd; =20 @@ -4635,7 +4635,7 @@ void report_add_sta_event(struct adapter *padapter, u= nsigned char *MacAddr, int =20 INIT_LIST_HEAD(&pcmd_obj->list); =20 - pcmd_obj->cmdcode =3D GEN_CMD_CODE(_Set_MLME_EVT); + pcmd_obj->cmdcode =3D SET_MLME_EVT; pcmd_obj->cmdsz =3D cmdsz; pcmd_obj->parmbuf =3D pevtcmd; =20 @@ -5085,7 +5085,7 @@ void survey_timer_hdl(struct timer_list *t) return; } =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, psurveyPara, GEN_CMD_CODE(_SiteSurvey)); + init_h2fwcmd_w_parm_no_rsp(ph2c, psurveyPara, SITE_SURVEY); rtw_enqueue_cmd(pcmdpriv, ph2c); } } @@ -5696,7 +5696,7 @@ u8 chk_bmc_sleepq_cmd(struct adapter *padapter) goto exit; } =20 - init_h2fwcmd_w_parm_no_parm_rsp(ph2c, GEN_CMD_CODE(_ChkBMCSleepq)); + init_h2fwcmd_w_parm_no_parm_rsp(ph2c, CHK_BMC_SLEEPQ); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); =20 @@ -5734,7 +5734,7 @@ u8 set_tx_beacon_cmd(struct adapter *padapter) pmlmeinfo->hidden_ssid_mode); ptxBeacon_parm->network.ie_length +=3D len_diff; =20 - init_h2fwcmd_w_parm_no_rsp(ph2c, ptxBeacon_parm, GEN_CMD_CODE(_TX_Beacon)= ); + init_h2fwcmd_w_parm_no_rsp(ph2c, ptxBeacon_parm, TX_BEACON); =20 res =3D rtw_enqueue_cmd(pcmdpriv, ph2c); =20 diff --git a/drivers/staging/rtl8723bs/include/rtw_cmd.h b/drivers/staging/= rtl8723bs/include/rtw_cmd.h index c4c3edee809d..2e071b5f12d9 100644 --- a/drivers/staging/rtl8723bs/include/rtw_cmd.h +++ b/drivers/staging/rtl8723bs/include/rtw_cmd.h @@ -551,9 +551,6 @@ struct RunInThread_param { }; =20 =20 -#define GEN_CMD_CODE(cmd) cmd ## _CMD_ - - /* =20 Result: @@ -624,92 +621,87 @@ extern void rtw_setassocsta_cmdrsp_callback(struct ad= apter *padapter, struct c extern void rtw_getrttbl_cmdrsp_callback(struct adapter *padapter, struc= t cmd_obj *pcmd); =20 =20 -struct _cmd_callback { - u32 cmd_code; - void (*callback)(struct adapter *padapter, struct cmd_obj *cmd); -}; - enum { - GEN_CMD_CODE(_Read_MACREG), /*0*/ - GEN_CMD_CODE(_Write_MACREG), - GEN_CMD_CODE(_Read_BBREG), - GEN_CMD_CODE(_Write_BBREG), - GEN_CMD_CODE(_Read_RFREG), - GEN_CMD_CODE(_Write_RFREG), /*5*/ - GEN_CMD_CODE(_Read_EEPROM), - GEN_CMD_CODE(_Write_EEPROM), - GEN_CMD_CODE(_Read_EFUSE), - GEN_CMD_CODE(_Write_EFUSE), - - GEN_CMD_CODE(_Read_CAM), /*10*/ - GEN_CMD_CODE(_Write_CAM), - GEN_CMD_CODE(_setBCNITV), - GEN_CMD_CODE(_setMBIDCFG), - GEN_CMD_CODE(_JoinBss), /*14*/ - GEN_CMD_CODE(_DisConnect), /*15*/ - GEN_CMD_CODE(_CreateBss), - GEN_CMD_CODE(_SetOpMode), - GEN_CMD_CODE(_SiteSurvey), /*18*/ - GEN_CMD_CODE(_SetAuth), - - GEN_CMD_CODE(_SetKey), /*20*/ - GEN_CMD_CODE(_SetStaKey), - GEN_CMD_CODE(_SetAssocSta), - GEN_CMD_CODE(_DelAssocSta), - GEN_CMD_CODE(_SetStaPwrState), - GEN_CMD_CODE(_SetBasicRate), /*25*/ - GEN_CMD_CODE(_GetBasicRate), - GEN_CMD_CODE(_SetDataRate), - GEN_CMD_CODE(_GetDataRate), - GEN_CMD_CODE(_SetPhyInfo), - - GEN_CMD_CODE(_GetPhyInfo), /*30*/ - GEN_CMD_CODE(_SetPhy), - GEN_CMD_CODE(_GetPhy), - GEN_CMD_CODE(_readRssi), - GEN_CMD_CODE(_readGain), - GEN_CMD_CODE(_SetAtim), /*35*/ - GEN_CMD_CODE(_SetPwrMode), - GEN_CMD_CODE(_JoinbssRpt), - GEN_CMD_CODE(_SetRaTable), - GEN_CMD_CODE(_GetRaTable), - - GEN_CMD_CODE(_GetCCXReport), /*40*/ - GEN_CMD_CODE(_GetDTMReport), - GEN_CMD_CODE(_GetTXRateStatistics), - GEN_CMD_CODE(_SetUsbSuspend), - GEN_CMD_CODE(_SetH2cLbk), - GEN_CMD_CODE(_AddBAReq), /*45*/ - GEN_CMD_CODE(_SetChannel), /*46*/ - GEN_CMD_CODE(_SetTxPower), - GEN_CMD_CODE(_SwitchAntenna), - GEN_CMD_CODE(_SetCrystalCap), - GEN_CMD_CODE(_SetSingleCarrierTx), /*50*/ - - GEN_CMD_CODE(_SetSingleToneTx),/*51*/ - GEN_CMD_CODE(_SetCarrierSuppressionTx), - GEN_CMD_CODE(_SetContinuousTx), - GEN_CMD_CODE(_SwitchBandwidth), /*54*/ - GEN_CMD_CODE(_TX_Beacon), /*55*/ - - GEN_CMD_CODE(_Set_MLME_EVT), /*56*/ - GEN_CMD_CODE(_Set_Drv_Extra), /*57*/ - GEN_CMD_CODE(_Set_H2C_MSG), /*58*/ - - GEN_CMD_CODE(_SetChannelPlan), /*59*/ - - GEN_CMD_CODE(_SetChannelSwitch), /*60*/ - GEN_CMD_CODE(_TDLS), /*61*/ - GEN_CMD_CODE(_ChkBMCSleepq), /*62*/ - - GEN_CMD_CODE(_RunInThreadCMD), /*63*/ + READ_MACREG, + WRITE_MACREG, + READ_BBREG, + WRITE_BBREG, + READ_RFREG, + WRITE_RFREG, + READ_EEPROM, + WRITE_EEPROM, + READ_EFUSE, + WRITE_EFUSE, + + READ_CAM, + WRITE_CAM, + SET_BCNITV, + SET_MBIDCFG, + JOIN_BSS, + DISCONNECT, + CREATE_BSS, + SET_OP_MODE, + SITE_SURVEY, + SET_AUTH, + + SET_KEY, + SET_STA_KEY, + SET_ASSOC_STA, + DEL_ASSOC_STA, + SET_STA_PWR_STATE, + SET_BASIC_RATE, + GET_BASIC_RATE, + SET_DATA_RATE, + GET_DATA_RATE, + SET_PHY_INFO, + + GET_PHY_INFO, + SET_PHY, + GET_PHY, + READ_RSSI, + READ_GAIN, + SET_ATIM, + SET_PWR_MODE, + JOIN_BSS_RPT, + SET_RA_TABLE, + GET_RA_TABLE, + + GET_CCX_REPORT, + GET_DTM_REPORT, + GET_TX_RATE_STATISTICS, + SET_USB_SUSPEND, + SET_H2C_LBK, + ADD_BA_REQ, + SET_CHANNEL, + SET_TX_POWER, + SWITCH_ANTENNA, + SET_CRYSTAL_CAP, + SET_SINGLE_CARRIER_TX, + + SET_SINGLE_TONE_TX, + SET_CARRIER_SUPPRESSION_TX, + SET_CONTINUOUS_TX, + SWITCH_BANDWIDTH, + TX_BEACON, + + SET_MLME_EVT, + SET_DRV_EXTRA, + SET_H2C_MSG, + + SET_CHANNEL_PLAN, + + SET_CHANNEL_SWITCH, + TDLS, + CHK_BMC_SLEEPQ, + + RUN_IN_THREAD_CMD, =20 MAX_H2CCMD }; =20 -#define _GetBBReg_CMD_ _Read_BBREG_CMD_ -#define _SetBBReg_CMD_ _Write_BBREG_CMD_ -#define _GetRFReg_CMD_ _Read_RFREG_CMD_ -#define _SetRFReg_CMD_ _Write_RFREG_CMD_ +#define GET_BB_REG READ_BBREG +#define SET_BB_REG WRITE_BBREG +#define GET_RF_REG READ_RFREG +#define SET_RF_REG WRITE_RFREG =20 -#endif /* _CMD_H_ */ +#endif /* _CMD_H_ */ diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h b/drivers/sta= ging/rtl8723bs/include/rtw_mlme_ext.h index 95769f90d196..4f508c91cecd 100644 --- a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h +++ b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h @@ -679,18 +679,18 @@ void rtw_dummy_event_callback(struct adapter *adapter= , u8 *pbuf); void rtw_fwdbg_event_callback(struct adapter *adapter, u8 *pbuf); =20 enum { - GEN_EVT_CODE(_Read_MACREG) =3D 0, /*0*/ - GEN_EVT_CODE(_Read_BBREG), - GEN_EVT_CODE(_Read_RFREG), - GEN_EVT_CODE(_Read_EEPROM), - GEN_EVT_CODE(_Read_EFUSE), - GEN_EVT_CODE(_Read_CAM), /*5*/ + GEN_EVT_CODE(READ_MACREG) =3D 0, /*0*/ + GEN_EVT_CODE(READ_BBREG), + GEN_EVT_CODE(READ_RFREG), + GEN_EVT_CODE(READ_EEPROM), + GEN_EVT_CODE(READ_EFUSE), + GEN_EVT_CODE(READ_CAM), /*5*/ GEN_EVT_CODE(_Get_BasicRate), GEN_EVT_CODE(_Get_DataRate), GEN_EVT_CODE(_Survey), /*8*/ GEN_EVT_CODE(_SurveyDone), /*9*/ =20 - GEN_EVT_CODE(_JoinBss), /*10*/ + GEN_EVT_CODE(JOIN_BSS), /*10*/ GEN_EVT_CODE(_AddSTA), GEN_EVT_CODE(_DelSTA), GEN_EVT_CODE(_AtimDone), --=20 2.53.0