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

PM To: Daniel Vetter <dan...@ffwll.ch>; Jindal, Sonika <sonika.jin...@intel.com>; intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH 4/5] drm/i915: Check live status before reading edid On Tue, Jul 12, 2016 at 02:39:47PM +0300, David Weinehall wrote: > I'm feeling

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

On Tue, Jul 12, 2016 at 02:39:47PM +0300, David Weinehall wrote: > I'm feeling sorely tempted to treat this as a proper regression, > and simply submit a revert patch, seeing as it slows down resume by > something like 200ms, but seeing as there was mention of a case where > incorrect

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

On Tue, Jul 12, 2016 at 02:39:47PM +0300, David Weinehall wrote: > On Thu, Jul 09, 2015 at 07:27:57PM +0200, Daniel Vetter wrote: > > On Thu, Jul 09, 2015 at 05:34:29PM +0530, Sonika Jindal wrote: > > > Adding this for SKL onwards. > > > > > > Signed-off-by: Sonika Jindal

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

On Wed, Jul 13, 2016 at 01:17:40PM +0100, Chris Wilson wrote: > On Wed, Jul 13, 2016 at 01:09:28PM +0100, Chris Wilson wrote: > > On Wed, Jul 13, 2016 at 01:59:52PM +0200, Daniel Vetter wrote: > > > I think the proper solution should be to make all the probing and fbdev > > > restoring async. As

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

On Wed, Jul 13, 2016 at 01:09:28PM +0100, Chris Wilson wrote: > On Wed, Jul 13, 2016 at 01:59:52PM +0200, Daniel Vetter wrote: > > I think the proper solution should be to make all the probing and fbdev > > restoring async. As soon as we have working async atomic commit for > > modeset we should

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

On Wed, Jul 13, 2016 at 01:59:52PM +0200, Daniel Vetter wrote: > I think the proper solution should be to make all the probing and fbdev > restoring async. As soon as we have working async atomic commit for > modeset we should be able to do all that. We're not just blocked on the mode change here

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

On Tue, Jul 12, 2016 at 02:39:47PM +0300, David Weinehall wrote: > On Thu, Jul 09, 2015 at 07:27:57PM +0200, Daniel Vetter wrote: > > On Thu, Jul 09, 2015 at 05:34:29PM +0530, Sonika Jindal wrote: > > > Adding this for SKL onwards. > > > > > > Signed-off-by: Sonika Jindal

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

On Thu, Jul 09, 2015 at 07:27:57PM +0200, Daniel Vetter wrote: > On Thu, Jul 09, 2015 at 05:34:29PM +0530, Sonika Jindal wrote: > > Adding this for SKL onwards. > > > > Signed-off-by: Sonika Jindal > > I think this is the critical piece really, and I'd like to roll it

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

On Thu, Jul 09, 2015 at 05:34:29PM +0530, Sonika Jindal wrote: Adding this for SKL onwards. Signed-off-by: Sonika Jindal sonika.jin...@intel.com I think this is the critical piece really, and I'd like to roll it out for all platforms. git has the code for all the old ones, so no big deal to

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

On 7/9/2015 10:57 PM, Daniel Vetter wrote: On Thu, Jul 09, 2015 at 05:34:29PM +0530, Sonika Jindal wrote: Adding this for SKL onwards. Signed-off-by: Sonika Jindal sonika.jin...@intel.com I think this is the critical piece really, and I'd like to roll it out for all platforms. git has the

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

Adding this for SKL onwards. Signed-off-by: Sonika Jindal sonika.jin...@intel.com --- drivers/gpu/drm/i915/intel_hdmi.c | 49 ++--- 1 file changed, 45 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_hdmi.c