[Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-14 Thread deepak . s
From: Deepak S Added media/render/common well VLV force wake routines to help bring up the WELLS before access the register - Refactor current vlv_forcewake get/put and added MEDIA or RENDER specific Forcewake. - Added VLV Check to bring up MEDIA and RENDER WELL base on the register accessed

Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-19 Thread Jesse Barnes
On Thu, 14 Nov 2013 19:02:15 +0530 deepa...@intel.com wrote: > From: Deepak S > > Added media/render/common well VLV force wake routines to help > bring up the WELLS before access the register > - Refactor current vlv_forcewake get/put and added MEDIA or > RENDER specific Forcewake. > - Added

Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-19 Thread S, Deepak
scenarios (GL Beanchmark & Media playback). Thanks, Deepak -Original Message- From: Jesse Barnes [mailto:jbar...@virtuousgeek.org] Sent: Tuesday, November 19, 2013 11:36 PM To: S, Deepak Cc: intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific fo

Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-20 Thread Daniel Vetter
On Wed, Nov 20, 2013 at 7:00 AM, S, Deepak wrote: >>- have you done measurements on this? given how infrequently we > ought to be waking the wells when they're idle, and how long we > generally keep them awake, is this a real power win? > [Deepak] By Individually controlling the wells we

Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-20 Thread S, Deepak
Thanks Jesse, I wil split the patches and send it for review. Thanks Deepak -Original Message- From: Jesse Barnes [mailto:jbar...@virtuousgeek.org] Sent: Wednesday, November 20, 2013 9:35 PM To: S, Deepak Cc: intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH] drm/i915/vlv

Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-20 Thread Jesse Barnes
On Wed, 20 Nov 2013 06:00:24 + "S, Deepak" wrote: > Hi Jesse, > > Thanks for the review. Below is my response. > > > - why not use the callback to __vlv_force_wake_* from > gen6_gt_force_wake_*? i.e. why is VLV special here? > [Deepak] Gen6 has a single power well whereas the VLV

Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-22 Thread Jesse Barnes
y, November 20, 2013 9:35 PM > To: S, Deepak > Cc: intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake > routines. > > On Wed, 20 Nov 2013 06:00:24 + > "S, Deepak" wrote: > > > Hi Jesse,

Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake routines.

2013-11-22 Thread S, Deepak
sgeek.org] > Sent: Wednesday, November 20, 2013 9:35 PM > To: S, Deepak > Cc: intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i915/vlv: Add VLV specific force wake > routines. > > On Wed, 20 Nov 2013 06:00:24 + > "S, Deepak" w