On Tue, 1 Dec 2015, Jonathan Gray wrote: > On Tue, Dec 01, 2015 at 07:00:19AM +0000, Scot Doyle wrote: > > Should we include "INTEL_INFO(dev)->gen >= 8"? There is at least one known > > regression (booting with blank screen) described in commit > > 8749be86a8f0c8a956d20f7d3c768ed6d1e356a1. > > I think we can safely skip that one as it concerns skylake and > we don't have any code for skylake at the moment. > > > > > > + intel_dp->dpcd[DP_DPCD_REV] >= 0x12) > > > + max_link_bw = DP_LINK_BW_5_4; > > > > No need to set max_link_bw to same value? Negate previous condition, > > assign DP_LINK_BW_2_7 and delete following else clause? > > It's harmless though, I'd prefer to leave it as then it is potentially > one less conflict to deal with later.
Sounds good. I've confirmed Ultra HD at 60Hz with the suggested patches.
