drm/amd/powerplay: fix static checker warning for return meaningless value.

The return value should be either negative or zero, no positive.

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

diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c
index d166fd925dbb2e19dcc4ce667c05dce63ba6b7e2..ebdb43a8daeff7c7d0636edb63e38944a81bc156 100644 (file)
--- a/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c
+++ b/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c
@@ -199,7 +199,7 @@ static int tonga_send_msg_to_smc(struct pp_smumgr *smumgr, uint16_t msg)
        PP_ASSERT_WITH_CODE(
                1 == SMUM_READ_FIELD(smumgr->device, SMC_RESP_0, SMC_RESP),
                "Failed to send Previous Message.",
-               return 1);
+               );
 
        cgs_write_register(smumgr->device, mmSMC_MESSAGE_0, msg);
 
@@ -207,7 +207,7 @@ static int tonga_send_msg_to_smc(struct pp_smumgr *smumgr, uint16_t msg)
        PP_ASSERT_WITH_CODE(
                1 == SMUM_READ_FIELD(smumgr->device, SMC_RESP_0, SMC_RESP),
                "Failed to send Message.",
-               return 1);
+               );
 
        return 0;
 }
@@ -229,7 +229,7 @@ static int tonga_send_msg_to_smc_without_waiting
        PP_ASSERT_WITH_CODE(
                1 == SMUM_READ_FIELD(smumgr->device, SMC_RESP_0, SMC_RESP),
                "Failed to send Previous Message.",
-               return 0);
+               );
        cgs_write_register(smumgr->device, mmSMC_MESSAGE_0, msg);
 
        return 0;