gpio: ixp4xx: remove redundant of_match_ptr()
authorZhu Wang <wangzhu9@huawei.com>
Wed, 2 Aug 2023 02:44:55 +0000 (10:44 +0800)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Thu, 3 Aug 2023 13:58:48 +0000 (15:58 +0200)
The driver depends on CONFIG_OF, so it is not necessary to use
of_match_ptr() here.

Even for drivers that do not depend on CONFIG_OF, it's almost always
better to leave out the of_match_ptr(), since the only thing it can
possibly do is to save a few bytes of .text if a driver can be used both
with and without it. Hence we remove of_match_ptr().

Signed-off-by: Zhu Wang <wangzhu9@huawei.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
drivers/gpio/gpio-ixp4xx.c

index 1e29de1..dde6cf3 100644 (file)
@@ -302,7 +302,7 @@ static const struct of_device_id ixp4xx_gpio_of_match[] = {
 static struct platform_driver ixp4xx_gpio_driver = {
        .driver = {
                .name           = "ixp4xx-gpio",
-               .of_match_table = of_match_ptr(ixp4xx_gpio_of_match),
+               .of_match_table = ixp4xx_gpio_of_match,
        },
        .probe = ixp4xx_gpio_probe,
 };