Re: [Intel-gfx] [PATCH 1/3] drm/i915: make DP configuration vars less confusing in ironlake_crtc_mode_se

2012-04-11 Thread Daniel Vetter
On Tue, Apr 10, 2012 at 10:22:09PM +0100, Chris Wilson wrote: On Tue, 10 Apr 2012 11:58:03 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: Both PCH and CPU eDP are DP, so set the is_dp flag to true. Add is_cpu_edp and is_pch_edp bools to make checking for each less verbose (rather

Re: [Intel-gfx] [PATCH 1/3] drm/i915: make DP configuration vars less confusing in ironlake_crtc_mode_se

2012-04-11 Thread Jesse Barnes
On Wed, 11 Apr 2012 11:53:55 +0200 Daniel Vetter dan...@ffwll.ch wrote: On Tue, Apr 10, 2012 at 10:22:09PM +0100, Chris Wilson wrote: On Tue, 10 Apr 2012 11:58:03 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: Both PCH and CPU eDP are DP, so set the is_dp flag to true. Add

[Intel-gfx] [PATCH 1/3] drm/i915: make DP configuration vars less confusing in ironlake_crtc_mode_se

2012-04-10 Thread Jesse Barnes
Both PCH and CPU eDP are DP, so set the is_dp flag to true. Add is_cpu_edp and is_pch_edp bools to make checking for each less verbose (rather than has_edp_encoder !intel_encoder_is_pch_edp() sprinkled everywhere). And rename the has_edp_encoder variable to just edp_encoder. With the above

Re: [Intel-gfx] [PATCH 1/3] drm/i915: make DP configuration vars less confusing in ironlake_crtc_mode_se

2012-04-10 Thread Chris Wilson
On Tue, 10 Apr 2012 11:58:03 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: Both PCH and CPU eDP are DP, so set the is_dp flag to true. Add is_cpu_edp and is_pch_edp bools to make checking for each less verbose (rather than has_edp_encoder !intel_encoder_is_pch_edp() sprinkled