i2c: gpio: Fix potential unused warning for 'i2c_gpio_dt_ids'
authorBinbin Zhou <zhoubinbin@loongson.cn>
Wed, 30 Nov 2022 05:55:51 +0000 (13:55 +0800)
committerWolfram Sang <wsa@kernel.org>
Mon, 5 Dec 2022 09:28:46 +0000 (10:28 +0100)
Dropping a matching #ifdef check along with dropping of_match_ptr()
is just a cleanup, while dropping of_match_ptr() that has no
corresponding #ifdef fixes an actual warning.

Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Binbin Zhou <zhoubinbin@loongson.cn>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
drivers/i2c/busses/i2c-gpio.c

index b1985c1..0e4385a 100644 (file)
@@ -482,19 +482,17 @@ static int i2c_gpio_remove(struct platform_device *pdev)
        return 0;
 }
 
-#if defined(CONFIG_OF)
 static const struct of_device_id i2c_gpio_dt_ids[] = {
        { .compatible = "i2c-gpio", },
        { /* sentinel */ }
 };
 
 MODULE_DEVICE_TABLE(of, i2c_gpio_dt_ids);
-#endif
 
 static struct platform_driver i2c_gpio_driver = {
        .driver         = {
                .name   = "i2c-gpio",
-               .of_match_table = of_match_ptr(i2c_gpio_dt_ids),
+               .of_match_table = i2c_gpio_dt_ids,
        },
        .probe          = i2c_gpio_probe,
        .remove         = i2c_gpio_remove,