diff options
author | Ivan Lipski <[email protected]> | 2024-05-29 13:47:53 -0400 |
---|---|---|
committer | Alex Deucher <[email protected]> | 2024-06-14 16:18:25 -0400 |
commit | fe6f61d4729780269b2a454fdd5b5e038f0db0e4 (patch) | |
tree | fa3bf680ace693b18b5d77dfb0eebba253af7053 /tools/perf/scripts/python/stackcollapse.py | |
parent | 989947e90563eee58f37fbbad8a5bb94a3d8af8c (diff) |
drm/amd/display: Remove reduntant comparisons by Coverity
[WHY]
Coverity analysis of the Upstream display driver code
(amd-staging-drm-next) flagged these three functions as
containing 'CONSTANT_EXPRESSION_RESULT' errors, i. e. the
conditionals are reduntant since their result is predetermined.
fixpt31_32.c:
The two flagged 'ASSERT' lines redundant since they are always true:
- LONG_MAX equals to the largest 'signed long long' number
- res.value is type 'signed long long',
So, the condition inside the 'ASSERTS's is always
link_dp_training.c:
The flagged conditional statement is always false:
- 'max_requested.FFE_PRESET.settings.level' is 4 bits, so its max
possible value is 15
- 'DP_FFE_PRESET_MAX_LEVEL' equals to 15.
So, the conditional statement is always false and the wrapped code
never executes.
[HOW]
Removed lines flagged by Coverity analysis.
Reviewed-by: Sun peng Li <[email protected]>
Acked-by: Zaeem Mohamed <[email protected]>
Signed-off-by: Ivan Lipski <[email protected]>
Tested-by: Daniel Wheeler <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions