From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH] drm/i915: Attach a PSR property on eDP Date: Sat, 14 Jun 2014 13:28:23 +0100 Message-ID: <20140614122823.GA6041@nuc-i3427.alporthouse.com> References: <1402691284-24458-1-git-send-email-chris@chris-wilson.co.uk> <1402691567-24531-1-git-send-email-chris@chris-wilson.co.uk> <20140614083956.GA4537@strange.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from fireflyinternet.com (mail.fireflyinternet.com [87.106.93.118]) by gabe.freedesktop.org (Postfix) with ESMTP id E45A66E02F for ; Sat, 14 Jun 2014 05:28:28 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140614083956.GA4537@strange.amr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Damien Lespiau Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Sat, Jun 14, 2014 at 09:39:56AM +0100, Damien Lespiau wrote: > On Fri, Jun 13, 2014 at 09:32:47PM +0100, Chris Wilson wrote: > > Let userspace know the status of Panel Self-Refresh by virtue of a > > property on the appropriate connector. > > > > v2: Only attach the property if the driver is capable of PSR. > > You probably want to expose it only if the panel is PSR-capable as well? > (is_edp_psr()) I wanted to expose it nevertheless as there was hints of supporting interchangeable panels. -Chris -- Chris Wilson, Intel Open Source Technology Centre