video: display: refactor display_read_timing to avoid code duplication

Commit 2dcf143398ad ("dm: video: Repurpose the 'displayport' uclass to 'display'")
left the display_read_edid() function unused by mistake.

Mark the function as static and reuse it within display_read_timing() to
avoid code duplication.

Signed-off-by: Julien Stephan <jstephan@baylibre.com>

diff --git a/drivers/video/display-uclass.c b/drivers/video/display-uclass.c
index 61a73e1bc2a629eae6a9210434641a9babaf9dea..57e730538dfd70513c4746653719440d973aed6a 100644 (file)
--- a/drivers/video/display-uclass.c
+++ b/drivers/video/display-uclass.c
@@ -10,7 +10,7 @@
 #include <edid.h>
 #include <errno.h>
 
-int display_read_edid(struct udevice *dev, u8 *buf, int buf_size)
+static int display_read_edid(struct udevice *dev, u8 *buf, int buf_size)
 {
        struct dm_display_ops *ops = display_get_ops(dev);
 
@@ -59,9 +59,7 @@ int display_read_timing(struct udevice *dev, struct display_timing *timing)
        if (ops && ops->read_timing)
                return ops->read_timing(dev, timing);
 
-       if (!ops || !ops->read_edid)
-               return -ENOSYS;
-       ret = ops->read_edid(dev, buf, sizeof(buf));
+       ret = display_read_edid(dev, buf, sizeof(buf));
        if (ret < 0)
                return ret;