drm/radeon/ci add comment to document intentionally unreachable code
authorNicholas Mc Guire <hofrat@osadl.org>
Tue, 19 Jul 2016 19:54:06 +0000 (21:54 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 29 Jul 2016 18:36:45 +0000 (14:36 -0400)
 commit d967be9b80a5 ("drm/radeon/ci: disable needless sclk changes")
 introduces an unreachable if(C != C) conditional code section
 flagged by coccinelle script bad_conditional.cocci:

 Add a comment to make it clear that this is intentional.

 Fixes: d967be9b80a5 ("drm/radeon/ci: disable needless sclk changes")

Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/radeon/ci_dpm.c

index 35e0fc3ae8a71d34b03f03ab175bf71db17fb45c..7ba450832e6b7a59db9499a0919a8bb5f4a3fd1a 100644 (file)
@@ -3843,7 +3843,10 @@ static void ci_find_dpm_states_clocks_in_dpm_table(struct radeon_device *rdev,
        if (i >= sclk_table->count) {
                pi->need_update_smu7_dpm_table |= DPMTABLE_OD_UPDATE_SCLK;
        } else {
-               /* XXX check display min clock requirements */
+               /* XXX The current code always reprogrammed the sclk levels,
+                * but we don't currently handle disp sclk requirements
+                * so just skip it.
+                */
                if (CISLAND_MINIMUM_ENGINE_CLOCK != CISLAND_MINIMUM_ENGINE_CLOCK)
                        pi->need_update_smu7_dpm_table |= DPMTABLE_UPDATE_SCLK;
        }