[v2,30/50] wilc1000: use more descriptive variable names

Message ID	20211223011358.4031459-31-davidm@egauge.net (mailing list archive)
State	Changes Requested
Delegated to:	Kalle Valo
Headers	show Return-Path: <linux-wireless-owner@kernel.org> From: David Mosberger-Tang <davidm@egauge.net> Subject: [PATCH v2 30/50] wilc1000: use more descriptive variable names Date: Thu, 23 Dec 2021 01:14:06 +0000 (UTC) Message-Id: <20211223011358.4031459-31-davidm@egauge.net> In-Reply-To: <20211223011358.4031459-1-davidm@egauge.net> References: <20211223011358.4031459-1-davidm@egauge.net> MIME-Version: 1.0 To: Ajay Singh <ajay.kathat@microchip.com> Cc: Claudiu Beznea <claudiu.beznea@microchip.com>, Kalle Valo <kvalo@kernel.org>, "David S. Miller" <davem@davemloft.net>, Jakub Kicinski <kuba@kernel.org>, linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, David Mosberger-Tang <davidm@egauge.net> Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=us-ascii Precedence: bulk
Series	wilc1000: rework tx path to use sk_buffs throughout \| expand [v2,00/50] wilc1000: rework tx path to use sk_buffs throughout [v2,04/50] wilc1000: factor common code in wilc_wlan_cfg_set() and wilc_wlan_cfg_get() [v2,01/50] wilc1000: don't hold txq_spinlock while initializing AC queue limits [v2,02/50] wilc1000: switch txq_event from completion to waitqueue [v2,31/50] wilc1000: eliminate another magic constant [v2,30/50] wilc1000: use more descriptive variable names [v2,29/50] wilc1000: factor header length calculation into a new function [v2,28/50] wilc1000: improve send_packets() a bit [v2,27/50] wilc1000: simplify ac_balance() a bit [v2,25/50] wilc1000: replace txq_spinlock with ack_filter_lock mutex [v2,24/50] wilc1000: protect tx_q_limit with a mutex instead of a spinlock [v2,23/50] wilc1000: introduce symbolic names for two tx-related control bits [v2,22/50] wilc1000: minor syntax cleanup [v2,21/50] wilc1000: declare read-only ac_preserve_ratio as static and const [v2,20/50] wilc1000: eliminate "max_size_over" variable in fill_vmm_table [v2,19/50] wilc1000: don't tell the chip to go to sleep while copying tx packets [v2,18/50] wilc1000: split huge tx handler into subfunctions [v2,16/50] wilc1000: switch tx queue to normal sk_buff entries [v2,03/50] wilc1000: move receive-queue stats from txq to wilc structure [v2,06/50] wilc1000: move tx packet drop code into its own function [v2,07/50] wilc1000: increment tx_dropped stat counter on tx packet drop [v2,08/50] wilc1000: fix management packet type inconsistency [v2,09/50] wilc1000: prepare wilc_wlan_tx_packet_done() for sk_buff changes [v2,10/50] wilc1000: factor initialization of tx queue-specific packet fields [v2,11/50] wilc1000: convert tqx_entries from "int" to "atomic_t" [v2,12/50] wilc1000: refactor wilc_wlan_cfg_commit() a bit [v2,13/50] wilc1000: sanitize config packet sequence number management a bit [v2,14/50] wilc1000: if there is no tx packet, don't increment packets-sent counter [v2,15/50] wilc1000: add struct wilc_skb_tx_cb as an alias of struct txq_entry_t [v2,17/50] wilc1000: remove no longer used "vif" argument from init_txq_entry() [v2,49/50] wilc1000: implement zero-copy transmit support for SPI [v2,50/50] wilc1000: add module parameter "disable_zero_copy_tx" to SPI driver [v2,48/50] wilc1000: introduce function to find and check DMA response [v2,47/50] wilc1000: factor SPI DMA command initialization code into a function [v2,46/50] wilc1000: remove duplicate CRC calculation code [v2,45/50] wilc1000: move struct wilc_spi declaration [v2,44/50] wilc1000: don't allocate tx_buffer when zero-copy is available [v2,43/50] wilc1000: add support for zero-copy transmit of tx packets [v2,42/50] wilc1000: simplify code by adding header/padding to skb [v2,41/50] wilc1000: use more descriptive variable name [v2,40/50] wilc1000: introduce schedule_packets() function [v2,39/50] wilc1000: eliminate txq_add_to_head_cs mutex [v2,38/50] wilc1000: take advantage of chip queue [v2,37/50] wilc1000: introduce set_header() function [v2,36/50] wilc1000: introduce transmit path chip queue [v2,35/50] wilc1000: introduce copy_and_send_packets() helper function [v2,34/50] wilc1000: restructure wilc-wlan_handle_txq() for clarity [v2,33/50] wilc1000: move ac_desired_ratio calculation to where its needed [v2,32/50] wilc1000: introduce vmm_table_entry() helper function [v2,26/50] wilc1000: reduce amount of time ack_filter_lock is held [v2,05/50] wilc1000: add wilc_wlan_tx_packet_done() function

Message ID

20211223011358.4031459-31-davidm@egauge.net (mailing list archive)

State

Changes Requested

Delegated to:

Kalle Valo

Headers

From: David Mosberger-Tang <davidm@egauge.net>
Subject: [PATCH v2 30/50] wilc1000: use more descriptive variable names
Date: Thu, 23 Dec 2021 01:14:06 +0000 (UTC)
Message-Id: <20211223011358.4031459-31-davidm@egauge.net>
In-Reply-To: <20211223011358.4031459-1-davidm@egauge.net>
References: <20211223011358.4031459-1-davidm@egauge.net>
MIME-Version: 1.0
To: Ajay Singh <ajay.kathat@microchip.com>
Cc: Claudiu Beznea <claudiu.beznea@microchip.com>,
        Kalle Valo <kvalo@kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        David Mosberger-Tang <davidm@egauge.net>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset=us-ascii
Precedence: bulk

Series

wilc1000: rework tx path to use sk_buffs throughout | expand

Commit Message

David Mosberger-Tang Dec. 23, 2021, 1:14 a.m. UTC

Rename "i" to "vmm_table_len" to improve readability and update
kernel-doc for send_vmm_table() as well.

Signed-off-by: David Mosberger-Tang <davidm@egauge.net>
---
 .../net/wireless/microchip/wilc1000/wlan.c    | 21 ++++++++++---------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/drivers/net/wireless/microchip/wilc1000/wlan.c b/drivers/net/wireless/microchip/wilc1000/wlan.c
index 1cd9a7761343a..a4523b0860878 100644
--- a/drivers/net/wireless/microchip/wilc1000/wlan.c
+++ b/drivers/net/wireless/microchip/wilc1000/wlan.c
@@ -712,9 +712,9 @@  static int fill_vmm_table(const struct wilc *wilc,
 }
 
 /**
- * send_vmm_table() - Send the VMM table to the chip
+ * send_vmm_table() - send the VMM table to the chip
  * @wilc: Pointer to the wilc structure.
- * @i: The number of entries in the VMM table.
+ * @vmm_table_len: The number of entries in the VMM table.
  * @vmm_table: The VMM table to send.
  *
  * Send the VMM table to the chip and get back the number of entries
@@ -723,10 +723,10 @@  static int fill_vmm_table(const struct wilc *wilc,
  * Context: The bus must have been acquired before calling this
  * function.
  *
- * Return:
- *	The number of VMM table entries the chip can accept.
+ * Return: The number of VMM table entries the chip can accept.
  */
-static int send_vmm_table(struct wilc *wilc, int i, const u32 *vmm_table)
+static int send_vmm_table(struct wilc *wilc,
+			  int vmm_table_len, const u32 *vmm_table)
 {
 	const struct wilc_hif_func *func;
 	int ret, counter, entries, timeout;
@@ -758,7 +758,8 @@  static int send_vmm_table(struct wilc *wilc, int i, const u32 *vmm_table)
 	timeout = 200;
 	do {
 		ret = func->hif_block_tx(wilc, WILC_VMM_TBL_RX_SHADOW_BASE,
-					 (u8 *)vmm_table, (i + 1) * 4);
+					 (u8 *)vmm_table,
+					 (vmm_table_len + 1) * 4);
 		if (ret)
 			break;
 
@@ -899,7 +900,7 @@  static int send_packets(struct wilc *wilc, int len)
 
 int wilc_wlan_handle_txq(struct wilc *wilc, u32 *txq_count)
 {
-	int i, entries, len;
+	int vmm_table_len, entries, len;
 	u8 ac_desired_ratio[NQUEUES] = {0, 0, 0, 0};
 	u8 vmm_entries_ac[WILC_VMM_TBL_SIZE];
 	int ret = 0;
@@ -919,13 +920,13 @@  int wilc_wlan_handle_txq(struct wilc *wilc, u32 *txq_count)
 		wilc_wlan_txq_filter_dup_tcp_ack(vif->ndev);
 	srcu_read_unlock(&wilc->srcu, srcu_idx);
 
-	i = fill_vmm_table(wilc, ac_desired_ratio, vmm_table, vmm_entries_ac);
-	if (i == 0)
+	vmm_table_len = fill_vmm_table(wilc, ac_desired_ratio, vmm_table, vmm_entries_ac);
+	if (vmm_table_len == 0)
 		goto out_unlock;
 
 	acquire_bus(wilc, WILC_BUS_ACQUIRE_AND_WAKEUP);
 
-	ret = send_vmm_table(wilc, i, vmm_table);
+	ret = send_vmm_table(wilc, vmm_table_len, vmm_table);
 	if (ret <= 0) {
 		if (ret == 0)
 			/* No VMM space available in firmware.  Inform

[v2,30/50] wilc1000: use more descriptive variable names

Commit Message

Patch