<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style> </head> <body dir="ltr"> <p style="font-family:Arial;font-size:10pt;color:#317100;margin:15pt;" align="Left"> [AMD Public Use]<br> </p> <br> <div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> That's fine we can take care of testing other platforms.</div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> <br> </div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Thanks,<br> </div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> <br> </div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Alex<br> </div> <div id="appendonsend"></div> <div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)"> <br> </div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Matt Coffin <mcoffin13@gmail.com><br> <b>Sent:</b> Tuesday, February 25, 2020 7:03 PM<br> <b>To:</b> Alex Deucher <alexdeucher@gmail.com><br> <b>Cc:</b> Das, Nirmoy <Nirmoy.Das@amd.com>; amd-gfx list <amd-gfx@lists.freedesktop.org><br> <b>Subject:</b> Re: [PATCH 1/2] drm/amdgpu/powerplay: Refactor SMU message handling for safety</font> <div> </div> </div> <div class="BodyFragment"><font size="2"><span style="font-size:11pt"> <div class="PlainText">Hey Alex,<br> <br> The reason I didn't transition the other code is that I have no hardware<br> to test with.<br> <br> I'll go ahead and do it, but someone with the hardware should at least<br> ensure it boots properly (vega20, and something on smu_v12_0).<br> <br> With that, I'm good with doing it, I just don't want to risk breaking<br> other ASICs with a change that is supposed to increase stability.<br> <br> On 2/25/20 12:30 PM, Alex Deucher wrote:<br> > Drop the unsafe ones. Let's convert everyone.<br> <br> Thanks for the review; I'll make the necessary changes. Sorry I keep<br> forgetting my Signed-off-by... I should really just add a git hook for it.<br> <br> ~Matt<br> _______________________________________________<br> amd-gfx mailing list<br> amd-gfx@lists.freedesktop.org<br> <a href="https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Calexander.deucher%40amd.com%7C479c9ea7ef3c4067dcf508d7ba4f4ee2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637182722163693285&sdata=nRAVFh8evB4fIOmPh%2F9ikOr3Aa52lc78wNgBxJChupw%3D&reserved=0">https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Calexander.deucher%40amd.com%7C479c9ea7ef3c4067dcf508d7ba4f4ee2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637182722163693285&sdata=nRAVFh8evB4fIOmPh%2F9ikOr3Aa52lc78wNgBxJChupw%3D&reserved=0</a><br> </div> </span></font></div> </div> </body> </html>