From: "Assadian, Navid" <navid.assad...@amd.com>

The mismatch type comparison/assignment may cause data loss. Since the
values are always non-negative, it is safe to use unsigned variables to
resolve the mismatch.

Signed-off-by: Navid Assadian <navid.assad...@amd.com>
Reviewed-by: Joshua Aberback <joshua.aberb...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/sspl/dc_spl.c       | 4 ++--
 drivers/gpu/drm/amd/display/dc/sspl/dc_spl_types.h | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/sspl/dc_spl.c 
b/drivers/gpu/drm/amd/display/dc/sspl/dc_spl.c
index 72a79288ab79..3d85732cc0f5 100644
--- a/drivers/gpu/drm/amd/display/dc/sspl/dc_spl.c
+++ b/drivers/gpu/drm/amd/display/dc/sspl/dc_spl.c
@@ -944,8 +944,8 @@ static bool spl_get_optimal_number_of_taps(
          bool *enable_isharp)
 {
        int num_part_y, num_part_c;
-       int max_taps_y, max_taps_c;
-       int min_taps_y, min_taps_c;
+       unsigned int max_taps_y, max_taps_c;
+       unsigned int min_taps_y, min_taps_c;
        enum lb_memory_config lb_config;
        bool skip_easf = false;
        bool is_subsampled = spl_is_subsampled_format(spl_in->basic_in.format);
diff --git a/drivers/gpu/drm/amd/display/dc/sspl/dc_spl_types.h 
b/drivers/gpu/drm/amd/display/dc/sspl/dc_spl_types.h
index 0130673ceee6..1c3949b24611 100644
--- a/drivers/gpu/drm/amd/display/dc/sspl/dc_spl_types.h
+++ b/drivers/gpu/drm/amd/display/dc/sspl/dc_spl_types.h
@@ -493,7 +493,7 @@ struct spl_sharpness_range {
 };
 struct adaptive_sharpness {
        bool enable;
-       int sharpness_level;
+       unsigned int sharpness_level;
        struct spl_sharpness_range sharpness_range;
 };
 enum linear_light_scaling      {       // convert it in translation logic
-- 
2.34.1

Reply via email to