[PATCH v7 05/15] drm/bridge: analogix_dp: Remove CONFIG_PM related check in analogix_dp_bind()/analogix_dp_unbind()

[Damon Ding]

Remove the check related to CONFIG_PM in order to make the code more
concise, as the CONFIG_PM should be a required option for many drivers.

In addition, it is preferable to use devm_pm_runtime_enable() instead of
manually invoking pm_runtime_enable() followed by pm_runtime_disable().

Signed-off-by: Damon Ding <damon.d...@rock-chips.com>
---
 .../drm/bridge/analogix/analogix_dp_core.c    | 30 ++++---------------
 1 file changed, 6 insertions(+), 24 deletions(-)

diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c 
b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
index d9dafb038e7a..174e7011bba9 100644
--- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
+++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
@@ -1698,15 +1698,11 @@ int analogix_dp_bind(struct analogix_dp_device *dp, 
struct drm_device *drm_dev)
        dp->drm_dev = drm_dev;
        dp->encoder = dp->plat_data->encoder;
 
-       if (IS_ENABLED(CONFIG_PM)) {
-               pm_runtime_use_autosuspend(dp->dev);
-               pm_runtime_set_autosuspend_delay(dp->dev, 100);
-               pm_runtime_enable(dp->dev);
-       } else {
-               ret = analogix_dp_resume(dp);
-               if (ret)
-                       return ret;
-       }
+       pm_runtime_use_autosuspend(dp->dev);
+       pm_runtime_set_autosuspend_delay(dp->dev, 100);
+       ret = devm_pm_runtime_enable(dp->dev);
+       if (ret)
+               return ret;
 
        dp->aux.name = "DP-AUX";
        dp->aux.transfer = analogix_dpaux_transfer;
@@ -1716,7 +1712,7 @@ int analogix_dp_bind(struct analogix_dp_device *dp, 
struct drm_device *drm_dev)
        ret = drm_dp_aux_register(&dp->aux);
        if (ret) {
                DRM_ERROR("failed to register AUX (%d)\n", ret);
-               goto err_disable_pm_runtime;
+               return ret;
        }
 
        ret = analogix_dp_create_bridge(drm_dev, dp);
@@ -1729,13 +1725,6 @@ int analogix_dp_bind(struct analogix_dp_device *dp, 
struct drm_device *drm_dev)
 
 err_unregister_aux:
        drm_dp_aux_unregister(&dp->aux);
-err_disable_pm_runtime:
-       if (IS_ENABLED(CONFIG_PM)) {
-               pm_runtime_dont_use_autosuspend(dp->dev);
-               pm_runtime_disable(dp->dev);
-       } else {
-               analogix_dp_suspend(dp);
-       }
 
        return ret;
 }
@@ -1752,13 +1741,6 @@ void analogix_dp_unbind(struct analogix_dp_device *dp)
        }
 
        drm_dp_aux_unregister(&dp->aux);
-
-       if (IS_ENABLED(CONFIG_PM)) {
-               pm_runtime_dont_use_autosuspend(dp->dev);
-               pm_runtime_disable(dp->dev);
-       } else {
-               analogix_dp_suspend(dp);
-       }
 }
 EXPORT_SYMBOL_GPL(analogix_dp_unbind);
 
-- 
2.34.1