Commit b5ffcb48 authored by Lee Jones's avatar Lee Jones Committed by Alex Deucher

drm/amd/display/dc/dce/dce_transform: Remove 3 unused/legacy variables

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: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent d031aa06
...@@ -493,7 +493,6 @@ static void dce60_transform_set_scaler( ...@@ -493,7 +493,6 @@ static void dce60_transform_set_scaler(
{ {
struct dce_transform *xfm_dce = TO_DCE_TRANSFORM(xfm); struct dce_transform *xfm_dce = TO_DCE_TRANSFORM(xfm);
bool is_scaling_required; bool is_scaling_required;
bool filter_updated = false;
const uint16_t *coeffs_v, *coeffs_h; const uint16_t *coeffs_v, *coeffs_h;
/*Use whole line buffer memory always*/ /*Use whole line buffer memory always*/
...@@ -558,7 +557,6 @@ static void dce60_transform_set_scaler( ...@@ -558,7 +557,6 @@ static void dce60_transform_set_scaler(
xfm_dce->filter_v = coeffs_v; xfm_dce->filter_v = coeffs_v;
xfm_dce->filter_h = coeffs_h; xfm_dce->filter_h = coeffs_h;
filter_updated = true;
} }
} }
...@@ -1037,34 +1035,23 @@ static void dce60_transform_set_pixel_storage_depth( ...@@ -1037,34 +1035,23 @@ static void dce60_transform_set_pixel_storage_depth(
const struct bit_depth_reduction_params *bit_depth_params) const struct bit_depth_reduction_params *bit_depth_params)
{ {
struct dce_transform *xfm_dce = TO_DCE_TRANSFORM(xfm); struct dce_transform *xfm_dce = TO_DCE_TRANSFORM(xfm);
int pixel_depth, expan_mode;
enum dc_color_depth color_depth; enum dc_color_depth color_depth;
switch (depth) { switch (depth) {
case LB_PIXEL_DEPTH_18BPP: case LB_PIXEL_DEPTH_18BPP:
color_depth = COLOR_DEPTH_666; color_depth = COLOR_DEPTH_666;
pixel_depth = 2;
expan_mode = 1;
break; break;
case LB_PIXEL_DEPTH_24BPP: case LB_PIXEL_DEPTH_24BPP:
color_depth = COLOR_DEPTH_888; color_depth = COLOR_DEPTH_888;
pixel_depth = 1;
expan_mode = 1;
break; break;
case LB_PIXEL_DEPTH_30BPP: case LB_PIXEL_DEPTH_30BPP:
color_depth = COLOR_DEPTH_101010; color_depth = COLOR_DEPTH_101010;
pixel_depth = 0;
expan_mode = 1;
break; break;
case LB_PIXEL_DEPTH_36BPP: case LB_PIXEL_DEPTH_36BPP:
color_depth = COLOR_DEPTH_121212; color_depth = COLOR_DEPTH_121212;
pixel_depth = 3;
expan_mode = 0;
break; break;
default: default:
color_depth = COLOR_DEPTH_101010; color_depth = COLOR_DEPTH_101010;
pixel_depth = 0;
expan_mode = 1;
BREAK_TO_DEBUGGER(); BREAK_TO_DEBUGGER();
break; break;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment