mirror of
https://github.com/nxp-imx/linux-imx.git
synced 2025-07-07 09:55:19 +02:00
drm/amd/display: Fix Coverity INTERGER_OVERFLOW within construct_integrated_info
[ Upstream commit 176abbcc71
]
[Why]
For substrcation, coverity reports integer overflow
warning message when variable type is uint32_t.
[How]
Change varaible type to int32_t.
Reviewed-by: Alex Hung <alex.hung@amd.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Acked-by: Tom Chung <chiahsuan.chung@amd.com>
Signed-off-by: Hersen Wu <hersenxs.wu@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
cb63090a17
commit
13faa9d401
|
@ -2552,8 +2552,8 @@ static enum bp_result construct_integrated_info(
|
|||
|
||||
/* Sort voltage table from low to high*/
|
||||
if (result == BP_RESULT_OK) {
|
||||
uint32_t i;
|
||||
uint32_t j;
|
||||
int32_t i;
|
||||
int32_t j;
|
||||
|
||||
for (i = 1; i < NUMBER_OF_DISP_CLK_VOLTAGE; ++i) {
|
||||
for (j = i; j > 0; --j) {
|
||||
|
|
|
@ -2935,8 +2935,11 @@ static enum bp_result construct_integrated_info(
|
|||
struct atom_common_table_header *header;
|
||||
struct atom_data_revision revision;
|
||||
|
||||
uint32_t i;
|
||||
uint32_t j;
|
||||
int32_t i;
|
||||
int32_t j;
|
||||
|
||||
if (!info)
|
||||
return result;
|
||||
|
||||
if (info && DATA_TABLES(integratedsysteminfo)) {
|
||||
header = GET_IMAGE(struct atom_common_table_header,
|
||||
|
|
Loading…
Reference in New Issue
Block a user