<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:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
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;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri",sans-serif;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
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="msipheader251902e5" align="Left" style="margin:0"><span style="font-size:10.0pt;font-family:Arial;color:#317100">[AMD Public Use]</span></p>
<br>
<div class="WordSection1">
<p class="MsoNormal">v1: During mode2 reset, PCI space is lost after message is sent.<o:p></o:p></p>
<p class="MsoNormal">Restore PCI space before waiting for response from firmware.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">v2: Move mode2 sequence to aldebaran and update PMFW version.<o:p></o:p></p>
<p class="MsoNormal">Handle generic sequence in smu13 without PMFW version check.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Signed-off-by: Lijo Lazar <a href="mailto:lijo.lazar@amd.com">
lijo.lazar@amd.com</a><o:p></o:p></p>
<p class="MsoNormal">---<o:p></o:p></p>
<p class="MsoNormal">.../drm/amd/pm/swsmu/smu13/aldebaran_ppt.c | 53 ++++++++++++++++++-<o:p></o:p></p>
<p class="MsoNormal">.../gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c | 15 +++---<o:p></o:p></p>
<p class="MsoNormal">drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c | 2 +-<o:p></o:p></p>
<p class="MsoNormal">drivers/gpu/drm/amd/pm/swsmu/smu_cmn.h | 2 +<o:p></o:p></p>
<p class="MsoNormal">4 files changed, 61 insertions(+), 11 deletions(-)<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c<o:p></o:p></p>
<p class="MsoNormal">index 7d38b92a78dc..ec485308b921 100644<o:p></o:p></p>
<p class="MsoNormal">--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c<o:p></o:p></p>
<p class="MsoNormal">+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c<o:p></o:p></p>
<p class="MsoNormal">@@ -1432,6 +1432,57 @@ static ssize_t aldebaran_get_gpu_metrics(struct smu_context *smu,<o:p></o:p></p>
<p class="MsoNormal"> return sizeof(struct gpu_metrics_v1_1);<o:p></o:p></p>
<p class="MsoNormal">}<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal">+int aldebaran_mode2_reset(struct smu_context *smu)<o:p></o:p></p>
<p class="MsoNormal">+{<o:p></o:p></p>
<p class="MsoNormal">+ u32 smu_version;<o:p></o:p></p>
<p class="MsoNormal">+ int ret = 0, index;<o:p></o:p></p>
<p class="MsoNormal">+ struct amdgpu_device *adev = smu->adev;<o:p></o:p></p>
<p class="MsoNormal">+ int timeout = 10;<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ smu_cmn_get_smc_version(smu, NULL, &smu_version);<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ index = smu_cmn_to_asic_specific_index(smu, CMN2ASIC_MAPPING_MSG,<o:p></o:p></p>
<p class="MsoNormal">+ SMU_MSG_GfxDeviceDriverReset);<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ mutex_lock(&smu->message_lock);<o:p></o:p></p>
<p class="MsoNormal">+ if (smu_version >= 0x00441400) {<o:p></o:p></p>
<p class="MsoNormal">+ ret = smu_cmn_send_msg_without_waiting(smu, (uint16_t)index, SMU_RESET_MODE_2);<o:p></o:p></p>
<p class="MsoNormal">+ /* This is similar to FLR, wait till max FLR timeout */<o:p></o:p></p>
<p class="MsoNormal">+ msleep(100);<o:p></o:p></p>
<p class="MsoNormal">+ dev_dbg(smu->adev->dev, "restore config space...\n");<o:p></o:p></p>
<p class="MsoNormal">+ /* Restore the config space saved during init */<o:p></o:p></p>
<p class="MsoNormal">+ amdgpu_device_load_pci_state(adev->pdev);<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ dev_dbg(smu->adev->dev, "wait for reset ack\n");<o:p></o:p></p>
<p class="MsoNormal">+ while (ret == -ETIME && timeout) {<o:p></o:p></p>
<p class="MsoNormal">+ ret = smu_cmn_wait_for_response(smu);<o:p></o:p></p>
<p class="MsoNormal">+ /* Wait a bit more time for getting ACK */<o:p></o:p></p>
<p class="MsoNormal">+ if (ret == -ETIME) {<o:p></o:p></p>
<p class="MsoNormal">+ --timeout;<o:p></o:p></p>
<p class="MsoNormal">+ usleep_range(500, 1000);<o:p></o:p></p>
<p class="MsoNormal">+ continue;<o:p></o:p></p>
<p class="MsoNormal">+ }<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ if (ret != 1) {<o:p></o:p></p>
<p class="MsoNormal">+ dev_err(adev->dev, "failed to send mode2 message \tparam: 0x%08x response %#x\n",<o:p></o:p></p>
<p class="MsoNormal">+ SMU_RESET_MODE_2, ret);<o:p></o:p></p>
<p class="MsoNormal">+ goto out;<o:p></o:p></p>
<p class="MsoNormal">+ }<o:p></o:p></p>
<p class="MsoNormal">+ }<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ } else {<o:p></o:p></p>
<p class="MsoNormal">+ dev_err(adev->dev, "smu fw 0x%x does not support MSG_GfxDeviceDriverReset MSG\n",<o:p></o:p></p>
<p class="MsoNormal">+ smu_version);<o:p></o:p></p>
<p class="MsoNormal">+ }<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ if (ret == 1)<o:p></o:p></p>
<p class="MsoNormal">+ ret = 0;<o:p></o:p></p>
<p class="MsoNormal">+out:<o:p></o:p></p>
<p class="MsoNormal">+ mutex_unlock(&smu->message_lock);<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ return ret;<o:p></o:p></p>
<p class="MsoNormal">+}<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">static bool aldebaran_is_mode1_reset_supported(struct smu_context *smu)<o:p></o:p></p>
<p class="MsoNormal">{<o:p></o:p></p>
<p class="MsoNormal">#if 0<o:p></o:p></p>
<p class="MsoNormal">@@ -1530,8 +1581,8 @@ static const struct pptable_funcs aldebaran_ppt_funcs = {<o:p></o:p></p>
<p class="MsoNormal"> .mode1_reset_is_support = aldebaran_is_mode1_reset_supported,<o:p></o:p></p>
<p class="MsoNormal"> .mode2_reset_is_support = aldebaran_is_mode2_reset_supported,<o:p></o:p></p>
<p class="MsoNormal"> .mode1_reset = smu_v13_0_mode1_reset,<o:p></o:p></p>
<p class="MsoNormal">- .mode2_reset = smu_v13_0_mode2_reset,<o:p></o:p></p>
<p class="MsoNormal"> .set_mp1_state = aldebaran_set_mp1_state,<o:p></o:p></p>
<p class="MsoNormal">+ .mode2_reset = aldebaran_mode2_reset,<o:p></o:p></p>
<p class="MsoNormal">};<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal"> void aldebaran_set_ppt_funcs(struct smu_context *smu)<o:p></o:p></p>
<p class="MsoNormal">diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c<o:p></o:p></p>
<p class="MsoNormal">index bd3a9c89dc44..fa0d8ca29982 100644<o:p></o:p></p>
<p class="MsoNormal">--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c<o:p></o:p></p>
<p class="MsoNormal">+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c<o:p></o:p></p>
<p class="MsoNormal">@@ -1376,17 +1376,14 @@ int smu_v13_0_mode1_reset(struct smu_context *smu)<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal"> int smu_v13_0_mode2_reset(struct smu_context *smu)<o:p></o:p></p>
<p class="MsoNormal">{<o:p></o:p></p>
<p class="MsoNormal">- u32 smu_version;<o:p></o:p></p>
<p class="MsoNormal">- int ret = 0;<o:p></o:p></p>
<p class="MsoNormal">- struct amdgpu_device *adev = smu->adev;<o:p></o:p></p>
<p class="MsoNormal">- smu_cmn_get_smc_version(smu, NULL, &smu_version);<o:p></o:p></p>
<p class="MsoNormal">- if (smu_version >= 0x00440700)<o:p></o:p></p>
<p class="MsoNormal">- ret = smu_cmn_send_smc_msg_with_param(smu, SMU_MSG_GfxDeviceDriverReset, SMU_RESET_MODE_2, NULL);<o:p></o:p></p>
<p class="MsoNormal">- else<o:p></o:p></p>
<p class="MsoNormal">- dev_err(adev->dev, "smu fw 0x%x does not support MSG_GfxDeviceDriverReset MSG\n", smu_version);<o:p></o:p></p>
<p class="MsoNormal">- /*TODO: mode2 reset wait time should be shorter, will modify it later*/<o:p></o:p></p>
<p class="MsoNormal">+ int ret;<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+ ret = smu_cmn_send_smc_msg_with_param(smu, SMU_MSG_GfxDeviceDriverReset,<o:p></o:p></p>
<p class="MsoNormal">+ SMU_RESET_MODE_2, NULL);<o:p></o:p></p>
<p class="MsoNormal">+ /*TODO: mode2 reset wait time should be shorter, add ASIC specific func if required */<o:p></o:p></p>
<p class="MsoNormal"> if (!ret)<o:p></o:p></p>
<p class="MsoNormal"> msleep(SMU13_MODE1_RESET_WAIT_TIME_IN_MS);<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal"> return ret;<o:p></o:p></p>
<p class="MsoNormal">}<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal">diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c b/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c<o:p></o:p></p>
<p class="MsoNormal">index 2d216f5b167d..b725f263092b 100644<o:p></o:p></p>
<p class="MsoNormal">--- a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c<o:p></o:p></p>
<p class="MsoNormal">+++ b/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.c<o:p></o:p></p>
<p class="MsoNormal">@@ -76,7 +76,7 @@ static void smu_cmn_read_arg(struct smu_context *smu,<o:p></o:p></p>
<p class="MsoNormal"> *arg = RREG32_SOC15(MP1, 0, mmMP1_SMN_C2PMSG_82);<o:p></o:p></p>
<p class="MsoNormal">}<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal">-static int smu_cmn_wait_for_response(struct smu_context *smu)<o:p></o:p></p>
<p class="MsoNormal">+int smu_cmn_wait_for_response(struct smu_context *smu)<o:p></o:p></p>
<p class="MsoNormal">{<o:p></o:p></p>
<p class="MsoNormal"> struct amdgpu_device *adev = smu->adev;<o:p></o:p></p>
<p class="MsoNormal"> uint32_t cur_value, i, timeout = adev->usec_timeout * 10;<o:p></o:p></p>
<p class="MsoNormal">diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.h b/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.h<o:p></o:p></p>
<p class="MsoNormal">index 155e2a68fa1c..da6ff6f024f9 100644<o:p></o:p></p>
<p class="MsoNormal">--- a/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.h<o:p></o:p></p>
<p class="MsoNormal">+++ b/drivers/gpu/drm/amd/pm/swsmu/smu_cmn.h<o:p></o:p></p>
<p class="MsoNormal">@@ -37,6 +37,8 @@ int smu_cmn_send_smc_msg(struct smu_context *smu,<o:p></o:p></p>
<p class="MsoNormal"> enum smu_message_type msg,<o:p></o:p></p>
<p class="MsoNormal"> uint32_t *read_arg);<o:p></o:p></p>
<p class="MsoNormal"><o:p></o:p></p>
<p class="MsoNormal">+int smu_cmn_wait_for_response(struct smu_context *smu);<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">int smu_cmn_to_asic_specific_index(struct smu_context *smu,<o:p></o:p></p>
<p class="MsoNormal"> enum smu_cmn2asic_mapping_type type,<o:p></o:p></p>
<p class="MsoNormal"> uint32_t index);<o:p></o:p></p>
<p class="MsoNormal">-- <o:p></o:p></p>
<p class="MsoNormal">2.17.1<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</body>
</html>