From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 00/14] drm/i915: edp vdd locking and prep for power sequencer kick Date: Tue, 19 Aug 2014 13:46:58 +0300 Message-ID: <20140819104658.GZ4193@intel.com> References: <1408389369-22898-1-git-send-email-ville.syrjala@linux.intel.com> <87oavghpfi.fsf@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id CFE246E590 for ; Tue, 19 Aug 2014 03:47:07 -0700 (PDT) Content-Disposition: inline In-Reply-To: <87oavghpfi.fsf@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Jani Nikula Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Aug 19, 2014 at 11:08:33AM +0300, Jani Nikula wrote: > On Mon, 18 Aug 2014, ville.syrjala@linux.intel.com wrote: > > From: Ville Syrj=E4l=E4 > > > > While wrestling with the VLV/CHV panel power sequencer I noticed the lo= cking > > in our edp vdd code was rather broken. This series aims to fix that by > > introducing a power seqeuencer mutex. I was already thinking about usin= g the > > aux.hw_mutex for this since it's already locked around the aux ->transf= er() > > function, but the VLV/CHV multiple power sequencer issue requires a sin= gle > > lock instead of per-port. > = > For extra kicks, see i915_save_display() and i915_restore_display(). Why > are we doing this to ourselves? Yeah, crap all around. I suppose someone needs to frob the lvds code a bit before we can kill the power sequencer stuff from those two functions. > = > BR, > Jani. > = > = > = > > > > At the end of the series there's a bit of reordering of the DP port > > enable/disable sequences to make subsequent power sequencer kick patches > > easier. The last patch fixes the wait_pipe_off() timeouts on my ILK. > > Strictly speaking it shouldn't be part of this series, but I couldn't > > really test this on my ILK without suffering tons of warnings so I > > included it here anyway. > > > > Ville Syrj=E4l=E4 (14): > > drm/i915: Parametrize PANEL_PORT_SELECT_VLV > > drm/i915: Reorganize vlv eDP reboot notifier > > drm/i915: Use intel_edp_panel_vdd_on() in intel_dp_probe_mst() > > drm/i915: Rename edp vdd funcs for consistency > > drm/i915: Add a note explaining vdd on/off handling in > > intel_dp_aux_ch() > > drm/i915: Replace big nested if block with early return > > drm/i915: Warn about want_panel_vdd in edp_panel_vdd_off_sync() > > drm/i915: Flatten intel_edp_panel_vdd_on() > > drm/i915: Fix edp vdd locking > > drm/i915: Track which port is using which pipe's power sequencer > > drm/i915: Be more careful when picking the initial power sequencer > > pipe > > drm/i915: Turn on panel power before doing aux transfers > > drm/i915: Enable DP port earlier > > drm/i915: Move DP port disable to post_disable for pch platforms > > > > drivers/gpu/drm/i915/i915_drv.h | 3 + > > drivers/gpu/drm/i915/i915_reg.h | 3 +- > > drivers/gpu/drm/i915/intel_display.c | 2 + > > drivers/gpu/drm/i915/intel_dp.c | 619 +++++++++++++++++++++++++--= -------- > > drivers/gpu/drm/i915/intel_drv.h | 6 + > > 5 files changed, 463 insertions(+), 170 deletions(-) > > > > -- = > > 1.8.5.5 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > = > -- = > Jani Nikula, Intel Open Source Technology Center -- = Ville Syrj=E4l=E4 Intel OTC