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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux