<html> <head> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> On 08/24/2016 10:54 PM, Sean Paul wrote: <blockquote cite="mid:1472050481-32724-1-git-send-email-seanpaul@chromium.org" type="cite"> <pre wrap="">From: Tomeu Vizoso <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:tomeu.vizoso@collabora.com"><tomeu.vizoso@collabora.com></a> Remove code for reading the EDID and DPCD fields and use the helpers instead. Besides the obvious code reduction, other helpers are being added to the core that could be used in this driver and will be good to be able to use them instead of duplicating them. Signed-off-by: Tomeu Vizoso <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:tomeu.vizoso@collabora.com"><tomeu.vizoso@collabora.com></a> Tested-by: Javier Martinez Canillas <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:javier@osg.samsung.com"><javier@osg.samsung.com></a> Tested-by: Sean Paul <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:seanpaul@chromium.org"><seanpaul@chromium.org></a> Reviewed-by: Sean Paul <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:seanpaul@chromium.org"><seanpaul@chromium.org></a> Signed-off-by: Sean Paul <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:seanpaul@chromium.org"><seanpaul@chromium.org></a> Cc: Javier Martinez Canillas <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:javier@osg.samsung.com"><javier@osg.samsung.com></a> Cc: Mika Kahola <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:mika.kahola@intel.com"><mika.kahola@intel.com></a> Cc: Yakir Yang <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:ykk@rock-chips.com"><ykk@rock-chips.com></a> Cc: Daniel Vetter <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:daniel.vetter@intel.com"><daniel.vetter@intel.com></a> Cc: Archit Taneja <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:architt@codeaurora.org"><architt@codeaurora.org></a></pre> </blockquote> Reviewed-by: Yakir Yang <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:ykk@rock-chips.com"><ykk@rock-chips.com></a> <blockquote cite="mid:1472050481-32724-1-git-send-email-seanpaul@chromium.org" type="cite"> <pre wrap=""> --- v2: - A bunch of good fixes from Sean and Yakir - Moved the transfer function to analogix_dp_reg.c - Removed reference to the EDID from the dp struct v3: - Rebased on top of Yakir's PSR series - Fixed checkpatch warning about line length drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 274 ++++--------- drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 40 +- drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 451 ++++++--------------- 3 files changed, 210 insertions(+), 555 deletions(-)</pre> </blockquote> </body> </html>