<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 Michel,</p>
<p><br>
</p>
<p>The code change has been submitted into our internal git server.</p>
<p>I have a follow up commit in another email thread.</p>
<p>The commit fixes more <span>errors</span> in comments.<br>
</p>
<p><br>
</p>
<p>Thanks,</p>
<p>Alex Bin<br>
</p>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" color="#000000" face="Calibri, sans-serif"><b>From:</b> Michel Dänzer <michel@daenzer.net><br>
<b>Sent:</b> Monday, May 8, 2017 9:13 PM<br>
<b>To:</b> Xie, AlexBin<br>
<b>Cc:</b> amd-gfx@lists.freedesktop.org<br>
<b>Subject:</b> Re: [PATCH] drm/amdgpu: fix errors in comments.</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">On 09/05/17 12:32 AM, Alex Xie wrote:<br>
> Signed-off-by: Alex Xie <AlexBin.Xie@amd.com><br>
> ---<br>
>  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 7 +++----<br>
>  1 file changed, 3 insertions(+), 4 deletions(-)<br>
> <br>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c<br>
> index 66bb60e..aab3206 100644<br>
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c<br>
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c<br>
> @@ -479,9 +479,8 @@ void amdgpu_doorbell_get_kfd_info(struct amdgpu_device *adev,<br>
>  <br>
>  /*<br>
>   * amdgpu_wb_*()<br>
> - * Writeback is the the method by which the the GPU updates special pages<br>
> - * in memory with the status of certain GPU events (fences, ring pointers,<br>
> - * etc.).<br>
> + * Writeback is the method by which GPU updates special pages in memory<br>
<br>
This should still say "the GPU".<br>
<br>
<br>
> + * with the status of certain GPU events (fences, ring pointers,etc.).<br>
>   */<br>
>  <br>
>  /**<br>
> @@ -507,7 +506,7 @@ static void amdgpu_wb_fini(struct amdgpu_device *adev)<br>
>   *<br>
>   * @adev: amdgpu_device pointer<br>
>   *<br>
> - * Disables Writeback and frees the Writeback memory (all asics).<br>
> + * Initialize writeback and allocates writeback memory (all asics).<br>
<br>
For consistency, this should say either "Initialize writeback and<br>
allocate writeback memory" or "Initializes writeback and allocates<br>
writeback memory".<br>
<br>
<br>
-- <br>
Earthling Michel Dänzer               |               <a href="http://www.amd.com" id="LPlnk618937" previewremoved="true">
http://www.amd.com</a><br>
Libre software enthusiast             |             Mesa and X developer<br>
</div>
</span></font></div>
</div>
</body>
</html>