Re: Drop "drm/amd/display: Add AV mute wait frames to dce110_set_avmute" from stable queues
Greg Kroah-Hartman <[email protected]>
| Newsgroups | org.kernel.vger.stable |
|---|---|
| Message-ID | <2026080937-reason-unlucky-9839@gregkh> |
On Sun, Aug 09, 2026 at 06:33:56PM +0200, Viktor Jägersküpper wrote: > On 8/7/26 15:13, Greg Kroah-Hartman wrote: > > On Thu, Aug 06, 2026 at 10:10:00PM +0200, Viktor Jägersküpper wrote: > >> Hi Greg and Sasha, > >> > >> I noticed that the following patch is in the queues for the 7.1, 6.18 and 6.12 > >> stable trees: > >> > >> >From 443290d70b01e9c35830c300e3247c06581b594c Mon Sep 17 00:00:00 2001 > >> From: Ray Wu <[email protected]> > >> Date: Mon, 13 Jul 2026 22:23:34 +0800 > >> Subject: drm/amd/display: Add AV mute wait frames to dce110_set_avmute > >> > >> From: Ray Wu <[email protected]> > >> > >> commit 443290d70b01e9c35830c300e3247c06581b594c upstream. > >> > >> Port the three-frame wait logic from dcn30_set_avmute to > >> dce110_set_avmute so that older DCN versions (1.0, 2.0) also > >> wait for GCP packets to be sent out before proceeding. > >> > >> This ensures HDMI sinks properly process the mute state, > >> preventing garbled display after link re-establishment. > >> > >> Closes: https://gitlab.freedesktop.org/drm/amd/-/work_items/5167 > >> Reviewed-by: Wayne Lin <[email protected]> > >> Signed-off-by: Ray Wu <[email protected]> > >> Signed-off-by: Fangzhi Zuo <[email protected]> > >> Tested-by: Dan Wheeler <[email protected]> > >> Signed-off-by: Alex Deucher <[email protected]> > >> (cherry picked from commit 414da24137ace80d8c59fefd43ba3ec9f5f854ba) > >> Cc: [email protected] > >> Signed-off-by: Greg Kroah-Hartman <[email protected]> > >> --- > > (...) > > >> Please drop this patch because it leads to a regression which I reported for the > >> mainline kernel here: > >> https://gitlab.freedesktop.org/drm/amd/-/work_items/5557 > >> > >> A fix for the regression is already available, but it hasn't landed in the > >> mainline tree yet. Since the patch above fixes an annoying issue, I propose to > >> postpone it until the mainline tree has the fix so that both patches can be > >> applied to the stable trees in the same release. I will notify you when that > >> can be done. > > > > Now dropped from all queues, thanks. > > > > greg k-h > > The fix for the regression has landed in the mainline tree, so you can > now backport > > 443290d70b01e9c35830c300e3247c06581b594c > drm/amd/display: Add AV mute wait frames to dce110_set_avmute > > and > > 3141e3d61469bba2624a91c5e2407f110b33b29e > drm/amd/display: Check for tg ops in dce110_set_avmute > > to the stable kernels (7.1, 6.18 and 6.12), both have 'Cc: stable'. Note > that the 'Fixes' tag in the second commit refers to a commit on the > drm-next branch and not the corresponding commit from the drm-fixes > branch, which might be confusing. It's more than "confusing", it's flat out wrong and breaks all of our tooling, which guarantees that we miss fixes like this... {sigh} I'll consider queueing these up in a future release if we get the chance. thanks, greg k-h