Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-04-07 Thread Daniel Vetter
On Thu, Apr 02, 2015 at 11:02:44AM +0530, Sonika Jindal wrote: > We make use of HW tracking for Selective update region and enable frame sync > on > sink. We use hardware's hardcoded data values for frame sync and GTC. > > v2: Add 3200x2000 resolution restriction with PSR2, move psr2_support to

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-04-01 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6095 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -2 270/270

[Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-04-01 Thread Sonika Jindal
We make use of HW tracking for Selective update region and enable frame sync on sink. We use hardware's hardcoded data values for frame sync and GTC. v2: Add 3200x2000 resolution restriction with PSR2, move psr2_support to i915_psr struct, add aux_frame_sync to independently control aux frame syn

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-31 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6095 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -2 270/270

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-31 Thread Daniel Vetter
On Tue, Mar 31, 2015 at 11:28:33AM +0530, Sonika Jindal wrote: > We make use of HW tracking for Selective update region and enable frame sync > on > sink. We use hardware's hardcoded data values for frame sync and GTC. > > v2: Add 3200x2000 resolution restriction with PSR2, move psr2_support to

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-30 Thread R, Durgadoss
>-Original Message- >From: Jindal, Sonika >Sent: Tuesday, March 31, 2015 11:29 AM >To: intel-gfx@lists.freedesktop.org >Cc: Jindal, Sonika; R, Durgadoss; Vivi, Rodrigo >Subject: [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync > >We make use of HW tracking for Selective update region

[Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-30 Thread Sonika Jindal
We make use of HW tracking for Selective update region and enable frame sync on sink. We use hardware's hardcoded data values for frame sync and GTC. v2: Add 3200x2000 resolution restriction with PSR2, move psr2_support to i915_psr struct, add aux_frame_sync to independently control aux frame syn

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-30 Thread Jindal, Sonika
/skl: Enabling PSR2 SU with frame sync We make use of HW tracking for Selective update region and enable frame sync on sink. We use hardware's hardcoded data values for frame sync and GTC. v2: Add 3200x2000 resolution restriction with PSR2, move psr2_support to i915_psr struct, add aux_frame

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-26 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6057 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -1 276/276

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-26 Thread R, Durgadoss
>-Original Message- >From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of >Sonika Jindal >Sent: Thursday, March 26, 2015 1:57 PM >To: intel-gfx@lists.freedesktop.org >Cc: Vivi, Rodrigo >Subject: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PS

[Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-26 Thread Sonika Jindal
We make use of HW tracking for Selective update region and enable frame sync on sink. We use hardware's hardcoded data values for frame sync and GTC. v2: Add 3200x2000 resolution restriction with PSR2, move psr2_support to i915_psr struct, add aux_frame_sync to independently control aux frame syn

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-23 Thread sonika
On Monday 23 March 2015 02:10 PM, Sivakumar Thulasimani wrote: On 3/20/2015 11:27 AM, Sonika Jindal wrote: +#define EDP_MAX_SU_DISABLE_TIME_MASK (0x1f<<20) +#define EDP_PSR2_TP2_TIME_500(0<<8) +#define EDP_PSR2_TP2_TIME_100(1<<8) +#define EDP_PSR2_TP2_TI

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-23 Thread Vivi, Rodrigo
On Mon, 2015-03-23 at 10:14 +0100, Daniel Vetter wrote: > On Fri, Mar 20, 2015 at 11:26:19PM +, Vivi, Rodrigo wrote: > > Hi Sonika, > > > > on the previous email I forgot to tell that we also need to add > > somewhere a check to avoid PSR for resolutions bigger than 3200x2000. > > > > Althoug

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-23 Thread Daniel Vetter
On Fri, Mar 20, 2015 at 11:26:19PM +, Vivi, Rodrigo wrote: > Hi Sonika, > > on the previous email I forgot to tell that we also need to add > somewhere a check to avoid PSR for resolutions bigger than 3200x2000. > > Although most of displays that we have that support psr are 3200x1800 it > is

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-23 Thread sonika
On Saturday 21 March 2015 04:56 AM, Vivi, Rodrigo wrote: Hi Sonika, on the previous email I forgot to tell that we also need to add somewhere a check to avoid PSR for resolutions bigger than 3200x2000. Yes, I will add that check before enabling PSR2. Although most of displays that we have tha

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-23 Thread Sivakumar Thulasimani
On 3/20/2015 11:27 AM, Sonika Jindal wrote: +#define EDP_MAX_SU_DISABLE_TIME_MASK (0x1f<<20) +#define EDP_PSR2_TP2_TIME_500(0<<8) +#define EDP_PSR2_TP2_TIME_100(1<<8) +#define EDP_PSR2_TP2_TIME_250(2<<8) Better to make all values inline, 5

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-22 Thread sonika
On Saturday 21 March 2015 02:50 AM, Vivi, Rodrigo wrote: On Fri, 2015-03-20 at 11:27 +0530, Sonika Jindal wrote: We make use of HW tracking for Selective update region and enable frame sync on sink. We use hardware's hardcoded data values for frame sync and GTC. Before enabling HW tracking for

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-20 Thread Vivi, Rodrigo
Hi Sonika, on the previous email I forgot to tell that we also need to add somewhere a check to avoid PSR for resolutions bigger than 3200x2000. Although most of displays that we have that support psr are 3200x1800 it is better to think a way to protect now than later. Thanks, Rodrigo. On Fri,

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-20 Thread Vivi, Rodrigo
On Fri, 2015-03-20 at 11:27 +0530, Sonika Jindal wrote: > We make use of HW tracking for Selective update region and enable frame sync > on > sink. We use hardware's hardcoded data values for frame sync and GTC. Before enabling HW tracking for PSR2 I'd like to know if all known bad cases of bad H

Re: [Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-20 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6014 -Summary- Platform Delta drm-intel-nightly Series Applied PNV -1 274/274

[Intel-gfx] [PATCH] drm/i915/skl: Enabling PSR2 SU with frame sync

2015-03-19 Thread Sonika Jindal
We make use of HW tracking for Selective update region and enable frame sync on sink. We use hardware's hardcoded data values for frame sync and GTC. Signed-off-by: Sonika Jindal --- drivers/gpu/drm/i915/i915_reg.h | 14 ++ drivers/gpu/drm/i915/intel_dp.c | 16