Re: [Intel-gfx] [PATCH 3/5] drm/i915: Fixes to support the DP Compliance EDID tests.

On Monday 23 May 2016 01:48 PM, Ander Conselvan De Oliveira wrote: On Fri, 2016-04-29 at 18:28 -0700, Manasi Navare wrote: This patch addresses a few issues from the original patch for DP Compliance EDID test support submitted by Todd Previte Do you mean commit

Re: [Intel-gfx] [PATCH 3/5] drm/i915: Fixes to support the DP Compliance EDID tests.

On Monday 23 May 2016 01:48 PM, Ander Conselvan De Oliveira wrote: On Fri, 2016-04-29 at 18:28 -0700, Manasi Navare wrote: This patch addresses a few issues from the original patch for DP Compliance EDID test support submitted by Todd Previte Do you mean commit

[Intel-gfx] [PATCH 2/2] drm/i915: Lane count change detection

-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/i915_irq.c | 2 ++ drivers/gpu/drm/i915/intel_dp

[Intel-gfx] [PATCH 1/2] drm/i915: Always perform link training when requested

com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 4c564ba..d5

Re: [Intel-gfx] [PATCH 2/5] drm/i915: Read test values for lane_count and link_rate

automated testing (test numbers from the CTS usite) in the commit message. Regards, Manasi Navare Graphics Kernel Developer OTC, Intel Corporation -Original Message- From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of Shubhangi Shrivastava Sent: Monday, April 25,

[Intel-gfx] [PATCH 3/5] drm/i915: Cleanup panel IRQ handling

to clear the bits only when handled, so this patch moves the IRQ clearing bit post handling code. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/int

[Intel-gfx] [PATCH 5/5] drm/i915: Delay acknowledging of link training for ATR

alreay ack'ed the request. So send the ack response only when performing link training with the given parameters. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i91

[Intel-gfx] [PATCH 2/5] drm/i915: Read test values for lane_count and link_rate

sim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 25 + 1 file changed, 25 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 1b26c59..38

[Intel-gfx] [PATCH 1/5] drm/i915: Handle automated test requests for short pulse hpd

This patch adds support for automated test requests during short pulse handling in gen platforms. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 12 +++

[Intel-gfx] [PATCH 4/5] drm/i915: Extract test reply to separate function

Replying for automated test request must be done only when we are ready for the request. This patch extracts this reply to separate function as it is needed in other places in the upcoming patches. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shu

[Intel-gfx] [PATCH 2/5] drm/i915: Read test values for lane_count and link_rate

sim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 25 + 1 file changed, 25 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 2a14603..04

[Intel-gfx] [PATCH 5/5] drm/i915: Delay acknowledging of link training for ATR

alreay ack'ed the request. So send the ack response only when performing link training with the given parameters. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i91

[Intel-gfx] [PATCH 1/5] drm/i915: Handle automated test requests for short pulse hpd

This patch adds support for automated test requests during short pulse handling in gen platforms. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 12 +++

[Intel-gfx] [PATCH 3/5] drm/i915: Cleanup panel IRQ handling

to clear the bits only when handled, so this patch moves the IRQ clearing bit post handling code. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/int

[Intel-gfx] [PATCH 4/5] drm/i915: Extract test reply to separate function

Replying for automated test request must be done only when we are ready for the request. This patch extracts this reply to separate function as it is needed in other places in the upcoming patches. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shu

[Intel-gfx] [PATCH] drm/i915: Use fail safe mode when edid is corrupt

anytime by using edid_corrupt flag instead of checking for Automated test request flag. Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/int

[Intel-gfx] [PATCH] drm/i915: Fixing eDP detection on certain platforms

...@linux.intel.com> Reported-by: Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Tested-by: Tvrtko Ursulin <tvrtko.ursu...@intel.c

[Intel-gfx] [PATCH] drm/i915: Fixing eDP detection on certain platforms

d9aa4265fe ("drm/i915: Read sink_count dpcd always") Cc: Ander Conselvan De Oliveira <conselv...@gmail.com> Cc: Jani Nikula <jani.nik...@linux.intel.com> Reported-by: Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulas

[Intel-gfx] [PATCH] drm/i915: Fixing eDP detection on certain platforms

Sink count was read for eDP as well as DP. But in some cases the sink count for eDP is received as zero, as against the expected non zero value. This is seen in: commit 30d9aa4265fe4b2b28239934770b2c2d59858df3 Author: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Date: Wed Mar 30

Re: [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/5] drm/i915: Splitting intel_dp_detect

On Thursday 07 April 2016 03:26 PM, Thulasimani, Sivakumar wrote: On 4/7/2016 1:54 PM, Ander Conselvan De Oliveira wrote: On Thu, 2016-04-07 at 10:58 +0300, Jani Nikula wrote: On Wed, 06 Apr 2016, Tvrtko Ursulin wrote: On 04/04/16 12:41, Tvrtko Ursulin

[Intel-gfx] [PATCH] drm/i915: Fixing eDP detection on certain platforms

Sink count was read for eDP as well as DP. But in some cases the sink count for eDP is received as zero, as against the expected non zero value. This is seen in: commit 30d9aa4265fe4b2b28239934770b2c2d59858df3 Author: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Date: Wed Mar 30

Re: [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/5] drm/i915: Splitting intel_dp_detect

On Friday 01 April 2016 01:11 PM, Ander Conselvan De Oliveira wrote: On Thu, 2016-03-31 at 12:38 +, Patchwork wrote: == Series Details == Series: series starting with [1/5] drm/i915: Splitting intel_dp_detect URL : https://patchwork.freedesktop.org/series/5044/ State : success I

Re: [Intel-gfx] [PATCH 5/5] drm/i915: force full detect on sink count change

by this series, but at this point is probably better to rebase and resend. CI seems to be a lot happier lately. :) Ander Alright.. Rebased and resent.. Finally, CI has become happy.. :) Thanks Ander!! Shubhangi Thanks and Regards, Shubhangi Shrivastava. -Original Message- From: Ander

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

On Wednesday 30 March 2016 08:18 PM, Jani Nikula wrote: On Fri, 25 Mar 2016, Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> wrote: This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

for the register and added required breaks. (Ville) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ville Syrjälä <ville.sy

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 1/5] drm/i915: Splitting intel_dp_detect

iob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gpu/drm/i915/intel_dp.c | 63 +

[Intel-gfx] [PATCH 4/5] drm/i915: Read sink_count dpcd always

anu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gp

[Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

multiple detects on hotplug. (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <

[Intel-gfx] [PATCH 3/5] drm/i915: Reorganizing intel_dp_check_link_status

; Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gpu/drm/i915/intel_dp.c | 65 +++-- 1 file changed, 36 insertions(+), 29 deletions(-) diff --git a/drivers/g

[Intel-gfx] [PATCH 5/5] drm/i915: force full detect on sink count change

in intel_dp_short_pulse (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

for the register and added required breaks. (Ville) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ville Syrjälä <ville.sy

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

for the register Changed logic to incorporate required breaks. (Ville) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ville S

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

sim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 2 ++ drivers/gpu/drm/i915/i915_irq.c | 20 drivers/gpu/drm/i915/i915_reg.h | 8

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

sim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/i915_irq.c | 21 drivers/gpu/drm/i915/i915_reg.h | 8

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

sim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/i915_irq.c | 21 drivers/gpu/drm/i915/i915_reg.h | 8

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

. (Jani) v3: Moved setting invert bit logic to bxt_hpd_irq_setup() and changed its logic to avoid looping twice. (Ville) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi

[Intel-gfx] [PATCH 5/5] drm/i915: force full detect on sink count change

in intel_dp_short_pulse (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <

[Intel-gfx] [PATCH 1/5] drm/i915: Splitting intel_dp_detect

iob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gpu/drm/i915/intel_dp.c | 63 +

[Intel-gfx] [PATCH 3/5] drm/i915: Reorganizing intel_dp_check_link_status

; Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gpu/drm/i915/intel_dp.c | 65 +++-- 1 file changed, 36 insertions(+), 29 deletions(-) diff --git a/drivers/g

[Intel-gfx] [PATCH 4/5] drm/i915: Read sink_count dpcd always

anu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gp

[Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

multiple detects on hotplug. (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <

[Intel-gfx] [PATCH] drm/i915: Set invert bit for hpd based on VBT

. (Jani) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

. (Jani) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915

[Intel-gfx] [PATCH 5/5] drm/i915: force full detect on sink count change

in intel_dp_short_pulse (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <

[Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 72 +--- drivers/gpu/drm/i915/intel_drv.h | 1 + 2 files changed, 47

[Intel-gfx] [PATCH 4/5] drm/i915: Save sink_count for tracking changes to it and read sink_count dpcd always

its into one. (Ander) v3: Added comment to explain the need of early return when sink count is 0. (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shr

[Intel-gfx] [PATCH 3/5] drm/i915: Reorganizing intel_dp_check_link_status

; Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gpu/drm/i915/intel_dp.c | 65 +++-- 1 file changed, 36 insertions(+), 29 deletions(-) diff --git a/drivers/g

[Intel-gfx] [PATCH 1/5] drm/i915: Splitting intel_dp_detect

com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <conselv...@gmail.com> --- drivers/gpu/drm/i915/intel_dp.c | 63 +

[Intel-gfx] [PATCH 2/2] drm/i915: Set invert bit for hpd based on VBT

sim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/i915_irq.c | 49 + drivers/gpu/drm/i915/i915_reg.h | 9 2 files chan

[Intel-gfx] [PATCH 1/2] drm/i915: Update VBT fields for child devices

This patch adds new fields that are not yet added in drm code in child devices struct Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Durgadoss R <durgados...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>

[Intel-gfx] [PATCH 3/5] drm/i915: Reorganizing intel_dp_check_link_status

; Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 65 +++-- 1 file changed, 36 insertions(+), 29 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index ce99d

[Intel-gfx] [PATCH 4/5] drm/i915: Save sink_count for tracking changes to it and read sink_count dpcd always

its into one. (Ander) v3: Added comment to explain the need of early return when sink count is 0. (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shr

[Intel-gfx] [PATCH 5/5] drm/i915: force full detect on sink count change

in intel_dp_short_pulse (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> Reviewed-by: Ander Conselvan de Oliveira <

[Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 65 + 1 file changed, 40 insertions(+), 25 deletions(-) diff --git

[Intel-gfx] [PATCH 1/5] drm/i915: Splitting intel_dp_detect

com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 63 - 1 file changed, 43 insertions(+), 20 deletions(-) di

Re: [Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

On Tuesday 26 January 2016 06:52 PM, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-19 at 16:07 +0530, Shubhangi Shrivastava wrote: Current DP detection has DPCD operations split across intel_dp_hpd_pulse and intel_dp_detect which contains duplicates as well. Also intel_dp_detect

Re: [Intel-gfx] [PATCH 6/6] drm/i915: force full detect on sink count change

On Thursday 14 January 2016 07:20 PM, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-05 at 18:20 +0530, Shubhangi Shrivastava wrote: This patch checks for changes in sink count between short pulse hpds and forces full detect when there is a change. This will allow both detection

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Splitting intel_dp_detect

On Friday 15 January 2016 03:37 PM, Ander Conselvan De Oliveira wrote: On Thu, 2016-01-14 at 19:20 +0530, Shubhangi Shrivastava wrote: On Wednesday 13 January 2016 07:03 PM, Ander Conselvan De Oliveira wrote: On Wed, 2016-01-13 at 13:20 +0200, Ander Conselvan De Oliveira wrote: On Tue, 2016

Re: [Intel-gfx] [PATCH 5/6] drm/i915: read sink_count dpcd always

On Monday 18 January 2016 06:30 PM, Ander Conselvan De Oliveira wrote: On Mon, 2016-01-18 at 18:14 +0530, Shubhangi Shrivastava wrote: On Thursday 14 January 2016 06:34 PM, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-05 at 18:20 +0530, Shubhangi Shrivastava wrote: This patch reads

Re: [Intel-gfx] [PATCH 3/6] drm/i915: Splitting intel_dp_check_link_status

On Wednesday 13 January 2016 08:34 PM, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-05 at 18:20 +0530, Shubhangi Shrivastava wrote: When created originally intel_dp_check_link_status() was supposed to handle only link training for short pulse but has grown into handler for short pulse

Re: [Intel-gfx] [PATCH 4/6] drm/i915: Save sink_count for tracking changes to it

On Thursday 14 January 2016 06:30 PM, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-05 at 18:20 +0530, Shubhangi Shrivastava wrote: Sink count can change between short pulse hpd hence this patch adds a member variable to intel_dp so we can track any changes between short pulse interrupts

[Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 71 + 1 file changed, 44 insertions(+), 27 deletions(-) diff --git

[Intel-gfx] [PATCH 4/5] drm/i915: Save sink_count for tracking changes to it and read sink_count dpcd always

..@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 18 +++--- drivers/gpu/drm/i915/intel_drv.h | 1 + 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b

[Intel-gfx] [PATCH 5/5] drm/i915: force full detect on sink count change

in intel_dp_short_pulse (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i9

[Intel-gfx] [PATCH 1/5] drm/i915: Splitting intel_dp_detect

.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 62 ++--- 1 file changed, 39 insertions(+), 23 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i91

[Intel-gfx] [PATCH 3/5] drm/i915: Reorganizing intel_dp_check_link_status

() (Ander) v3: Changed commit message to explain need of link status being read before performing encoder checks (Daniel) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi

[Intel-gfx] [PATCH 1/5] drm/i915: Splitting intel_dp_detect

.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 62 ++--- 1 file changed, 39 insertions(+), 23 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i91

[Intel-gfx] [PATCH 3/5] drm/i915: Reorganizing intel_dp_check_link_status

() (Ander) v3: Changed commit message to explain need of link status being read before performing encoder checks (Daniel) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi

[Intel-gfx] [PATCH 4/5] drm/i915: Save sink_count for tracking changes to it and read sink_count dpcd always

..@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 18 +++--- drivers/gpu/drm/i915/intel_drv.h | 1 + 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b

[Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 71 + 1 file changed, 44 insertions(+), 27 deletions(-) diff --git

[Intel-gfx] [PATCH 5/5] drm/i915: force full detect on sink count change

in intel_dp_short_pulse (Ander) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i9

[Intel-gfx] [PATCH] drm/i915: Reorganizing intel_dp_check_link_status

() (Ander) v3: Changed commit message to explain need of link status being read before performing encoder checks (Daniel) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi

[Intel-gfx] [PATCH] drm/i915: Splitting intel_dp_check_link_status

ed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 65 +++-- 1 file changed, 36 insertions(+), 29 deletions(-) diff --git a/dr

[Intel-gfx] [PATCH] drm/i915: Splitting intel_dp_detect

.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 62 ++--- 1 file changed, 39 insertions(+), 23 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i91

Re: [Intel-gfx] [PATCH 5/6] drm/i915: read sink_count dpcd always

On Thursday 14 January 2016 06:34 PM, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-05 at 18:20 +0530, Shubhangi Shrivastava wrote: This patch reads sink_count dpcd always and removes its read operation based on values in downstream port dpcd. SINK_COUNT dpcd is not dependent

[Intel-gfx] [PATCH] drm/i915: Save sink_count for tracking changes to it and read sink_count dpcd always

..@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 18 +++--- drivers/gpu/drm/i915/intel_drv.h | 1 + 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b

Re: [Intel-gfx] [PATCH 5/6] drm/i915: read sink_count dpcd always

On Monday 18 January 2016 06:14 PM, Shubhangi Shrivastava wrote: On Thursday 14 January 2016 06:34 PM, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-05 at 18:20 +0530, Shubhangi Shrivastava wrote: This patch reads sink_count dpcd always and removes its read operation based on values

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Splitting intel_dp_detect

On Wednesday 13 January 2016 07:03 PM, Ander Conselvan De Oliveira wrote: On Wed, 2016-01-13 at 13:20 +0200, Ander Conselvan De Oliveira wrote: On Tue, 2016-01-05 at 18:20 +0530, Shubhangi Shrivastava wrote: intel_dp_detect() is called for not just detection but during modes enumeration

Re: [Intel-gfx] Intel-gfx Digest, Vol 96, Issue 26

Hi, Can someone review the patches in the below mail? PFB the link to the same: https://patchwork.freedesktop.org/series/369/#rev5 Thanks and Regards, Shubhangi Shrivastava. On Tuesday 05 January 2016 06:28 PM, intel-gfx-requ...@lists.freedesktop.org wrote: Send Intel-gfx

[Intel-gfx] [PATCH 6/6] drm/i915: force full detect on sink count change

) return immediately if perform_full_detect is set(Siva) v3: changed method of determining full detection from using pointer to return code (Siva) Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-b

[Intel-gfx] [PATCH 1/6] drm/i915: Splitting intel_dp_detect

.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 56 + 1 file changed, 35 insertions(+), 21 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i91

[Intel-gfx] [PATCH 2/6] drm/i915: Cleaning up intel_dp_hpd_pulse

Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 71 + 1 file changed, 44 insertions(+), 27 deletions(-) diff --git

[Intel-gfx] [PATCH 3/6] drm/i915: Splitting intel_dp_check_link_status

of intel_dp_short_pulse. This change is required to avoid performing all DPCD related operations on performing link retraining. Tested-by: Nathan D Ciobanu <nathan.d.ciob...@intel.com> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shu

[Intel-gfx] [PATCH 4/6] drm/i915: Save sink_count for tracking changes to it

; Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com> --- drivers/gpu/drm/i915/intel_dp.c | 7 +++ drivers/gpu/drm/i915/intel_drv.h | 1 + 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_

[Intel-gfx] [PATCH 0/6] Fixing sink count related detection over

appropriately. Shubhangi Shrivastava (6): drm/i915: Splitting intel_dp_detect drm/i915: Cleaning up intel_dp_hpd_pulse drm/i915: Splitting intel_dp_check_link_status drm/i915: Save sink_count for tracking changes to it drm/i915: read sink_count dpcd always drm/i915: force full detect

Re: [Intel-gfx] [PATCH 0/6] Fixing sink count related detection over

On Wednesday 18 November 2015 02:53 PM, Ander Conselvan De Oliveira wrote: On Wed, 2015-11-18 at 11:12 +0530, Shubhangi Shrivastava wrote: On Wednesday 18 November 2015 01:23 AM, Daniel Vetter wrote: On Mon, Nov 02, 2015 at 06:25:10PM +0530, Shubhangi Shrivastava wrote: This patch set

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Splitting intel_dp_detect

On Tuesday 17 November 2015 02:52 PM, Ander Conselvan De Oliveira wrote: On Tue, 2015-11-17 at 11:47 +0530, Shubhangi Shrivastava wrote: On Monday 16 November 2015 07:10 PM, Ander Conselvan De Oliveira wrote: On Fri, 2015-11-06 at 17:58 +0530, Shubhangi Shrivastava wrote: This patch moves

Re: [Intel-gfx] [PATCH 0/6] Fixing sink count related detection over

On Wednesday 18 November 2015 01:23 AM, Daniel Vetter wrote: On Mon, Nov 02, 2015 at 06:25:10PM +0530, Shubhangi Shrivastava wrote: This patch set cleans up DP detection logic to bring all DPCD operations at one place and to create a clear demarcation between handling of long and short pulses

Re: [Intel-gfx] [PATCH 1/6] drm/i915: Splitting intel_dp_detect

On Monday 16 November 2015 07:10 PM, Ander Conselvan De Oliveira wrote: On Fri, 2015-11-06 at 17:58 +0530, Shubhangi Shrivastava wrote: This patch moves probing for panel, DPCD read etc to another function intel_dp_long_pulse, while intel_dp_detect returns the status as connected

[Intel-gfx] [PATCH] drm/i915: Splitting intel_dp_detect

multiple DPCD operations and removing their duplication. v2: Moved a hunk to next patch of the series. Moved intel_dp_unset_edid to out. (Ander) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@

Re: [Intel-gfx] [PATCH 2/6] drm/i915: Cleaning up intel_dp_hpd_pulse

On Monday 16 November 2015 08:03 PM, Ander Conselvan De Oliveira wrote: On Fri, 2015-11-06 at 17:58 +0530, Shubhangi Shrivastava wrote: Current DP detection has DPCD operations split across intel_dp_hpd_pulse and intel_dp_detect which contains duplicates as well. Also intel_dp_detect

[Intel-gfx] [PATCH] drm/i915: Cleaning up intel_dp_hpd_pulse

DPCD operations in one single function and make intel_dp_detect use existing values instead of repeating same steps. v2: Pulled in a hunk from last patch of the series to this patch. (Ander) Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com> Signed-off-by: Shu

  1   2   >