From: Alex Hung <alex.h...@amd.com>

CalculateSwathAndDETConfiguration_params_st's UnboundedRequestEnabled
is a pointer (i.e. dml_bool_t *UnboundedRequestEnabled), and thus
if (p->UnboundedRequestEnabled) checks its address, not bool value.

This fixes 1 REVERSE_INULL issue reported by Coverity.

Reviewed-by: Harry Wentland <harry.wentl...@amd.com>
Acked-by: Hamza Mahfooz <hamza.mahf...@amd.com>
Signed-off-by: Alex Hung <alex.h...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c 
b/drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c
index 791f1725b62b..547dfcc80fde 100644
--- a/drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c
+++ b/drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c
@@ -4283,7 +4283,7 @@ static void CalculateSwathAndDETConfiguration(struct 
display_mode_lib_scratch_st
        }
 
        *p->compbuf_reserved_space_64b = 2 * p->PixelChunkSizeInKByte * 1024 / 
64;
-       if (p->UnboundedRequestEnabled) {
+       if (*p->UnboundedRequestEnabled) {
                *p->compbuf_reserved_space_64b = 
dml_max(*p->compbuf_reserved_space_64b,
                                (dml_float_t)(p->ROBBufferSizeInKByte * 1024/64)
                                - 
(dml_float_t)(RoundedUpSwathSizeBytesY[SurfaceDoingUnboundedRequest] * 
TTUFIFODEPTH / MAXIMUMCOMPRESSION/64));
-- 
2.45.1

Reply via email to