mbox series

[0/3,SRU,OEM-5.10] backlight parsing for VBT 234+

Message ID 20210119093404.34490-1-acelan.kao@canonical.com
Headers show
Series backlight parsing for VBT 234+ | expand

Message

AceLan Kao Jan. 19, 2021, 9:34 a.m. UTC 
From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>

BugLink: https://bugs.launchpad.net/bugs/1912157

[Impact]
Current driver fails to parse VBT 234+ and gets wrong brightness levels.
It leads to screen dim completely when brightness is set to lowest.

[Fix]
We need the commits to parse VBT 234+
https://patchwork.freedesktop.org/patch/394304/

[Test]
Verified on Dell machines

[Where problems could occur]
The patches are simple and with a separate flow control for VBT 234,
should have no impact for current brightness control.

José Roberto de Souza (3):
  drm/i915/vbt: Fix backlight parsing for VBT 234+
  drm/i915/vbt: Update the version and expected size of
    BDB_GENERAL_DEFINITIONS map
  drm/i915/vbt: Add VRR VBT toggle

 drivers/gpu/drm/i915/display/intel_bios.c     | 32 +++++++++++++++++--
 drivers/gpu/drm/i915/display/intel_vbt_defs.h | 13 ++++++--
 2 files changed, 40 insertions(+), 5 deletions(-)

Comments

Timo Aaltonen Jan. 21, 2021, 10:58 a.m. UTC | #1 
On 19.1.2021 11.34, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1912157
> 
> [Impact]
> Current driver fails to parse VBT 234+ and gets wrong brightness levels.
> It leads to screen dim completely when brightness is set to lowest.
> 
> [Fix]
> We need the commits to parse VBT 234+
> https://patchwork.freedesktop.org/patch/394304/
> 
> [Test]
> Verified on Dell machines
> 
> [Where problems could occur]
> The patches are simple and with a separate flow control for VBT 234,
> should have no impact for current brightness control.
> 
> José Roberto de Souza (3):
>    drm/i915/vbt: Fix backlight parsing for VBT 234+
>    drm/i915/vbt: Update the version and expected size of
>      BDB_GENERAL_DEFINITIONS map
>    drm/i915/vbt: Add VRR VBT toggle
> 
>   drivers/gpu/drm/i915/display/intel_bios.c     | 32 +++++++++++++++++--
>   drivers/gpu/drm/i915/display/intel_vbt_defs.h | 13 ++++++--
>   2 files changed, 40 insertions(+), 5 deletions(-)
> 

applied to oem-5.10, thanks