Looks reasonable to me, adding a few more AMD people which probably want to take a look as well.
Patch #3 and #4 are Reviewed-by: Christian König <christian.koenig at amd.com> since they touch the radeon/amdgpu driver. Patch #1, #2 and #5 - #8 are Acked-by: Christian König <christian.koenig at amd.com>. Regards, Christian. Am 03.08.2016 um 08:33 schrieb ville.syrjala at linux.intel.com: > From: Ville Syrjälä <ville.syrjala at linux.intel.com> > > This series aims to clean up the way we fill out the display_info > structure a bit. Mainly doing a clean separation of the audio and > video related bits. > > My aim is getting i915 to respect the HDMI sink TMDS clock limit > (currently we respect only the DP++ dongle limit, and the source > limit). But while doing that, I noticed that the display_info stuff > was a bit of mess, hence it resulted in this bigger series. > > Entire series available here: > git://github.com/vsyrjala/linux.git hdmi_sink_tmds_limit_3 > > Ville Syrjälä (10): > drm/edid: Clear old audio latency values before parsing the new EDID > drm/edid: Clear old dvi_dual/max_tmds_clock before parsing the new > EDID > drm/edid: Make max_tmds_clock kHz instead of MHz > drm/edid: Move dvi_dual/max_tmds_clock to drm_display_info > drm/edid: Don't pass around drm_display_info needlessly > drm/edid: Reduce the number of times we parse the CEA extension block > drm/edid: Clear the old cea_rev when there's no CEA extension in the > new EDID > drm/edid: Move dvi_dual/max_tmds_clock parsing out from > drm_edid_to_eld() > drm/i915: Replace a bunch of connector->base.display_info with a local > variable > drm/i915: Account for sink max TMDS clock when checking the port clock > > drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | 4 +- > drivers/gpu/drm/drm_edid.c | 258 > +++++++++++++------------ > drivers/gpu/drm/i915/intel_display.c | 14 +- > drivers/gpu/drm/i915/intel_hdmi.c | 9 +- > drivers/gpu/drm/radeon/radeon_connectors.c | 4 +- > include/drm/drm_crtc.h | 7 +- > 6 files changed, 152 insertions(+), 144 deletions(-) >
