From: Philipp Hortmann <philipp.g.hortmann@gmail.com>
To: Tree Davies <tdavies@darkphysics.net>,
gregkh@linuxfoundation.org, anjan@momi.ca
Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] Staging: rtl8192e: Rename variable pNetwork
Date: Fri, 30 Aug 2024 21:11:19 +0200 [thread overview]
Message-ID: <252c48f2-37f4-4a55-845f-8fa0a046a31a@gmail.com> (raw)
In-Reply-To: <20240830020508.532945-2-tdavies@darkphysics.net>
On 8/30/24 04:05, Tree Davies wrote:
> Rename variable pNetwork to network
> to fix checkpatch warning Avoid CamelCase.
>
> Signed-off-by: Tree Davies <tdavies@darkphysics.net>
> ---
> drivers/staging/rtl8192e/rtl819x_HTProc.c | 34 +++++++++++------------
> drivers/staging/rtl8192e/rtllib.h | 4 +--
> 2 files changed, 19 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c b/drivers/staging/rtl8192e/rtl819x_HTProc.c
> index dfdb7a7dc14f..188e1fd9554d 100644
> --- a/drivers/staging/rtl8192e/rtl819x_HTProc.c
> +++ b/drivers/staging/rtl8192e/rtl819x_HTProc.c
> @@ -559,7 +559,7 @@ void ht_initialize_bss_desc(struct bss_ht *bss_ht)
> }
>
> void ht_reset_self_and_save_peer_setting(struct rtllib_device *ieee,
> - struct rtllib_network *pNetwork)
> + struct rtllib_network *network)
> {
> struct rt_hi_throughput *ht_info = ieee->ht_info;
> u8 bIOTAction = 0;
> @@ -567,32 +567,32 @@ void ht_reset_self_and_save_peer_setting(struct rtllib_device *ieee,
> /* unmark enable_ht flag here is the same reason why unmarked in
> * function rtllib_softmac_new_net. WB 2008.09.10
> */
> - if (pNetwork->bssht.bd_support_ht) {
> + if (network->bssht.bd_support_ht) {
> ht_info->current_ht_support = true;
> - ht_info->peer_ht_spec_ver = pNetwork->bssht.bd_ht_spec_ver;
> + ht_info->peer_ht_spec_ver = network->bssht.bd_ht_spec_ver;
>
> - if (pNetwork->bssht.bd_ht_cap_len > 0 &&
> - pNetwork->bssht.bd_ht_cap_len <= sizeof(ht_info->peer_ht_cap_buf))
> + if (network->bssht.bd_ht_cap_len > 0 &&
> + network->bssht.bd_ht_cap_len <= sizeof(ht_info->peer_ht_cap_buf))
> memcpy(ht_info->peer_ht_cap_buf,
> - pNetwork->bssht.bd_ht_cap_buf,
> - pNetwork->bssht.bd_ht_cap_len);
> + network->bssht.bd_ht_cap_buf,
> + network->bssht.bd_ht_cap_len);
>
> - if (pNetwork->bssht.bd_ht_info_len > 0 &&
> - pNetwork->bssht.bd_ht_info_len <=
> + if (network->bssht.bd_ht_info_len > 0 &&
> + network->bssht.bd_ht_info_len <=
> sizeof(ht_info->peer_ht_info_buf))
> memcpy(ht_info->peer_ht_info_buf,
> - pNetwork->bssht.bd_ht_info_buf,
> - pNetwork->bssht.bd_ht_info_len);
> + network->bssht.bd_ht_info_buf,
> + network->bssht.bd_ht_info_len);
>
> ht_info->current_rt2rt_aggregation =
> - pNetwork->bssht.bd_rt2rt_aggregation;
> + network->bssht.bd_rt2rt_aggregation;
> ht_info->current_rt2rt_long_slot_time =
> - pNetwork->bssht.bd_rt2rt_long_slot_time;
> + network->bssht.bd_rt2rt_long_slot_time;
>
> ht_iot_peer_determine(ieee);
>
> ht_info->iot_action = 0;
> - bIOTAction = ht_iot_act_is_mgnt_use_cck_6m(ieee, pNetwork);
> + bIOTAction = ht_iot_act_is_mgnt_use_cck_6m(ieee, network);
> if (bIOTAction)
> ht_info->iot_action |= HT_IOT_ACT_MGNT_USE_CCK_6M;
> bIOTAction = ht_iot_act_is_ccd_fsync(ieee);
> @@ -609,14 +609,14 @@ void ht_reset_self_and_save_peer_setting(struct rtllib_device *ieee,
> }
>
> void HT_update_self_and_peer_setting(struct rtllib_device *ieee,
> - struct rtllib_network *pNetwork)
> + struct rtllib_network *network)
> {
> struct rt_hi_throughput *ht_info = ieee->ht_info;
> struct ht_info_ele *pPeerHTInfo =
> - (struct ht_info_ele *)pNetwork->bssht.bd_ht_info_buf;
> + (struct ht_info_ele *)network->bssht.bd_ht_info_buf;
>
> if (ht_info->current_ht_support) {
> - if (pNetwork->bssht.bd_ht_info_len != 0)
> + if (network->bssht.bd_ht_info_len != 0)
> ht_info->current_op_mode = pPeerHTInfo->opt_mode;
> }
> }
> diff --git a/drivers/staging/rtl8192e/rtllib.h b/drivers/staging/rtl8192e/rtllib.h
> index 7a1a68317a49..82fef04e5784 100644
> --- a/drivers/staging/rtl8192e/rtllib.h
> +++ b/drivers/staging/rtl8192e/rtllib.h
> @@ -1733,9 +1733,9 @@ void ht_on_assoc_rsp(struct rtllib_device *ieee);
> void ht_initialize_ht_info(struct rtllib_device *ieee);
> void ht_initialize_bss_desc(struct bss_ht *bss_ht);
> void ht_reset_self_and_save_peer_setting(struct rtllib_device *ieee,
> - struct rtllib_network *pNetwork);
> + struct rtllib_network *network);
> void HT_update_self_and_peer_setting(struct rtllib_device *ieee,
> - struct rtllib_network *pNetwork);
> + struct rtllib_network *network);
> u8 ht_get_highest_mcs_rate(struct rtllib_device *ieee, u8 *pMCSRateSet,
> u8 *pMCSFilter);
> extern u8 MCS_FILTER_ALL[];
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
next prev parent reply other threads:[~2024-08-30 19:11 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-30 2:05 [PATCH 0/3] Staging: rtl8192e: Yet another variable rename and cleanup series Tree Davies
2024-08-30 2:05 ` [PATCH 1/3] Staging: rtl8192e: Rename variable pNetwork Tree Davies
2024-08-30 19:11 ` Philipp Hortmann [this message]
2024-08-30 2:05 ` [PATCH 2/3] Staging: rtl8192e: Rename variable bHwError Tree Davies
2024-08-30 19:11 ` Philipp Hortmann
2024-08-30 2:05 ` [PATCH 3/3] Staging: rtl8192e: Add spaces around operators Tree Davies
2024-08-30 19:26 ` Philipp Hortmann
2024-09-03 8:34 ` Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=252c48f2-37f4-4a55-845f-8fa0a046a31a@gmail.com \
--to=philipp.g.hortmann@gmail.com \
--cc=anjan@momi.ca \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=tdavies@darkphysics.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox