Commit 052991d7 authored by Inaky Perez-Gonzalez's avatar Inaky Perez-Gonzalez

wimax/i2400m: remove redundant readiness checks from i2400m_report_tlv_*()

Functions i2400m_report_tlv*() are only called from
i2400m_report_hook(), called in a workqueue by
i2400m_report_hook_work(). The scheduler checks for device readiness
before scheduling.

Added an extra check for readiness in i2400m_report_hook_work(), which
makes all the checks down the line redundant.

Obviously the device state could change in the middle, but error
handling would take care of that.
Signed-off-by: default avatarInaky Perez-Gonzalez <inaky@linux.intel.com>
parent 8ac1101f
...@@ -292,8 +292,6 @@ void i2400m_report_tlv_system_state(struct i2400m *i2400m, ...@@ -292,8 +292,6 @@ void i2400m_report_tlv_system_state(struct i2400m *i2400m,
d_fnstart(3, dev, "(i2400m %p ss %p [%u])\n", i2400m, ss, i2400m_state); d_fnstart(3, dev, "(i2400m %p ss %p [%u])\n", i2400m, ss, i2400m_state);
if (unlikely(i2400m->ready == 0)) /* act if up */
goto out;
if (i2400m->state != i2400m_state) { if (i2400m->state != i2400m_state) {
i2400m->state = i2400m_state; i2400m->state = i2400m_state;
wake_up_all(&i2400m->state_wq); wake_up_all(&i2400m->state_wq);
...@@ -341,7 +339,6 @@ void i2400m_report_tlv_system_state(struct i2400m *i2400m, ...@@ -341,7 +339,6 @@ void i2400m_report_tlv_system_state(struct i2400m *i2400m,
i2400m->bus_reset(i2400m, I2400M_RT_WARM); i2400m->bus_reset(i2400m, I2400M_RT_WARM);
break; break;
}; };
out:
d_fnend(3, dev, "(i2400m %p ss %p [%u]) = void\n", d_fnend(3, dev, "(i2400m %p ss %p [%u]) = void\n",
i2400m, ss, i2400m_state); i2400m, ss, i2400m_state);
} }
...@@ -372,8 +369,6 @@ void i2400m_report_tlv_media_status(struct i2400m *i2400m, ...@@ -372,8 +369,6 @@ void i2400m_report_tlv_media_status(struct i2400m *i2400m,
d_fnstart(3, dev, "(i2400m %p ms %p [%u])\n", i2400m, ms, status); d_fnstart(3, dev, "(i2400m %p ms %p [%u])\n", i2400m, ms, status);
if (unlikely(i2400m->ready == 0)) /* act if up */
goto out;
switch (status) { switch (status) {
case I2400M_MEDIA_STATUS_LINK_UP: case I2400M_MEDIA_STATUS_LINK_UP:
netif_carrier_on(net_dev); netif_carrier_on(net_dev);
...@@ -393,7 +388,6 @@ void i2400m_report_tlv_media_status(struct i2400m *i2400m, ...@@ -393,7 +388,6 @@ void i2400m_report_tlv_media_status(struct i2400m *i2400m,
dev_err(dev, "HW BUG? unknown media status %u\n", dev_err(dev, "HW BUG? unknown media status %u\n",
status); status);
}; };
out:
d_fnend(3, dev, "(i2400m %p ms %p [%u]) = void\n", d_fnend(3, dev, "(i2400m %p ms %p [%u]) = void\n",
i2400m, ms, status); i2400m, ms, status);
} }
......
...@@ -177,6 +177,7 @@ void i2400m_report_hook_work(struct work_struct *ws) ...@@ -177,6 +177,7 @@ void i2400m_report_hook_work(struct work_struct *ws)
struct i2400m_work *iw = struct i2400m_work *iw =
container_of(ws, struct i2400m_work, ws); container_of(ws, struct i2400m_work, ws);
struct i2400m_report_hook_args *args = (void *) iw->pl; struct i2400m_report_hook_args *args = (void *) iw->pl;
if (iw->i2400m->ready)
i2400m_report_hook(iw->i2400m, args->l3l4_hdr, args->size); i2400m_report_hook(iw->i2400m, args->l3l4_hdr, args->size);
kfree_skb(args->skb_rx); kfree_skb(args->skb_rx);
i2400m_put(iw->i2400m); i2400m_put(iw->i2400m);
......
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