<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" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif" dir="ltr">
<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 style="font-size:11pt" color="#000000" face="Calibri, sans-serif"><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" 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)" cellspacing="0">
<tbody>
<tr style="border-spacing:0px" valign="top">
<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" style="text-decoration:none" target="_blank">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>