drm/tidss: Make use of the helper macro SET_RUNTIME_PM_OPS()
authorCai Huoqing <caihuoqing@baidu.com>
Tue, 7 Sep 2021 03:35:26 +0000 (11:35 +0800)
committerTomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Wed, 10 Nov 2021 11:51:26 +0000 (13:51 +0200)
Use the helper macro SET_RUNTIME_PM_OPS() instead of the verbose
operators ".runtime_suspend/.runtime_resume", because the
SET_RUNTIME_PM_OPS() is a nice helper macro that could be brought
in to make code a little more concise.

Signed-off-by: Cai Huoqing <caihuoqing@baidu.com>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210907033526.1612-1-caihuoqing@baidu.com
drivers/gpu/drm/tidss/tidss_drv.c

index d620f35688da86a5b811440a16895372bf1c5c1e..4366b5c798e0a5e55449f634a44d07e610c36457 100644 (file)
@@ -88,16 +88,11 @@ static int __maybe_unused tidss_resume(struct device *dev)
        return drm_mode_config_helper_resume(&tidss->ddev);
 }
 
-#ifdef CONFIG_PM
-
 static const struct dev_pm_ops tidss_pm_ops = {
-       .runtime_suspend = tidss_pm_runtime_suspend,
-       .runtime_resume = tidss_pm_runtime_resume,
        SET_SYSTEM_SLEEP_PM_OPS(tidss_suspend, tidss_resume)
+       SET_RUNTIME_PM_OPS(tidss_pm_runtime_suspend, tidss_pm_runtime_resume, NULL)
 };
 
-#endif /* CONFIG_PM */
-
 /* DRM device Information */
 
 static void tidss_release(struct drm_device *ddev)
@@ -250,9 +245,7 @@ static struct platform_driver tidss_platform_driver = {
        .shutdown       = tidss_shutdown,
        .driver         = {
                .name   = "tidss",
-#ifdef CONFIG_PM
-               .pm     = &tidss_pm_ops,
-#endif
+               .pm     = pm_ptr(&tidss_pm_ops),
                .of_match_table = tidss_of_table,
                .suppress_bind_attrs = true,
        },