Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-12 Thread S, Deepak
On 6/13/2014 12:10 AM, Ville Syrjälä wrote: On Thu, Jun 12, 2014 at 11:45:03PM +0530, S, Deepak wrote: >> I was polling VLV_GTLC_SURVIVABILITY_REG and VLV_GTLC_PW_STATUS to make >> sure both render and media wells and the gfx clock remain off, and I >> was also monitoring vnn via svid. W

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-12 Thread Ville Syrjälä
On Thu, Jun 12, 2014 at 11:45:03PM +0530, S, Deepak wrote: > >> I was polling VLV_GTLC_SURVIVABILITY_REG and VLV_GTLC_PW_STATUS to make > >> sure both render and media wells and the gfx clock remain off, and I > >> was also monitoring vnn via svid. While that was going on I just rewrote > >> PU

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-12 Thread S, Deepak
>> I was polling VLV_GTLC_SURVIVABILITY_REG and VLV_GTLC_PW_STATUS to make >> sure both render and media wells and the gfx clock remain off, and I >> was also monitoring vnn via svid. While that was going on I just rewrote >> PUNIT_REG_GPU_FREQ_REQ to make Punit change the frequency, and sure >> e

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-12 Thread Lee, Chon Ming
On 06/12 11:11, Ville Syrjälä wrote: > On Thu, Jun 12, 2014 at 03:37:18PM +0800, Lee, Chon Ming wrote: > > On 06/12 08:14, S, Deepak wrote: > > > > > > > > > On 6/11/2014 10:26 PM, Ville Syrjälä wrote: > > > >On Fri, Jun 06, 2014 at 08:03:20AM -0700, Jesse Barnes wrote: > > > >>On Fri, 6 Jun 2014

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-12 Thread Ville Syrjälä
On Thu, Jun 12, 2014 at 03:37:18PM +0800, Lee, Chon Ming wrote: > On 06/12 08:14, S, Deepak wrote: > > > > > > On 6/11/2014 10:26 PM, Ville Syrjälä wrote: > > >On Fri, Jun 06, 2014 at 08:03:20AM -0700, Jesse Barnes wrote: > > >>On Fri, 6 Jun 2014 11:29:24 +0300 > > >>Ville Syrjälä wrote: > > >>

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-12 Thread Daniel Vetter
On Thu, Jun 12, 2014 at 03:37:18PM +0800, Lee, Chon Ming wrote: > On 06/12 08:14, S, Deepak wrote: > > > > > > On 6/11/2014 10:26 PM, Ville Syrjälä wrote: > > >On Fri, Jun 06, 2014 at 08:03:20AM -0700, Jesse Barnes wrote: > > >>On Fri, 6 Jun 2014 11:29:24 +0300 > > >>Ville Syrjälä wrote: > > >>

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-12 Thread Lee, Chon Ming
On 06/12 08:14, S, Deepak wrote: > > > On 6/11/2014 10:26 PM, Ville Syrjälä wrote: > >On Fri, Jun 06, 2014 at 08:03:20AM -0700, Jesse Barnes wrote: > >>On Fri, 6 Jun 2014 11:29:24 +0300 > >>Ville Syrjälä wrote: > >> > >>>On Thu, Jun 05, 2014 at 01:49:34PM -0700, Jesse Barnes wrote: > This ma

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-11 Thread S, Deepak
On 6/11/2014 10:26 PM, Ville Syrjälä wrote: On Fri, Jun 06, 2014 at 08:03:20AM -0700, Jesse Barnes wrote: On Fri, 6 Jun 2014 11:29:24 +0300 Ville Syrjälä wrote: On Thu, Jun 05, 2014 at 01:49:34PM -0700, Jesse Barnes wrote: This may take awhile (~10ms), and we don't need to make noise about

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-11 Thread Ville Syrjälä
On Fri, Jun 06, 2014 at 08:03:20AM -0700, Jesse Barnes wrote: > On Fri, 6 Jun 2014 11:29:24 +0300 > Ville Syrjälä wrote: > > > On Thu, Jun 05, 2014 at 01:49:34PM -0700, Jesse Barnes wrote: > > > This may take awhile (~10ms), and we don't need to make noise about it. > > > > > > References: https

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-06 Thread Jesse Barnes
On Fri, 6 Jun 2014 11:29:24 +0300 Ville Syrjälä wrote: > On Thu, Jun 05, 2014 at 01:49:34PM -0700, Jesse Barnes wrote: > > This may take awhile (~10ms), and we don't need to make noise about it. > > > > References: https://bugs.freedesktop.org/show_bug.cgi?id=75244 > > Tested-by: huax...@intel.c

Re: [Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-06 Thread Ville Syrjälä
On Thu, Jun 05, 2014 at 01:49:34PM -0700, Jesse Barnes wrote: > This may take awhile (~10ms), and we don't need to make noise about it. > > References: https://bugs.freedesktop.org/show_bug.cgi?id=75244 > Tested-by: huax...@intel.com > Signed-off-by: Jesse Barnes > --- > drivers/gpu/drm/i915/int

[Intel-gfx] [PATCH] drm/i915/vlv: drop punit freq staus read after setting idle

2014-06-05 Thread Jesse Barnes
This may take awhile (~10ms), and we don't need to make noise about it. References: https://bugs.freedesktop.org/show_bug.cgi?id=75244 Tested-by: huax...@intel.com Signed-off-by: Jesse Barnes --- drivers/gpu/drm/i915/intel_pm.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/d