<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:11pt;color:#0078D7;margin:5pt;" align="Left"> [AMD Official Use Only - Internal Distribution Only]<br> </p> <br> <div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);"> Hi Siqueira,</div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);"> <br> Sorry about the change id, posted a v2<br> </div> <div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> <br> </div> <div id="Signature"> <div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> --</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 & Regards,</div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Aurabindo Pillai<br> </div> <div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> <br> <hr tabindex="-1" style="display:inline-block; width:98%;"> <b>From:</b> Siqueira, Rodrigo<br> <b>Sent:</b> Monday, March 29, 2021 9:44 AM<br> <b>To:</b> Aurabindo Pillai<br> <b>Cc:</b> igt-dev@lists.freedesktop.org; Pillai, Aurabindo<br> <b>Subject:</b> Re: [PATCH] tests/amdgpu: Add test description for amd_abm <div><br> </div> </div> <div class="BodyFragment"><font size="2"><span style="font-size:11pt;"> <div class="PlainText">Hi Aurabindo,<br> <br> On 03/29, Aurabindo Pillai wrote:<br> > From: Aurabindo Pillai <aurabindo.pillai@amd.com><br> > <br> > Add overall test description for amd_abm<br> > <br> > Change-Id: I4baa52ea66189688b12a82befbbd842ba3d3a24d<br> <br> We don't need this Change-id in IGT.<br> <br> Aside from the above comment, everything LGTM.<br> <br> Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com><br> <br> Let me know if you need help for applying this change in IGT.<br> <br> Thanks<br> Siqueira<br> <br> > Signed-off-by: Aurabindo Pillai <aurabindo.pillai@amd.com><br> > ---<br> > tests/amdgpu/amd_abm.c | 2 ++<br> > 1 file changed, 2 insertions(+)<br> > <br> > diff --git a/tests/amdgpu/amd_abm.c b/tests/amdgpu/amd_abm.c<br> > index 0505f17e..a1e48533 100644<br> > --- a/tests/amdgpu/amd_abm.c<br> > +++ b/tests/amdgpu/amd_abm.c<br> > @@ -32,6 +32,8 @@<br> > #include <fcntl.h><br> > #include <time.h><br> > <br> > +IGT_TEST_DESCRIPTION("Tests Adaptive Backlight Modulation on supported hardware");<br> > +<br> > #define BACKLIGHT_PATH "/sys/class/backlight/amdgpu_bl0"<br> > <br> > typedef struct data {<br> > -- <br> > 2.31.0<br> > <br> <br> -- <br> Rodrigo Siqueira<br> <a href="https://siqueira.tech" target="_blank" rel="noopener noreferrer" data-auth="NotApplicable">https://siqueira.tech</a><br> </div> </span></font></div> </div> </div> </div> </div> </div> </body> </html>