Re: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Set D3_hot for vlv during runtime_suspend

2014-06-11 Thread Sagar Arun Kamble
On Wed, 2014-06-11 at 12:17 +0200, Daniel Vetter wrote: > On Wed, Jun 11, 2014 at 01:53:49PM +0530, Sagar Arun Kamble wrote: > > This patch can be marked as "abandoned". > > Have verified locally that pci driver is taking care of D0ix transitions > > and state save/restore. > > That still leaves t

Re: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Set D3_hot for vlv during runtime_suspend

2014-06-11 Thread Daniel Vetter
On Wed, Jun 11, 2014 at 01:53:49PM +0530, Sagar Arun Kamble wrote: > This patch can be marked as "abandoned". > Have verified locally that pci driver is taking care of D0ix transitions > and state save/restore. That still leaves the question why you originally thought this is required. What did bl

Re: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Set D3_hot for vlv during runtime_suspend

2014-06-11 Thread Sagar Arun Kamble
This patch can be marked as "abandoned". Have verified locally that pci driver is taking care of D0ix transitions and state save/restore. Thanks Imre for the clarification. On Tue, 2014-06-10 at 20:51 +0300, Imre Deak wrote: > On Tue, 2014-06-10 at 23:05 +0530, Sagar Arun Kamble wrote: > > On Tue

Re: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Set D3_hot for vlv during runtime_suspend

2014-06-10 Thread Imre Deak
On Tue, 2014-06-10 at 23:05 +0530, Sagar Arun Kamble wrote: > On Tue, 2014-06-10 at 15:43 +0300, Imre Deak wrote: > > On Tue, 2014-06-10 at 00:27 +0530, sagar.a.kam...@intel.com wrote: > > > From: Sagar Kamble > > > > > > To do a platform wide S0i3 transition, Gfx is required to go > > > to D3_ho

Re: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Set D3_hot for vlv during runtime_suspend

2014-06-10 Thread Sagar Arun Kamble
On Tue, 2014-06-10 at 15:43 +0300, Imre Deak wrote: > On Tue, 2014-06-10 at 00:27 +0530, sagar.a.kam...@intel.com wrote: > > From: Sagar Kamble > > > > To do a platform wide S0i3 transition, Gfx is required to go > > to D3_hot state. pci_save_state and pci_restore_state needed to avoid ring > > h

Re: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Set D3_hot for vlv during runtime_suspend

2014-06-10 Thread Imre Deak
On Tue, 2014-06-10 at 00:27 +0530, sagar.a.kam...@intel.com wrote: > From: Sagar Kamble > > To do a platform wide S0i3 transition, Gfx is required to go > to D3_hot state. pci_save_state and pci_restore_state needed to avoid ring > hangs across D3_hot transitions. > > Cc: Daniel Vetter (support

[Intel-gfx] [PATCH 2/2] drm/i915/vlv: Set D3_hot for vlv during runtime_suspend

2014-06-09 Thread sagar . a . kamble
From: Sagar Kamble To do a platform wide S0i3 transition, Gfx is required to go to D3_hot state. pci_save_state and pci_restore_state needed to avoid ring hangs across D3_hot transitions. Cc: Daniel Vetter (supporter:INTEL DRM DRIVERS...) Cc: Jani Nikula (supporter:INTEL DRM DRIVERS...) Signed