Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-13 Thread Wang, Gary C
@intel.com> Cc: Jindal, Sonika <sonika.jin...@intel.com>; intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking On Fri, Dec 11, 2015 at 07:10:35AM +, Wang, Gary C wrote: > I will upload new version of pa

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-11 Thread Ville Syrjälä
op.org] On Behalf Of > Wang, Gary C > Sent: Friday, December 11, 2015 7:39 AM > To: intel-gfx@lists.freedesktop.org > Subject: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug > live status checking > > The total delay of HDMI hotplug detecting with 30ms should have

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-11 Thread Daniel Vetter
On Fri, Dec 11, 2015 at 03:12:05PM +0800, Gary Wang wrote: > The total delay of HDMI hotplug detecting with 30ms should have > been split into a resolution of 3 retries of 10ms each, for the worst > cases. But it still suffered from only waiting 10ms at most in > intel_hdmi_detect(). This patch

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-11 Thread Daniel Vetter
r 11, 2015 2:23 PM > To: Jindal, Sonika <sonika.jin...@intel.com>; intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug > live status checking > > No problem. > > Please refer following information. > Lenovo Mode:

[Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-10 Thread Gary Wang
The total delay of HDMI hotplug detecting with 30ms should have been split into a resolution of 3 retries of 10ms each, for the worst cases. But it still suffered from only waiting 10ms at most in intel_hdmi_detect(). This patch corrects it by reading hotplug status with 4 times at most for 30ms

[Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-10 Thread Gary Wang
The total delay of HDMI hotplug detecting with 30ms should have been split into a resolution of 3 retries of 10ms each, for the worst cases. But it still suffered from only waiting 10ms at most in intel_hdmi_detect(). This patch corrects it by reading hotplug status with 4 times at most for 30ms

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-10 Thread Wang, Gary C
op.org Subject: Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking No problem. Please refer following information. Lenovo Mode: L246 1xwA (4/30 failed, necessary hot-plug delay: 58/40/60/40ms) Philips Model : HH2AP (9/30 failed, necessary hot-plug delay: 80/50

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-10 Thread Jindal, Sonika
(--retry); Regards, Sonika -Original Message- From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of Wang, Gary C Sent: Friday, December 11, 2015 7:39 AM To: intel-gfx@lists.freedesktop.org Subject: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-10 Thread Wang, Gary C
Friday, December 11, 2015 11:34 AM To: Jindal, Sonika <sonika.jin...@intel.com>; intel-gfx@lists.freedesktop.org Subject: RE: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking Hi Sonika, Thanks for your comment, and it's more readability on code. But it

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-10 Thread Wang, Gary C
-Original Message- From: Jindal, Sonika Sent: Friday, December 11, 2015 1:05 PM To: Wang, Gary C <gary.c.w...@intel.com>; intel-gfx@lists.freedesktop.org Subject: RE: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking How about following instead

Re: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking

2015-12-10 Thread Jindal, Sonika
, 2015 11:34 AM To: Jindal, Sonika <sonika.jin...@intel.com>; intel-gfx@lists.freedesktop.org Subject: RE: [Intel-gfx] [PATCH] drm/i915: Correct max delay for HDMI hotplug live status checking Hi Sonika, Thanks for your comment, and it's more readability on code. But it only delay 2 integ