]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/amd/display: Remove redundant if conditions
authorIvan Lipski <ivlipski@amd.com>
Fri, 31 May 2024 18:48:41 +0000 (14:48 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 14 Jun 2024 20:17:13 +0000 (16:17 -0400)
[WHY]
Coverity analysis flagged the two if conditions in
dcn30_dpp.c and dcn401_dpp.c as DEADCODE since they
are never true, since the variable
'program_prealpha_dealpha' is initialized at 0 and
never chagnes.

[HOW]
Removed the variable 'program_prealpha_dealpha' and
the if conditions.

Acked-by: Zaeem Mohamed <zaeem.mohamed@amd.com>
Signed-off-by: Ivan Lipski <ivlipski@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dpp/dcn30/dcn30_dpp.c
drivers/gpu/drm/amd/display/dc/dpp/dcn401/dcn401_dpp.c

index abdf6a132e66539bcec0c676ef4ed90b9b550506..40acebd13e46dc1fa8b7ed691171bd3ebf2403ef 100644 (file)
@@ -219,7 +219,6 @@ void dpp3_cnv_setup (
        uint32_t alpha_plane_enable = 0;
        uint32_t dealpha_en = 0, dealpha_ablnd_en = 0;
        uint32_t realpha_en = 0, realpha_ablnd_en = 0;
-       uint32_t program_prealpha_dealpha = 0;
        struct out_csc_color_matrix tbl_entry;
        int i;
 
@@ -347,10 +346,6 @@ void dpp3_cnv_setup (
                        CNVC_ALPHA_PLANE_ENABLE, alpha_plane_enable);
        REG_UPDATE(FORMAT_CONTROL, FORMAT_CONTROL__ALPHA_EN, alpha_en);
 
-       if (program_prealpha_dealpha) {
-               dealpha_en = 1;
-               realpha_en = 1;
-       }
        REG_SET_2(PRE_DEALPHA, 0,
                        PRE_DEALPHA_EN, dealpha_en,
                        PRE_DEALPHA_ABLND_EN, dealpha_ablnd_en);
index eee64d8e10130b36e1ea24f6072c0b2a1607da8d..7cae18fd7be9c48b2d1aa7ec6099eb529acba4ab 100644 (file)
@@ -68,7 +68,6 @@ void dpp401_dpp_setup(
        uint32_t alpha_plane_enable = 0;
        uint32_t dealpha_en = 0, dealpha_ablnd_en = 0;
        uint32_t realpha_en = 0, realpha_ablnd_en = 0;
-       uint32_t program_prealpha_dealpha = 0;
        struct out_csc_color_matrix tbl_entry;
        int i;
 
@@ -192,10 +191,6 @@ void dpp401_dpp_setup(
                CNVC_ALPHA_PLANE_ENABLE, alpha_plane_enable);
        REG_UPDATE(FORMAT_CONTROL, FORMAT_CONTROL__ALPHA_EN, alpha_en);
 
-       if (program_prealpha_dealpha) {
-               dealpha_en = 1;
-               realpha_en = 1;
-       }
        REG_SET_2(PRE_DEALPHA, 0,
                PRE_DEALPHA_EN, dealpha_en,
                PRE_DEALPHA_ABLND_EN, dealpha_ablnd_en);