Commit 60bd99fd authored by zhengbin's avatar zhengbin Committed by Alex Deucher

drm/amd/display: use true, false for bool variable in display_rq_dlg_calc_21.c

Fixes coccicheck warning:

drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:85:6-13: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:88:2-9: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:225:6-14: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:226:6-14: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:251:3-11: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:252:3-11: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:256:3-11: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:257:3-11: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:267:3-11: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:269:3-11: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:682:6-14: WARNING: Assignment of 0/1 to bool variable
drivers/gpu/drm/amd/display/dc/dml/dcn21/display_rq_dlg_calc_21.c:1013:1-9: WARNING: Assignment of 0/1 to bool variable
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 d94af04b
...@@ -82,10 +82,10 @@ static unsigned int get_bytes_per_element(enum source_format_class source_format ...@@ -82,10 +82,10 @@ static unsigned int get_bytes_per_element(enum source_format_class source_format
static bool is_dual_plane(enum source_format_class source_format) static bool is_dual_plane(enum source_format_class source_format)
{ {
bool ret_val = 0; bool ret_val = false;
if ((source_format == dm_420_8) || (source_format == dm_420_10)) if ((source_format == dm_420_8) || (source_format == dm_420_10))
ret_val = 1; ret_val = true;
return ret_val; return ret_val;
} }
...@@ -222,8 +222,8 @@ static void handle_det_buf_split( ...@@ -222,8 +222,8 @@ static void handle_det_buf_split(
unsigned int swath_bytes_c = 0; unsigned int swath_bytes_c = 0;
unsigned int full_swath_bytes_packed_l = 0; unsigned int full_swath_bytes_packed_l = 0;
unsigned int full_swath_bytes_packed_c = 0; unsigned int full_swath_bytes_packed_c = 0;
bool req128_l = 0; bool req128_l = false;
bool req128_c = 0; bool req128_c = false;
bool surf_linear = (pipe_src_param.sw_mode == dm_sw_linear); bool surf_linear = (pipe_src_param.sw_mode == dm_sw_linear);
bool surf_vert = (pipe_src_param.source_scan == dm_vert); bool surf_vert = (pipe_src_param.source_scan == dm_vert);
unsigned int log2_swath_height_l = 0; unsigned int log2_swath_height_l = 0;
...@@ -248,13 +248,13 @@ static void handle_det_buf_split( ...@@ -248,13 +248,13 @@ static void handle_det_buf_split(
total_swath_bytes = 2 * full_swath_bytes_packed_l + 2 * full_swath_bytes_packed_c; total_swath_bytes = 2 * full_swath_bytes_packed_l + 2 * full_swath_bytes_packed_c;
if (total_swath_bytes <= detile_buf_size_in_bytes) { //full 256b request if (total_swath_bytes <= detile_buf_size_in_bytes) { //full 256b request
req128_l = 0; req128_l = false;
req128_c = 0; req128_c = false;
swath_bytes_l = full_swath_bytes_packed_l; swath_bytes_l = full_swath_bytes_packed_l;
swath_bytes_c = full_swath_bytes_packed_c; swath_bytes_c = full_swath_bytes_packed_c;
} else { //128b request (for luma only for yuv420 8bpc) } else { //128b request (for luma only for yuv420 8bpc)
req128_l = 1; req128_l = true;
req128_c = 0; req128_c = false;
swath_bytes_l = full_swath_bytes_packed_l / 2; swath_bytes_l = full_swath_bytes_packed_l / 2;
swath_bytes_c = full_swath_bytes_packed_c; swath_bytes_c = full_swath_bytes_packed_c;
} }
...@@ -264,9 +264,9 @@ static void handle_det_buf_split( ...@@ -264,9 +264,9 @@ static void handle_det_buf_split(
total_swath_bytes = 2 * full_swath_bytes_packed_l; total_swath_bytes = 2 * full_swath_bytes_packed_l;
if (total_swath_bytes <= detile_buf_size_in_bytes) if (total_swath_bytes <= detile_buf_size_in_bytes)
req128_l = 0; req128_l = false;
else else
req128_l = 1; req128_l = true;
swath_bytes_l = total_swath_bytes; swath_bytes_l = total_swath_bytes;
swath_bytes_c = 0; swath_bytes_c = 0;
...@@ -679,7 +679,7 @@ static void get_surf_rq_param( ...@@ -679,7 +679,7 @@ static void get_surf_rq_param(
const display_pipe_params_st pipe_param, const display_pipe_params_st pipe_param,
bool is_chroma) bool is_chroma)
{ {
bool mode_422 = 0; bool mode_422 = false;
unsigned int vp_width = 0; unsigned int vp_width = 0;
unsigned int vp_height = 0; unsigned int vp_height = 0;
unsigned int data_pitch = 0; unsigned int data_pitch = 0;
...@@ -1010,7 +1010,7 @@ static void dml_rq_dlg_get_dlg_params( ...@@ -1010,7 +1010,7 @@ static void dml_rq_dlg_get_dlg_params(
// Source // Source
// dcc_en = src.dcc; // dcc_en = src.dcc;
dual_plane = is_dual_plane((enum source_format_class) (src->source_format)); dual_plane = is_dual_plane((enum source_format_class) (src->source_format));
mode_422 = 0; // FIXME mode_422 = false; // FIXME
access_dir = (src->source_scan == dm_vert); // vp access direction: horizontal or vertical accessed access_dir = (src->source_scan == dm_vert); // vp access direction: horizontal or vertical accessed
// bytes_per_element_l = get_bytes_per_element(source_format_class(src.source_format), 0); // bytes_per_element_l = get_bytes_per_element(source_format_class(src.source_format), 0);
// bytes_per_element_c = get_bytes_per_element(source_format_class(src.source_format), 1); // bytes_per_element_c = get_bytes_per_element(source_format_class(src.source_format), 1);
......
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