]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: fix inconsistent brightness after resume
authorJeremiah Mahler <jmmahler@gmail.com>
Mon, 12 Jan 2015 19:01:03 +0000 (11:01 -0800)
committerJani Nikula <jani.nikula@intel.com>
Mon, 26 Jan 2015 09:00:33 +0000 (11:00 +0200)
commit061258cb9a3d633456954127e2a5c19d18ea45a5
treea9e72161012cd8c49ead349766424e726070400e
parentb1ec55e5ce02c9c75e75bf8bd64a7238bbd8234a
drm/i915: fix inconsistent brightness after resume

commit f960a9250d8f8f75662eaa25abb6eed0f3cd1384
Author: Jani Nikula <jani.nikula@intel.com>
Date:   Tue Jun 24 18:27:40 2014 +0300

    drm/i915: respect the VBT minimum backlight brightness

introduced a bug which resulted in inconsistent brightness levels on
different machines. If a suspended was entered with the screen off some
machines would resume with the screen at minimum brightness and others
at maximum brightness.

The following commands can be used to produce this behavior.

  xset dpms force off
  sleep 1
  sudo systemctl suspend
  (resume ...)

The root cause of this problem is a comparison which checks to see if
the backlight level is zero when the panel is enabled.  If it is zero,
it is set to the maximum level.  Unfortunately, not all machines have a
minimum level of zero. On those machines the level is left at the
minimum instead of begin set to the maximum.

Fix the bug by updating the comparison to check for the minimum
backlight level instead of zero.  Also, expand the comparison for
the possible case when the level is less than the minimum.

Fixes: f960a9250d8f ("respect the VBT minimum backlight brightness")
Signed-off-by: Jeremiah Mahler <jmmahler@gmail.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/intel_panel.c