drm/rockchip: remove redundant of_match_ptr
authorZhu Wang <wangzhu9@huawei.com>
Mon, 31 Jul 2023 12:53:04 +0000 (20:53 +0800)
committerHeiko Stuebner <heiko@sntech.de>
Mon, 9 Oct 2023 19:46:36 +0000 (21:46 +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>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20230731125304.87059-1-wangzhu9@huawei.com
drivers/gpu/drm/rockchip/cdn-dp-core.c
drivers/gpu/drm/rockchip/rockchip_lvds.c
drivers/gpu/drm/rockchip/rockchip_vop2_reg.c

index 3793863c210ebd4413864a08cda2b44f26ed5c40..21254e4e107a9f080e3577a3ca2064a9a691009f 100644 (file)
@@ -1261,7 +1261,7 @@ struct platform_driver cdn_dp_driver = {
        .driver = {
                   .name = "cdn-dp",
                   .owner = THIS_MODULE,
-                  .of_match_table = of_match_ptr(cdn_dp_dt_ids),
+                  .of_match_table = cdn_dp_dt_ids,
                   .pm = &cdn_dp_pm_ops,
        },
 };
index 5828593877923f2157ee6444c3fb17a8cd13e044..f0f47e9abf5aad7767ee7abb27ab67861479cabe 100644 (file)
@@ -752,6 +752,6 @@ struct platform_driver rockchip_lvds_driver = {
        .remove_new = rockchip_lvds_remove,
        .driver = {
                   .name = "rockchip-lvds",
-                  .of_match_table = of_match_ptr(rockchip_lvds_dt_ids),
+                  .of_match_table = rockchip_lvds_dt_ids,
        },
 };
index 62b573f282a7ae00bb72ded614e86a0946f0b42b..fcd4cf3072cd9e8f963169b6db4d74a6fb826577 100644 (file)
@@ -274,6 +274,6 @@ struct platform_driver vop2_platform_driver = {
        .remove_new = vop2_remove,
        .driver = {
                .name = "rockchip-vop2",
-               .of_match_table = of_match_ptr(vop2_dt_match),
+               .of_match_table = vop2_dt_match,
        },
 };