Re: [igt-dev] [PATCH v2 1/1] i915/pm_backlight: Do not turn off DPMS before system suspend

["Souza, Jose" <jose.souza@intel.com>]

[-- Attachment #1.1: Type: text/plain, Size: 2034 bytes --]

On Wed, 2019-04-03 at 18:26 -0700, Harish Chegondi wrote:
> backlight fade with suspend test turns off dpms which turns off the
> edp
> backlight. Then it does a system suspend and resume. After resume,
> the edp backlight would still be off, but the test sets the
> brightness
> value and reads it back. Since the edp backlight is off, the
> brightness
> values written and read are different causing the test to fail.
> 
> Do not turn off the DPMS before suspend so that after system resume,
> the edp backlight would be on and setting the brightness value would
> be successful.
> 
> v2: Remove "DPMS off" before system suspend instead of adding
>     "DPMS on" after system resume.

Reviewed-by: José Roberto de Souza <jose.souza@intel.com>

And going to push it.

> 
> Cc: Jyoti Yadav <jyoti.r.yadav@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Daniel Vetter <daniel.vetter@intel.com>
> Cc: Clinton Taylor <clinton.a.taylor@intel.com>
> Signed-off-by: Harish Chegondi <harish.chegondi@intel.com>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=107820
> Fixes: 377752242995 ("Brightness test with DPMS and System suspend.")
> ---
>  tests/i915/i915_pm_backlight.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/tests/i915/i915_pm_backlight.c
> b/tests/i915/i915_pm_backlight.c
> index 054300f6..4c1bff5b 100644
> --- a/tests/i915/i915_pm_backlight.c
> +++ b/tests/i915/i915_pm_backlight.c
> @@ -175,13 +175,6 @@ test_fade_with_dpms(struct context *context,
> igt_output_t *output)
>  static void
>  test_fade_with_suspend(struct context *context, igt_output_t
> *output)
>  {
> -	igt_require(igt_setup_runtime_pm());
> -
> -	kmstest_set_connector_dpms(output->display->drm_fd,
> -				   output->config.connector,
> -				   DRM_MODE_DPMS_OFF);
> -	igt_require(igt_wait_for_pm_status(IGT_RUNTIME_PM_STATUS_SUSPEN
> DED));
> -
>  	igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
> SUSPEND_TEST_NONE);
>  
>  	test_fade(context);

[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev