On Thu, Mar 12, 2015 at 12:03:51PM -0700, Jesse Barnes wrote: > On 03/12/2015 11:36 AM, Jani Nikula wrote: > > On Thu, 12 Mar 2015, Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> wrote: > >> On 02/11/2015 09:43 AM, Damien Lespiau wrote: > >>> v2: Use the recently introduced INTEL_REVID() and SKL_REVID defines > >>> (Nick Hoath) > >>> > >>> Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > >>> --- > >>> drivers/gpu/drm/i915/intel_dp.c | 5 ++++- > >>> 1 file changed, 4 insertions(+), 1 deletion(-) > >>> > >>> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > >>> index d4c82d7..a7bc3e8 100644 > >>> --- a/drivers/gpu/drm/i915/intel_dp.c > >>> +++ b/drivers/gpu/drm/i915/intel_dp.c > >>> @@ -129,7 +129,10 @@ intel_dp_max_link_bw(struct intel_dp *intel_dp) > >>> case DP_LINK_BW_2_7: > >>> break; > >>> case DP_LINK_BW_5_4: /* 1.2 capable displays may advertise higher bw */ > >>> - if (((IS_HASWELL(dev) && !IS_HSW_ULX(dev)) || > >>> + if (IS_SKYLAKE(dev) && INTEL_REVID(dev) <= SKL_REVID_B0) > >>> + /* WaDisableHBR2:skl */ > >>> + max_link_bw = DP_LINK_BW_2_7; > >>> + else if (((IS_HASWELL(dev) && !IS_HSW_ULX(dev)) || > >>> INTEL_INFO(dev)->gen >= 8) && > >>> intel_dp->dpcd[DP_DPCD_REV] >= 0x12) > >>> max_link_bw = DP_LINK_BW_5_4; > >>> > >> > >> Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> > >> Tested-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> > > > > Potentially > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89554 > > Yeah hopefully. I definitely need it here for eDP to come up reliably. Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
