From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 18/21] drm/i915: Disable eDP VDD in a delayed work proc instead of synchronously Date: Fri, 30 Sep 2011 11:31:45 +0100 Message-ID: References: <20110923085243.6e4b7b4c@jbarnes-x220> <1317344993-24945-1-git-send-email-keithp@keithp.com> <1317344993-24945-19-git-send-email-keithp@keithp.com> Return-path: In-Reply-To: <1317344993-24945-19-git-send-email-keithp@keithp.com> Sender: linux-kernel-owner@vger.kernel.org To: Dave Airlie Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, Keith Packard List-Id: intel-gfx@lists.freedesktop.org On Thu, 29 Sep 2011 18:09:50 -0700, Keith Packard wrote: > +static void ironlake_panel_vdd_work(struct work_struct *__work) > +{ > + struct intel_dp *intel_dp = container_of(to_delayed_work(__work), > + struct intel_dp, panel_vdd_work); > + struct drm_device *dev = intel_dp->base.base.dev; > + > + mutex_lock(&dev->struct_mutex); > + ironlake_panel_vdd_off_sync(intel_dp); > + mutex_unlock(&dev->struct_mutex); I think you want the dev->mode_config.mutex here. -Chris -- Chris Wilson, Intel Open Source Technology Centre