<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 id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p style="margin-top:0;margin-bottom:0">Is this required? Are the harvesting fuses incorrect? If the blocks are harvested, we should bail out of the blocks properly during init. Also, please make this more explicit if we still need it. E.g.,</p>
<p style="margin-top:0;margin-bottom:0"><font size="2"><span style="font-size:11pt;"><br>
if ((adev->pdev->device == 0x67df) &&<br>
(adev->pdev->revision == 0xf7)) {<br>
</span></font></p>
<p style="margin-top:0;margin-bottom:0"><font size="2"><span style="font-size:11pt;">
<font size="2"><span style="font-size:11pt;">/* Some polaris12 variants don't support UVD/VCE */</span></font><br>
</span></font></p>
<p style="margin-top:0;margin-bottom:0"><font size="2"><span style="font-size:11pt;"> } else {<br>
</span></font></p>
<p style="margin-top:0;margin-bottom:0"><font size="2"><span style="font-size:11pt;"> amdgpu_device_ip_block_add(adev, &uvd_v6_3_ip_block);</span></font></p>
<p style="margin-top:0;margin-bottom:0"><font size="2"><span style="font-size:11pt;"> amdgpu_device_ip_block_add(adev, &vce_v3_4_ip_block);</span></font></p>
<p style="margin-top:0;margin-bottom:0"> }</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">That way if we re-arrange the order later, it will be easier to track.</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">Alex<br>
</p>
</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 Junwei Zhang <Jerry.Zhang@amd.com><br>
<b>Sent:</b> Friday, November 23, 2018 3:32:27 AM<br>
<b>To:</b> amd-gfx@lists.freedesktop.org<br>
<b>Cc:</b> Zhang, Jerry<br>
<b>Subject:</b> [PATCH] drm/amdgpu: disable UVD/VCE for some polaris 12 variants</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Some variants don't support UVD and VCE.<br>
<br>
Signed-off-by: Junwei Zhang <Jerry.Zhang@amd.com><br>
---<br>
drivers/gpu/drm/amd/amdgpu/vi.c | 4 ++++<br>
1 file changed, 4 insertions(+)<br>
<br>
diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c<br>
index f3a4cf1f013a..3338b013ded4 100644<br>
--- a/drivers/gpu/drm/amd/amdgpu/vi.c<br>
+++ b/drivers/gpu/drm/amd/amdgpu/vi.c<br>
@@ -1660,6 +1660,10 @@ int vi_set_ip_blocks(struct amdgpu_device *adev)<br>
amdgpu_device_ip_block_add(adev, &dce_v11_2_ip_block);<br>
amdgpu_device_ip_block_add(adev, &gfx_v8_0_ip_block);<br>
amdgpu_device_ip_block_add(adev, &sdma_v3_1_ip_block);<br>
+ /* Some polaris12 variants don't support UVD/VCE */<br>
+ if ((adev->pdev->device == 0x67df) &&<br>
+ (adev->pdev->revision == 0xf7))<br>
+ break;<br>
amdgpu_device_ip_block_add(adev, &uvd_v6_3_ip_block);<br>
amdgpu_device_ip_block_add(adev, &vce_v3_4_ip_block);<br>
break;<br>
-- <br>
2.17.1<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><br>
</div>
</span></font></div>
</body>
</html>