diff options
| author | Arend van Spriel <arend@broadcom.com> | 2013-10-15 09:44:48 -0400 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2013-10-18 14:06:57 -0400 |
| commit | 3f782744f99773889ad698eb3ea8f6bfb2d254d2 (patch) | |
| tree | b4a9535d6add10b7ef59567c6882862e9e8d9292 | |
| parent | 76584ece258dd71066b536fe8636e64537d21011 (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.c | 11 |
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; |
