From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vandana Kannan Subject: Re: [PATCH 0/5] Enabling DRRS support in the kernel Date: Wed, 18 Dec 2013 15:45:06 +0530 Message-ID: <52B175AA.4050105@intel.com> References: <1387258107-19232-1-git-send-email-vandana.kannan@intel.com> <20131218092400.GF26371@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id A4AA0FBEFE for ; Wed, 18 Dec 2013 02:15:09 -0800 (PST) In-Reply-To: <20131218092400.GF26371@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Daniel Vetter Cc: "intel-gfx@lists.freedesktop.org" List-Id: intel-gfx@lists.freedesktop.org On Dec-18-2013 2:54 PM, Daniel Vetter wrote: > On Tue, Dec 17, 2013 at 10:58:22AM +0530, Vandana Kannan wrote: >> Dynamic Refresh Rate Switching (DRRS) is a power conservation feature which >> enables switching between low and high refresh rates based on the usage >> scenario. This feature is applicable for internal eDP panel. Indication that >> the panel can support DRRS is given by the panel EDID, which would list >> multiple refresh rates for one resolution. >> The patch series supports idleness detection in display i915 driver and switch >> to low refresh rate. > > Just one overall comment on process: > > When resending patches _always_ mention what you've changed compared to > the last version. Specifically call out which review comments you've > incorporated and even more important which one's you've decided to > implement differently or rejected. > > The preferred form for that is with a per-patch changelog in the commit > message. Also mentioned the reviewer's name if the change is due to > review. > > For bigger reworks that affect the entire patch series please also mention > them in the cover letter. > > I'll update our internal BKMs with this information. > > Thanks, Daniel Thanks Daniel. I will incorporate these comments while resending the patches. - Vandana >> >> Pradeep Bhat (3): >> [Intel-gfx] drm/i915: Adding VBT fields to support eDP DRRS feature >> [Intel-gfx] drm/i915: Parse EDID probed modes for DRRS support >> [Intel-gfx] drm/i915: Add support for DRRS to switch RR >> >> Vandana Kannan (2): >> [Intel-gfx] drm/i915: Idleness detection for DRRS >> [Intel-gfx] drm/i915/bdw: Add support for DRRS to switch RR >> >> drivers/gpu/drm/i915/i915_drv.h | 30 ++++++ >> drivers/gpu/drm/i915/i915_reg.h | 1 + >> drivers/gpu/drm/i915/intel_bios.c | 23 +++++ >> drivers/gpu/drm/i915/intel_bios.h | 29 ++++++ >> drivers/gpu/drm/i915/intel_display.c | 13 +++ >> drivers/gpu/drm/i915/intel_dp.c | 180 ++++++++++++++++++++++++++++++++++ >> drivers/gpu/drm/i915/intel_drv.h | 35 ++++++- >> drivers/gpu/drm/i915/intel_pm.c | 112 +++++++++++++++++++++ >> drivers/gpu/drm/i915/intel_sprite.c | 3 + >> 9 files changed, 425 insertions(+), 1 deletion(-) >> >> -- >> 1.7.9.5 >> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx >