Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2016-03-10 Thread Sharma, Shashank
, Sonika; Chris Wilson Cc: intel-gfx@lists.freedesktop.org; Mukherjee, Indranil Subject: RE: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid We are procuring the DVI-single-link cable, as we don't have one with us. Sonika tested the Dual link cable, and that was working well.

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2016-03-08 Thread Sharma, Shashank
l-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid On Tue, Sep 15, 2015 at 09:44:20AM +0530, Sonika Jindal wrote: > The Bspec is very clear that Live status must be checked about before > trying to read EDID over DDC channel. This patch make

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2016-03-08 Thread Jindal, Sonika
Subject: Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid On Tue, Sep 15, 2015 at 09:44:20AM +0530, Sonika Jindal wrote: > The Bspec is very clear that Live status must be checked about before > trying to read EDID over DDC channel. This patch makes sure that HDMI >

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2016-03-08 Thread Chris Wilson
On Tue, Sep 15, 2015 at 09:44:20AM +0530, Sonika Jindal wrote: > The Bspec is very clear that Live status must be checked about before > trying to read EDID over DDC channel. This patch makes sure that HDMI > EDID is read only when live status is up. > > The live status doesn't seem to perform ver

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-23 Thread Daniel Vetter
On Tue, Sep 15, 2015 at 09:44:20AM +0530, Sonika Jindal wrote: > The Bspec is very clear that Live status must be checked about before > trying to read EDID over DDC channel. This patch makes sure that HDMI > EDID is read only when live status is up. > > The live status doesn't seem to perform ver

[Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-14 Thread Sonika Jindal
The Bspec is very clear that Live status must be checked about before trying to read EDID over DDC channel. This patch makes sure that HDMI EDID is read only when live status is up. The live status doesn't seem to perform very consistent across various platforms when tested with different monitors

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-14 Thread Daniel Vetter
On Mon, Sep 14, 2015 at 02:44:01PM +0530, Jindal, Sonika wrote: > > > On 9/14/2015 2:12 PM, Daniel Vetter wrote: > >On Fri, Sep 11, 2015 at 05:56:47PM +, Rodrigo Vivi wrote: > >>Thanks > >> > >>Reviewed-by: Rodrigo Vivi > >> > >> > >>On Fri, Sep 11, 2015 at 4:38 AM Sonika Jindal > >>wrote:

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-14 Thread Jindal, Sonika
On 9/14/2015 2:12 PM, Daniel Vetter wrote: On Fri, Sep 11, 2015 at 05:56:47PM +, Rodrigo Vivi wrote: Thanks Reviewed-by: Rodrigo Vivi On Fri, Sep 11, 2015 at 4:38 AM Sonika Jindal wrote: The Bspec is very clear that Live status must be checked about before trying to read EDID over D

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-14 Thread Daniel Vetter
On Fri, Sep 11, 2015 at 05:56:47PM +, Rodrigo Vivi wrote: > Thanks > > Reviewed-by: Rodrigo Vivi > > > On Fri, Sep 11, 2015 at 4:38 AM Sonika Jindal > wrote: > > > The Bspec is very clear that Live status must be checked about before > > trying to read EDID over DDC channel. This patch ma

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-11 Thread Rodrigo Vivi
Thanks Reviewed-by: Rodrigo Vivi On Fri, Sep 11, 2015 at 4:38 AM Sonika Jindal wrote: > The Bspec is very clear that Live status must be checked about before > trying to read EDID over DDC channel. This patch makes sure that HDMI > EDID is read only when live status is up. > > The live status

[Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-11 Thread Sonika Jindal
The Bspec is very clear that Live status must be checked about before trying to read EDID over DDC channel. This patch makes sure that HDMI EDID is read only when live status is up. The live status doesn't seem to perform very consistent across various platforms when tested with different monitors

[Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-08 Thread Sonika Jindal
The Bspec is very clear that Live status must be checked about before trying to read EDID over DDC channel. This patch makes sure that HDMI EDID is read only when live status us up. The live status doesn't seem to perform very consistent across various platforms when tested with different monitors

[Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-09-06 Thread Sonika Jindal
Adding this for SKL onwards. v2: Adding checks for VLV/CHV as well. Reusing old ibx and g4x functions to check digital port status. Adding a separate function to get bxt live status (Daniel) v3: Using intel_encoder->hpd_pin to check the live status (Siva) Moving the live status read to intel_hdmi_

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-07-14 Thread Daniel Vetter
On Tue, Jul 14, 2015 at 10:16:17AM +0530, Jindal, Sonika wrote: > > > On 7/13/2015 8:25 PM, Daniel Vetter wrote: > >On Mon, Jul 13, 2015 at 04:35:00PM +0530, Sonika Jindal wrote: > >>Adding this for SKL onwards. > >> > >>v2: Adding checks for VLV/CHV as well. Reusing old ibx and g4x functions > >

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-07-13 Thread Jindal, Sonika
On 7/13/2015 8:25 PM, Daniel Vetter wrote: On Mon, Jul 13, 2015 at 04:35:00PM +0530, Sonika Jindal wrote: Adding this for SKL onwards. v2: Adding checks for VLV/CHV as well. Reusing old ibx and g4x functions to check digital port status. Adding a separate function to get bxt live status (Dani

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-07-13 Thread Daniel Vetter
On Mon, Jul 13, 2015 at 04:35:00PM +0530, Sonika Jindal wrote: > Adding this for SKL onwards. > > v2: Adding checks for VLV/CHV as well. Reusing old ibx and g4x functions > to check digital port status. Adding a separate function to get bxt live > status (Daniel) > > Signed-off-by: Sonika Jindal

[Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-07-13 Thread Sonika Jindal
Adding this for SKL onwards. v2: Adding checks for VLV/CHV as well. Reusing old ibx and g4x functions to check digital port status. Adding a separate function to get bxt live status (Daniel) Signed-off-by: Sonika Jindal --- drivers/gpu/drm/i915/intel_dp.c |4 ++-- drivers/gpu/drm/i915/int