[AMD Public Use]

Nitpick - you may want to send the message only during enable workflow. Apart 
from that,

Series is Reviewed-by: Lijo Lazar <lijo.la...@amd.com>

-----Original Message-----
From: Quan, Evan <evan.q...@amd.com> 
Sent: Wednesday, December 9, 2020 9:49 AM
To: amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Lazar, Lijo 
<lijo.la...@amd.com>; Quan, Evan <evan.q...@amd.com>
Subject: [PATCH 5/5] drm/amd/pm: fulfill sienna cichlid 2nd usb2.0 port 
workaround

Fulfill the 2nd usb2.0 port workaround for sienna cichlid.

Change-Id: Id5a89a468787846ed0050b56cd318a9574185567
Signed-off-by: Evan Quan <evan.q...@amd.com>
---
 .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.c   | 38 ++++++++++++++++++-
 1 file changed, 37 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
index 47d4f92d5ead..74cf027e4a41 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
@@ -3247,6 +3247,42 @@ static int sienna_cichlid_gpo_control(struct smu_context 
*smu,
 
        return ret;
 }
+
+static int sienna_cichlid_notify_2nd_usb20_port(struct smu_context 
+*smu) {
+       uint32_t smu_version;
+       int ret = 0;
+
+       ret = smu_cmn_get_smc_version(smu, NULL, &smu_version);
+       if (ret)
+               return ret;
+
+       /*
+        * Message SMU_MSG_Enable2ndUSB20Port is supported by 58.45
+        * onwards PMFWs.
+        */
+       if (smu_version < 0x003A2D00)
+               return 0;
+
+       return smu_cmn_send_smc_msg_with_param(smu,
+                                              SMU_MSG_Enable2ndUSB20Port,
+                                              
smu->smu_table.boot_values.firmware_caps & 
ATOM_FIRMWARE_CAP_ENABLE_2ND_USB20PORT ?
+                                              1 : 0,
+                                              NULL);
+}
+
+static int sienna_cichlid_system_features_control(struct smu_context *smu,
+                                                 bool en)
+{
+       int ret = 0;
+
+       ret = sienna_cichlid_notify_2nd_usb20_port(smu);
+       if (ret)
+               return ret;
+
+       return smu_v11_0_system_features_control(smu, en); }
+
 static const struct pptable_funcs sienna_cichlid_ppt_funcs = {
        .get_allowed_feature_mask = sienna_cichlid_get_allowed_feature_mask,
        .set_default_dpm_table = sienna_cichlid_set_default_dpm_table,
@@ -3287,7 +3323,7 @@ static const struct pptable_funcs 
sienna_cichlid_ppt_funcs = {
        .set_driver_table_location = smu_v11_0_set_driver_table_location,
        .set_tool_table_location = smu_v11_0_set_tool_table_location,
        .notify_memory_pool_location = smu_v11_0_notify_memory_pool_location,
-       .system_features_control = smu_v11_0_system_features_control,
+       .system_features_control = sienna_cichlid_system_features_control,
        .send_smc_msg_with_param = smu_cmn_send_smc_msg_with_param,
        .send_smc_msg = smu_cmn_send_smc_msg,
        .init_display_count = NULL,
--
2.29.0
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to