On Thu, Apr 02, 2015 at 06:58:22PM +0300, Imre Deak wrote: > On ke, 2015-04-01 at 16:22 +0530, Animesh Manna wrote: > > From: "A.Sunil Kamath" <sunil.kamath@xxxxxxxxx> > > > > This patch just implements the basic enable and disable > > functions of DC5 state which is needed for both SKL and BXT. > > Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx> For the record, this patch generates compilation warnings when applied on its own: drivers/gpu/drm/i915/intel_runtime_pm.c:368:13: warning: ‘gen9_enable_dc5’ defined but not used [-Wunused-function] static void gen9_enable_dc5(struct drm_i915_private *dev_priv) ^ drivers/gpu/drm/i915/intel_runtime_pm.c:386:13: warning: ‘gen9_disable_dc5’ defined but not used [-Wunused-function] static void gen9_disable_dc5(struct drm_i915_private *dev_priv) ^ Generally speaking, in a series, each step should compile without warning and result in a working driver (for bisectability). -- Damien _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx