wimax/i2400m: rename misleading I2400M_PL_PAD to I2400M_PL_ALIGN
authorInaky Perez-Gonzalez <inaky@linux.intel.com>
Wed, 20 May 2009 23:53:30 +0000 (16:53 -0700)
committerInaky Perez-Gonzalez <inaky@linux.intel.com>
Thu, 11 Jun 2009 10:30:20 +0000 (03:30 -0700)
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>
drivers/net/wimax/i2400m/fw.c
drivers/net/wimax/i2400m/rx.c
drivers/net/wimax/i2400m/tx.c
include/linux/wimax/i2400m.h

index 7ee1b99b3843efa44ac12a0d3a59df9e52a76442..26924f17f19df096cfdf9822115ed7c3730d51da 100644 (file)
@@ -397,7 +397,7 @@ static int i2400m_download_chunk(struct i2400m *i2400m, const void *chunk,
                                 unsigned int direct, unsigned int do_csum)
 {
        int ret;
-       size_t chunk_len = ALIGN(__chunk_len, I2400M_PL_PAD);
+       size_t chunk_len = ALIGN(__chunk_len, I2400M_PL_ALIGN);
        struct device *dev = i2400m_dev(i2400m);
        struct {
                struct i2400m_bootrom_header cmd;
index 7643850a6fb81717f09c6b8169c3b475fe99691c..07c32e68909f93c33ceb08fd58d332c848193fa9 100644 (file)
@@ -1148,7 +1148,7 @@ int i2400m_rx(struct i2400m *i2400m, struct sk_buff *skb)
        num_pls = le16_to_cpu(msg_hdr->num_pls);
        pl_itr = sizeof(*msg_hdr) +     /* Check payload descriptor(s) */
                num_pls * sizeof(msg_hdr->pld[0]);
-       pl_itr = ALIGN(pl_itr, I2400M_PL_PAD);
+       pl_itr = ALIGN(pl_itr, I2400M_PL_ALIGN);
        if (pl_itr > skb->len) {        /* got all the payload descriptors? */
                dev_err(dev, "RX: HW BUG? message too short (%u bytes) for "
                        "%u payload descriptors (%zu each, total %zu)\n",
@@ -1166,7 +1166,7 @@ int i2400m_rx(struct i2400m *i2400m, struct sk_buff *skb)
                single_last = num_pls == 1 || i == num_pls - 1;
                i2400m_rx_payload(i2400m, skb, single_last, &msg_hdr->pld[i],
                                  skb->data + pl_itr);
-               pl_itr += ALIGN(pl_size, I2400M_PL_PAD);
+               pl_itr += ALIGN(pl_size, I2400M_PL_ALIGN);
                cond_resched();         /* Don't monopolize */
        }
        kfree_skb(skb);
index 8ef724d31fbf34d839dd9fa1cd9a3005451d9a85..a635fd720f3e2f931fc6457f1e3ecaefba4b5666 100644 (file)
@@ -491,7 +491,7 @@ void i2400m_tx_close(struct i2400m *i2400m)
         */
        hdr_size = sizeof(*tx_msg)
                + le16_to_cpu(tx_msg->num_pls) * sizeof(tx_msg->pld[0]);
-       hdr_size = ALIGN(hdr_size, I2400M_PL_PAD);
+       hdr_size = ALIGN(hdr_size, I2400M_PL_ALIGN);
        tx_msg->offset = I2400M_TX_PLD_SIZE - hdr_size;
        tx_msg_moved = (void *) tx_msg + tx_msg->offset;
        memmove(tx_msg_moved, tx_msg, hdr_size);
@@ -574,7 +574,7 @@ int i2400m_tx(struct i2400m *i2400m, const void *buf, size_t buf_len,
 
        d_fnstart(3, dev, "(i2400m %p skb %p [%zu bytes] pt %u)\n",
                  i2400m, buf, buf_len, pl_type);
-       padded_len = ALIGN(buf_len, I2400M_PL_PAD);
+       padded_len = ALIGN(buf_len, I2400M_PL_ALIGN);
        d_printf(5, dev, "padded_len %zd buf_len %zd\n", padded_len, buf_len);
        /* If there is no current TX message, create one; if the
         * current one is out of payload slots or we have a singleton,
index d5148a7889a6350f43b4bae0ae757545fd1621e6..433693ef2bb083c0b81cf1bb5fa49625ef0045c6 100644 (file)
@@ -266,7 +266,7 @@ enum i2400m_ro_type {
 
 /* Misc constants */
 enum {
-       I2400M_PL_PAD = 16,     /* Payload data size alignment */
+       I2400M_PL_ALIGN = 16,   /* Payload data size alignment */
        I2400M_PL_SIZE_MAX = 0x3EFF,
        I2400M_MAX_PLS_IN_MSG = 60,
        /* protocol barkers: sync sequences; for notifications they