[PATCH/RFC 0/5] Decouple EDID retrieval from drm_connector

Laurent Pinchart laurent.pinchart at ideasonboard.com
Wed Aug 21 18:50:00 UTC 2019


Hello,

This small patch series attemps at decoupling EDID retrieval from
drm_connector, following a discussion with Daniel Vetter [1]. While
working on this I noticed a few issues with EDID retrieval, which I have
attempted to fix in patches 1/5 to 4/5. Patch 5/5 then tries to decouple
the EDID retrieval from drm_connector, in what I think is a pretty bad
way. I would like to discuss this further to see if a) there's an
interest, and b) someone has a better idea :-)

Regardless of the outcome of the discussion on patch 5/5, the first four
patches are candidates for upstreaming as they try to address existing
issues (they may also introduce new bugs, but hopefully they will be
pointed out during review).

The patches are available at

	git://linuxtv.org/pinchartl/media.git omapdrm/edid

[1] https://lists.freedesktop.org/archives/dri-devel/2019-August/231930.html

Laurent Pinchart (5):
  drm/edid: Reorganise the DisplayID parsing code
  drm/edid: Move functions to avoid forward declaration
  drm/edid: Move DisplayID tile parsing to drm_connector.c
  drm/edid: Honour connector->force in drm_do_get_edid()
  [HACK] drm/edid: Decouple EDID retrieval from connector

 drivers/gpu/drm/drm_connector.c       | 137 +++-
 drivers/gpu/drm/drm_dp_mst_topology.c |   3 +-
 drivers/gpu/drm/drm_edid.c            | 857 ++++++++++++--------------
 include/drm/drm_connector.h           |   3 +-
 include/drm/drm_displayid.h           |   2 +
 include/drm/drm_edid.h                |   2 +
 6 files changed, 534 insertions(+), 470 deletions(-)

-- 
Regards,

Laurent Pinchart



More information about the dri-devel mailing list