提交 b4bd07e3 编写于 作者: I Inaky Perez-Gonzalez

wimax/i2400m: don't reset device on i2400m_dev_shutdown()

i2400m_dev_shutdown() tried to reset the device to put it in a known
state before shutting down.

But that turned out to be pointless. We reach this case in two paths:

 1 - when the device resets, to clean up state
 2 - when the driver is unloaded, for the same

however, in both cases it is pointless; in (1) the device is already
reset, why do it again? in (2) we can't -- the USB stack, for example,
doesn't allow communicating with the device when the driver is being
unbound and if the device is disconnected, the device is gone already.

So just remove it. Leave the function as a placeholder for future
cleanups that will be done from data allocated by the driver during
device operation.
Signed-off-by: NInaky Perez-Gonzalez <inaky@linux.intel.com>
上级 2971a5ba
...@@ -1396,16 +1396,16 @@ int i2400m_dev_initialize(struct i2400m *i2400m) ...@@ -1396,16 +1396,16 @@ int i2400m_dev_initialize(struct i2400m *i2400m)
* *
* @i2400m: device descriptor * @i2400m: device descriptor
* *
* Gracefully stops the device, moving it to the lowest power * Release resources acquired during the running of the device; in
* consumption state possible. * theory, should also tell the device to go to sleep, switch off the
* radio, all that, but at this point, in most cases (driver
* disconnection, reset handling) we can't even talk to the device.
*/ */
void i2400m_dev_shutdown(struct i2400m *i2400m) void i2400m_dev_shutdown(struct i2400m *i2400m)
{ {
int result = -ENODEV;
struct device *dev = i2400m_dev(i2400m); struct device *dev = i2400m_dev(i2400m);
d_fnstart(3, dev, "(i2400m %p)\n", i2400m); d_fnstart(3, dev, "(i2400m %p)\n", i2400m);
result = i2400m->bus_reset(i2400m, I2400M_RT_WARM); d_fnend(3, dev, "(i2400m %p) = void\n", i2400m);
d_fnend(3, dev, "(i2400m %p) = void [%d]\n", i2400m, result);
return; return;
} }
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册