aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArend van Spriel <arend@broadcom.com>2013-10-15 09:44:48 -0400
committerJohn W. Linville <linville@tuxdriver.com>2013-10-18 14:06:57 -0400
commit3f782744f99773889ad698eb3ea8f6bfb2d254d2 (patch)
treeb4a9535d6add10b7ef59567c6882862e9e8d9292
parent76584ece258dd71066b536fe8636e64537d21011 (diff)
brcmfmac: rename variable max_seg_sz to max_seg_cnt for clarity
The variable max_seg_sz in brcmf_sdio_buffrw() respresents the maximum number of buffers that can be sent in one MMC transfer request. Rename it to max_seg_cnt to avoid confusion. Reviewed-by: Franky Lin <frankyl@broadcom.com> Reviewed-by: Hante Meuleman <meuleman@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r--drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c11
1 files changed, 6 insertions, 5 deletions
diff --git a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c
index e13b1a65c65f..5dfc96cea364 100644
--- a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c
+++ b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c
@@ -332,7 +332,7 @@ static int brcmf_sdio_buffrw(struct brcmf_sdio_dev *sdiodev, uint fn,
332{ 332{
333 unsigned int req_sz, func_blk_sz, sg_cnt, sg_data_sz, pkt_offset; 333 unsigned int req_sz, func_blk_sz, sg_cnt, sg_data_sz, pkt_offset;
334 unsigned int max_blks, max_req_sz, orig_offset, dst_offset; 334 unsigned int max_blks, max_req_sz, orig_offset, dst_offset;
335 unsigned short max_seg_sz, seg_sz; 335 unsigned short max_seg_cnt, seg_sz;
336 unsigned char *pkt_data, *orig_data, *dst_data; 336 unsigned char *pkt_data, *orig_data, *dst_data;
337 struct sk_buff *pkt_next = NULL, *local_pkt_next; 337 struct sk_buff *pkt_next = NULL, *local_pkt_next;
338 struct sk_buff_head local_list, *target_list; 338 struct sk_buff_head local_list, *target_list;
@@ -406,13 +406,14 @@ static int brcmf_sdio_buffrw(struct brcmf_sdio_dev *sdiodev, uint fn,
406 max_blks = min_t(unsigned int, host->max_blk_count, 511u); 406 max_blks = min_t(unsigned int, host->max_blk_count, 511u);
407 max_req_sz = min_t(unsigned int, host->max_req_size, 407 max_req_sz = min_t(unsigned int, host->max_req_size,
408 max_blks * func_blk_sz); 408 max_blks * func_blk_sz);
409 max_seg_sz = min_t(unsigned short, host->max_segs, SG_MAX_SINGLE_ALLOC); 409 max_seg_cnt = min_t(unsigned short, host->max_segs,
410 max_seg_sz = min_t(unsigned short, max_seg_sz, target_list->qlen); 410 SG_MAX_SINGLE_ALLOC);
411 max_seg_cnt = min_t(unsigned short, max_seg_cnt, target_list->qlen);
411 seg_sz = target_list->qlen; 412 seg_sz = target_list->qlen;
412 pkt_offset = 0; 413 pkt_offset = 0;
413 pkt_next = target_list->next; 414 pkt_next = target_list->next;
414 415
415 if (sg_alloc_table(&st, max_seg_sz, GFP_KERNEL)) { 416 if (sg_alloc_table(&st, max_seg_cnt, GFP_KERNEL)) {
416 ret = -ENOMEM; 417 ret = -ENOMEM;
417 goto exit; 418 goto exit;
418 } 419 }
@@ -444,7 +445,7 @@ static int brcmf_sdio_buffrw(struct brcmf_sdio_dev *sdiodev, uint fn,
444 pkt_next = pkt_next->next; 445 pkt_next = pkt_next->next;
445 } 446 }
446 447
447 if (req_sz >= max_req_sz || sg_cnt >= max_seg_sz) 448 if (req_sz >= max_req_sz || sg_cnt >= max_seg_cnt)
448 break; 449 break;
449 } 450 }
450 seg_sz -= sg_cnt; 451 seg_sz -= sg_cnt;