Re: [Intel-gfx] [PATCH] drm/i915/chv: calculate rc6 residency correctly

2014-07-12 Thread Daniel Vetter
On Wed, Jul 09, 2014 at 02:55:56PM +0300, Mika Kuoppala wrote: The register to read cz count is different from vlv. Also the counts returned from CCK_CTL1 for BSW are (ticks in 30ns - 1). czcount_30ns of value 1 is a special case for 320Mhz. Bugzilla:

Re: [Intel-gfx] [drm-intel:for-linux-next 190/192] drivers/gpu/drm/i915/intel_pm.c:6929:5: sparse: symbol 'byt_gpu_freq' was not declared. Should it be static?

2014-07-12 Thread Daniel Vetter
On Sat, Jul 12, 2014 at 01:54:34AM +0800, kbuild test robot wrote: tree: git://anongit.freedesktop.org/drm-intel for-linux-next head: 7707df4ad6c73e005098c4b4db2f86494e9d404d commit: 22b1b2f866b2089d8264e367121c9c9ee0689da4 [190/192] drm/i915: CHV GPU frequency to opcode functions

Re: [Intel-gfx] [RFC v2 1/1] drm/i915: Power gating display wells during i915_pm_suspend

2014-07-12 Thread Daniel Vetter
On Sat, Jul 12, 2014 at 10:02:27AM +0530, sagar.a.kam...@intel.com wrote: From: Borun Fu borun...@intel.com On VLV, after i915_pm_suspend display power wells are staying power ungated. So, after initiating mem sleep echo mem /sys/power/state Display is staing D0 State. There might be better

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Daniel Vetter
On Fri, Jul 11, 2014 at 10:30:19AM -0700, Rodrigo Vivi wrote: Panel Self Refresh is an eDP power saving feature specified by VESA's eDP v1.3, that allows some panel componets to shutdown while you still see static images on the screen. Besides being supported on the platform it must be

Re: [Intel-gfx] [PATCH] drm/i915: Ignore VBT backlight presence check on HP Chromebook 14

2014-07-12 Thread Daniel Vetter
On Fri, Jul 11, 2014 at 10:16:30PM +, Scot Doyle wrote: commit c675949ec58ca50d5a3ae3c757892f1560f6e896 drm/i915: do not setup backlight if not available according to VBT caused a regression on the HP Chromebook 14 (with Celeron 2955U CPU), which has a misconfigured VBT. Apply quirk

Re: [Intel-gfx] [PATCH] igt/gem_partial_pwrite_pread: Add set-cache subtest to validate JIRA VIZ-3721

2014-07-12 Thread Daniel Vetter
On Fri, Jul 11, 2014 at 05:55:41PM +, Reese, Armin C wrote: Which of the I-G-Ts would be the best to house the set-cache subtest? Is gem_partial_pwrite_pread a good candidate? I guess you could just create a new test binary if there's nothing suitable. Especially for tests I'm leaning

Re: [Intel-gfx] [Xen-devel] [RFC][PATCH] gpu:drm:i915:intel_detect_pch: back to check devfn instead of check class type

2014-07-12 Thread Daniel Vetter
On Fri, Jul 11, 2014 at 08:30:59PM +, Tian, Kevin wrote: From: Konrad Rzeszutek Wilk [mailto:konrad.w...@oracle.com] Sent: Friday, July 11, 2014 12:42 PM On Fri, Jul 11, 2014 at 08:29:56AM +0200, Daniel Vetter wrote: On Thu, Jul 10, 2014 at 09:08:24PM +, Tian, Kevin wrote:

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Chris Wilson
On Sat, Jul 12, 2014 at 12:05:13PM +0200, Daniel Vetter wrote: On Fri, Jul 11, 2014 at 10:30:19AM -0700, Rodrigo Vivi wrote: Panel Self Refresh is an eDP power saving feature specified by VESA's eDP v1.3, that allows some panel componets to shutdown while you still see static images on

Re: [Intel-gfx] [PATCH 0/2] Low hanging fruit to reduce the driver size

2014-07-12 Thread Daniel Vetter
On Fri, Jul 11, 2014 at 06:34:12PM +0100, Damien Lespiau wrote: Being able to target a single platform to reduce the driver size has been voiced a few times. These patches provide a Kconfig option to provide the opportunity. Let's start small, and, along side the generic Multi-platform

Re: [Intel-gfx] [PATCH 0/2] Low hanging fruit to reduce the driver size

2014-07-12 Thread Damien Lespiau
On Sat, Jul 12, 2014 at 01:01:42PM +0200, Daniel Vetter wrote: And I'm really not terribly convinced that kb shaving is worth the effort overall, especially if it comes at a fairly steep maintainance burden of piles of new config options and #defines. I wanted to try, I tried, probably not

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Daniel Vetter
On Sat, Jul 12, 2014 at 12:42 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Sat, Jul 12, 2014 at 12:05:13PM +0200, Daniel Vetter wrote: On Fri, Jul 11, 2014 at 10:30:19AM -0700, Rodrigo Vivi wrote: Panel Self Refresh is an eDP power saving feature specified by VESA's eDP v1.3, that

[Intel-gfx] [PATCH 2/3] drm/i915: wait for all DSI FIFOs to be empty

2014-07-12 Thread Shobhit Kumar
Ensure that the DSI packets for a particular sequence are completely sent before going ahead in the enabling or disabling of the panel Signed-off-by: Shobhit Kumar shobhit.ku...@intel.com --- drivers/gpu/drm/i915/intel_dsi.c | 8 drivers/gpu/drm/i915/intel_dsi_cmd.c | 16

[Intel-gfx] [PATCH 0/3] Fixing last of few known issues in DSI and Burst mode Support

2014-07-12 Thread Shobhit Kumar
Hi, This pacth set addresses a couple WARN dumps as DSI encoder did not yet implement the -get_config for state tracking. This is tried to be addressed here. Most likely I have missed few things but atleast all WARNs are taken care of. Also better check is added to ensure all data has been sent

[Intel-gfx] [PATCH 3/3] drm/i915: Add support for Video Burst Mode for MIPI DSI

2014-07-12 Thread Shobhit Kumar
Signed-off-by: Shobhit Kumar shobhit.ku...@intel.com --- drivers/gpu/drm/i915/intel_bios.h | 3 ++- drivers/gpu/drm/i915/intel_dsi.c | 22 ++--- drivers/gpu/drm/i915/intel_dsi.h | 2 ++ drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 38

[Intel-gfx] [PATCH 1/3] drm/i915: Add get_config implementation for DSI encoder

2014-07-12 Thread Shobhit Kumar
Call to vlv_crtc_clock_get is not needed for DSI and was causing dpio read WARN dumps as well. Absence of -get_config was casuing othet WARN dumps as well. With this the last of the known WARN dumps for DSI should be fixed. Signed-off-by: Shobhit Kumar shobhit.ku...@intel.com ---

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Add get_config implementation for DSI encoder

2014-07-12 Thread Daniel Vetter
On Sat, Jul 12, 2014 at 1:47 PM, Shobhit Kumar shobhit.ku...@intel.com wrote: Call to vlv_crtc_clock_get is not needed for DSI and was causing dpio read WARN dumps as well. Absence of -get_config was casuing othet WARN dumps as well. With this the last of the known WARN dumps for DSI should be

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Chris Wilson
On Sat, Jul 12, 2014 at 01:20:10PM +0200, Daniel Vetter wrote: On Sat, Jul 12, 2014 at 12:42 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Sat, Jul 12, 2014 at 12:05:13PM +0200, Daniel Vetter wrote: On Fri, Jul 11, 2014 at 10:30:19AM -0700, Rodrigo Vivi wrote: Panel Self Refresh is

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Daniel Vetter
On Sat, Jul 12, 2014 at 2:02 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: But PSR is definitely an output property... Please-dont-touch-me is indeed a plane property. Well I guess we also want this for fbc, and fbc isn't an output property. I guess in the end it doesn't really matter that

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Chris Wilson
On Sat, Jul 12, 2014 at 07:14:33PM +0200, Daniel Vetter wrote: On Sat, Jul 12, 2014 at 2:02 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: But PSR is definitely an output property... Please-dont-touch-me is indeed a plane property. Well I guess we also want this for fbc, and fbc isn't

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Daniel Vetter
On Sat, Jul 12, 2014 at 7:51 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Sat, Jul 12, 2014 at 07:14:33PM +0200, Daniel Vetter wrote: On Sat, Jul 12, 2014 at 2:02 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: But PSR is definitely an output property... Please-dont-touch-me is

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Chris Wilson
On Sat, Jul 12, 2014 at 08:15:42PM +0200, Daniel Vetter wrote: On Sat, Jul 12, 2014 at 7:51 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Sat, Jul 12, 2014 at 07:14:33PM +0200, Daniel Vetter wrote: On Sat, Jul 12, 2014 at 2:02 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: But

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

2014-07-12 Thread Daniel Vetter
On Sat, Jul 12, 2014 at 8:55 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Sat, Jul 12, 2014 at 08:15:42PM +0200, Daniel Vetter wrote: On Sat, Jul 12, 2014 at 7:51 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Sat, Jul 12, 2014 at 07:14:33PM +0200, Daniel Vetter wrote: On Sat,