diff mbox series

[-next] drm/ingenic: remove redundant of_match_ptr

Message ID 20230731122400.250848-1-wangzhu9@huawei.com (mailing list archive)
State New, archived
Headers show
Series [-next] drm/ingenic: remove redundant of_match_ptr | expand

Commit Message

wangzhu July 31, 2023, 12:24 p.m. UTC
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>
---
 drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
index 8dbd4847d3a6..8821b98fd232 100644
--- a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
+++ b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
@@ -1608,7 +1608,7 @@  static struct platform_driver ingenic_drm_driver = {
 	.driver = {
 		.name = "ingenic-drm",
 		.pm = pm_sleep_ptr(&ingenic_drm_pm_ops),
-		.of_match_table = of_match_ptr(ingenic_drm_of_match),
+		.of_match_table = ingenic_drm_of_match,
 	},
 	.probe = ingenic_drm_probe,
 	.remove = ingenic_drm_remove,