On Wed, 6 Mar 2013 21:00:11 +0200 Ville Syrj?l? <ville.syrjala at linux.intel.com> wrote: > On Fri, Mar 01, 2013 at 01:14:23PM -0800, Jesse Barnes wrote: > > Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org> > > --- > > drivers/gpu/drm/i915/intel_display.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > > index fd4a0d4..60397e8 100644 > > --- a/drivers/gpu/drm/i915/intel_display.c > > +++ b/drivers/gpu/drm/i915/intel_display.c > > @@ -8503,7 +8503,8 @@ static void intel_setup_outputs(struct drm_device *dev) > > I915_WRITE(PFIT_CONTROL, 0); > > } > > > > - if (!(HAS_DDI(dev) && (I915_READ(DDI_BUF_CTL(PORT_A)) & DDI_A_4_LANES))) > > + if (!(HAS_DDI(dev) && (I915_READ(DDI_BUF_CTL(PORT_A)) & DDI_A_4_LANES)) && > > + !IS_VALLEYVIEW(dev)) > > intel_crt_init(dev); > > I was under the impression that VLV still has this. Was I mistaken? Not exposed anymore I think. And yeah this can be dropped since the DDI check should dtrt. -- Jesse Barnes, Intel Open Source Technology Center
