Commit 2b0619c1 authored by zhengbin's avatar zhengbin Committed by Alex Deucher

drm/amd/display: Remove set but not used variables 'h_ratio_chroma', 'v_ratio_chroma'

Fixes gcc '-Wunused-but-set-variable' warning:

drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dwb_scl.c: In function dwb_program_horz_scalar:
drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dwb_scl.c:725:11: warning: variable h_ratio_chroma set but not used [-Wunused-but-set-variable]
drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dwb_scl.c: In function dwb_program_vert_scalar:
drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dwb_scl.c:806:11: warning: variable v_ratio_chroma set but not used [-Wunused-but-set-variable]

They are not used since commit 345429a6 ("drm/amd/display:
Add DCN2 DWB")
Reviewed-by: default avatarHarry Wentland <harry.wentland@amd.com>
Reported-by: default avatarHulk Robot <hulkci@huawei.com>
Signed-off-by: default avatarzhengbin <zhengbin13@huawei.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent a1fc5147
...@@ -722,7 +722,6 @@ bool dwb_program_horz_scalar(struct dcn20_dwbc *dwbc20, ...@@ -722,7 +722,6 @@ bool dwb_program_horz_scalar(struct dcn20_dwbc *dwbc20,
struct scaling_taps num_taps) struct scaling_taps num_taps)
{ {
uint32_t h_ratio_luma = 1; uint32_t h_ratio_luma = 1;
uint32_t h_ratio_chroma = 1;
uint32_t h_taps_luma = num_taps.h_taps; uint32_t h_taps_luma = num_taps.h_taps;
uint32_t h_taps_chroma = num_taps.h_taps_c; uint32_t h_taps_chroma = num_taps.h_taps_c;
int32_t h_init_phase_luma = 0; int32_t h_init_phase_luma = 0;
...@@ -747,7 +746,6 @@ bool dwb_program_horz_scalar(struct dcn20_dwbc *dwbc20, ...@@ -747,7 +746,6 @@ bool dwb_program_horz_scalar(struct dcn20_dwbc *dwbc20,
h_ratio_luma = -1; h_ratio_luma = -1;
else else
h_ratio_luma = dc_fixpt_u3d19(tmp_h_ratio_luma) << 5; h_ratio_luma = dc_fixpt_u3d19(tmp_h_ratio_luma) << 5;
h_ratio_chroma = h_ratio_luma * 2;
/*Program ratio*/ /*Program ratio*/
REG_UPDATE(WBSCL_HORZ_FILTER_SCALE_RATIO, WBSCL_H_SCALE_RATIO, h_ratio_luma); REG_UPDATE(WBSCL_HORZ_FILTER_SCALE_RATIO, WBSCL_H_SCALE_RATIO, h_ratio_luma);
...@@ -803,7 +801,6 @@ bool dwb_program_vert_scalar(struct dcn20_dwbc *dwbc20, ...@@ -803,7 +801,6 @@ bool dwb_program_vert_scalar(struct dcn20_dwbc *dwbc20,
enum dwb_subsample_position subsample_position) enum dwb_subsample_position subsample_position)
{ {
uint32_t v_ratio_luma = 1; uint32_t v_ratio_luma = 1;
uint32_t v_ratio_chroma = 1;
uint32_t v_taps_luma = num_taps.v_taps; uint32_t v_taps_luma = num_taps.v_taps;
uint32_t v_taps_chroma = num_taps.v_taps_c; uint32_t v_taps_chroma = num_taps.v_taps_c;
int32_t v_init_phase_luma = 0; int32_t v_init_phase_luma = 0;
...@@ -827,7 +824,6 @@ bool dwb_program_vert_scalar(struct dcn20_dwbc *dwbc20, ...@@ -827,7 +824,6 @@ bool dwb_program_vert_scalar(struct dcn20_dwbc *dwbc20,
v_ratio_luma = -1; v_ratio_luma = -1;
else else
v_ratio_luma = dc_fixpt_u3d19(tmp_v_ratio_luma) << 5; v_ratio_luma = dc_fixpt_u3d19(tmp_v_ratio_luma) << 5;
v_ratio_chroma = v_ratio_luma * 2;
/*Program ratio*/ /*Program ratio*/
REG_UPDATE(WBSCL_VERT_FILTER_SCALE_RATIO, WBSCL_V_SCALE_RATIO, v_ratio_luma); REG_UPDATE(WBSCL_VERT_FILTER_SCALE_RATIO, WBSCL_V_SCALE_RATIO, v_ratio_luma);
......
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