diff options
author | Inaky Perez-Gonzalez <inaky@linux.intel.com> | 2009-05-20 19:53:30 -0400 |
---|---|---|
committer | Inaky Perez-Gonzalez <inaky@linux.intel.com> | 2009-06-11 06:30:20 -0400 |
commit | 8593a1967fb9746d318dde88a0a39a36dbfc3445 (patch) | |
tree | 6e3d5e259ae08eab71c8442b2f7fa66bd1d39867 /drivers/net/wimax/i2400m/rx.c | |
parent | 10b1de6b774a531c9054ee01e734a85ffbab179e (diff) |
wimax/i2400m: rename misleading I2400M_PL_PAD to I2400M_PL_ALIGN
The constant is being use as an alignment factor, not as a padding
factor; made reading/reviewing the code quite confusing.
Signed-off-by: Inaky Perez-Gonzalez <inaky@linux.intel.com>
Diffstat (limited to 'drivers/net/wimax/i2400m/rx.c')
-rw-r--r-- | drivers/net/wimax/i2400m/rx.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wimax/i2400m/rx.c b/drivers/net/wimax/i2400m/rx.c index 7643850a6fb8..07c32e68909f 100644 --- a/drivers/net/wimax/i2400m/rx.c +++ b/drivers/net/wimax/i2400m/rx.c | |||
@@ -1148,7 +1148,7 @@ int i2400m_rx(struct i2400m *i2400m, struct sk_buff *skb) | |||
1148 | num_pls = le16_to_cpu(msg_hdr->num_pls); | 1148 | num_pls = le16_to_cpu(msg_hdr->num_pls); |
1149 | pl_itr = sizeof(*msg_hdr) + /* Check payload descriptor(s) */ | 1149 | pl_itr = sizeof(*msg_hdr) + /* Check payload descriptor(s) */ |
1150 | num_pls * sizeof(msg_hdr->pld[0]); | 1150 | num_pls * sizeof(msg_hdr->pld[0]); |
1151 | pl_itr = ALIGN(pl_itr, I2400M_PL_PAD); | 1151 | pl_itr = ALIGN(pl_itr, I2400M_PL_ALIGN); |
1152 | if (pl_itr > skb->len) { /* got all the payload descriptors? */ | 1152 | if (pl_itr > skb->len) { /* got all the payload descriptors? */ |
1153 | dev_err(dev, "RX: HW BUG? message too short (%u bytes) for " | 1153 | dev_err(dev, "RX: HW BUG? message too short (%u bytes) for " |
1154 | "%u payload descriptors (%zu each, total %zu)\n", | 1154 | "%u payload descriptors (%zu each, total %zu)\n", |
@@ -1166,7 +1166,7 @@ int i2400m_rx(struct i2400m *i2400m, struct sk_buff *skb) | |||
1166 | single_last = num_pls == 1 || i == num_pls - 1; | 1166 | single_last = num_pls == 1 || i == num_pls - 1; |
1167 | i2400m_rx_payload(i2400m, skb, single_last, &msg_hdr->pld[i], | 1167 | i2400m_rx_payload(i2400m, skb, single_last, &msg_hdr->pld[i], |
1168 | skb->data + pl_itr); | 1168 | skb->data + pl_itr); |
1169 | pl_itr += ALIGN(pl_size, I2400M_PL_PAD); | 1169 | pl_itr += ALIGN(pl_size, I2400M_PL_ALIGN); |
1170 | cond_resched(); /* Don't monopolize */ | 1170 | cond_resched(); /* Don't monopolize */ |
1171 | } | 1171 | } |
1172 | kfree_skb(skb); | 1172 | kfree_skb(skb); |