staging: most: i2c: remove unnecessary poison_channel call
authorChristian Gromm <christian.gromm@microchip.com>
Tue, 8 May 2018 09:44:52 +0000 (11:44 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 8 May 2018 11:41:48 +0000 (13:41 +0200)
This removes call of the poison_channel that is:
  - not allowed after most_deregister_interface;
  - is made during the most_deregister_interface call.

Signed-off-by: Andrey Shvetsov <andrey.shvetsov@k2l.de>
Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/i2c/i2c.c

index 501eec0783a1588a1eb195c7782a7326b61d6d51..8ec660bd299023ff6fa4ae051019e9bbd0c0fa3d 100644 (file)
@@ -376,16 +376,11 @@ static int i2c_probe(struct i2c_client *client, const struct i2c_device_id *id)
 static int i2c_remove(struct i2c_client *client)
 {
        struct hdm_i2c *dev = i2c_get_clientdata(client);
-       int i;
 
        if (!dev->polling_mode)
                free_irq(client->irq, dev);
 
        most_deregister_interface(&dev->most_iface);
-
-       for (i = 0 ; i < NUM_CHANNELS; i++)
-               if (dev->is_open[i])
-                       poison_channel(&dev->most_iface, i);
        cancel_delayed_work_sync(&dev->rx.dwork);
        kfree(dev);