drm/amd/powerplay: don't add invalid voltage.

if atomctrl_get_voltage_evv_on_sclk_ai returns
non zero (fail) in the expansion of the
PP_ASSERT_WITH_CODE macro the continue will
actually do nothing, So invalid voltage will be
added to ppbable.

Signed-off-by: Rex Zhu <Rex.Zhu@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

1 files changed, 6 insertions, 6 deletions

diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c
index 69be3e59694f..9d764c4d253e 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c
@@ -2871,12 +2871,12 @@ static int polaris10_get_evv_voltages(struct pp_hwmgr *hwmgr)
                                 }
                         }
 
-
-                        PP_ASSERT_WITH_CODE(0 == atomctrl_get_voltage_evv_on_sclk_ai(hwmgr,
-                                                        VOLTAGE_TYPE_VDDC, sclk, vv_id, &vddc),
-                                                "Error retrieving EVV voltage value!",
-                                                continue);
-
+                        if (atomctrl_get_voltage_evv_on_sclk_ai(hwmgr,
+                                                VOLTAGE_TYPE_VDDC,
+                                                sclk, vv_id, &vddc) != 0) {
+                                printk(KERN_WARNING "failed to retrieving EVV voltage!\n");
+                                continue;
+                        }
 
                         /* need to make sure vddc is less than 2v or else, it could burn the ASIC. */
                         PP_ASSERT_WITH_CODE((vddc < 2000 && vddc != 0),