public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: Imre Deak <imre.deak@intel.com>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>
Subject: Re: Power saving using Display port HPD
Date: Tue, 15 Apr 2014 21:32:43 +0200	[thread overview]
Message-ID: <20140415193243.GF1023@phenom.ffwll.local> (raw)
In-Reply-To: <1397463473.2995.9.camel@intelbox>

On Mon, Apr 14, 2014 at 11:17:53AM +0300, Imre Deak wrote:
> On Mon, 2014-04-14 at 09:47 +0200, Daniel Vetter wrote:
> > On Mon, Apr 14, 2014 at 9:43 AM, Arun Chandran <achandran@mvista.com> wrote:
> > > 1)  Revert the commit 77961eb984c7e5394bd29cc7be2ab0bf0cc7e7b1.
> > > With this commit DP hotplug events are not coming after doing "xset dpms
> > > force off"
> > >
> > > commit bfcbf45b5b458ebdc38118ca67279a1cd90e085d
> > > Author: Arun Chandran <achandran@mvista.com>
> > > Date:   Fri Apr 11 16:16:32 2014 +0530
> > >
> > >     Revert "drm/i915: power domains: add vlv power wells"
> > >
> > >     This reverts commit 77961eb984c7e5394bd29cc7be2ab0bf0cc7e7b1.
> > 
> > So this breaks DP hotplug detection? Imre?
> 
> Yes, unfortunately. I made this clear in my patchset [1] and it was also
> discussed on IRC. If there isn't any (e)DP,HDMI pipe active we power
> down the DPIO HW block responsible normally for DP and HDMI hotplug
> detection.
> 
> There is one possible solution: the pin that is used for HPD detection
> can be used either normally in the above way, where it's controlled by
> the DPIO block, or as a GPIO where it's controlled by the GPIO HW block
> which is on even if we power down the DPIO. So during power down periods
> we could reconfigure that pin to work as a GPIO and treat any interrupts
> arriving it as an HPD event. I haven't had yet time to investigate this.
> 
> Another way is to turn on polling while powered down. This would also
> make VGA hotplug work, for which we don't have a GPIO alternative as
> above.
> 
> [1] 
> http://lists.freedesktop.org/archives/intel-gfx/2014-February/040232.html

Iirc we've agreed that when all screens are off it's ok to no longer
support hotplug. Or is this only the case when _only_ the DP port is off
but e.g. another port (edp or mipi) is on?

I'm asking since currently on hsw/bdw hotplug also doesn't work when you
switch everything off ...
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

  reply	other threads:[~2014-04-15 19:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-10  6:36 Power saving using Display port HPD Arun Chandran
2014-04-10  7:22 ` Jani Nikula
2014-04-10  7:24 ` Daniel Vetter
2014-04-10  7:39   ` Jani Nikula
2014-04-10 10:28   ` Arun Chandran
2014-04-14  7:43     ` Arun Chandran
2014-04-14  7:47       ` Daniel Vetter
2014-04-14  8:17         ` Imre Deak
2014-04-15 19:32           ` Daniel Vetter [this message]
2014-04-15 20:01             ` Imre Deak
2014-04-15 21:13               ` Daniel Vetter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140415193243.GF1023@phenom.ffwll.local \
    --to=daniel@ffwll.ch \
    --cc=imre.deak@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox