<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Acked-by: Alex Deucher <alexander.deucher@amd.com><br>
</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Liu, Shaoyun <Shaoyun.Liu@amd.com><br>
<b>Sent:</b> Monday, March 25, 2019 4:14 PM<br>
<b>To:</b> amd-gfx@lists.freedesktop.org<br>
<b>Cc:</b> Liu, Shaoyun<br>
<b>Subject:</b> [PATCH] drm/amdgpu: Adjust TMR address alignment as per HW requirement</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">According to HW engineer, they prefer the TMR address be "naturally aligned", e.g. the start address<br>
must be an integer divide of TME size.<br>
<br>
Change-Id: Ie01b3d41e564fc8f416048e001d75edb64c045e3<br>
Signed-off-by: shaoyunl <shaoyun.liu@amd.com><br>
---<br>
 drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 6 +++---<br>
 1 file changed, 3 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c<br>
index 2206bb4..905cce1 100644<br>
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c<br>
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c<br>
@@ -187,13 +187,13 @@ static int psp_tmr_init(struct psp_context *psp)<br>
         int ret;<br>
 <br>
         /*<br>
-        * Allocate 3M memory aligned to 1M from Frame Buffer (local<br>
-        * physical).<br>
+        * According to HW engineer, they prefer the TMR address be "naturally<br>
+        * aligned" , e.g. the start address be an integer divide of TMR size.<br>
          *<br>
          * Note: this memory need be reserved till the driver<br>
          * uninitializes.<br>
          */<br>
-       ret = amdgpu_bo_create_kernel(psp->adev, PSP_TMR_SIZE, 0x100000,<br>
+       ret = amdgpu_bo_create_kernel(psp->adev, PSP_TMR_SIZE, PSP_TMR_SIZE,<br>
                                       AMDGPU_GEM_DOMAIN_VRAM,<br>
                                       &psp->tmr_bo, &psp->tmr_mc_addr, &psp->tmr_buf);<br>
 <br>
-- <br>
2.7.4<br>
<br>
_______________________________________________<br>
amd-gfx mailing list<br>
amd-gfx@lists.freedesktop.org<br>
<a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a></div>
</span></font></div>
</body>
</html>