Hi,
commit 872b8f14d772 ("drm/amd/display: Validate backlight caps are sane") was added to stable trees to fix a brightness problem on one laptop on a buggy firmware but with how aggressive it was it caused a problem on another.
Fortunately the problem on the other was already fixed in 6.12 though!
commit 87d749a6aab7 ("drm/amd/display: Allow backlight to go below `AMDGPU_DM_DEFAULT_MIN_BACKLIGHT`")
Can that commit please be brought everywhere that 872b8f14d772 went?
Thanks!
On 8/10/24 03:33, Mario Limonciello wrote:
Hi,
commit 872b8f14d772 ("drm/amd/display: Validate backlight caps are sane") was added to stable trees to fix a brightness problem on one laptop on a buggy firmware but with how aggressive it was it caused a problem on another.
Fortunately the problem on the other was already fixed in 6.12 though!
commit 87d749a6aab7 ("drm/amd/display: Allow backlight to go below `AMDGPU_DM_DEFAULT_MIN_BACKLIGHT`")
Can that commit please be brought everywhere that 872b8f14d772 went?
Thanks!
So far commit 872b8f14d772 got added to 6.11.2, 6.10.13 and 6.6.54. It is also queued up for upcoming 6.1.113 and 5.15.168.
On 08. 10. 24, 3:35, Philip Müller wrote:
On 8/10/24 03:33, Mario Limonciello wrote:
Hi,
commit 872b8f14d772 ("drm/amd/display: Validate backlight caps are sane") was added to stable trees to fix a brightness problem on one laptop on a buggy firmware but with how aggressive it was it caused a problem on another.
Fortunately the problem on the other was already fixed in 6.12 though!
commit 87d749a6aab7 ("drm/amd/display: Allow backlight to go below `AMDGPU_DM_DEFAULT_MIN_BACKLIGHT`")
Can that commit please be brought everywhere that 872b8f14d772 went?
Thanks!
So far commit 872b8f14d772 got added to 6.11.2, 6.10.13 and 6.6.54. It is also queued up for upcoming 6.1.113 and 5.15.168.
This is confusing -- Mario used a wrong SHA.
The (correct) bad one is: 327e62f47eb5 drm/amd/display: Validate backlight caps are sane and is present: queue-5.15/drm-amd-display-validate-backlight-caps-are-sane.patch queue-6.1/drm-amd-display-validate-backlight-caps-are-sane.patch releases/6.10.13/drm-amd-display-validate-backlight-caps-are-sane.patch releases/6.11.2/drm-amd-display-validate-backlight-caps-are-sane.patch releases/6.6.54/drm-amd-display-validate-backlight-caps-are-sane.patch
The good one is: 87d749a6aab7 drm/amd/display: Allow backlight to go below `AMDGPU_DM_DEFAULT_MIN_BACKLIGHT`
and is nowhere.
thanks,
On Tue, Oct 08, 2024 at 08:35:33AM +0700, Philip Müller wrote:
On 8/10/24 03:33, Mario Limonciello wrote:
Hi,
commit 872b8f14d772 ("drm/amd/display: Validate backlight caps are sane") was added to stable trees to fix a brightness problem on one laptop on a buggy firmware but with how aggressive it was it caused a problem on another.
Fortunately the problem on the other was already fixed in 6.12 though!
commit 87d749a6aab7 ("drm/amd/display: Allow backlight to go below `AMDGPU_DM_DEFAULT_MIN_BACKLIGHT`")
Can that commit please be brought everywhere that 872b8f14d772 went?
Thanks!
So far commit 872b8f14d772 got added to 6.11.2, 6.10.13 and 6.6.54. It is also queued up for upcoming 6.1.113 and 5.15.168.
I've queued up the fix too, thanks!
linux-stable-mirror@lists.linaro.org
-
Jiri Slaby -
Mario Limonciello -
Philip Müller -
Sasha Levin