[Intel-gfx] [PATCH] [drm][i915] Increase LSPCON timeout

[Sharma, Shashank]

Regards

Shashank


On 8/16/2018 12:47 PM, Jani Nikula wrote:
> On Wed, 15 Aug 2018, Rodrigo Vivi <rodrigo.vivi at intel.com> wrote:
>> On Wed, Aug 15, 2018 at 03:39:40PM -0700, Rodrigo Vivi wrote:
>>> On Mon, Aug 13, 2018 at 07:51:33PM +0200, Fredrik Schön wrote:
>>>
>>> First of all we need to fix the commit subject:
>>>
>>> drm/i915: Increase LSPCON timeout
>>>
>>> (this can be done when merging, no need to resend)
>>>
>>>> 100 ms is not enough time for the LSPCON adapter on Intel NUC devices to
>>>> settle. This causes dropped display modes at driver initialisation.
>>>>
>>>> Increase timeout to 1000 ms.
>>>>
>>>> Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1570392
>>> Missusage of "Fixes:" tag, please read
>>>
>>> https://01.org/linuxgraphics/gfx-docs/maintainer-tools/drm-intel.html#fixes
>>>
>>> But also no need for resending... could be fixed when merging
>>>
>>> The right one would be:
>>>
>>> Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1570392
>>> Fixes: 357c0ae9198a ("drm/i915/lspcon: Wait for expected LSPCON mode to settle")
>>> Cc: Shashank Sharma <shashank.sharma at intel.com>
>>> Cc: Imre Deak <imre.deak at intel.com>
>>> Cc: Jani Nikula <jani.nikula at intel.com>
>>> Cc: <stable at vger.kernel.org> # v4.11+
>>>
>>> Since initial 100 seemed to be empirical and this increase seems to
>>> help other cases I'm in favor of this move so
>>>
>>> Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
>>>
>>> However I will wait a bit before merging it
>>> so Imre, Shashank, and/or Jani can take a look here...
>> now, really cc'ing them...
> Shashank? Does this slow down non-LSPCON paths?
This will slow down the lspcon probing and resume part, but both of them 
happen only when LSPCON device is found. So to answer your question, 
this will not slow down the non-lspcon path, but will slow down the 
LSPCON connector resume and probe time. but I would recommend, instead 
of increasing it to 1000 ms in a single shot, we might want to gradually 
pick this up, on a wake-and-check way.

- Shashank
>
> BR,
> Jani.
>
>
>>>> Signed-off-by: Fredrik Schön <fredrik.schon at gmail.com>
>>>> ---
>>>>   drivers/gpu/drm/i915/intel_lspcon.c | 2 +-
>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/gpu/drm/i915/intel_lspcon.c b/drivers/gpu/drm/i915/intel_lspcon.c
>>>> index 8ae8f42f430a..be1b08f589a4 100644
>>>> --- a/drivers/gpu/drm/i915/intel_lspcon.c
>>>> +++ b/drivers/gpu/drm/i915/intel_lspcon.c
>>>> @@ -74,7 +74,7 @@ static enum drm_lspcon_mode lspcon_wait_mode(struct intel_lspcon *lspcon,
>>>>   	DRM_DEBUG_KMS("Waiting for LSPCON mode %s to settle\n",
>>>>   		      lspcon_mode_name(mode));
>>>>   
>>>> -	wait_for((current_mode = lspcon_get_current_mode(lspcon)) == mode, 100);
>>>> +	wait_for((current_mode = lspcon_get_current_mode(lspcon)) == mode, 1000);
>>>>   	if (current_mode != mode)
>>>>   		DRM_ERROR("LSPCON mode hasn't settled\n");
>>>>   
>>>> -- 
>>>> 2.17.1
>>>>
>>>> _______________________________________________
>>>> Intel-gfx mailing list
>>>> Intel-gfx at lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx