[PATCH 1/2] video: display: refactor display_read_timing to avoid code duplication

[Julien Stephan]

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 at baylibre.com>
---
 drivers/video/display-uclass.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/video/display-uclass.c b/drivers/video/display-uclass.c
index 61a73e1bc2a629eae6a9210434641a9babaf9dea..57e730538dfd70513c4746653719440d973aed6a 100644
--- 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;
 

-- 
2.49.0