On Thu, May 29, 2014 at 12:27:55AM +0200, Daniel Vetter wrote: > - if (IS_HASWELL(dev) && intel_crtc->config.pch_pfit.enabled) > + if (IS_HASWELL(dev) && intel_crtc->config.pch_pfit.enabled || > + intel_crtc->config.pch_pfit.force_thru) > temp |= TRANS_DDI_EDP_INPUT_A_ONOFF; My gcc warns here, suggesting the addition of (). And indeed, it seems that we want: if (IS_HASWELL(dev) && (intel_crtc->config.pch_pfit.enabled || intel_crtc->config.pch_pfit.force_thru)) temp |= TRANS_DDI_EDP_INPUT_A_ONOFF; -- Damien _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
