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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to