[Intel-gfx] [PATCH 2/2] drm/i915: Don't give up waiting on INVALID_MODE
Sharma, Shashank
shashank.sharma at intel.com
Wed Aug 16 15:50:24 UTC 2017
Regards
Shashank
On 8/15/2017 5:51 AM, Pandiyan, Dhinakaran wrote:
> On Fri, 2017-08-11 at 18:58 +0530, Shashank Sharma wrote:
>> Our current logic to read LSPCON's current mode, stops retries and
>> breaks wait-loop, if it gets LSPCON_MODE_INVALID as return from the
>> core function. This doesn't allow us to try reading the mode again.
>>
>> This patch removes this condition and allows retries reading
>> the currnt mode until timeout.
>>
>> This also fixes/prevents some of the noise in form of debug messages
>> while running IGT CI test cases.
>>
>> Cc: Imre Deak <imre.deak at intel.com>
>> Cc: Daniel Vetter <daniel.vetter at intel.com>
>> Signed-off-by: Shashank Sharma <shashank.sharma at intel.com>
>> Signed-off-by: Mahesh Kumar <Mahesh1.kumar at intel.com>
>> ---
>> drivers/gpu/drm/i915/intel_lspcon.c | 5 ++---
>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_lspcon.c b/drivers/gpu/drm/i915/intel_lspcon.c
>> index 8413a4c..e64a336 100644
>> --- a/drivers/gpu/drm/i915/intel_lspcon.c
>> +++ b/drivers/gpu/drm/i915/intel_lspcon.c
>> @@ -118,14 +118,13 @@ static enum drm_lspcon_mode lspcon_wait_mode(struct intel_lspcon *lspcon,
>> enum drm_lspcon_mode current_mode;
>>
>> current_mode = lspcon_get_current_mode(lspcon);
>> - if (current_mode == mode || current_mode == DRM_LSPCON_MODE_INVALID)
>> + if (current_mode == mode)
>> goto out;
>>
>> 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 ||
>> - current_mode == DRM_LSPCON_MODE_INVALID, 100);
>> + wait_for((current_mode = lspcon_get_current_mode(lspcon)) == mode, 100);
> Doesn't this do the job of retries that you are adding in patch 1/2 ?
I think Imre has responded on this comment already, and explained better
than me :)
- Shashank
>> if (current_mode != mode)
>> DRM_DEBUG_KMS("LSPCON mode hasn't settled\n");
>>
More information about the Intel-gfx
mailing list