Re: [Intel-gfx] [PATCH 00/66] runtime pm for DPMS

2014-05-16 Thread Naresh Kumar Kachhi
interludes to move a bit of the hdmi/audio state into the pipe config. Reviewer: Naresh Kumar reviewed 10-20, looks good. Reviewed-by: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com drm/i915/lvds: Remove -mode_set callback drm/i915/ddi: Remove -mode_set callback drm/i915/dsi: Remove

[Intel-gfx] [PATCH v1 0/2] disable rings before sync flush

2014-03-12 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com initial version: stop rings before sync flush v1: address the comments from Chris Wilson on http://www.spinics.net/lists/intel-gfx/msg41025.html. Rather than adding stop and start ring, we are moving ring disable before hw_status_page_setup

[Intel-gfx] [PATCH v1 1/2] drm/i915: disable rings before HW status page setup

2014-03-12 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Rings should be idle before issuing sync_flush (in intel_ring_setup_status_page). This patch moves the ring disabling before doing the HW status page setup. Signed-off-by: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com --- drivers/gpu

[Intel-gfx] [PATCH v1 2/2] drm/i915: warn if ring is active before sync flush

2014-03-12 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Based on Bspec the command parser must be stopped prior to issuing sync flush. This should be done by the caller of intel_ring_setup_status_page. Patch adds a warning if it is not done. Signed-off-by: Naresh Kumar Kachhi naresh.kumar.kac

[Intel-gfx] [PATCH v2 0/3] disable rings before sync flush

2014-03-12 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com initial version: stop rings before sync flush v1: address the comments from Chris Wilson on http://www.spinics.net/lists/intel-gfx/msg41025.html. Rather than adding stop and start ring, we are moving ring disable before hw_status_page_setup

[Intel-gfx] [PATCH v2 2/3] drm/i915: wait for rings to become idle once disabled

2014-03-12 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com make sure we wait for rings to become idle once they are disabled. In case of timeout print an error message Signed-off-by: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com --- drivers/gpu/drm/i915/i915_reg.h | 2 ++ drivers/gpu

[Intel-gfx] [PATCH v2 3/3] drm/i915: warn if ring is active before sync flush

2014-03-12 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Based on Bspec the command parser must be stopped prior to issuing sync flush. This should be done by the caller of intel_ring_setup_status_page. Patch adds a warning if it is not done. v2: rebased based on new patch (wait for ring

[Intel-gfx] [PATCH v2 1/3] drm/i915: disable rings before HW status page setup

2014-03-12 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Rings should be idle before issuing sync_flush (in intel_ring_setup_status_page). This patch moves the ring disabling before doing the HW status page setup. Signed-off-by: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Reviewed-by: Chris

[Intel-gfx] [PATCH] drm/i915: stop command parser before sync flush

2014-03-03 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Based on Bspec the command parser must be stopped prior to issuing sync flush. Only after observing Rings Idle set in MI_MODE can a Sync Flush be issued. Once sync flush has finished the command parser is re-enabled by clearing Stop Rings

Re: [Intel-gfx] [RFC 1/6] drm/i915: cover ioctls with runtime_get/put

2014-02-23 Thread Naresh Kumar Kachhi
On 01/24/2014 09:26 PM, Paulo Zanoni wrote: 2014/1/24 Naresh Kumar Kachhi naresh.kumar.kac...@intel.com: On 01/22/2014 06:53 PM, Imre Deak wrote: On Wed, 2014-01-22 at 13:51 +0100, Daniel Vetter wrote: On Wed, Jan 22, 2014 at 05:34:17PM +0530, naresh.kumar.kac...@intel.com wrote: From

Re: [Intel-gfx] [RFC 3/6] drm/i915: introduce runtime get/put based on display activity

2014-02-23 Thread Naresh Kumar Kachhi
On 01/22/2014 07:10 PM, Paulo Zanoni wrote: Hi 2014/1/22 naresh.kumar.kac...@intel.com: From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Once the display is disabled, we need to call runtime_put to make sure Runtime framework triggers runtime_suspend based on idleness. Similarly when

Re: [Intel-gfx] [RFC 1/6] drm/i915: cover ioctls with runtime_get/put

2014-02-23 Thread Naresh Kumar Kachhi
On 02/22/2014 01:37 AM, Jesse Barnes wrote: On Fri, 24 Jan 2014 19:23:54 +0200 Imre Deak imre.d...@intel.com wrote: On Fri, 2014-01-24 at 20:35 +0530, Naresh Kumar Kachhi wrote: On 01/22/2014 06:53 PM, Imre Deak wrote: On Wed, 2014-01-22 at 13:51 +0100, Daniel Vetter wrote: On Wed, Jan 22

Re: [Intel-gfx] [RFC 6/6] FOR_UPSTREAM [VPG]: drm/i915: call init_runtime_pm before gem_init

2014-01-24 Thread Naresh Kumar Kachhi
On 01/22/2014 07:14 PM, Paulo Zanoni wrote: 2014/1/22naresh.kumar.kac...@intel.com: From: Naresh Kumar Kachhinaresh.kumar.kac...@intel.com with current code intel_runtime_pm_gpu_idle is getting called even before runtime_pm is initialized. Moving runtime_pm_init before i915_gem_init Following

Re: [Intel-gfx] [RFC 5/6] drm/i915: device specific runtime suspend/resume

2014-01-24 Thread Naresh Kumar Kachhi
On 01/22/2014 06:28 PM, Daniel Vetter wrote: On Wed, Jan 22, 2014 at 05:34:21PM +0530, naresh.kumar.kac...@intel.com wrote: From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com we might need special handling for different platforms. for ex. baytrail. To handle this this patch creates

[Intel-gfx] [RFC 4/6] drm/i915/vlv: call runtime get/put based on disp activity

2014-01-22 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com do a runtime_get/put based on the display activity for valleyview platrform Signed-off-by: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com --- drivers/gpu/drm/i915/intel_display.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[Intel-gfx] [RFC 3/6] drm/i915: introduce runtime get/put based on display activity

2014-01-22 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com Once the display is disabled, we need to call runtime_put to make sure Runtime framework triggers runtime_suspend based on idleness. Similarly when display gets enabled, runtime_get should be called. We have similiar function for pc8 feature

[Intel-gfx] [RFC 0/6] enabling runtime on BYT

2014-01-22 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com This is the first set of patches to enable runtime pm for BYT. These patches are created keeping in mind that PC8 feature will be replaced with runtime pm. So all the patches are on assumption that HAS_RUNTIME_PM is true while HAS_PC8

[Intel-gfx] [RFC 5/6] drm/i915: device specific runtime suspend/resume

2014-01-22 Thread naresh . kumar . kachhi
From: Naresh Kumar Kachhi naresh.kumar.kac...@intel.com we might need special handling for different platforms. for ex. baytrail. To handle this this patch creates function pointers for runtime suspend/resume which are assigned during driver load time Signed-off-by: Naresh Kumar Kachhi