<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<p style="font-family:Calibri;font-size:10pt;color:#0000FF;margin:5pt;font-style:normal;font-weight:normal;text-decoration:none;" align="Left">
[AMD Official Use Only - AMD Internal Distribution Only]<br>
</p>
<br>
<div>
<div style="font-family: "Segoe UI", "Segoe UI Web (West European)", "Helvetica Neue", sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
Okay and will do</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Alex Deucher <alexdeucher@gmail.com><br>
<b>Sent:</b> 18 August 2025 11:03<br>
<b>To:</b> Hung, Alex <Alex.Hung@amd.com><br>
<b>Cc:</b> Timur Kristóf <timur.kristof@gmail.com>; amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org><br>
<b>Subject:</b> Re: [PATCH 0/3] DC: Fix page flip timeouts on DCE 6</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">@Alex Hung<br>
can you pick these up as well?<br>
<br>
Thanks,<br>
<br>
Alex<br>
<br>
On Mon, Aug 18, 2025 at 12:23 AM Alex Hung <alex.hung@amd.com> wrote:<br>
><br>
> The series is Reviewed-by: Alex Hung <alex.hung@amd.com><br>
><br>
> The patch series is also tested in promotion test and CI without regression.<br>
><br>
> On 8/2/25 10:05, Timur Kristóf wrote:<br>
> > Currently when using DC on DCE 6, it produces a page flip timeout<br>
> > after a suspend/resume.<br>
> ><br>
> > After some investigation, it turns out that the issue is because<br>
> > DC uses different interrupts between DCE 6 and newer HW versions,<br>
> > and it assumes that the VUPDATE interrupt is always present, when<br>
> > in fact it's not. It also assumes that the HW is capable of VRR<br>
> > when a VRR capable monitor is plugged in, when in fact this is<br>
> > not the case.<br>
> ><br>
> > This series marks VRR as unsupported on DCE 6 even when a VRR<br>
> > capable monitor is plugged in. Furthermore, it fixes the code<br>
> > trying to access the unregistered VUPDATE interrupt.<br>
> ><br>
> > As a side note, I also attempted to actually use the same<br>
> > interrupts as newer DCE versions, but this didn't work.<br>
> > Apparently DCE 6 was not advertised with VRR support anyway.<br>
> ><br>
> > Finally, there is also a patch to disable fast boot mode<br>
> > on DCE 6. The rationale is that this already didn't work<br>
> > on DCE 8, and even if it did I have no means to test it.<br>
> ><br>
> > Timur Kristóf (3):<br>
> > drm/amd/display: Disable fastboot on DCE 6 too.<br>
> > drm/amd/display: Disable VRR on DCE 6<br>
> > drm/amd/display: Don't use non-registered VUPDATE on DCE 6<br>
> ><br>
> > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 26 ++++++++++++-------<br>
> > .../amd/display/amdgpu_dm/amdgpu_dm_crtc.c | 16 +++++++-----<br>
> > drivers/gpu/drm/amd/display/dc/dc_helper.c | 5 ++++<br>
> > drivers/gpu/drm/amd/display/dc/dm_services.h | 2 ++<br>
> > .../amd/display/dc/hwss/dce110/dce110_hwseq.c | 6 ++---<br>
> > 5 files changed, 35 insertions(+), 20 deletions(-)<br>
> ><br>
><br>
</div>
</span></font></div>
</div>
</body>
</html>