OSDN Git Service

platform/x86: i2c-multi-instantiate: Accept errors of i2c_acpi_new_device()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 28 Nov 2018 11:45:23 +0000 (13:45 +0200)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Mon, 3 Dec 2018 19:40:01 +0000 (21:40 +0200)
In the future i2c_acpi_new_device() will return error pointer in some cases.
Prepare i2c-multi-instantiate driver to support that.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/i2c-multi-instantiate.c

index 5456581..e3345da 100644 (file)
@@ -72,9 +72,14 @@ static int i2c_multi_inst_probe(struct platform_device *pdev)
                        board_info.irq = ret;
                }
                multi->clients[i] = i2c_acpi_new_device(dev, i, &board_info);
-               if (!multi->clients[i]) {
-                       dev_err(dev, "Error creating i2c-client, idx %d\n", i);
+               if (IS_ERR(multi->clients[i]))
+                       ret = PTR_ERR(multi->clients[i]);
+               else if (!multi->clients[i])
                        ret = -ENODEV;
+               else
+                       ret = 0;
+               if (ret) {
+                       dev_err(dev, "Error creating i2c-client, idx %d\n", i);
                        goto error;
                }
        }