Merge patch series "video: display: refactor display_read_timing to avoid code duplication"

Julien Stephan <[email protected]> says:

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

This series addresses that oversight and introduces a new useful cmd.

Patch 1:
 - Refactors display_read_timing() to use the existing
   display_read_edid() function, eliminating redundant code.
 - Marks display_read_edid() as static since it is not used outside of
   the file.

Patch 2:
 - Adds a new read_edid command, which can be very useful for debugging
   or developing new display drivers.
 - As this command uses display_read_edid(), the function is made
   non-static again.

Link: https://lore.kernel.org/r/[email protected]

1 files changed, 10 insertions, 0 deletions

diff --git a/include/display.h b/include/display.h
index e8d8aaa15fb..26b965daba9 100644
--- a/include/display.h
+++ b/include/display.h
@@ -26,6 +26,16 @@ struct display_plat {
 };
 
 /**
+ * display_read_edid() - Read edid from display
+ *
+ * @dev:	Device to read from
+ * @buf:	Buffer to read into (should be EDID_SIZE bytes)
+ * @buf_size:	Buffer size (should be EDID_SIZE)
+ * Return number of bytes read, <= 0 for error
+ */
+int display_read_edid(struct udevice *dev, u8 *buf, int buf_size);
+
+/**
  * display_read_timing() - Read timing information
  *
  * @dev:	Device to read from