On Tue, May 27, 2014 at 10:12:07AM +0200, Daniel Vetter wrote: > diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h > index 0542de982260..6d86820d350b 100644 > --- a/drivers/gpu/drm/i915/intel_drv.h > +++ b/drivers/gpu/drm/i915/intel_drv.h > @@ -327,6 +327,7 @@ struct intel_crtc_config { > u32 pos; > u32 size; > bool enabled; > + bool disallow_bypass; > } pch_pfit; (haven't tested anything or looked deeply into it, but...) Having struggled with negative variables in the past, this one really looks awful. disallow *and* bypass. Can we just have 'force' or similar? As a side note, pch_pfit isn't really correct these days now (the pch_ part). -- Damien _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx