Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f3b5cb3e authored by Rex Zhu's avatar Rex Zhu Committed by Alex Deucher
Browse files

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: default avatarRex Zhu <Rex.Zhu@amd.com>
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 5f232365

drivers/gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c

+3 −3
Original line number Diff line number Diff line
@@ -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;