Commit cb64a610 authored by Kalle Valo's avatar Kalle Valo

ath6kl: use ath6kl_credit prefix consistently

Not all credit functions used that prefix, fix that.
Signed-off-by: default avatarKalle Valo <kvalo@qca.qualcomm.com>
parent f2f92195
...@@ -23,10 +23,9 @@ ...@@ -23,10 +23,9 @@
#define CALC_TXRX_PADDED_LEN(dev, len) (__ALIGN_MASK((len), (dev)->block_mask)) #define CALC_TXRX_PADDED_LEN(dev, len) (__ALIGN_MASK((len), (dev)->block_mask))
/* Functions for Tx credit handling */ /* Functions for Tx credit handling */
static void ath6kl_deposit_credit_to_ep(struct ath6kl_htc_credit_info static void ath6kl_credit_deposit(struct ath6kl_htc_credit_info *cred_info,
*cred_info, struct htc_endpoint_credit_dist *ep_dist,
struct htc_endpoint_credit_dist int credits)
*ep_dist, int credits)
{ {
ep_dist->credits += credits; ep_dist->credits += credits;
ep_dist->cred_assngd += credits; ep_dist->cred_assngd += credits;
...@@ -52,7 +51,7 @@ static void ath6kl_credit_init(struct ath6kl_htc_credit_info *cred_info, ...@@ -52,7 +51,7 @@ static void ath6kl_credit_init(struct ath6kl_htc_credit_info *cred_info,
if (tot_credits > 4) { if (tot_credits > 4) {
if ((cur_ep_dist->svc_id == WMI_DATA_BK_SVC) || if ((cur_ep_dist->svc_id == WMI_DATA_BK_SVC) ||
(cur_ep_dist->svc_id == WMI_DATA_BE_SVC)) { (cur_ep_dist->svc_id == WMI_DATA_BE_SVC)) {
ath6kl_deposit_credit_to_ep(cred_info, ath6kl_credit_deposit(cred_info,
cur_ep_dist, cur_ep_dist,
cur_ep_dist->cred_min); cur_ep_dist->cred_min);
cur_ep_dist->dist_flags |= HTC_EP_ACTIVE; cur_ep_dist->dist_flags |= HTC_EP_ACTIVE;
...@@ -60,7 +59,7 @@ static void ath6kl_credit_init(struct ath6kl_htc_credit_info *cred_info, ...@@ -60,7 +59,7 @@ static void ath6kl_credit_init(struct ath6kl_htc_credit_info *cred_info,
} }
if (cur_ep_dist->svc_id == WMI_CONTROL_SVC) { if (cur_ep_dist->svc_id == WMI_CONTROL_SVC) {
ath6kl_deposit_credit_to_ep(cred_info, cur_ep_dist, ath6kl_credit_deposit(cred_info, cur_ep_dist,
cur_ep_dist->cred_min); cur_ep_dist->cred_min);
/* /*
* Control service is always marked active, it * Control service is always marked active, it
...@@ -113,7 +112,7 @@ static void ath6kl_credit_init(struct ath6kl_htc_credit_info *cred_info, ...@@ -113,7 +112,7 @@ static void ath6kl_credit_init(struct ath6kl_htc_credit_info *cred_info,
} }
/* initialize and setup credit distribution */ /* initialize and setup credit distribution */
int ath6kl_setup_credit_dist(void *htc_handle, int ath6kl_credit_setup(void *htc_handle,
struct ath6kl_htc_credit_info *cred_info) struct ath6kl_htc_credit_info *cred_info)
{ {
u16 servicepriority[5]; u16 servicepriority[5];
...@@ -133,7 +132,7 @@ int ath6kl_setup_credit_dist(void *htc_handle, ...@@ -133,7 +132,7 @@ int ath6kl_setup_credit_dist(void *htc_handle,
} }
/* reduce an ep's credits back to a set limit */ /* reduce an ep's credits back to a set limit */
static void ath6kl_reduce_credits(struct ath6kl_htc_credit_info *cred_info, static void ath6kl_credit_reduce(struct ath6kl_htc_credit_info *cred_info,
struct htc_endpoint_credit_dist *ep_dist, struct htc_endpoint_credit_dist *ep_dist,
int limit) int limit)
{ {
...@@ -164,18 +163,18 @@ static void ath6kl_credit_update(struct ath6kl_htc_credit_info *cred_info, ...@@ -164,18 +163,18 @@ static void ath6kl_credit_update(struct ath6kl_htc_credit_info *cred_info,
cur_dist_list->cred_to_dist = 0; cur_dist_list->cred_to_dist = 0;
if (cur_dist_list->credits > if (cur_dist_list->credits >
cur_dist_list->cred_assngd) cur_dist_list->cred_assngd)
ath6kl_reduce_credits(cred_info, ath6kl_credit_reduce(cred_info,
cur_dist_list, cur_dist_list,
cur_dist_list->cred_assngd); cur_dist_list->cred_assngd);
if (cur_dist_list->credits > if (cur_dist_list->credits >
cur_dist_list->cred_norm) cur_dist_list->cred_norm)
ath6kl_reduce_credits(cred_info, cur_dist_list, ath6kl_credit_reduce(cred_info, cur_dist_list,
cur_dist_list->cred_norm); cur_dist_list->cred_norm);
if (!(cur_dist_list->dist_flags & HTC_EP_ACTIVE)) { if (!(cur_dist_list->dist_flags & HTC_EP_ACTIVE)) {
if (cur_dist_list->txq_depth == 0) if (cur_dist_list->txq_depth == 0)
ath6kl_reduce_credits(cred_info, ath6kl_credit_reduce(cred_info,
cur_dist_list, 0); cur_dist_list, 0);
} }
} }
...@@ -186,7 +185,7 @@ static void ath6kl_credit_update(struct ath6kl_htc_credit_info *cred_info, ...@@ -186,7 +185,7 @@ static void ath6kl_credit_update(struct ath6kl_htc_credit_info *cred_info,
* HTC has an endpoint that needs credits, ep_dist is the endpoint in * HTC has an endpoint that needs credits, ep_dist is the endpoint in
* question. * question.
*/ */
static void ath6kl_seek_credits(struct ath6kl_htc_credit_info *cred_info, static void ath6kl_credit_seek(struct ath6kl_htc_credit_info *cred_info,
struct htc_endpoint_credit_dist *ep_dist) struct htc_endpoint_credit_dist *ep_dist)
{ {
struct htc_endpoint_credit_dist *curdist_list; struct htc_endpoint_credit_dist *curdist_list;
...@@ -239,7 +238,7 @@ static void ath6kl_seek_credits(struct ath6kl_htc_credit_info *cred_info, ...@@ -239,7 +238,7 @@ static void ath6kl_seek_credits(struct ath6kl_htc_credit_info *cred_info,
* above it's minimum to fulfill our need try to * above it's minimum to fulfill our need try to
* take away just enough to fulfill our need. * take away just enough to fulfill our need.
*/ */
ath6kl_reduce_credits(cred_info, curdist_list, ath6kl_credit_reduce(cred_info, curdist_list,
curdist_list->cred_assngd - need); curdist_list->cred_assngd - need);
if (cred_info->cur_free_credits >= if (cred_info->cur_free_credits >=
...@@ -256,13 +255,13 @@ static void ath6kl_seek_credits(struct ath6kl_htc_credit_info *cred_info, ...@@ -256,13 +255,13 @@ static void ath6kl_seek_credits(struct ath6kl_htc_credit_info *cred_info,
out: out:
/* did we find some credits? */ /* did we find some credits? */
if (credits) if (credits)
ath6kl_deposit_credit_to_ep(cred_info, ep_dist, credits); ath6kl_credit_deposit(cred_info, ep_dist, credits);
ep_dist->seek_cred = 0; ep_dist->seek_cred = 0;
} }
/* redistribute credits based on activity change */ /* redistribute credits based on activity change */
static void ath6kl_redistribute_credits(struct ath6kl_htc_credit_info *info, static void ath6kl_credit_redistribute(struct ath6kl_htc_credit_info *info,
struct list_head *ep_dist_list) struct list_head *ep_dist_list)
{ {
struct htc_endpoint_credit_dist *curdist_list; struct htc_endpoint_credit_dist *curdist_list;
...@@ -278,9 +277,9 @@ static void ath6kl_redistribute_credits(struct ath6kl_htc_credit_info *info, ...@@ -278,9 +277,9 @@ static void ath6kl_redistribute_credits(struct ath6kl_htc_credit_info *info,
if ((curdist_list->svc_id != WMI_CONTROL_SVC) && if ((curdist_list->svc_id != WMI_CONTROL_SVC) &&
!(curdist_list->dist_flags & HTC_EP_ACTIVE)) { !(curdist_list->dist_flags & HTC_EP_ACTIVE)) {
if (curdist_list->txq_depth == 0) if (curdist_list->txq_depth == 0)
ath6kl_reduce_credits(info, curdist_list, 0); ath6kl_credit_reduce(info, curdist_list, 0);
else else
ath6kl_reduce_credits(info, ath6kl_credit_reduce(info,
curdist_list, curdist_list,
curdist_list->cred_min); curdist_list->cred_min);
} }
...@@ -304,7 +303,7 @@ static void ath6kl_credit_distribute(struct ath6kl_htc_credit_info *cred_info, ...@@ -304,7 +303,7 @@ static void ath6kl_credit_distribute(struct ath6kl_htc_credit_info *cred_info,
ath6kl_credit_update(cred_info, ep_dist_list); ath6kl_credit_update(cred_info, ep_dist_list);
break; break;
case HTC_CREDIT_DIST_ACTIVITY_CHANGE: case HTC_CREDIT_DIST_ACTIVITY_CHANGE:
ath6kl_redistribute_credits(cred_info, ep_dist_list); ath6kl_credit_redistribute(cred_info, ep_dist_list);
break; break;
default: default:
break; break;
...@@ -529,7 +528,7 @@ static int htc_check_credits(struct htc_target *target, ...@@ -529,7 +528,7 @@ static int htc_check_credits(struct htc_target *target,
ath6kl_dbg(ATH6KL_DBG_HTC, "htc creds ctxt 0x%p dist 0x%p\n", ath6kl_dbg(ATH6KL_DBG_HTC, "htc creds ctxt 0x%p dist 0x%p\n",
target->credit_info, &ep->cred_dist); target->credit_info, &ep->cred_dist);
ath6kl_seek_credits(target->credit_info, &ep->cred_dist); ath6kl_credit_seek(target->credit_info, &ep->cred_dist);
ep->cred_dist.seek_cred = 0; ep->cred_dist.seek_cred = 0;
...@@ -552,7 +551,7 @@ static int htc_check_credits(struct htc_target *target, ...@@ -552,7 +551,7 @@ static int htc_check_credits(struct htc_target *target,
ath6kl_dbg(ATH6KL_DBG_HTC, "htc creds ctxt 0x%p dist 0x%p\n", ath6kl_dbg(ATH6KL_DBG_HTC, "htc creds ctxt 0x%p dist 0x%p\n",
target->credit_info, &ep->cred_dist); target->credit_info, &ep->cred_dist);
ath6kl_seek_credits(target->credit_info, &ep->cred_dist); ath6kl_credit_seek(target->credit_info, &ep->cred_dist);
/* see if we were successful in getting more */ /* see if we were successful in getting more */
if (ep->cred_dist.credits < ep->cred_dist.cred_per_msg) { if (ep->cred_dist.credits < ep->cred_dist.cred_per_msg) {
......
...@@ -570,7 +570,7 @@ int ath6kl_htc_add_rxbuf_multiple(struct htc_target *target, ...@@ -570,7 +570,7 @@ int ath6kl_htc_add_rxbuf_multiple(struct htc_target *target,
int ath6kl_htc_rxmsg_pending_handler(struct htc_target *target, int ath6kl_htc_rxmsg_pending_handler(struct htc_target *target,
u32 msg_look_ahead, int *n_pkts); u32 msg_look_ahead, int *n_pkts);
int ath6kl_setup_credit_dist(void *htc_handle, int ath6kl_credit_setup(void *htc_handle,
struct ath6kl_htc_credit_info *cred_info); struct ath6kl_htc_credit_info *cred_info);
static inline void set_htc_pkt_info(struct htc_packet *packet, void *context, static inline void set_htc_pkt_info(struct htc_packet *packet, void *context,
......
...@@ -1506,7 +1506,7 @@ static int ath6kl_init(struct ath6kl *ar) ...@@ -1506,7 +1506,7 @@ static int ath6kl_init(struct ath6kl *ar)
ath6kl_refill_amsdu_rxbufs(ar, ATH6KL_MAX_AMSDU_RX_BUFFERS); ath6kl_refill_amsdu_rxbufs(ar, ATH6KL_MAX_AMSDU_RX_BUFFERS);
/* setup credit distribution */ /* setup credit distribution */
ath6kl_setup_credit_dist(ar->htc_target, &ar->credit_state_info); ath6kl_credit_setup(ar->htc_target, &ar->credit_state_info);
ath6kl_cookie_init(ar); ath6kl_cookie_init(ar);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment