https://patchwork.freedesktop.org/patch/582411/?series=130780&rev=2 provides the scenario and the details of DMESG in the commit message on rev#1. I'll assume we'll go with rev#2. Thanks, Gareth -----Original Message----- From: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> Sent: Monday, March 18, 2024 9:45 PM To: Dolan Liu <liuyong5@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>; Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Cc: Yu, Gareth <gareth.yu@xxxxxxxxx>; intel-gfx@xxxxxxxxxxxxxxxxxxxxx Subject: Re: [PATCH] drm/i915/display: Fixed a screen flickering when turning on display from off ... As far as the change goes, the original patch is pretty much the only one we should consider. I only ever asked for 1) an issue to be reported at fdo gitlab, and 2) a commit message properly detailing the issue. I never asked for functional changes in the patch. Frankly, all the alternative versions are nonsense. I think we can try to go with the patch, but please understand that if it regresses some silly userspace, the change will be reverted instead of fixing that silly userspace, because that's the rule in kernel development. BR, Jani. PS. Please try to reply inline instead of top-posting in public mailing lists.
