On Wed, Sep 10, 2014 at 06:17:07PM +0300, Imre Deak wrote: > The suspend_late handler saves some registers and powers off the device, > so it doesn't have a big overhead. Calling it at S4 poweroff_late time > makes the power off handling identical to the S3 suspend and S4 freeze > handling, so do this for consistency. Heh. I was wondering where .poweroff_late() was when reading the earlier patches. I like consistency. Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_drv.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index f6157a1..907a027 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -1520,6 +1520,7 @@ static const struct dev_pm_ops i915_pm_ops = { > .thaw_early = i915_pm_resume_early, > .thaw = i915_pm_resume, > .poweroff = i915_pm_suspend, > + .poweroff_late = i915_pm_suspend_late, > .restore_early = i915_pm_resume_early, > .restore = i915_pm_resume, > .runtime_suspend = intel_runtime_suspend, > -- > 1.8.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx