Commit ce19bbe4 authored by Chandan Vurdigere Nataraj's avatar Chandan Vurdigere Nataraj Committed by Alex Deucher

drm/amdgpu/display: Remove unnecessary typecasts and fix build issues

[Why]
Getting below errors:
drivers/gpu/drm/amd/amdgpu/../display/dc/dml/dcn32/display_mode_vba_32.c:1414:5: error: implicit conversion from enumeration type 'enum scan_direction_class' to different enumeration type 'enum dm_rotation_angle' [-Werror,-Wenum-conversion]
                                mode_lib->vba.SourceScan[k],
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/dc/dml/dcn32/display_mode_vba_32.c:1744:22: error: implicit conversion from enumeration type 'enum scan_direction_class' to different enumeration type 'enum dm_rotation_angle' [-Werror,-Wenum-conversion]
                        && (!(!IsVertical(mode_lib->vba.SourceScan[k])) || mode_lib->vba.DCCEnable[k] == true)) {
                               ~~~~~~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.

drivers/gpu/drm/amd/amdgpu/../display/dc/dml/dcn32/display_mode_vba_util_32.c:5484:18: error: implicit conversion from enumeration type 'RequestType' to different enumeration type 'enum RequestType' [-Werror,-Wenum-conversion]
                        RequestLuma = REQ_256Bytes;
                                    ~ ^~~~~~~~~~~~
18 errors of similar kind

[How]
1. Add typecast at relevant places
2. Move the enum RequestType definition ahead of declarations
Signed-off-by: default avatarChandan Vurdigere Nataraj <chandan.vurdigerenataraj@amd.com>
Reviewed-by: default avatarLeo Li <sunpeng.li@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 64f6516e
...@@ -1411,7 +1411,7 @@ static void DISPCLKDPPCLKDCFCLKDeepSleepPrefetchParametersWatermarksAndPerforman ...@@ -1411,7 +1411,7 @@ static void DISPCLKDPPCLKDCFCLKDeepSleepPrefetchParametersWatermarksAndPerforman
v->BytePerPixelC[k], v->BytePerPixelC[k],
v->BytePerPixelDETY[k], v->BytePerPixelDETY[k],
v->BytePerPixelDETC[k], v->BytePerPixelDETC[k],
mode_lib->vba.SourceScan[k], (enum dm_rotation_angle) mode_lib->vba.SourceScan[k],
/* Output */ /* Output */
&v->DCCYMaxUncompressedBlock[k], &v->DCCYMaxUncompressedBlock[k],
&v->DCCCMaxUncompressedBlock[k], &v->DCCCMaxUncompressedBlock[k],
...@@ -1741,7 +1741,8 @@ void dml32_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l ...@@ -1741,7 +1741,8 @@ void dml32_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l
mode_lib->vba.SourceFormatPixelAndScanSupport = true; mode_lib->vba.SourceFormatPixelAndScanSupport = true;
for (k = 0; k <= mode_lib->vba.NumberOfActiveSurfaces - 1; k++) { for (k = 0; k <= mode_lib->vba.NumberOfActiveSurfaces - 1; k++) {
if (mode_lib->vba.SurfaceTiling[k] == dm_sw_linear if (mode_lib->vba.SurfaceTiling[k] == dm_sw_linear
&& (!(!IsVertical(mode_lib->vba.SourceScan[k])) || mode_lib->vba.DCCEnable[k] == true)) { && (!(!IsVertical((enum dm_rotation_angle) mode_lib->vba.SourceScan[k]))
|| mode_lib->vba.DCCEnable[k] == true)) {
mode_lib->vba.SourceFormatPixelAndScanSupport = false; mode_lib->vba.SourceFormatPixelAndScanSupport = false;
} }
} }
......
...@@ -5314,9 +5314,15 @@ void dml32_CalculateDCCConfiguration( ...@@ -5314,9 +5314,15 @@ void dml32_CalculateDCCConfiguration(
unsigned int *IndependentBlockLuma, unsigned int *IndependentBlockLuma,
unsigned int *IndependentBlockChroma) unsigned int *IndependentBlockChroma)
{ {
typedef enum {
REQ_256Bytes,
REQ_128BytesNonContiguous,
REQ_128BytesContiguous,
REQ_NA
} RequestType;
enum RequestType RequestLuma; RequestType RequestLuma;
enum RequestType RequestChroma; RequestType RequestChroma;
unsigned int segment_order_horz_contiguous_luma; unsigned int segment_order_horz_contiguous_luma;
unsigned int segment_order_horz_contiguous_chroma; unsigned int segment_order_horz_contiguous_chroma;
...@@ -5350,13 +5356,6 @@ void dml32_CalculateDCCConfiguration( ...@@ -5350,13 +5356,6 @@ void dml32_CalculateDCCConfiguration(
double detile_buf_vp_horz_limit; double detile_buf_vp_horz_limit;
double detile_buf_vp_vert_limit; double detile_buf_vp_vert_limit;
typedef enum {
REQ_256Bytes,
REQ_128BytesNonContiguous,
REQ_128BytesContiguous,
REQ_NA
} RequestType;
yuv420 = ((SourcePixelFormat == dm_420_8 || SourcePixelFormat == dm_420_10 || yuv420 = ((SourcePixelFormat == dm_420_8 || SourcePixelFormat == dm_420_10 ||
SourcePixelFormat == dm_420_12) ? 1 : 0); SourcePixelFormat == dm_420_12) ? 1 : 0);
horz_div_l = 1; horz_div_l = 1;
...@@ -5527,11 +5526,11 @@ void dml32_CalculateDCCConfiguration( ...@@ -5527,11 +5526,11 @@ void dml32_CalculateDCCConfiguration(
RequestChroma = REQ_128BytesContiguous; RequestChroma = REQ_128BytesContiguous;
} }
if (RequestLuma == (enum RequestType) REQ_256Bytes) { if (RequestLuma == REQ_256Bytes) {
*MaxUncompressedBlockLuma = 256; *MaxUncompressedBlockLuma = 256;
*MaxCompressedBlockLuma = 256; *MaxCompressedBlockLuma = 256;
*IndependentBlockLuma = 0; *IndependentBlockLuma = 0;
} else if (RequestLuma == (enum RequestType) REQ_128BytesContiguous) { } else if (RequestLuma == REQ_128BytesContiguous) {
*MaxUncompressedBlockLuma = 256; *MaxUncompressedBlockLuma = 256;
*MaxCompressedBlockLuma = 128; *MaxCompressedBlockLuma = 128;
*IndependentBlockLuma = 128; *IndependentBlockLuma = 128;
...@@ -5541,11 +5540,11 @@ void dml32_CalculateDCCConfiguration( ...@@ -5541,11 +5540,11 @@ void dml32_CalculateDCCConfiguration(
*IndependentBlockLuma = 64; *IndependentBlockLuma = 64;
} }
if (RequestChroma == (enum RequestType) REQ_256Bytes) { if (RequestChroma == REQ_256Bytes) {
*MaxUncompressedBlockChroma = 256; *MaxUncompressedBlockChroma = 256;
*MaxCompressedBlockChroma = 256; *MaxCompressedBlockChroma = 256;
*IndependentBlockChroma = 0; *IndependentBlockChroma = 0;
} else if (RequestChroma == (enum RequestType) REQ_128BytesContiguous) { } else if (RequestChroma == REQ_128BytesContiguous) {
*MaxUncompressedBlockChroma = 256; *MaxUncompressedBlockChroma = 256;
*MaxCompressedBlockChroma = 128; *MaxCompressedBlockChroma = 128;
*IndependentBlockChroma = 128; *IndependentBlockChroma = 128;
......
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