From: Piotr Maziarz <[email protected]>

[Why]
Some variables are expected to be, and used as unsigned.

[How]
Explicitly declare them as unsigned

Reviewed-by: Dillon Varone <[email protected]>
Signed-off-by: Piotr Maziarz <[email protected]>
Signed-off-by: George Zhang <[email protected]>
---
 drivers/gpu/drm/amd/display/dc/core/dc_resource.c     | 11 +++++------
 .../amd/display/dc/dml/dcn31/display_mode_vba_31.c    |  2 +-
 .../amd/display/dc/dml/dcn314/display_mode_vba_314.c  |  2 +-
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
index 5f6cc1b1f788..b21d41df0fab 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
@@ -4412,14 +4412,13 @@ enum dc_status dc_validate_with_context(struct dc *dc,
        struct dc_stream_state *unchanged_streams[MAX_PIPES] = { 0 };
        struct dc_stream_state *del_streams[MAX_PIPES] = { 0 };
        struct dc_stream_state *add_streams[MAX_PIPES] = { 0 };
-       int old_stream_count = context->stream_count;
+       unsigned int old_stream_count = context->stream_count;
        enum dc_status res = DC_ERROR_UNEXPECTED;
-       int unchanged_streams_count = 0;
-       int del_streams_count = 0;
-       int add_streams_count = 0;
+       unsigned int unchanged_streams_count = 0;
+       unsigned int del_streams_count = 0;
+       unsigned int add_streams_count = 0;
        bool found = false;
-       int i, j;
-       unsigned int k;
+       unsigned int i, j, k;

        DC_LOGGER_INIT(dc->ctx->logger);

diff --git a/drivers/gpu/drm/amd/display/dc/dml/dcn31/display_mode_vba_31.c 
b/drivers/gpu/drm/amd/display/dc/dml/dcn31/display_mode_vba_31.c
index bd14ebea1111..8064c4b3fd25 100644
--- a/drivers/gpu/drm/amd/display/dc/dml/dcn31/display_mode_vba_31.c
+++ b/drivers/gpu/drm/amd/display/dc/dml/dcn31/display_mode_vba_31.c
@@ -5337,7 +5337,7 @@ void dml31_ModeSupportAndSystemConfigurationFull(struct 
display_mode_lib *mode_l
                for (j = 0; j <= 1; ++j) {
                        double VMDataOnlyReturnBWPerState;
                        double HostVMInefficiencyFactor = 1;
-                       int NextPrefetchModeState = MinPrefetchMode;
+                       unsigned int NextPrefetchModeState = MinPrefetchMode;
                        bool UnboundedRequestEnabledThisState = false;
                        unsigned int CompressedBufferSizeInkByteThisState = 0;
                        double dummy;
diff --git a/drivers/gpu/drm/amd/display/dc/dml/dcn314/display_mode_vba_314.c 
b/drivers/gpu/drm/amd/display/dc/dml/dcn314/display_mode_vba_314.c
index 2ea5cf37f273..bf2dde26b98b 100644
--- a/drivers/gpu/drm/amd/display/dc/dml/dcn314/display_mode_vba_314.c
+++ b/drivers/gpu/drm/amd/display/dc/dml/dcn314/display_mode_vba_314.c
@@ -5421,7 +5421,7 @@ void dml314_ModeSupportAndSystemConfigurationFull(struct 
display_mode_lib *mode_
                for (j = 0; j <= 1; ++j) {
                        double VMDataOnlyReturnBWPerState;
                        double HostVMInefficiencyFactor = 1;
-                       int NextPrefetchModeState = MinPrefetchMode;
+                       unsigned int NextPrefetchModeState = MinPrefetchMode;
                        bool UnboundedRequestEnabledThisState = false;
                        unsigned int CompressedBufferSizeInkByteThisState = 0;
                        double dummy;
--
2.54.0

Reply via email to