Re: [Intel-gfx] [PATCH] drm/i915/ddi: set has_infoframe flag on DDI too v2

2014-11-20 Thread Daniel Vetter
Let's be good citizen and summarize the important points from the irc chat between Jesse&me. On Thu, Nov 20, 2014 at 8:54 PM, Jesse Barnes wrote: >> Hm I dind't look too closely apparently at this. You again rely upon sw >> state here, just encoder->type this time around. Which means you can't >>

Re: [Intel-gfx] [PATCH] drm/i915/ddi: set has_infoframe flag on DDI too v2

2014-11-20 Thread Jesse Barnes
On Thu, 20 Nov 2014 16:49:42 +0100 Daniel Vetter wrote: > On Tue, Nov 18, 2014 at 09:45:52AM -0800, Jesse Barnes wrote: > > Just like we do in the HDMI code, set the infoframe flag if we detect > > that infoframes are enabled. > > > > v2: check for actual infoframe status as in hdmi code (Daniel

Re: [Intel-gfx] [PATCH] drm/i915/ddi: set has_infoframe flag on DDI too v2

2014-11-20 Thread Daniel Vetter
On Tue, Nov 18, 2014 at 09:45:52AM -0800, Jesse Barnes wrote: > Just like we do in the HDMI code, set the infoframe flag if we detect > that infoframes are enabled. > > v2: check for actual infoframe status as in hdmi code (Daniel) > > Signed-off-by: Jesse Barnes > --- > drivers/gpu/drm/i915/in

Re: [Intel-gfx] [PATCH] drm/i915/ddi: set has_infoframe flag on DDI too v2

2014-11-19 Thread Daniel Vetter
On Tue, Nov 18, 2014 at 09:45:52AM -0800, Jesse Barnes wrote: > Just like we do in the HDMI code, set the infoframe flag if we detect > that infoframes are enabled. > > v2: check for actual infoframe status as in hdmi code (Daniel) > > Signed-off-by: Jesse Barnes Queued for -next, thanks for th

Re: [Intel-gfx] [PATCH] drm/i915/ddi: set has_infoframe flag on DDI too

2014-11-19 Thread Daniel Vetter
On Tue, Nov 18, 2014 at 04:00:58PM -0800, shuang...@intel.com wrote: > Tested-By: PRC QA PRTS (Patch Regression Test System Contact: > shuang...@intel.com) > -Summary- > Platform: baseline_drm_intel_nightly_pass_rate->patch_ap

Re: [Intel-gfx] [PATCH] drm/i915/ddi: set has_infoframe flag on DDI too

2014-11-18 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) -Summary- Platform: baseline_drm_intel_nightly_pass_rate->patch_applied_pass_rate BYT: pass/total=290/290->290/290 PNV: pass/total=362/362->362

[Intel-gfx] [PATCH] drm/i915/ddi: set has_infoframe flag on DDI too v2

2014-11-18 Thread Jesse Barnes
Just like we do in the HDMI code, set the infoframe flag if we detect that infoframes are enabled. v2: check for actual infoframe status as in hdmi code (Daniel) Signed-off-by: Jesse Barnes --- drivers/gpu/drm/i915/intel_ddi.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/