From mboxrd@z Thu Jan  1 00:00:00 1970
From: Daniel Vetter <daniel@ffwll.ch>
Subject: Re: [PATCH 02/14] drm/i915: Reorganize vlv eDP reboot
 notifier
Date: Tue, 26 Aug 2014 15:36:07 +0200
Message-ID: <20140826133607.GY15520@phenom.ffwll.local>
References: <1408389369-22898-1-git-send-email-ville.syrjala@linux.intel.com>
 <1408389369-22898-3-git-send-email-ville.syrjala@linux.intel.com>
 <87egwdhsk8.fsf@intel.com> <20140826125851.GK4193@intel.com>
 <8761hfl743.fsf@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Received: from mail-we0-f174.google.com (mail-we0-f174.google.com
 [74.125.82.174])
 by gabe.freedesktop.org (Postfix) with ESMTP id 0C8AE6E50E
 for <intel-gfx@lists.freedesktop.org>; Tue, 26 Aug 2014 06:35:44 -0700 (PDT)
Received: by mail-we0-f174.google.com with SMTP id x48so14653321wes.5
 for <intel-gfx@lists.freedesktop.org>; Tue, 26 Aug 2014 06:35:43 -0700 (PDT)
Content-Disposition: inline
In-Reply-To: <8761hfl743.fsf@intel.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Errors-To: intel-gfx-bounces@lists.freedesktop.org
Sender: "Intel-gfx" <intel-gfx-bounces@lists.freedesktop.org>
To: Jani Nikula <jani.nikula@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

On Tue, Aug 26, 2014 at 04:21:00PM +0300, Jani Nikula wrote:
> On Tue, 26 Aug 2014, Ville Syrj=E4l=E4 <ville.syrjala@linux.intel.com> wr=
ote:
> > On Tue, Aug 19, 2014 at 10:00:55AM +0300, Jani Nikula wrote:
> >> On Mon, 18 Aug 2014, ville.syrjala@linux.intel.com wrote:
> >> > From: Ville Syrj=E4l=E4 <ville.syrjala@linux.intel.com>
> >> >
> >> > Move the vlv_power_sequencer_pipe() after the IS_VALLEYVIEW() check
> >> > and flatten the rest of the function.
> >> =

> >> Please imagine adding another platform there, and realize this just ad=
ds
> >> unnecessary churn.
> >
> > I'd just add another reboot notifier then.
> =

> Fair enough; it should be vlv_edp_notify_handler then. (No, don't send a
> patch to change that! ;)
> =

> > Frankly I don't understand the current one either. Why does it need to
> > set the delay to max for instance? And does this mean that the
> > PANEL_POWER_RESET bit doesn't actually work as advertised in the docs?
> =

> *shrug* experimental evidence?
> =

> commit 01527b3127997ef6370d5ad4fa25d96847fbf12a
> Author: Clint Taylor <clinton.a.taylor@intel.com>
> Date:   Mon Jul 7 13:01:46 2014 -0700
> =

>     drm/i915/vlv: T12 eDP panel timing enforcement during reboot
>     =

>     The panel power sequencer on vlv doesn't appear to accept changes to =
its
>     T12 power down duration during warm reboots. This change forces a del=
ay
>     for warm reboots to the T12 panel timing as defined in the VBT table =
for
>     the connected panel.

So if I remember this piece of lore correctly in the past the pp was
pessimistic, and enforced this delay on resume/boot-up, assuming you've
shut down _right_ before the machine was lit up again. Apparently people
where unhappy with that enforced delay and it was ditched on vlv, but then
it broke panels if you actually managed to reboot quickly enough.
-Daniel
-- =

Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch