Commit e84e40dc authored by Jiapeng Chong's avatar Jiapeng Chong Committed by Alex Deucher
Browse files

drm/amd/display: Simplify the calculation of variables 



./drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hwseq.c:586:37-39: WARNING !A || A && B is equivalent to !A || B.
./drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hwseq.c:595:37-39: WARNING !A || A && B is equivalent to !A || B.

Reported-by: default avatarAbaci Robot <abaci@linux.alibaba.com>
Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=4941


Signed-off-by: default avatarJiapeng Chong <jiapeng.chong@linux.alibaba.com>
Signed-off-by: default avatarHamza Mahfooz <hamza.mahfooz@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 82ad22bb

drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hwseq.c

+3 −3
Original line number Diff line number Diff line
@@ -583,8 +583,8 @@ void dcn32_update_force_pstate(struct dc *dc, struct dc_state *context) 
		struct pipe_ctx *pipe = &context->res_ctx.pipe_ctx[i];

		struct hubp *hubp = pipe->plane_res.hubp;



		if (!pipe->stream || (pipe->stream && !(pipe->stream->mall_stream_config.type == SUBVP_MAIN ||

						pipe->stream->fpo_in_use))) {

		if (!pipe->stream || !(pipe->stream->mall_stream_config.type == SUBVP_MAIN ||

		    pipe->stream->fpo_in_use)) {

			if (hubp && hubp->funcs->hubp_update_force_pstate_disallow)

				hubp->funcs->hubp_update_force_pstate_disallow(hubp, false);

		}
@@ -592,7 +592,7 @@ void dcn32_update_force_pstate(struct dc *dc, struct dc_state *context) 
		/* Today only FPO uses cursor P-State force. Only clear cursor P-State force

		 * if it's not FPO.

		 */

		if (!pipe->stream || (pipe->stream && !pipe->stream->fpo_in_use)) {

		if (!pipe->stream || !pipe->stream->fpo_in_use) {

			if (hubp && hubp->funcs->hubp_update_force_cursor_pstate_disallow)

				hubp->funcs->hubp_update_force_cursor_pstate_disallow(hubp, false);

		}