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

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

drm/amd/powerplay: fix static checker warnings in smu7_hwmgr.c 



variable dereferenced before check it

Signed-off-by: default avatarRex Zhu <Rex.Zhu@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent e0705324

drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c

+4 −2
Original line number Diff line number Diff line
@@ -3802,13 +3802,15 @@ static inline bool smu7_are_power_levels_equal(const struct smu7_performance_lev 


int smu7_check_states_equal(struct pp_hwmgr *hwmgr, const struct pp_hw_power_state *pstate1, const struct pp_hw_power_state *pstate2, bool *equal)

{

	const struct smu7_power_state *psa = cast_const_phw_smu7_power_state(pstate1);

	const struct smu7_power_state *psb = cast_const_phw_smu7_power_state(pstate2);

	const struct smu7_power_state *psa;

	const struct smu7_power_state *psb;

	int i;



	if (pstate1 == NULL || pstate2 == NULL || equal == NULL)

		return -EINVAL;



	psa = cast_const_phw_smu7_power_state(pstate1);

	psb = cast_const_phw_smu7_power_state(pstate2);

	/* If the two states don't even have the same number of performance levels they cannot be the same state. */

	if (psa->performance_level_count != psb->performance_level_count) {

		*equal = false;