drm/bridge: ti-sn65dsi86: Split connector creation to a function
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Thu, 24 Jun 2021 00:03:04 +0000 (03:03 +0300)
committerRobert Foss <robert.foss@linaro.org>
Thu, 24 Jun 2021 11:38:44 +0000 (13:38 +0200)
To prepare for making connector creation option, move connector creation
out of ti_sn_bridge_attach to a separate function.

No functional change intended.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Signed-off-by: Robert Foss <robert.foss@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20210624000304.16281-7-laurent.pinchart+renesas@ideasonboard.com
drivers/gpu/drm/bridge/ti-sn65dsi86.c

index feeda88..9bf8893 100644 (file)
@@ -636,6 +636,25 @@ static const struct drm_connector_funcs ti_sn_bridge_connector_funcs = {
        .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
 };
 
+static int ti_sn_bridge_connector_init(struct ti_sn65dsi86 *pdata)
+{
+       int ret;
+
+       ret = drm_connector_init(pdata->bridge.dev, &pdata->connector,
+                                &ti_sn_bridge_connector_funcs,
+                                DRM_MODE_CONNECTOR_eDP);
+       if (ret) {
+               DRM_ERROR("Failed to initialize connector with drm\n");
+               return ret;
+       }
+
+       drm_connector_helper_add(&pdata->connector,
+                                &ti_sn_bridge_connector_helper_funcs);
+       drm_connector_attach_encoder(&pdata->connector, pdata->bridge.encoder);
+
+       return 0;
+}
+
 /*------------------------------------------------------------------------------
  * DRM Bridge
  */
@@ -669,17 +688,9 @@ static int ti_sn_bridge_attach(struct drm_bridge *bridge,
                return ret;
        }
 
-       ret = drm_connector_init(bridge->dev, &pdata->connector,
-                                &ti_sn_bridge_connector_funcs,
-                                DRM_MODE_CONNECTOR_eDP);
-       if (ret) {
-               DRM_ERROR("Failed to initialize connector with drm\n");
+       ret = ti_sn_bridge_connector_init(pdata);
+       if (ret < 0)
                goto err_conn_init;
-       }
-
-       drm_connector_helper_add(&pdata->connector,
-                                &ti_sn_bridge_connector_helper_funcs);
-       drm_connector_attach_encoder(&pdata->connector, bridge->encoder);
 
        /*
         * TODO: ideally finding host resource and dsi dev registration needs