* Regression in 6.11.2
@ 2024-10-07 20:33 Mario Limonciello
2024-10-08 1:35 ` Philip Müller
0 siblings, 1 reply; 4+ messages in thread
From: Mario Limonciello @ 2024-10-07 20:33 UTC (permalink / raw)
To: stable@vger.kernel.org
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!
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Regression in 6.11.2
2024-10-07 20:33 Regression in 6.11.2 Mario Limonciello
@ 2024-10-08 1:35 ` Philip Müller
2024-10-08 4:35 ` Jiri Slaby
2024-10-08 6:05 ` Sasha Levin
0 siblings, 2 replies; 4+ messages in thread
From: Philip Müller @ 2024-10-08 1:35 UTC (permalink / raw)
To: Mario Limonciello, stable@vger.kernel.org; +Cc: 'Roman Gilg'
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.
--
Best, Philip
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Regression in 6.11.2
2024-10-08 1:35 ` Philip Müller
@ 2024-10-08 4:35 ` Jiri Slaby
2024-10-08 6:05 ` Sasha Levin
1 sibling, 0 replies; 4+ messages in thread
From: Jiri Slaby @ 2024-10-08 4:35 UTC (permalink / raw)
To: Philip Müller, Mario Limonciello, stable@vger.kernel.org
Cc: 'Roman Gilg'
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,
--
js
suse labs
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Regression in 6.11.2
2024-10-08 1:35 ` Philip Müller
2024-10-08 4:35 ` Jiri Slaby
@ 2024-10-08 6:05 ` Sasha Levin
1 sibling, 0 replies; 4+ messages in thread
From: Sasha Levin @ 2024-10-08 6:05 UTC (permalink / raw)
To: Philip Müller
Cc: Mario Limonciello, stable@vger.kernel.org, 'Roman Gilg'
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!
--
Thanks,
Sasha
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-10-08 6:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-07 20:33 Regression in 6.11.2 Mario Limonciello
2024-10-08 1:35 ` Philip Müller
2024-10-08 4:35 ` Jiri Slaby
2024-10-08 6:05 ` Sasha Levin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).