drm/amd/display: do not compare integers of different widths
authorJosip Pavic <Josip.Pavic@amd.com>
Thu, 8 Sep 2022 16:01:41 +0000 (12:01 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 19 Sep 2022 19:15:30 +0000 (15:15 -0400)
[Why & How]
Increase width of some variables to avoid comparing integers of
different widths

Reviewed-by: Alvin Lee <alvin.lee2@amd.com>
Acked-by: Wayne Lin <wayne.lin@amd.com>
Signed-off-by: Josip Pavic <Josip.Pavic@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dcn30/dcn30_hwseq.c
drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hwseq.c
drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource_helpers.c

index 4a1954f..8c50457 100644 (file)
@@ -940,7 +940,7 @@ bool dcn30_does_plane_fit_in_mall(struct dc *dc, struct dc_plane_state *plane, s
 void dcn30_hardware_release(struct dc *dc)
 {
        bool subvp_in_use = false;
-       uint8_t i;
+       uint32_t i;
 
        dc_dmub_srv_p_state_delegate(dc, false, NULL);
        dc_dmub_setup_subvp_dmub_command(dc, dc->current_state, false);
index c1261cb..b8326d4 100644 (file)
@@ -1354,7 +1354,7 @@ void dcn32_update_phantom_vp_position(struct dc *dc,
                struct dc_state *context,
                struct pipe_ctx *phantom_pipe)
 {
-       uint8_t i;
+       uint32_t i;
        struct dc_plane_state *phantom_plane = phantom_pipe->plane_state;
 
        for (i = 0; i < dc->res_pool->pipe_count; i++) {
index 5490b37..160aefa 100644 (file)
@@ -265,7 +265,8 @@ void dcn32_determine_det_override(struct dc *dc,
                display_e2e_pipe_params_st *pipes,
                uint8_t *is_pipe_split_expected)
 {
-       uint8_t i, j, pipe_plane_count, stream_segments, plane_segments, pipe_segments[MAX_PIPES] = {0};
+       uint32_t i, j;
+       uint8_t pipe_plane_count, stream_segments, plane_segments, pipe_segments[MAX_PIPES] = {0};
        uint8_t pipe_counted[MAX_PIPES] = {0};
        uint8_t pipe_cnt = 0;
        struct dc_plane_state *current_plane = NULL;