From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH v2 0/3] drm/i915: Split VLV forcewake routines. Date: Thu, 28 Nov 2013 08:36:10 +0100 Message-ID: <20131128073610.GE27344@phenom.ffwll.local> References: <1385610643-8352-1-git-send-email-deepak.s@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ea0-f173.google.com (mail-ea0-f173.google.com [209.85.215.173]) by gabe.freedesktop.org (Postfix) with ESMTP id 28934FAE52 for ; Wed, 27 Nov 2013 23:35:28 -0800 (PST) Received: by mail-ea0-f173.google.com with SMTP id g15so5415180eak.4 for ; Wed, 27 Nov 2013 23:35:27 -0800 (PST) Content-Disposition: inline In-Reply-To: <1385610643-8352-1-git-send-email-deepak.s@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: deepak.s@intel.com Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Nov 28, 2013 at 09:20:40AM +0530, deepak.s@intel.com wrote: > From: Deepak S > > Valleyview has power wells MEDIA & RENDER and by spliting vlv force wake > routines and individually controling Media/Render well, We have seen > power savings in the lower sub-1W range on different workloads, e.g. > glbenchmark, media playback > > v2: Addressed review comments. I've alredy pulled it all in and fixed things up while applying ;-) And iirc this is v3 of this series if we count the not-split patch as v1. Please double-check that I didn't miss anything in latest -nightly. -Daniel > > Deepak S (3): > drm/i915: Add power well arguments to force wake routines. > drm/i915/vlv: Valleyview support for forcewake Individual power wells. > v2 > drm/i915: Enabling DebugFS for valleyview forcewake counts. v2 > > drivers/gpu/drm/i915/i915_debugfs.c | 22 ++-- > drivers/gpu/drm/i915/i915_drv.h | 32 ++++- > drivers/gpu/drm/i915/i915_reg.h | 4 +- > drivers/gpu/drm/i915/intel_display.c | 4 +- > drivers/gpu/drm/i915/intel_pm.c | 22 ++-- > drivers/gpu/drm/i915/intel_ringbuffer.c | 14 ++- > drivers/gpu/drm/i915/intel_uncore.c | 217 +++++++++++++++++++++++++------- > 7 files changed, 246 insertions(+), 69 deletions(-) > > -- > 1.8.4.2 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch