drm/amd/display/dc/dce/dce_transform: Remove 3 unused/legacy variables
authorLee Jones <lee.jones@linaro.org>
Fri, 8 Jan 2021 20:14:45 +0000 (20:14 +0000)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 11 Jan 2021 21:09:14 +0000 (16:09 -0500)
Fixes the following W=1 kernel build warning(s):

 drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_transform.c: In function ‘dce60_transform_set_scaler’:
 drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_transform.c:496:7: warning: variable ‘filter_updated’ set but not used [-Wunused-but-set-variable]
 drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_transform.c: In function ‘dce60_transform_set_pixel_storage_depth’:
 drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_transform.c:1040:19: warning: variable ‘expan_mode’ set but not used [-Wunused-but-set-variable]
 drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_transform.c:1040:6: warning: variable ‘pixel_depth’ set but not used [-Wunused-but-set-variable]

Cc: Harry Wentland <harry.wentland@amd.com>
Cc: Leo Li <sunpeng.li@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: "Christian König" <christian.koenig@amd.com>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Mauro Rossi <issor.oruam@gmail.com>
Cc: amd-gfx@lists.freedesktop.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dce/dce_transform.c

index 130a0a0..6121bb7 100644 (file)
@@ -493,7 +493,6 @@ static void dce60_transform_set_scaler(
 {
        struct dce_transform *xfm_dce = TO_DCE_TRANSFORM(xfm);
        bool is_scaling_required;
-       bool filter_updated = false;
        const uint16_t *coeffs_v, *coeffs_h;
 
        /*Use whole line buffer memory always*/
@@ -558,7 +557,6 @@ static void dce60_transform_set_scaler(
 
                        xfm_dce->filter_v = coeffs_v;
                        xfm_dce->filter_h = coeffs_h;
-                       filter_updated = true;
                }
        }
 
@@ -1037,34 +1035,23 @@ static void dce60_transform_set_pixel_storage_depth(
        const struct bit_depth_reduction_params *bit_depth_params)
 {
        struct dce_transform *xfm_dce = TO_DCE_TRANSFORM(xfm);
-       int pixel_depth, expan_mode;
        enum dc_color_depth color_depth;
 
        switch (depth) {
        case LB_PIXEL_DEPTH_18BPP:
                color_depth = COLOR_DEPTH_666;
-               pixel_depth = 2;
-               expan_mode  = 1;
                break;
        case LB_PIXEL_DEPTH_24BPP:
                color_depth = COLOR_DEPTH_888;
-               pixel_depth = 1;
-               expan_mode  = 1;
                break;
        case LB_PIXEL_DEPTH_30BPP:
                color_depth = COLOR_DEPTH_101010;
-               pixel_depth = 0;
-               expan_mode  = 1;
                break;
        case LB_PIXEL_DEPTH_36BPP:
                color_depth = COLOR_DEPTH_121212;
-               pixel_depth = 3;
-               expan_mode  = 0;
                break;
        default:
                color_depth = COLOR_DEPTH_101010;
-               pixel_depth = 0;
-               expan_mode  = 1;
                BREAK_TO_DEBUGGER();
                break;
        }