Implement the bridge connector-related .get_edid() operation, and report
the related bridge capabilities and type. The .get_edid() operation is
implemented with the same backend as the EDID retrieval from the
connector .get_modes() operation.

Signed-off-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
---
 drivers/gpu/drm/bridge/ti-sn65dsi86.c | 30 ++++++++++++++++++++++-----
 1 file changed, 25 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c 
b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
index dc300fab4319..6f6e075544e8 100644
--- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
+++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
@@ -261,6 +261,18 @@ static void ti_sn_debugfs_remove(struct ti_sn_bridge 
*pdata)
        pdata->debugfs = NULL;
 }
 
+static struct edid *__ti_sn_bridge_get_edid(struct ti_sn_bridge *pdata,
+                                           struct drm_connector *connector)
+{
+       struct edid *edid;
+
+       pm_runtime_get_sync(pdata->dev);
+       edid = drm_get_edid(connector, &pdata->aux.ddc);
+       pm_runtime_put(pdata->dev);
+
+       return edid;
+}
+
 /* 
-----------------------------------------------------------------------------
  * DRM Connector Operations
  */
@@ -277,11 +289,8 @@ static int ti_sn_bridge_connector_get_modes(struct 
drm_connector *connector)
        struct edid *edid = pdata->edid;
        int num, ret;
 
-       if (!edid) {
-               pm_runtime_get_sync(pdata->dev);
-               edid = pdata->edid = drm_get_edid(connector, &pdata->aux.ddc);
-               pm_runtime_put(pdata->dev);
-       }
+       if (!edid)
+               edid = pdata->edid = __ti_sn_bridge_get_edid(pdata, connector);
 
        if (edid && drm_edid_is_valid(edid)) {
                ret = drm_connector_update_edid_property(connector, edid);
@@ -871,12 +880,21 @@ static void ti_sn_bridge_post_disable(struct drm_bridge 
*bridge)
        pm_runtime_put_sync(pdata->dev);
 }
 
+static struct edid *ti_sn_bridge_get_edid(struct drm_bridge *bridge,
+                                         struct drm_connector *connector)
+{
+       struct ti_sn_bridge *pdata = bridge_to_ti_sn_bridge(bridge);
+
+       return __ti_sn_bridge_get_edid(pdata, connector);
+}
+
 static const struct drm_bridge_funcs ti_sn_bridge_funcs = {
        .attach = ti_sn_bridge_attach,
        .pre_enable = ti_sn_bridge_pre_enable,
        .enable = ti_sn_bridge_enable,
        .disable = ti_sn_bridge_disable,
        .post_disable = ti_sn_bridge_post_disable,
+       .get_edid = ti_sn_bridge_get_edid,
 };
 
 /* 
-----------------------------------------------------------------------------
@@ -1335,6 +1353,8 @@ static int ti_sn_bridge_probe(struct i2c_client *client,
 
        pdata->bridge.funcs = &ti_sn_bridge_funcs;
        pdata->bridge.of_node = client->dev.of_node;
+       pdata->bridge.ops = DRM_BRIDGE_OP_EDID;
+       pdata->bridge.type = DRM_MODE_CONNECTOR_eDP;
 
        drm_bridge_add(&pdata->bridge);
 
-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to