OSDN Git Service

i2c: core: Tidy up handling of init_irq
authorCharles Keepax <ckeepax@opensource.cirrus.com>
Thu, 27 Jun 2019 09:24:11 +0000 (10:24 +0100)
committerWolfram Sang <wsa@the-dreams.de>
Sat, 29 Jun 2019 11:18:01 +0000 (13:18 +0200)
Only set init_irq during i2c_device_new and only handle client->irq on
the probe/remove paths.

Suggested-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/i2c-core-base.c

index b70c2a9..d9b6341 100644 (file)
@@ -322,6 +322,8 @@ static int i2c_device_probe(struct device *dev)
 
        driver = to_i2c_driver(dev->driver);
 
+       client->irq = client->init_irq;
+
        if (!client->irq && !driver->disable_i2c_core_irq_mapping) {
                int irq = -ENOENT;
 
@@ -432,7 +434,7 @@ static int i2c_device_remove(struct device *dev)
        dev_pm_clear_wake_irq(&client->dev);
        device_init_wakeup(&client->dev, false);
 
-       client->irq = client->init_irq;
+       client->irq = 0;
        if (client->flags & I2C_CLIENT_HOST_NOTIFY)
                pm_runtime_put(&client->adapter->dev);
 
@@ -749,7 +751,6 @@ i2c_new_client_device(struct i2c_adapter *adap, struct i2c_board_info const *inf
        if (!client->init_irq)
                client->init_irq = i2c_dev_irq_from_resources(info->resources,
                                                         info->num_resources);
-       client->irq = client->init_irq;
 
        strlcpy(client->name, info->type, sizeof(client->name));