diff options
author | Alex Hung <alex.hung@amd.com> | 2023-12-01 06:25:34 -0700 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2023-12-06 15:22:35 -0500 |
commit | 86ecd796a88e26e025f184ff6a2e8872a6dc9ac7 (patch) | |
tree | c03c8c397bacc20ec3be943762152b5baa5d2c08 /drivers/gpu/drm/amd | |
parent | c09919e6ea5fefd49d8b7b54aa5b222937163108 (diff) | |
download | linux-stable-86ecd796a88e26e025f184ff6a2e8872a6dc9ac7.tar.gz linux-stable-86ecd796a88e26e025f184ff6a2e8872a6dc9ac7.tar.bz2 linux-stable-86ecd796a88e26e025f184ff6a2e8872a6dc9ac7.zip |
drm/amd/display: Validate hw_points_num before using it
[WHAT]
hw_points_num is 0 before ogam LUT is programmed; however, function
"dwb3_program_ogam_pwl" assumes hw_points_num is always greater than 0,
i.e. substracting it by 1 as an array index.
[HOW]
Check hw_points_num is not equal to 0 before using it.
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Alex Hung <alex.hung@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd')
-rw-r--r-- | drivers/gpu/drm/amd/display/dc/dcn30/dcn30_dwb_cm.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_dwb_cm.c b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_dwb_cm.c index 701c7d8bc038..03a50c32fcfe 100644 --- a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_dwb_cm.c +++ b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_dwb_cm.c @@ -243,6 +243,9 @@ static bool dwb3_program_ogam_lut( return false; } + if (params->hw_points_num == 0) + return false; + REG_SET(DWB_OGAM_CONTROL, 0, DWB_OGAM_MODE, 2); current_mode = dwb3_get_ogam_current(dwbc30); |