drm/bridge: lt8912b: use drm_bridge_edid_read()
authorJani Nikula <jani.nikula@intel.com>
Tue, 23 Jan 2024 19:37:10 +0000 (21:37 +0200)
committerJani Nikula <jani.nikula@intel.com>
Thu, 8 Feb 2024 15:11:12 +0000 (17:11 +0200)
Prefer using the struct drm_edid based functions.

cc: Adrien Grassein <adrien.grassein@gmail.com>
Cc: Andrzej Hajda <andrzej.hajda@intel.com>
Cc: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Robert Foss <rfoss@kernel.org>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Jernej Skrabec <jernej.skrabec@gmail.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/32c9b52fe6fa7cbad6bfd0ff00041876977e02ea.1706038510.git.jani.nikula@intel.com
drivers/gpu/drm/bridge/lontium-lt8912b.c

index 273157428c8270741d5cafcac33d44695eb774a4..4dc748d5d1ee0ae03d043ec618a98d9fc6beac75 100644 (file)
@@ -440,16 +440,16 @@ lt8912_connector_mode_valid(struct drm_connector *connector,
 
 static int lt8912_connector_get_modes(struct drm_connector *connector)
 {
-       struct edid *edid;
+       const struct drm_edid *drm_edid;
        int ret = -1;
        int num = 0;
        struct lt8912 *lt = connector_to_lt8912(connector);
        u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24;
 
-       edid = drm_bridge_get_edid(lt->hdmi_port, connector);
-       if (edid) {
-               drm_connector_update_edid_property(connector, edid);
-               num = drm_add_edid_modes(connector, edid);
+       drm_edid = drm_bridge_edid_read(lt->hdmi_port, connector);
+       if (drm_edid) {
+               drm_edid_connector_update(connector, drm_edid);
+               num = drm_edid_connector_add_modes(connector);
        } else {
                return ret;
        }
@@ -459,7 +459,7 @@ static int lt8912_connector_get_modes(struct drm_connector *connector)
        if (ret)
                num = ret;
 
-       kfree(edid);
+       drm_edid_free(drm_edid);
        return num;
 }