From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915: fix pm init ordering Date: Fri, 6 Dec 2013 13:08:53 +0100 Message-ID: <20131206120853.GJ27344@phenom.ffwll.local> References: <1386321473-27423-1-git-send-email-daniel.vetter@ffwll.ch> 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 B3F38FA4F4 for ; Fri, 6 Dec 2013 04:08:05 -0800 (PST) Received: by mail-ea0-f173.google.com with SMTP id o10so262040eaj.32 for ; Fri, 06 Dec 2013 04:08:05 -0800 (PST) Content-Disposition: inline In-Reply-To: <1386321473-27423-1-git-send-email-daniel.vetter@ffwll.ch> 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: Intel Graphics Development Cc: Daniel Vetter List-Id: intel-gfx@lists.freedesktop.org On Fri, Dec 06, 2013 at 10:17:53AM +0100, Daniel Vetter wrote: > Shovel a bit more of the the code into the setup function, and call > it earlier. Otherwise lockdep is unhappy since we cancel the delayed > resume work before it's initialized. > > While at it also shovel the pc8 setup code into the same functions. > I wanted to also ditch the header declaration of the hws pc8 functions, > but for unfathomable reasons that stuff is in intel_display.c instead > of intel_pm.c. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71980 > Tested-by: Guo Jinxian > Signed-off-by: Daniel Vetter Merged to -fixes with Damien's irc r-b. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch