From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <ville.syrjala@linux.intel.com>
Subject: Re: [PATCH] drm/i915: Add Baytrail PSR Support.
Date: Fri, 24 Jan 2014 19:41:04 +0200
Message-ID: <20140124174104.GJ9454@intel.com>
References: <1390501010-4641-4-git-send-email-rodrigo.vivi@gmail.com>
	<1390504793-6256-1-git-send-email-rodrigo.vivi@gmail.com>
	<20140124145316.GH9454@intel.com>
	<CABVU7+s3bRffjuo2oGo8pzMu6zEtJoifVcWpSQ5_hfRbbPa_xg@mail.gmail.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 mga02.intel.com (mga02.intel.com [134.134.136.20])
	by gabe.freedesktop.org (Postfix) with ESMTP id 4FBF1FA2D6
	for <intel-gfx@lists.freedesktop.org>;
	Fri, 24 Jan 2014 09:41:10 -0800 (PST)
Content-Disposition: inline
In-Reply-To: <CABVU7+s3bRffjuo2oGo8pzMu6zEtJoifVcWpSQ5_hfRbbPa_xg@mail.gmail.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>
Sender: intel-gfx-bounces@lists.freedesktop.org
Errors-To: intel-gfx-bounces@lists.freedesktop.org
To: Rodrigo Vivi <rodrigo.vivi@gmail.com>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>
List-Id: intel-gfx@lists.freedesktop.org

On Fri, Jan 24, 2014 at 02:05:57PM -0200, Rodrigo Vivi wrote:
> On Fri, Jan 24, 2014 at 12:53 PM, Ville Syrj=E4l=E4
> <ville.syrjala@linux.intel.com> wrote:
> > On Thu, Jan 23, 2014 at 05:19:53PM -0200, Rodrigo Vivi wrote:
> > <snip>
> >> index 76126e0..f5501ab 100644
> >> --- a/drivers/gpu/drm/i915/i915_reg.h
> >> +++ b/drivers/gpu/drm/i915/i915_reg.h
> >> @@ -1969,6 +1969,40 @@
> >>  #define BCLRPAT(pipe) _PIPE(pipe, _BCLRPAT_A, _BCLRPAT_B)
> >>  #define VSYNCSHIFT(trans) _TRANSCODER(trans, _VSYNCSHIFT_A, _VSYNCSHI=
FT_B)
> >>
> >> +/* VLV eDP PSR registers */
> >> +#define VLV_EDP_PSR_CTL                              (VLV_DISPLAY_BAS=
E + 0x60090)
> >
> > VLV has per-pipe PSR registers. The ones you have here are just for
> > pipe A. Seems like some rework is needed to make it work on either
> > pipe.
> =

> Yes, but since I don't have any hw with two eDPs here I decided to let
> the limitation we had for HSW, PSR only on pipe A.
> In my point of view we could go ahead with this one eDP scenario and
> implement psr on pipe b support later.

I don't see any pipe checks in the code, so you will happily enable PSR
on pipe A even if eDP is being fed by pipe B at the time.

Also even if you enable PSR on pipe A, you set the trunk clock gate
disable for pipe B, which seems weird. Setting that bit might actually
be the reason the pipe, port and PLL remain enabled during PSR.

-- =

Ville Syrj=E4l=E4
Intel OTC