<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:DengXian;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"\@DengXian";
panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:#0563C1;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri",sans-serif;
color:windowtext;}
p.xmsonormal, li.xmsonormal, div.xmsonormal
{mso-style-name:x_msonormal;
margin:0cm;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri",sans-serif;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word">
<p class="msipheaderdf3d92d6" align="Left" style="margin:0"><span style="font-size:10.0pt;font-family:Arial;color:#0000FF">[AMD Official Use Only - General]</span></p>
<br>
<div class="WordSection1">
<p class="xmsonormal">drm/amdgpu: Differentiate between Raven2 and Raven/Picasso according to revision id.<o:p></o:p></p>
<p class="xmsonormal"> <o:p></o:p></p>
<p class="xmsonormal"> Due to the raven2 and raven/picasso maybe have the same GC_HWIP version.<o:p></o:p></p>
<p class="xmsonormal"> So differentiate them by revision id.<o:p></o:p></p>
<p class="xmsonormal"> <o:p></o:p></p>
<p class="xmsonormal"> Signed-off-by: shanshengwang <<a href="mailto:shansheng.wang@amd.com">shansheng.wang@amd.com</a>><o:p></o:p></p>
<p class="xmsonormal"> Signed-off-by: Jesse Zhang <<a href="mailto:Jesse.Zhang@amd.com">Jesse.Zhang@amd.com</a>><o:p></o:p></p>
<p class="xmsonormal"> <o:p></o:p></p>
<p class="xmsonormal">diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c<o:p></o:p></p>
<p class="xmsonormal">index e093e83ae739..1f4edfb96636 100644<o:p></o:p></p>
<p class="xmsonormal">--- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c<o:p></o:p></p>
<p class="xmsonormal">+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c<o:p></o:p></p>
<p class="xmsonormal">@@ -4000,30 +4000,27 @@ static uint64_t gfx_v9_0_get_gpu_clock_counter(struct amdgpu_device *adev)<o:p></o:p></p>
<p class="xmsonormal"> clock = clock_lo | (clock_hi << 32ULL);<o:p></o:p></p>
<p class="xmsonormal"> break;<o:p></o:p></p>
<p class="xmsonormal"> case IP_VERSION(9, 1, 0):<o:p></o:p></p>
<p class="xmsonormal">+ case IP_VERSION(9, 2, 2):<o:p></o:p></p>
<p class="xmsonormal"> preempt_disable();<o:p></o:p></p>
<p class="xmsonormal">- clock_hi = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven);<o:p></o:p></p>
<p class="xmsonormal">- clock_lo = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_LOWER_Raven);<o:p></o:p></p>
<p class="xmsonormal">- hi_check = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven);<o:p></o:p></p>
<p class="xmsonormal">- /* The PWR TSC clock frequency is 100MHz, which sets 32-bit carry over<o:p></o:p></p>
<p class="xmsonormal">- * roughly every 42 seconds.<o:p></o:p></p>
<p class="xmsonormal">- */<o:p></o:p></p>
<p class="xmsonormal">- if (hi_check != clock_hi) {<o:p></o:p></p>
<p class="xmsonormal">+ if (adev->rev_id >= 0x8)<o:p></o:p></p>
<p class="xmsonormal">+ {<o:p></o:p></p>
<p class="xmsonormal">+ clock_hi = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven2);<o:p></o:p></p>
<p class="xmsonormal">+ clock_lo = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_LOWER_Raven2);<o:p></o:p></p>
<p class="xmsonormal">+ hi_check = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven2);<o:p></o:p></p>
<p class="xmsonormal">+ }else{<o:p></o:p></p>
<p class="xmsonormal">+ clock_hi = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven);<o:p></o:p></p>
<p class="xmsonormal"> clock_lo = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_LOWER_Raven);<o:p></o:p></p>
<p class="xmsonormal">- clock_hi = hi_check;<o:p></o:p></p>
<p class="xmsonormal">+ hi_check = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven);<o:p></o:p></p>
<p class="xmsonormal"> }<o:p></o:p></p>
<p class="xmsonormal">- preempt_enable();<o:p></o:p></p>
<p class="xmsonormal">- clock = clock_lo | (clock_hi << 32ULL);<o:p></o:p></p>
<p class="xmsonormal">- break;<o:p></o:p></p>
<p class="xmsonormal">- case IP_VERSION(9, 2, 2):<o:p></o:p></p>
<p class="xmsonormal">- preempt_disable();<o:p></o:p></p>
<p class="xmsonormal">- clock_hi = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven2);<o:p></o:p></p>
<p class="xmsonormal">- clock_lo = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_LOWER_Raven2);<o:p></o:p></p>
<p class="xmsonormal">- hi_check = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_UPPER_Raven2);<o:p></o:p></p>
<p class="xmsonormal"> /* The PWR TSC clock frequency is 100MHz, which sets 32-bit carry over<o:p></o:p></p>
<p class="xmsonormal">- * roughly every 42 seconds.<o:p></o:p></p>
<p class="xmsonormal">- */<o:p></o:p></p>
<p class="xmsonormal">+ * roughly every 42 seconds.<o:p></o:p></p>
<p class="xmsonormal">+ */<o:p></o:p></p>
<p class="xmsonormal"> if (hi_check != clock_hi) {<o:p></o:p></p>
<p class="xmsonormal">- clock_lo = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_LOWER_Raven2);<o:p></o:p></p>
<p class="xmsonormal">+ if (adev->rev_id >= 0x8) {<o:p></o:p></p>
<p class="xmsonormal">+ clock_lo = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_LOWER_Raven2);<o:p></o:p></p>
<p class="xmsonormal">+ }else{<o:p></o:p></p>
<p class="xmsonormal">+ clock_lo = RREG32_SOC15_NO_KIQ(PWR, 0, mmGOLDEN_TSC_COUNT_LOWER_Raven);<o:p></o:p></p>
<p class="xmsonormal">+ }<o:p></o:p></p>
<p class="xmsonormal"> clock_hi = hi_check;<o:p></o:p></p>
<p class="xmsonormal"> }<o:p></o:p></p>
<p class="xmsonormal"> preempt_enable();<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</body>
</html>