drm/amd/display: remove set but not used variable 'min_content'
authorzhengbin <zhengbin13@huawei.com>
Thu, 14 Nov 2019 12:36:27 +0000 (20:36 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 19 Nov 2019 15:12:51 +0000 (10:12 -0500)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/gpu/drm/amd/display/modules/color/color_gamma.c: In function build_freesync_hdr:
drivers/gpu/drm/amd/display/modules/color/color_gamma.c:830:20: warning: variable min_content set but not used [-Wunused-but-set-variable]

It is not used since commit 50575eb5b339 ("drm/amd/display:
Only use EETF when maxCL > max display")

Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: zhengbin <zhengbin13@huawei.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/modules/color/color_gamma.c

index 1de4805cb8c7fc23dc949263b54d3cdac662995a..9b121b08c80633ffd2c662b71b085710e908751c 100644 (file)
@@ -937,7 +937,6 @@ static bool build_freesync_hdr(struct pwl_float_data_ex *rgb_regamma,
        struct fixed31_32 max_display;
        struct fixed31_32 min_display;
        struct fixed31_32 max_content;
-       struct fixed31_32 min_content;
        struct fixed31_32 clip = dc_fixpt_one;
        struct fixed31_32 output;
        bool use_eetf = false;
@@ -951,7 +950,6 @@ static bool build_freesync_hdr(struct pwl_float_data_ex *rgb_regamma,
        max_display = dc_fixpt_from_int(fs_params->max_display);
        min_display = dc_fixpt_from_fraction(fs_params->min_display, 10000);
        max_content = dc_fixpt_from_int(fs_params->max_content);
-       min_content = dc_fixpt_from_fraction(fs_params->min_content, 10000);
        sdr_white_level = dc_fixpt_from_int(fs_params->sdr_white_level);
 
        if (fs_params->min_display > 1000) // cap at 0.1 at the bottom