I looked into potentially applying a fast-and-narrow link config policy on eDP, and realized it's pretty difficult to do this sensibly with the current code. The intel_dp_compute_config() function has grown out of hands. Refactor. The functional changes should be minimal, apart from some debug logging and potential compliance test fixes. I think this should make it easier to add the DSC code too. BR, Jani. Jani Nikula (7): drm/i915/dp: remove stale comment about bw constants drm/i915/dp: move link_bw and rate_select debugging where used drm/i915/dp: abstract dp link config computation from the rest drm/i915/dp: move eDP VBT bpp claming code to intel_dp_compute_bpp() drm/i915/dp: group link config limits in a struct drm/i915/dp: abstract link config selection drm/i915/dp: fix compliance test adjustments drivers/gpu/drm/i915/intel_dp.c | 276 +++++++++++++++----------- drivers/gpu/drm/i915/intel_dp_link_training.c | 5 + 2 files changed, 168 insertions(+), 113 deletions(-) -- 2.11.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx