Re: [REGRESSION] bisected: 7.1rc1 breaks backlight control on Thinkpad T480

Stefan Seyfried <[email protected]>
Newsgroups dev.linux.lists.regressions,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
Am 15.05.26 um 14:13 schrieb Thorsten Leemhuis:
> On 5/15/26 13:47, Stefan Seyfried wrote:
>>
>> I found that 7.1rc1 broke the backlight control on my Thinkpad T480.
>> Bisected it to
>>
>>> 40d2f5820951dee818d05c14677277048bd85f9f is the first bad commit
>>> commit 40d2f5820951dee818d05c14677277048bd85f9f
>>> Author: Suraj Kandpal <[email protected]>
>>> Date:   Tue Feb 24 08:43:22 2026 +0530
>>>
>>>      drm/i915/backlight: Remove try_vesa_interface
> 
> FWIW: Here are two bug reports that contain proposed fixes for what
> might or might not be the issue your face:
> 
> https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16043
> https://gitlab.freedesktop.org/drm/i915/kernel/-/work_items/16015

Thanks, will look into testing the patches floating around there.

For now, this

  diff --git a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c 
b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
  index a8d56ebf06a2..c4ac46559669 100644
  --- a/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
  +++ b/drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
  @@ -745,7 +745,7 @@ int intel_dp_aux_init_backlight_funcs(struct 
intel_connector *connector)
                  return 0;
          }

  -       if (intel_dp_aux_supports_vesa_backlight(connector)) {
  +       if (0 && intel_dp_aux_supports_vesa_backlight(connector)) {
                  drm_dbg_kms(dev, "[CONNECTOR:%d:%s] Using VESA eDP 
backlight controls\n",
                              connector->base.base.id, 
connector->base.name);
                  panel->backlight.funcs = &intel_dp_vesa_bl_funcs;

is enough to get the backlight control on the T480 going again.

I know nothing about that code (obviously), but maybe 
"intel_dp_aux_supports_vesa_backlight()" is wrongly determining VESA 
backlight support on my machine?

-- 
Stefan Seyfried

"For a successful technology, reality must take precedence over
  public relations, for nature cannot be fooled." -- Richard Feynman
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.