<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">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;" dir="ltr">
<p>Hi Alex,</p>
<p><br>
</p>
<p>I'll wait on pushing it until I hear back from one of them.</p>
<p><br>
</p>
<p>Cheers,</p>
<p>Tom</p>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Alex Deucher <alexdeucher@gmail.com><br>
<b>Sent:</b> Wednesday, December 7, 2016 12:32<br>
<b>To:</b> StDenis, Tom<br>
<b>Cc:</b> Edward O'Callaghan; amd-gfx@lists.freedesktop.org<br>
<b>Subject:</b> Re: GPR read support via debugfs (v2)</font>
<div> </div>
</div>
<div>
<div dir="ltr">
<div>Patches look fine to me.  Assuming they are useful to Marek and Nicolai, the patches are:<br>
</div>
Reviewed-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com">alexander.deucher@amd.com</a>><br>
<div>
<div>
<div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Wed, Dec 7, 2016 at 12:05 PM, StDenis, Tom <span dir="ltr">
<<a href="mailto:Tom.StDenis@amd.com" target="_blank">Tom.StDenis@amd.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
<div dir="ltr">
<div id="m_1549130000763319934divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Arial,Helvetica,sans-serif">
<p>Thanks.</p>
<p><br>
</p>
<p>Could use a RB/NAK from Christian or Alex when they get the time :-)</p>
<p><br>
</p>
<p>Tom</p>
<br>
<br>
<div style="color:rgb(0,0,0)">
<div>
<hr style="display:inline-block; width:98%">
<div id="m_1549130000763319934x_divRplyFwdMsg" dir="ltr"><font color="#000000" face="Calibri, sans-serif" style="font-size:11pt"><b>From:</b> amd-gfx <<a href="mailto:amd-gfx-bounces@lists.freedesktop.org" target="_blank">amd-gfx-bounces@lists.<wbr>freedesktop.org</a>>
 on behalf of Edward O'Callaghan <<a href="mailto:funfunctor@folklore1984.net" target="_blank">funfunctor@folklore1984.net</a>><br>
<b>Sent:</b> Tuesday, December 6, 2016 17:15<br>
<b>To:</b> Tom St Denis; <a href="mailto:amd-gfx@lists.freedesktop.org" target="_blank">
amd-gfx@lists.freedesktop.org</a><br>
<b>Subject:</b> Re: GPR read support via debugfs (v2)</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt">
<div class="m_1549130000763319934PlainText">
<div>
<div class="h5">This series is,<br>
Acked-by: Edward O'Callaghan <<a href="mailto:funfunctor@folklore1984.net" target="_blank">funfunctor@folklore1984.net</a>><br>
<br>
On 12/07/2016 01:50 AM, Tom St Denis wrote:<br>
> These patches introduce debugfs access to read GPRs (SGPRs for now).<br>
> <br>
> (v2): Fixes white space, addressing is in dwords not bytes, and removes<br>
>       the buggy sanity checking.<br>
> <br>
> ______________________________<wbr>_________________<br>
> amd-gfx mailing list<br>
> <a href="mailto:amd-gfx@lists.freedesktop.org" target="_blank">amd-gfx@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx" id="m_1549130000763319934LPlnk539856" target="_blank">
https://lists.freedesktop.org/<wbr>mailman/listinfo/amd-gfx</a> </div>
</div>
<div id="m_1549130000763319934LPBorder_GT_14811303134020.3066773959785267" style="margin-bottom:20px; overflow:auto; width:100%; text-indent:0px">
<table id="m_1549130000763319934LPContainer_14811303133990.7273202165353794" cellspacing="0" style="width:90%; background-color:rgb(255,255,255); overflow:auto; padding-top:20px; padding-bottom:20px; margin-top:20px; border-top:1px dotted rgb(200,200,200); border-bottom:1px dotted rgb(200,200,200)">
<tbody>
<tr valign="top" style="border-spacing:0px">
<td id="m_1549130000763319934TextCell_14811303134010.6228712278926285" colspan="2" style="vertical-align:top; padding:0px; display:table-cell">
<div id="m_1549130000763319934LPRemovePreviewContainer_14811303134010.667042105097579">
</div>
<div id="m_1549130000763319934LPTitle_14811303134010.2987133567172171" style="color:rgb(59,87,119); font-weight:normal; font-size:21px; font-family:wf_segoe-ui_light,"Segoe UI Light","Segoe WP Light","Segoe UI","Segoe WP",Tahoma,Arial,sans-serif; line-height:21px">
<a id="m_1549130000763319934LPUrlAnchor_14811303134010.6835253784315398" href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx" target="_blank" style="text-decoration:none">amd-gfx Info Page - lists.freedesktop.org</a></div>
<div id="m_1549130000763319934LPMetadata_14811303134010.9344851329159858" style="margin:10px 0px 16px; color:rgb(102,102,102); font-weight:normal; font-family:wf_segoe-ui_normal,"Segoe UI","Segoe WP",Tahoma,Arial,sans-serif; font-size:14px; line-height:14px">
<a href="http://lists.freedesktop.org" target="_blank">lists.freedesktop.org</a></div>
<div id="m_1549130000763319934LPDescription_14811303134020.7740425410367178" style="display:block; color:rgb(102,102,102); font-weight:normal; font-family:wf_segoe-ui_normal,"Segoe UI","Segoe WP",Tahoma,Arial,sans-serif; font-size:14px; line-height:20px; max-height:100px; overflow:hidden">
To see the collection of prior postings to the list, visit the amd-gfx Archives. Using amd-gfx: To post a message to all the list members, send email ...</div>
</td>
</tr>
</tbody>
</table>
</div>
<br>
<br>
> <br>
<br>
</div>
</span></font></div>
</div>
</div>
<br>
______________________________<wbr>_________________<br>
amd-gfx mailing list<br>
<a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/amd-gfx</a><br>
<br>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>