From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [PATCH] i915: send D1 opregion notification Date: Tue, 14 Jan 2014 15:08:25 -0800 Message-ID: <20140114150825.0d145e47@jbarnes-desktop> References: <1389648765-10904-1-git-send-email-kristen@linux.intel.com> <20140114092602.GR4770@phenom.ffwll.local> <20140114111944.61abeefb@jbarnes-desktop> <20140114150646.6322dffd@kcaccard-desk.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from alt-proxy17.mail.unifiedlayer.com (alt-proxy17.mail.unifiedlayer.com [66.147.241.60]) by gabe.freedesktop.org (Postfix) with SMTP id 6603C10682C for ; Tue, 14 Jan 2014 15:08:21 -0800 (PST) In-Reply-To: <20140114150646.6322dffd@kcaccard-desk.amr.corp.intel.com> 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: Kristen Carlson Accardi Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, 14 Jan 2014 15:06:46 -0800 Kristen Carlson Accardi wrote: > On Tue, 14 Jan 2014 11:19:44 -0800 > Jesse Barnes wrote: > > > On Tue, 14 Jan 2014 10:26:02 +0100 > > Daniel Vetter wrote: > > > > > On Mon, Jan 13, 2014 at 01:32:45PM -0800, Kristen Carlson Accardi wrote: > > > > The opregion notification for runtime suspend should be D1, not D3. > > > > This is to maintain compatibility with firmware. > > > > > > > > Signed-off-by: Kristen Carlson Accardi > > > > --- > > > > drivers/gpu/drm/i915/i915_drv.c | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > > > > index 61fb9fc..a6c370a 100644 > > > > --- a/drivers/gpu/drm/i915/i915_drv.c > > > > +++ b/drivers/gpu/drm/i915/i915_drv.c > > > > @@ -925,7 +925,7 @@ static int i915_runtime_suspend(struct device *device) > > > > > > > > del_timer_sync(&dev_priv->gpu_error.hangcheck_timer); > > > > dev_priv->pm.suspended = true; > > > > - intel_opregion_notify_adapter(dev, PCI_D3cold); > > > > + intel_opregion_notify_adapter(dev, PCI_D1); > > > > > > This needs a bigger comment and some mention on which platforms we exactly > > > need this ... Or do the bios people plan to stick to this for bdw/vlv and > > > all the further stuff? > > > > Doc # would help too, even if it's CDI or internal only... I don't know > > where to find this info, or if I did I've forgotten. > > > > Thanks, > > hahhah, you crack me up with all this "doc" stuff. There is no doc. Can you file an HSD or something on that internally? Seems like it should at least be part of the OpRegion spec or something... -- Jesse Barnes, Intel Open Source Technology Center