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

[Souza, Jose]

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 at intel.com>

And going to push it.

> 
> Cc: Jyoti Yadav <jyoti.r.yadav at intel.com>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Daniel Vetter <daniel.vetter at intel.com>
> Cc: Clinton Taylor <clinton.a.taylor at intel.com>
> Signed-off-by: Harish Chegondi <harish.chegondi at 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);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20190404/9c591599/attachment.sig>