public inbox for linux-staging@lists.linux.dev
 help / color / mirror / Atom feed
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>


  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