On Tue, Mar 17, 2015 at 04:22:33PM +0200, Imre Deak wrote: > On ti, 2015-03-17 at 14:51 +0100, Daniel Vetter wrote: > > On Tue, Mar 17, 2015 at 11:40:01AM +0200, Imre Deak wrote: > > > From: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> > > > > > > Broxton has the same panel fitter registers as Skylake. > > > > > > Signed-off-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> > > > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> > > > --- > > > drivers/gpu/drm/i915/intel_display.c | 6 +++--- > > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > > > index ba2d1ae..95ce0a8 100644 > > > --- a/drivers/gpu/drm/i915/intel_display.c > > > +++ b/drivers/gpu/drm/i915/intel_display.c > > > @@ -4532,7 +4532,7 @@ static void haswell_crtc_enable(struct drm_crtc *crtc) > > > > > > intel_ddi_enable_pipe_clock(intel_crtc); > > > > > > - if (IS_SKYLAKE(dev)) > > > + if (INTEL_INFO(dev)->gen == 9) > > > > Shouldn't we go with gen >= 9 here while at it? > > == 9 looks like the right choice based on bspec. Then maybe add a else if (gen > 9) MISSING_CASE(gen); afterwards? Just trying to avoid surprises when enabling skl/bxt+1 ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
