Re: [Intel-gfx] [PATCH] drm/i915: Add checks to i915_bind_vma

2015-04-22 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6249 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH] lib/scatterlist: fix kerneldoc for sg_pcopy_{to, from}_buffer()

2015-04-22 Thread Jani Nikula
On Wed, 22 Apr 2015, Dave Gordon david.s.gor...@intel.com wrote: The kerneldoc for the functions doesn't match the code; the last two parameters (buflen, skip) have been transposed, which is confusing, especially as they're both integral types and the compiler won't warn about swapping them.

Re: [Intel-gfx] [PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

2015-04-22 Thread John Hunter
Hi, I think maybe we should remove both of the two lines: #include intel_drv.h #include i915_drv.h Because we have* two* intel_drv.h and *one* i915_drv.h, and i915_drv.h has already been included in the intel_drv.h. I not sure whether i am right. If you need me to do the patch, let me know. On

Re: [Intel-gfx] [PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

2015-04-22 Thread yongjun_...@trendmicro.com.cn
Hi John, Feel free to submit a new patch. Regards, Yongjun Wei From: John Hunter [mailto:zhjw...@gmail.com] Sent: 2015年4月22日 10:30 To: weiyj...@163.com; Daniel Vetter; Jani Nikula; David Airlie; Yongjun Wei (RD-CN); intel-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org; open list

Re: [Intel-gfx] [PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

2015-04-22 Thread John Hunter
Sure, but I need Daniel to admit that, because maybe include the two header file make it easier to understand. And after checked other files in drm/i915, I found that a lot other file do the same thing(include both header file). So I will just wait Daniel to wait up and give me the order :-) On

Re: [Intel-gfx] [PATCH 2/5] drm: Add edid_corrupt flag for Displayport Link CTS 4.2.2.6

2015-04-22 Thread Alex Deucher
On Tue, Apr 21, 2015 at 2:09 PM, Todd Previte tprev...@gmail.com wrote: Displayport compliance test 4.2.2.6 requires that a source device be capable of detecting a corrupt EDID. The test specification states that the sink device sets up the EDID with an invalid checksum. To do this, the sink

[Intel-gfx] [PATCH RFC 1/5] drm/i915: Get rid of intel_crtc_disable and related code.

2015-04-22 Thread maarten . lankhorst
From: Maarten Lankhorst maarten.lankho...@linux.intel.com Now that the dpll updates are (mostly) atomic, the .off() code is no longer used, and there are no more callers for intel_put_shared_dpll. Move all the updates done in intel_crtc_disable to intel_modeset_update_state, one less special