public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: "chris@chris-wilson.co.uk" <chris@chris-wilson.co.uk>,
	"Vivi, Rodrigo" <rodrigo.vivi@intel.com>,
	"daniel@ffwll.ch" <daniel@ffwll.ch>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"rodrigo.vivi@gmail.com" <rodrigo.vivi@gmail.com>
Subject: Re: [PATCH 7/7] drm/i915: Enable PSR by default.
Date: Wed, 25 Mar 2015 14:53:39 +0100	[thread overview]
Message-ID: <20150325135339.GI1349@phenom.ffwll.local> (raw)
In-Reply-To: <20150324220503.GA9923@nuc-i3427.alporthouse.com>

On Tue, Mar 24, 2015 at 10:05:03PM +0000, chris@chris-wilson.co.uk wrote:
> On Tue, Mar 24, 2015 at 08:55:04PM +0000, Vivi, Rodrigo wrote:
> > On Tue, 2015-03-24 at 10:08 +0000, Chris Wilson wrote:
> > > On Tue, Mar 24, 2015 at 11:03:30AM +0100, Daniel Vetter wrote:
> > > > On Mon, Mar 23, 2015 at 01:20:07PM -0700, Rodrigo Vivi wrote:
> > > > > Hi Daniel,
> > > > > 
> > > > > Is something missing to enable it by default?
> > > > 
> > > > Patch 1 has a small comment from me and latest version of patch 2 lacks an
> > > > r-b afaict. Hence why I didn't pull in the series yet. But it's also a
> > > > longer discussion, so ca you please resend the entire series with r-b tags
> > > > added to make sure I don't pick up the wrong versions?
> > > 
> > > Do I yet have a method to tell when PSR is active on an output?
> > 
> > Is that pre computed pipe_config->psr_enabled enough or you mean the
> > immediate psr.active also exposed there on pipe_config?
> 
> I want a property exposed to userspace. Having a tristate that says
> unsupported, disabled, active would be most useful.

Imo we should get the current behind-the-scenes psr support working
correctly first. Later on we can add userspace abi to expose this to
compositors, but that should be a separate step.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2015-03-25 13:51 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-28  1:26 [PATCH 1/7] drm/i915: PSR: Remove wrong LINK_DISABLE Rodrigo Vivi
2015-02-28  1:26 ` [PATCH 2/7] drm/i915: PSR: Fix DP_PSR_NO_TRAIN_ON_EXIT logic Rodrigo Vivi
2015-03-16  5:24   ` R, Durgadoss
2015-03-16 17:35     ` [PATCH] " Rodrigo Vivi
2015-03-16 23:36       ` shuang.he
2015-03-24 15:29       ` Rodrigo Vivi
2015-03-25  0:39         ` Runyan, Arthur J
2015-04-10 18:10           ` Rodrigo Vivi
2015-04-11  1:22             ` shuang.he
2015-04-14 13:18             ` R, Durgadoss
2015-02-28  1:26 ` [PATCH 3/7] drm/i915: PSR: deprecate link_standby support for core platforms Rodrigo Vivi
2015-03-02 11:41   ` Jindal, Sonika
2015-03-02 20:27     ` Rodrigo Vivi
2015-03-16  5:28   ` R, Durgadoss
2015-03-16 17:37     ` [PATCH] " Rodrigo Vivi
2015-03-16 17:38     ` Rodrigo Vivi
2015-02-28  1:26 ` [PATCH 4/7] drm/i915: PSR VLV: Add single frame update Rodrigo Vivi
2015-03-05  2:48   ` Pandiyan, Dhinakaran
2015-02-28  1:26 ` [PATCH 5/7] drm/i915: Make sure we invalidate frontbuffer on fbcon Rodrigo Vivi
2015-03-02 17:59   ` Daniel Vetter
2015-03-02 18:35     ` Vivi, Rodrigo
2015-03-03  8:28       ` Daniel Vetter
2015-03-03 20:03         ` Vivi, Rodrigo
2015-03-04 14:30           ` Daniel Vetter
2015-03-04 23:05             ` Rodrigo Vivi
2015-03-05 12:06               ` Daniel Vetter
2015-03-10  0:57                 ` [PATCH] " Rodrigo Vivi
2015-03-10 10:08                   ` shuang.he
2015-03-10 10:23                   ` Daniel Vetter
2015-02-28  1:26 ` [PATCH 6/7] drm/i915: VLV/CHV PSR: Increase wait delay time before active PSR Rodrigo Vivi
2015-03-16  5:15   ` R, Durgadoss
2015-02-28  1:26 ` [PATCH 7/7] drm/i915: Enable PSR by default Rodrigo Vivi
2015-03-03  9:54   ` shuang.he
2015-03-16  5:31   ` R, Durgadoss
2015-03-23 20:20     ` Rodrigo Vivi
2015-03-24 10:03       ` Daniel Vetter
2015-03-24 10:08         ` Chris Wilson
2015-03-24 20:55           ` Vivi, Rodrigo
2015-03-24 22:05             ` chris
2015-03-25 13:53               ` Daniel Vetter [this message]
2015-03-25 19:27               ` Vivi, Rodrigo
2015-03-25 19:40                 ` chris
2015-03-02 17:56 ` [PATCH 1/7] drm/i915: PSR: Remove wrong LINK_DISABLE Daniel Vetter
2015-03-16  5:15 ` R, Durgadoss
2015-04-09 17:42 ` Matthew Garrett
2015-04-13 23:11   ` Rodrigo Vivi

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=20150325135339.GI1349@phenom.ffwll.local \
    --to=daniel@ffwll.ch \
    --cc=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=rodrigo.vivi@gmail.com \
    --cc=rodrigo.vivi@intel.com \
    /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