drm/amd/display: Remove unused dml32_CalculatedoublePipeDPPCLKAndSCLThroughput function
authorMaíra Canal <mairacanal@riseup.net>
Thu, 14 Jul 2022 16:44:59 +0000 (13:44 -0300)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 25 Jul 2022 13:31:02 +0000 (09:31 -0400)
Remove dml32_CalculatedoublePipeDPPCLKAndSCLThroughput function, which is not used in
the codebase.

This was pointed by clang with the following warning:

drivers/gpu/drm/amd/amdgpu/../display/dc/dml/dcn32/display_mode_vba_util_32.c:393:6:
warning: no previous prototype for function
'dml32_CalculatedoublePipeDPPCLKAndSCLThroughput' [-Wmissing-prototypes]
void dml32_CalculatedoublePipeDPPCLKAndSCLThroughput(
     ^
drivers/gpu/drm/amd/amdgpu/../display/dc/dml/dcn32/display_mode_vba_util_32.c:393:1:
note: declare 'static' if the function is not intended to be used outside of
this translation unit
void dml32_CalculatedoublePipeDPPCLKAndSCLThroughput(
^
static
1 warning generated.

Reviewed-by: André Almeida <andrealmeid@igalia.com>
Signed-off-by: Maíra Canal <mairacanal@riseup.net>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dml/dcn32/display_mode_vba_util_32.c

index febaff7..67cbc79 100644 (file)
@@ -390,60 +390,6 @@ void dml32_CalculateBytePerPixelAndBlockSizes(
 #endif
 } // CalculateBytePerPixelAndBlockSizes
 
-void dml32_CalculatedoublePipeDPPCLKAndSCLThroughput(
-               double HRatio,
-               double HRatioChroma,
-               double VRatio,
-               double VRatioChroma,
-               double MaxDCHUBToPSCLThroughput,
-               double MaxPSCLToLBThroughput,
-               double PixelClock,
-               enum source_format_class SourcePixelFormat,
-               unsigned int HTaps,
-               unsigned int HTapsChroma,
-               unsigned int VTaps,
-               unsigned int VTapsChroma,
-
-               /* output */
-               double *PSCL_THROUGHPUT,
-               double *PSCL_THROUGHPUT_CHROMA,
-               double *DPPCLKUsingdoubleDPP)
-{
-       double DPPCLKUsingdoubleDPPLuma;
-       double DPPCLKUsingdoubleDPPChroma;
-
-       if (HRatio > 1) {
-               *PSCL_THROUGHPUT = dml_min(MaxDCHUBToPSCLThroughput, MaxPSCLToLBThroughput * HRatio /
-                               dml_ceil((double) HTaps / 6.0, 1.0));
-       } else {
-               *PSCL_THROUGHPUT = dml_min(MaxDCHUBToPSCLThroughput, MaxPSCLToLBThroughput);
-       }
-
-       DPPCLKUsingdoubleDPPLuma = PixelClock * dml_max3(VTaps / 6 * dml_min(1, HRatio), HRatio * VRatio /
-                       *PSCL_THROUGHPUT, 1);
-
-       if ((HTaps > 6 || VTaps > 6) && DPPCLKUsingdoubleDPPLuma < 2 * PixelClock)
-               DPPCLKUsingdoubleDPPLuma = 2 * PixelClock;
-
-       if ((SourcePixelFormat != dm_420_8 && SourcePixelFormat != dm_420_10 && SourcePixelFormat != dm_420_12 &&
-                       SourcePixelFormat != dm_rgbe_alpha)) {
-               *PSCL_THROUGHPUT_CHROMA = 0;
-               *DPPCLKUsingdoubleDPP = DPPCLKUsingdoubleDPPLuma;
-       } else {
-               if (HRatioChroma > 1) {
-                       *PSCL_THROUGHPUT_CHROMA = dml_min(MaxDCHUBToPSCLThroughput, MaxPSCLToLBThroughput *
-                                       HRatioChroma / dml_ceil((double) HTapsChroma / 6.0, 1.0));
-               } else {
-                       *PSCL_THROUGHPUT_CHROMA = dml_min(MaxDCHUBToPSCLThroughput, MaxPSCLToLBThroughput);
-               }
-               DPPCLKUsingdoubleDPPChroma = PixelClock * dml_max3(VTapsChroma / 6 * dml_min(1, HRatioChroma),
-                               HRatioChroma * VRatioChroma / *PSCL_THROUGHPUT_CHROMA, 1);
-               if ((HTapsChroma > 6 || VTapsChroma > 6) && DPPCLKUsingdoubleDPPChroma < 2 * PixelClock)
-                       DPPCLKUsingdoubleDPPChroma = 2 * PixelClock;
-               *DPPCLKUsingdoubleDPP = dml_max(DPPCLKUsingdoubleDPPLuma, DPPCLKUsingdoubleDPPChroma);
-       }
-}
-
 void dml32_CalculateSwathAndDETConfiguration(
                unsigned int DETSizeOverride[],
                enum dm_use_mall_for_pstate_change_mode UseMALLForPStateChange[],