[PATCH 2/3] staging: brcm80211: made overlapping WMM functions more similar

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Code cleanup. Functions wlc_edcf_setparams() and
wlc_wme_setparams() duplicate code amongst each other. This commit
attempts to make life easier for the reviewer, by introducing non
functional changes that bring wlc_edcf_setparams() more in line
with wlc_wme_setparams(). As a result the subsequent commit can be
reviewed more easily.

Signed-off-by: Roland Vossen <rvossen@xxxxxxxxxxxx>
Reviewed-by: Arend van Spriel <arend@xxxxxxxxxxxx>
---
 drivers/staging/brcm80211/brcmsmac/wlc_main.c |   30 +++++++++++++++---------
 1 files changed, 19 insertions(+), 11 deletions(-)

diff --git a/drivers/staging/brcm80211/brcmsmac/wlc_main.c b/drivers/staging/brcm80211/brcmsmac/wlc_main.c
index 51bc3ee..8d38652 100644
--- a/drivers/staging/brcm80211/brcmsmac/wlc_main.c
+++ b/drivers/staging/brcm80211/brcmsmac/wlc_main.c
@@ -1434,16 +1434,21 @@ void wlc_wme_setparams(struct wlc_info *wlc, u16 aci,
 
 void wlc_edcf_setparams(struct wlc_info *wlc, bool suspend)
 {
-	uint aci, i, j;
+	u16 aci;
+	int i_ac, i;
 	edcf_acparam_t *edcf_acp;
 	shm_acparams_t acp_shm;
 	u16 *shm_entry;
+	struct ieee80211_tx_queue_params txq_pars;
+	struct ieee80211_tx_queue_params *params = &txq_pars;
 
 	ASSERT(wlc);
 
 	/* Only apply params if the core is out of reset and has clocks */
-	if (!wlc->clk)
+	if (!wlc->clk) {
+		WL_ERROR("wl%d: %s : no-clock\n", wlc->pub->unit, __func__);
 		return;
+	}
 
 	/*
 	 * AP uses AC params from wme_param_ie_ap.
@@ -1455,7 +1460,7 @@ void wlc_edcf_setparams(struct wlc_info *wlc, bool suspend)
 
 	wlc->wme_admctl = 0;
 
-	for (i = 0; i < AC_COUNT; i++, edcf_acp++) {
+	for (i_ac = 0; i_ac < AC_COUNT; i_ac++, edcf_acp++) {
 		memset((char *)&acp_shm, 0, sizeof(shm_acparams_t));
 		/* find out which ac this set of params applies to */
 		aci = (edcf_acp->ACI & EDCF_ACI_MASK) >> EDCF_ACI_SHIFT;
@@ -1466,12 +1471,13 @@ void wlc_edcf_setparams(struct wlc_info *wlc, bool suspend)
 		}
 
 		/* fill in shm ac params struct */
-		acp_shm.txop = le16_to_cpu(edcf_acp->TXOP);
+		params->txop = edcf_acp->TXOP;
+		acp_shm.txop = le16_to_cpu(params->txop);
 		/* convert from units of 32us to us for ucode */
-		wlc->edcf_txop[aci] = acp_shm.txop =
+		wlc->edcf_txop[aci & 0x3] = acp_shm.txop =
 		    EDCF_TXOP2USEC(acp_shm.txop);
-		acp_shm.aifs = (edcf_acp->ACI & EDCF_AIFSN_MASK);
-
+		params->aifs = edcf_acp->ACI;
+		acp_shm.aifs = (params->aifs & EDCF_AIFSN_MASK);
 		if (aci == AC_VI && acp_shm.txop == 0
 		    && acp_shm.aifs < EDCF_AIFSN_MAX)
 			acp_shm.aifs++;
@@ -1484,10 +1490,12 @@ void wlc_edcf_setparams(struct wlc_info *wlc, bool suspend)
 		}
 
 		/* CWmin = 2^(ECWmin) - 1 */
-		acp_shm.cwmin = EDCF_ECW2CW(edcf_acp->ECW & EDCF_ECWMIN_MASK);
+		params->cw_min = EDCF_ECW2CW(edcf_acp->ECW & EDCF_ECWMIN_MASK);
+		acp_shm.cwmin = params->cw_min;
 		/* CWmax = 2^(ECWmax) - 1 */
-		acp_shm.cwmax = EDCF_ECW2CW((edcf_acp->ECW & EDCF_ECWMAX_MASK)
+		params->cw_max = EDCF_ECW2CW((edcf_acp->ECW & EDCF_ECWMAX_MASK)
 					    >> EDCF_ECWMAX_SHIFT);
+		acp_shm.cwmax = params->cw_max;
 		acp_shm.cwcur = acp_shm.cwmin;
 		acp_shm.bslots =
 		    R_REG(&wlc->regs->tsf_random) & acp_shm.cwcur;
@@ -1501,10 +1509,10 @@ void wlc_edcf_setparams(struct wlc_info *wlc, bool suspend)
 
 		/* Fill in shm acparam table */
 		shm_entry = (u16 *) &acp_shm;
-		for (j = 0; j < (int)sizeof(shm_acparams_t); j += 2)
+		for (i = 0; i < (int)sizeof(shm_acparams_t); i += 2)
 			wlc_write_shm(wlc,
 				      M_EDCF_QINFO +
-				      wme_shmemacindex(aci) * M_EDCF_QLEN + j,
+				      wme_shmemacindex(aci) * M_EDCF_QLEN + i,
 				      *shm_entry++);
 	}
 
-- 
1.7.1


_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux